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

Chore/deploy workspaces heal preprod #2704

Merged
merged 2 commits into from
Mar 31, 2021

Conversation

em-ingram
Copy link
Contributor

Environments

  • HEAL Preprod

Description of changes

  • Deploy 2021.03 service versions
  • Deploy workspaces

@PlanXCyborg
Copy link
Contributor

preprod.healdata.org/manifest.json

Deployment changes

  • fence
    • REQUIRES A FENCE DB MIGRATION, since with this update there is now an additional column in the authorization_code table "refresh_token_expires_in" (uc-cdis/Feat/refresh expires in fence#848)
  • 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)

@williamhaley williamhaley force-pushed the chore/deploy-workspaces-heal-preprod branch from 979db7d to e2c23f9 Compare March 30, 2021 22:00
Copy link
Contributor

@williamhaley williamhaley left a comment

Choose a reason for hiding this comment

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

lgtm! Maybe double check my ecr updates to the containers @mpingram, but otherwise :shipit:

@em-ingram em-ingram merged commit cabf0cf into master Mar 31, 2021
@em-ingram em-ingram deleted the chore/deploy-workspaces-heal-preprod branch March 31, 2021 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants