Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge BDNS contributions from each partner and make them available as a centralized BDNS hosts file #123

Open
joaoluis-pdm opened this issue Oct 6, 2022 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@joaoluis-pdm
Copy link
Contributor

joaoluis-pdm commented Oct 6, 2022

On the meeting 2022-10-06 between @lukasosterheider , @TiagoV-PDMFC and @joaoluis-pdm , it was decided that the next stage UAT testing (Stage 5 testing?) will be more distributed (avoinding PDM centralized storage - on BDNS domain traceability - when possible).

This implies (at least):

  • BDNS host set of files so that each partner can add their own subdomain traceability.PARTNER-ID by contributing (with pull requests ?)
  • merge mechanism so that partner's constributions are merged into a single file, and the file made available by https (with a frozen URL) (needs to be a public URL ? If not on github, we can make that available on PDM)
  • configuration changes (both docker and https://github.com/PharmaLedger-IMI/helm-charts/tree/master/charts/fgt ) so that apihub pulls that BDNS hosts file.
  • build process must be modified to build appropriate DSUs on to appropriate domains. (Need extra config parsing to figure out the ID ? Or manual domain configuration)
  • test everything with a limited set of partners (Bayer?) so that all operational configs and procedures are ok to be announced to the other partners.
@joaoluis-pdm joaoluis-pdm added the enhancement New feature or request label Oct 6, 2022
@joaoluis-pdm joaoluis-pdm self-assigned this Oct 6, 2022
TiagoV-PDMFC added a commit that referenced this issue Oct 13, 2022
TiagoV-PDMFC added a commit that referenced this issue Oct 14, 2022
TiagoV-PDMFC added a commit that referenced this issue Oct 14, 2022
@joaoluis-pdm
Copy link
Contributor Author

Issue #122 might block one of the partners to use decentralised domain DSUs.

TiagoV-PDMFC added a commit that referenced this issue Nov 9, 2022
TiagoV-PDMFC added a commit that referenced this issue Nov 9, 2022
TiagoV-PDMFC added a commit that referenced this issue Nov 9, 2022
TiagoV-PDMFC added a commit that referenced this issue Nov 9, 2022
…er file to use environment variables instead of build args
TiagoV-PDMFC added a commit that referenced this issue Nov 9, 2022
TiagoV-PDMFC added a commit that referenced this issue Nov 9, 2022
joaoluis-pdm added a commit that referenced this issue Nov 9, 2022
joaoluis-pdm added a commit that referenced this issue Nov 9, 2022
joaoluis-pdm added a commit that referenced this issue Nov 9, 2022
joaoluis-pdm added a commit that referenced this issue Nov 9, 2022
TiagoV-PDMFC added a commit that referenced this issue Nov 9, 2022
TiagoV-PDMFC added a commit that referenced this issue Nov 9, 2022
TiagoV-PDMFC added a commit that referenced this issue Nov 9, 2022
TiagoV-PDMFC added a commit that referenced this issue Nov 9, 2022
TiagoV-PDMFC added a commit that referenced this issue Nov 10, 2022
TiagoV-PDMFC added a commit that referenced this issue Nov 10, 2022
TiagoV-PDMFC added a commit that referenced this issue Nov 10, 2022
TiagoV-PDMFC added a commit that referenced this issue Nov 10, 2022
TiagoV-PDMFC added a commit that referenced this issue Nov 10, 2022
TiagoV-PDMFC added a commit that referenced this issue Nov 10, 2022
TiagoV-PDMFC added a commit that referenced this issue Nov 10, 2022
joaoluis-pdm added a commit that referenced this issue Nov 10, 2022
joaoluis-pdm added a commit that referenced this issue Nov 10, 2022
joaoluis-pdm added a commit that referenced this issue Nov 10, 2022
joaoluis-pdm added a commit that referenced this issue Nov 10, 2022
joaoluis-pdm added a commit that referenced this issue Nov 11, 2022
joaoluis-pdm added a commit that referenced this issue Nov 14, 2022
joaoluis-pdm added a commit that referenced this issue Nov 14, 2022
joaoluis-pdm added a commit that referenced this issue Nov 15, 2022
joaoluis-pdm added a commit that referenced this issue Nov 21, 2022
joaoluis-pdm added a commit that referenced this issue Nov 21, 2022
@joaoluis-pdm
Copy link
Contributor Author

After a BDMS chamge, such as 61d1bae
all PDM hosted WHSs and PHAs need to be restarted to refresh DNS.

pharmaledger@fgt-tst-pl:~/fgt-workspace/docker/api$ docker compose rm --stop whs-1
pharmaledger@fgt-tst-pl:~/fgt-workspace/docker/api$ docker compose rm --stop --force whs-2
pharmaledger@fgt-tst-pl:~/fgt-workspace/docker/api$ docker compose rm --force --stop pha-1
pharmaledger@fgt-tst-pl:~/fgt-workspace/docker/api$ docker compose rm --force --stop pha-2
pharmaledger@fgt-tst-pl:~/fgt-workspace/docker/api$ docker compose rm --force --stop whs-zuellig
pharmaledger@fgt-tst-pl:~/fgt-workspace/docker/api$ docker compose rm --force --stop whs-takeda
pharmaledger@fgt-tst-pl:~/fgt-workspace/docker/api$ ./bootCompose.sh tst

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants