Make shared default admin demo user for GraphKB/IPR about pori

[creisle]

@brayden-bcgsc and @mathieulemieux when the tasks above are done can you post the applicable release version of the API container back here

[mathieulemieux]

In order to test these changes, I did the following on dockerdev server:

• strop and remove all pori containers
• clone a new copy of pori, pori_ipr_api and pori_grapgkb_api repos
• in pori_ipr_api:
  • checkout develop branch (includes changes to demo/ipr_demodb.postgres.dump)
• in pori_graphkb_api:
  • checkout feature/KBDEV-977-make-shared-default-admin-demo-user-for-graphkb-ipr-57 branch (includes changes to src/repo/schema.js)
• in pori:
  • checkout feature/make-shared-default-admin-demo-user-for-graphkb-ipr-57 branch (includes changes to demo/kc_realmexport.json)
  • made these changes to docker-compose.dev.yml
  • mkdir -p databases/{postgres,orientdb}/{backup,data} && mkdir keys
  • docker-compose -f docker-compose.dev.yml up -d --build
  • exported 3000, 5000, 8080, 8081 & 8888.

Here are the Keycloak users:

Here are the GraphKb users:

Errors I got:

• I can't log into IPR using pori_admin; only iprdemo will work
• Going from IPR to GraphKB while logged to IPR with iprdemo isn't possible without relogging with pori_admin user

[mathieulemieux]

Ok, in order to make it work, we need to update the ipr_demodb image too.
It's look like this: before starting the containers with docker-compose,

• in pori_ipr_api repo, run the following command: "docker build -f Dockerfile.db -t bcgsc/pori-ipr-demodb:pori_admin ." (Ty @brayden-bcgsc !)
• in pori repo, make an additional change to the docker-compose.dev.yml to make the ipr_db container based on image bcgsc/pori-ipr-demodb:pori_admin instead of bcgsc/pori-ipr-demodb:latest

Everything went well.
The pori_admin user allows to switch from IPR to GraphKB without the need to logout.

[mathieulemieux]

On pori_graphkb_api, replace feature/KBDEV-977-make-shared-default-admin-demo-user-for-graphkb-ipr-57 from develop by bugfix/KBDEV-977-make-shared-default-admin-demo-user-for-graphkb-ipr-57 from master. Commit the same changes as before and re-tested everything on dockerdev. Ready to PR.

[mathieulemieux]

For info, here is the previous PR for default db dump made by @RaphaelPletz to pori_ipr_api: bcgsc/pori_ipr_api#160.

[mathieulemieux]

Here are the different users in ipr, graphkb and keycloak:

Should we get rid of colab_demo and iprdemo?

[creisle]

Here are the different users in ipr, graphkb and keycloak: Should we get rid of colab_demo and iprdemo?

we should only remove the users that were replaced with the new account (pori_admin) or that are no longer created

• colab_demo should still be being created, it is used for the colab/jupyter notebooks in the docs
• please double check that iprdemo still exists in IPR, if it does it can be left as well

[mathieulemieux]

Looks like we are still using the iprdemo user in IPR so I'll let it there: https://github.com/bcgsc/pori_ipr_api/blob/b5038f806e183f0432a25078ed682b68ad9081f6/demo/clean_db_for_demo.js#L167-L206

[mathieulemieux]

Closed by PR #59