Skip to content

Conversation

csenet
Copy link
Contributor

@csenet csenet commented Jul 23, 2025

Fixed an issue where deployment log messages were incorrect for ECS native Blue/Green deployments

Expected behavior

When using ECS Blue/Green deployment strategy, ecspresso should log "deployment by ECS blue/green" to clearly indicate the deployment method being used.

Actual behavior

  • All deployments were logged as "deployment by ECS rolling update" regardless of the actual deployment strategy
  • This was misleading when Blue/Green deployment was actually being used

Solution

Added conditional logic to check DeploymentConfiguration.Strategy and log the appropriate message:

  • When Strategy == types.DeploymentStrategyBlueGreen: logs "deployment by ECS blue/green"
  • Otherwise: logs "deployment by ECS rolling update"

@fujiwara fujiwara merged commit 644a004 into kayac:v2 Jul 23, 2025
2 checks passed
@fujiwara
Copy link
Collaborator

@csenet Thank you!!

@github-actions github-actions bot mentioned this pull request Jul 22, 2025
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