-
Notifications
You must be signed in to change notification settings - Fork 9
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 references to "_aks" environment names #4693
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From the app point of view, looks good 👍
Is there anything we need to do on infra side or did they already remove references to _aks
as well?
Such as copy/moving the settings.yml for each environment and so on?
Could you also rename maintenance_page/manifests/env_aks to just env, or I can update on a separate pr if preferred. |
Done! I'm curious about how it knows how to pick up the changes though. The workflow file has only |
Thanks! The workflow would call env, but maintenance_page/scripts/failover.sh uses $CONFIG from global_config/env.sh which is now being changed from env_aks to env. |
Context
We've recently dropped "_aks" suffixes from AKS configurations.
We should remove the "_aks" suffixes from application code too.
Changes proposed in this pull request
_aks
suffix from environment names.Guidance to review
Let's see if
review
works :D