Skip to content

Conversation

bogsi17
Copy link
Contributor

@bogsi17 bogsi17 commented Apr 7, 2025

  • In cases where a new ECS service is deployed, while the loadbalancer target group that is controlled by Codedeploy points to the "wrong" target group, terraform apply would result into an error that the new service is not attached to a valid target group.

  • This can be taken care of by assigning the currently active target group to this variable

Previously, this could already be done by changing it temporarily in the terraform configuration, but having this as a variable allows for different values per environment

This is part of a PR chain with #3323 as parent

@bogsi17 bogsi17 force-pushed the MAV-818_default_lb_target_group branch from 8f5f6fe to 91d66f1 Compare April 8, 2025 07:26
@bogsi17 bogsi17 force-pushed the MAV-818_default_lb_target_group branch from 91d66f1 to d31cc6c Compare April 8, 2025 07:28
@bogsi17 bogsi17 changed the base branch from v2.1.3-wip to update_workflows_for_modularized_services April 8, 2025 07:29
@TheOneFromNorway TheOneFromNorway force-pushed the update_workflows_for_modularized_services branch 3 times, most recently from 27f6bf0 to 16ed7c2 Compare April 8, 2025 13:09
@TheOneFromNorway TheOneFromNorway self-requested a review April 8, 2025 13:11
@TheOneFromNorway TheOneFromNorway force-pushed the MAV-818_default_lb_target_group branch from d31cc6c to 377f5b1 Compare April 8, 2025 13:17
Base automatically changed from update_workflows_for_modularized_services to main April 8, 2025 13:20
bogsi17 added 2 commits April 8, 2025 14:28
* In cases where a new ECS service is deployed, while the loadbalancer target group that is controlled by Codedeploy points to the "wrong" target group, terraform apply would result into an error that the new service is not attached to a valid target group.
* This can be taken care of by assigning the currently active target group to this variable
@bogsi17 bogsi17 force-pushed the MAV-818_default_lb_target_group branch from e400829 to 0da26aa Compare April 8, 2025 13:29
* Set the variable for preview since the currently active target group is green
@bogsi17 bogsi17 force-pushed the MAV-818_default_lb_target_group branch from 6a1cf41 to e07be8f Compare April 8, 2025 14:05
@tvararu tvararu temporarily deployed to mavis-pr-3335 April 8, 2025 14:05 Inactive
Copy link

sonarqubecloud bot commented Apr 8, 2025

@TheOneFromNorway TheOneFromNorway added this to the v2.2.0 milestone Apr 8, 2025
@bogsi17 bogsi17 merged commit ce5f46c into main Apr 8, 2025
22 checks passed
@bogsi17 bogsi17 deleted the MAV-818_default_lb_target_group branch April 8, 2025 14:16
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.

3 participants