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

Feat/use stable image tag #2

Merged
merged 4 commits into from
Oct 15, 2024
Merged

Feat/use stable image tag #2

merged 4 commits into from
Oct 15, 2024

Conversation

jboix
Copy link
Contributor

@jboix jboix commented Oct 15, 2024

Description

Updated the ECR image tagging strategy and renamed environment variables for monitoring services. It also synchronizes the Grafana dashboard with the latest configuration.

Changes Made

  • Update remote submodules to reference the latest commit.
  • Changed environment variables from QOS to MONITORING.
  • Updated the grafana dashboard to the latest remote configuration.
  • Modified Terraform configuration to tag ECR images with both latest and stable.
  • Latest is automatically applied to the dev environment for the latest version.
  • stable is manually tagged to indicate a version that is ready for promotion to the production environment.

Checklist

  • I have followed the project's style and contribution guidelines.
  • I have performed a self-review of my own changes.
  • I have made corresponding changes to the documentation.
  • I have added tests that prove my fix is effective or that my feature works.

Update remote submodules to reference the latest commit.
Changed environment variables from `QOS` to `MONITORING`.
Updated the grafana dashboard to the latest remote configuration.
- Modified Terraform configuration to tag ECR images with both `latest` and `stable`.
- `Latest` is automatically applied to the dev environment for the latest version.
- `stable` is manually tagged to indicate a version that is ready for promotion to
  the production environment.
@jboix jboix self-assigned this Oct 15, 2024
@jboix jboix requested review from defagos and waliid October 15, 2024 12:14
@jboix jboix added this pull request to the merge queue Oct 15, 2024
Merged via the queue into main with commit d8d2610 Oct 15, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants