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

Dibbs ECR Viewer repo update, NBS_AUTH usage and database var changes 🎊 #20

Merged
merged 13 commits into from
Feb 19, 2025

Conversation

alismx
Copy link
Collaborator

@alismx alismx commented Feb 13, 2025

Related Issue

Changes Proposed

  • This PR updates the GHCR used for the dibbs-ecr-viewer docker images
  • NBS_AUTH has been added to enable and disable auth explicitly
  • The AWS provider version has been updated

Additional Info

  • Once this is merged, I'll be creating a new release which will publish this module to the terraform registry

Testing

  • Those that need to know how to test have been contacted in other channels.

This commit introduces more granular secret handling for database and SQL server parameters, adjusts the ECR repository for viewer, and adds 'nbs auth' variable. In the ECS task configuration, a few more properties are specified for container definitions, and the re-deployment trigger in ECS service has been updated to include version in addition to timestamp.
@alismx alismx changed the title update variables to set config name, allow for a service to be reacha… Dibbs ECR Viewer repo update and CONFIG_NAME env usage 🎊 Feb 13, 2025
@alismx alismx changed the title Dibbs ECR Viewer repo update and CONFIG_NAME env usage 🎊 Dibbs ECR Viewer repo update, NBS_AUTH env usage and database var changes 🎊 Feb 13, 2025
@alismx alismx changed the title Dibbs ECR Viewer repo update, NBS_AUTH env usage and database var changes 🎊 Dibbs ECR Viewer repo update, NBS_AUTH usage and database var changes 🎊 Feb 13, 2025
@alismx alismx self-assigned this Feb 13, 2025
@alismx alismx marked this pull request as draft February 13, 2025 21:50
@alismx alismx marked this pull request as ready for review February 18, 2025 20:57
@alismx alismx merged commit 1b73e62 into main Feb 19, 2025
2 checks passed
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.

AWS provider update Config name setup, add nbs_auth var
2 participants