Skip to content

feat(ui): Show eks cluster name in the main UI - #447

Open
ianis-c wants to merge 5 commits into
awslabs:mainfrom
ianis-c:show-eks-cluster-name
Open

ianis-c wants to merge 5 commits into
awslabs:mainfrom
ianis-c:show-eks-cluster-name

Conversation

@ianis-c

@ianis-c ianis-c commented Jun 25, 2025

Copy link
Copy Markdown

Issue #, if available:
No linked issue.

Description of changes:
This feature retrieve and show the current EKS cluster being shown in the eks-node-viewer main UI.
It can be useful when working with more than one cluster (one or more clusters per environment for example).

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ianis-c
ianis-c requested a review from a team as a code owner June 25, 2025 12:00
Comment thread cmd/eks-node-viewer/main.go Resolved
@ianis-c
ianis-c requested a review from tzneal June 25, 2025 18:22
@ianis-c

ianis-c commented Jul 4, 2025

Copy link
Copy Markdown
Author

Hi @tzneal, do you think this feature could be integrated ?

Comment thread cmd/eks-node-viewer/main.go Resolved
Comment thread pkg/model/uimodel.go Resolved
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BRz2ooES611HtEWW5Rmwga
…p line

Address review feedback:
- Get the current context name from the kubeconfig via client-go in
  getConfig instead of shelling out to kubectl. This also respects the
  --context flag and multi-file KUBECONFIG values.
- Show the cluster name after the "q: quit" help text instead of using
  an extra line at the top of the screen.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BRz2ooES611HtEWW5Rmwga
@ianis-c
ianis-c requested a review from tzneal November 3, 2026 18:29
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