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

Remove temporary config changes (rolling deployment chore) #811

Open
colleenXu opened this issue Apr 18, 2024 · 2 comments
Open

Remove temporary config changes (rolling deployment chore) #811

colleenXu opened this issue Apr 18, 2024 · 2 comments
Labels
On Test Related changes are deployed to Test server

Comments

@colleenXu
Copy link
Collaborator

colleenXu commented Apr 18, 2024

Often, we use temporary overrides or other config adjustments to handle the staggered deployment of features (Dev -> CI -> Test -> Prod). We direct BTE to use updated/changed SmartAPI yamls.

Once the features are deployed to Prod (on all instances), we have a chore to remove them:

  • make the changes into the default
    • merge the SmartAPI yaml changes to the master branch (registered yamls)
    • for TRAPI updates, update the registered yaml to latest TRAPI + all server urls. Then delete the temporary registrations used in the exclusions
  • remove the temporary overrides or other config adjustments
  • put into the next deployment cycle

We can't remove SmartAPI yaml branches used for the temporary overrides until the overrides have been removed from all instances.

@colleenXu
Copy link
Collaborator Author

For the current Translator Sprint (2/Octopus):

@colleenXu
Copy link
Collaborator Author

colleenXu commented Jun 14, 2024

Update: sprint 2/Octopus has deployed to Prod.

We've decided to add this chore to the patch we're doing to Octopus with this PR biothings/bte-server#30:

Once the patch is deployed on all instances, then I can merge this PR to update BioThings PFOCR's registered yaml (which all instances will be using): NCATS-Tangerine/translator-api-registry#153

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
On Test Related changes are deployed to Test server
Projects
None yet
Development

No branches or pull requests

1 participant