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

use a separate job to deploy US1 #2228

Merged
merged 2 commits into from
May 11, 2023
Merged

Conversation

BenoitZugmeyer
Copy link
Member

Motivation

We reached more than 100 orgs using the new CDN URL on US1. When doing a release, deploy US1 separately to limit the scope.

Changes

Split deploy-prod-all-dcs into deploy-prod-minor-dcs and deploy-prod-us1. I used gitlab extend to avoid repeating ourselves too much.

Testing

  • Local
  • Staging
  • Unit
  • End to end

I have gone over the contributing documentation.

@BenoitZugmeyer BenoitZugmeyer requested a review from a team as a code owner May 10, 2023 15:16
.gitlab-ci.yml Outdated
Comment on lines 289 to 301
deploy-prod-minor-dcs:
extends:
- .deploy-prod
variables:
UPLOAD_PATH: eu1,us3,us5,ap1

deploy-prod-us1:
extends:
- .deploy-prod
variables:
UPLOAD_PATH: us1

deploy-prod-root:
Copy link
Contributor

Choose a reason for hiding this comment

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

💭 thought: ‏FMU, the jobs of a stage are displayed in alphabetical order, it could be nice to have this order match the expected deployment order.
Maybe we could prefix the name by stepN_or something, wdyt?

Copy link
Contributor

@bcaudan bcaudan left a comment

Choose a reason for hiding this comment

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

Nice!

@BenoitZugmeyer BenoitZugmeyer merged commit 437516f into main May 11, 2023
17 checks passed
@BenoitZugmeyer BenoitZugmeyer deleted the benoit/add-deploy-prod-us1-job branch May 11, 2023 12:55
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.

None yet

3 participants