Skip to content

Ensure canary/green machines get their health checked - #4952

Merged
lubien merged 6 commits into
masterfrom
fix/bluegreen-canary-skiplaunck-healthcheck
Aug 6, 2026
Merged

Ensure canary/green machines get their health checked#4952
lubien merged 6 commits into
masterfrom
fix/bluegreen-canary-skiplaunck-healthcheck

Conversation

@lubien

@lubien lubien commented Jun 29, 2026

Copy link
Copy Markdown
Member

Change Summary

What and Why: fix green and canary machines being considered ready if they're stopped

How: ensure they start so we can actually see their healthchecks

Related to: internal forum post


Documentation

  • Fresh Produce (soon)
  • In superfly/docs, or asked for help from docs team
  • n/a

@lubien
lubien force-pushed the fix/bluegreen-canary-skiplaunck-healthcheck branch from 06e9fe0 to b7a732e Compare August 5, 2026 13:22
@lubien
lubien marked this pull request as ready for review August 5, 2026 13:22
@lubien
lubien requested review from dangra and rianmcguirefly August 5, 2026 13:22
@lubien lubien changed the title wip: bluegreen tweaks Ensure canary/green machines get their health checked Aug 5, 2026
@lubien
lubien force-pushed the fix/bluegreen-canary-skiplaunck-healthcheck branch from b7a732e to dc83e1c Compare August 5, 2026 13:28
@lubien lubien self-assigned this Aug 5, 2026

@dangra dangra left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is the rationale to start all green instances? shouldn't it be enough (and faster) if it mimics the started/stopped instances? At minimum, it could ensure at least one is started and passing checks from each process group.

@lubien
lubien requested a review from dangra August 6, 2026 12:09

@dangra dangra left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

@lubien
lubien merged commit 9aaec2f into master Aug 6, 2026
@lubien
lubien deleted the fix/bluegreen-canary-skiplaunck-healthcheck branch August 6, 2026 19:08
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