Skip to content

Update prod deployment flows to match new config format#641

Merged
ryanjjung merged 8 commits intomainfrom
prod-cluster
Mar 18, 2026
Merged

Update prod deployment flows to match new config format#641
ryanjjung merged 8 commits intomainfrom
prod-cluster

Conversation

@ryanjjung
Copy link
Contributor

This updates the Accounts deployment flow to update the new .accounts_image variable in the config, which gets applied to all task definitions that use it. Also, I did a pass on the prod config, fixed several things out of alphabetical order, and found a couple items that were missing.

@ryanjjung ryanjjung self-assigned this Mar 16, 2026
=======
.oidc_url_jwks: &VAR_OIDC_URL_JWKS {name: "OIDC_URL_JWKS", value: "https://auth-stage.tb.pro/realms/tbpro/protocol/openid-connect/certs"}
.oidc_url_logout: &VAR_OIDC_URL_LOGOUT {name: "OIDC_URL_LOGOUT", value: "https://auth-stage.tb.pro/realms/tbpro/protocol/openid-connect/logout"}
>>>>>>> main
Copy link
Member

Choose a reason for hiding this comment

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

Rebase conflict?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ugh, guess I missed one. I'll fix it.

@ryanjjung ryanjjung requested a review from mzeier March 17, 2026 15:45
Copy link
Member

@MelissaAutumn MelissaAutumn left a comment

Choose a reason for hiding this comment

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

Looks okay. Did we need that appointment secret?

@ryanjjung
Copy link
Contributor Author

There was an appointment-caldav-secret that we do need in stage. Davi and I made sure it was set up there yesterday.

@ryanjjung
Copy link
Contributor Author

On second review, it looks like the log level is being removed. Lemme add that back.

@ryanjjung ryanjjung merged commit 69c58a0 into main Mar 18, 2026
8 checks passed
@ryanjjung ryanjjung deleted the prod-cluster branch March 18, 2026 17:00
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.

2 participants