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

deploying 2020.12 to NCT PROD #2447

Merged
merged 1 commit into from
Jan 11, 2021
Merged

deploying 2020.12 to NCT PROD #2447

merged 1 commit into from
Jan 11, 2021

Conversation

atharvar28
Copy link
Contributor

This PR

  1. deploying 2020.12 to NCT prod
  2. adding metadata-service to the service block as with ssjdispatcher version 2020.12 it is mandatory to deploy metadata-service
  3. correcting the /workspace

@PlanXCyborg
Copy link
Contributor

accessclinicaldata.niaid.nih.gov/manifest.json

Deployment changes

  • ssjdispatcher
    • metadata-service must already be present or added to the "versions": { section of the cdis-manifest. After metadata-service has been confirmed to be in the cdis-manifest, the now-required Metadata Service creds for SSJDispatcher must be configured by running gen3 roll all with JENKINS_HOME not set. gen3 roll all calls gen3 kube-setup-ssjdispatcher, which now automatically configures the SSJDispatcher secret with Metadata Service creds. (uc-cdis/(PXP-6977): make mds creds required for service startup ssjdispatcher#32)

"ssjdispatcher": "quay.io/cdis/ssjdispatcher:2020.11"
"guppy": "quay.io/cdis/guppy:2020.12",
"ssjdispatcher": "quay.io/cdis/ssjdispatcher:2020.12",
"metadata": "quay.io/cdis/metadata-service:2020.12"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how come the qa environments doesn't have the mds?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh, it's because QA doesn't have ssjdispatcher. if you don't mind, could you sync QA with prod?

Copy link
Contributor Author

@atharvar28 atharvar28 Dec 16, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its doesnt have ssjdispatcher service. PROD had ssjdispatcher from the start so maybe a good idea to add metadata-service so not to break anything in PROD

@frickjack frickjack merged commit b362ee6 into master Jan 11, 2021
@frickjack frickjack deleted the chore/NCT_PROD_202012 branch January 11, 2021 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants