Skip to content

Fix federation status check when cluster has no context#430

Merged
yoctozepto merged 2 commits intomainfrom
no-context-means-federation-status-cannot-be-checked
Apr 15, 2026
Merged

Fix federation status check when cluster has no context#430
yoctozepto merged 2 commits intomainfrom
no-context-means-federation-status-cannot-be-checked

Conversation

@alec-w
Copy link
Copy Markdown
Contributor

@alec-w alec-w commented Apr 15, 2026

When a cluster has no context the federation status cannot be checked - so status should be "Unknown"

@alec-w alec-w self-assigned this Apr 15, 2026
@alec-w alec-w added this to the now milestone Apr 15, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a check to ensure a Kubernetes context is present for a cluster during federation status verification. The review feedback recommends enhancing the error message to include the cluster and trust zone names for better clarity and notes a potential logic improvement regarding how clusters are selected for status checks.

Comment thread cmd/cofidectl/cmd/federation/federation.go
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@yoctozepto yoctozepto enabled auto-merge (squash) April 15, 2026 14:59
@yoctozepto yoctozepto merged commit 762ce83 into main Apr 15, 2026
8 checks passed
@yoctozepto yoctozepto deleted the no-context-means-federation-status-cannot-be-checked branch April 15, 2026 15:01
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