Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding system arch to nodes view #2480

Merged
merged 3 commits into from Jan 17, 2024

Conversation

guanchzhou
Copy link
Contributor

This PR introduces system architecture column in node view.

It may be helpfull for activities like migration from amd64 to arm64 nodes.

Copy link
Owner

@derailed derailed left a comment

Choose a reason for hiding this comment

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

@guanchzhou Thank you for this PR!

internal/render/node.go Outdated Show resolved Hide resolved
@derailed derailed added enhancement New feature or request question Further information is requested labels Jan 16, 2024
@derailed
Copy link
Owner

@guanchzhou Thank you for the updates! Looks like the tests are not happy ;(

@guanchzhou
Copy link
Contributor Author

@guanchzhou Thank you for the updates! Looks like the tests are not happy ;(

Yes, my bad. FIxed that and test locally, all looks good.

@derailed derailed merged commit 5445ff4 into derailed:master Jan 17, 2024
3 checks passed
@derailed derailed mentioned this pull request Jan 18, 2024
@eldada
Copy link
Contributor

eldada commented Jan 18, 2024

Great feature, but I don't see it. Do I need to do anything to get it showing?

@guanchzhou
Copy link
Contributor Author

Great feature, but I don't see it. Do I need to do anything to get it showing?

basically, in nodes view press ctrl-w to toggle wide mode and you will see it there

Screenshot 2024-01-18 at 12 50 30

@guanchzhou guanchzhou deleted the adding-arch-to-nodes-view branch January 18, 2024 10:54
@eldada
Copy link
Contributor

eldada commented Jan 18, 2024

Great feature, but I don't see it. Do I need to do anything to get it showing?

basically, in nodes view press ctrl-w to toggle wide mode and you will see it there

Screenshot 2024-01-18 at 12 50 30

Yep. Works as expected 😄
Thank you. Great feature!

thejoeejoee pushed a commit to thejoeejoee/k9s that referenced this pull request Feb 23, 2024
* adding arch to nodes view

* hiding arch under wide view and fix test

* fixing tests

---------

Co-authored-by: Andrei Maltsev <amaltsev@alpha-sense.com>
placintaalexandru pushed a commit to placintaalexandru/k9s that referenced this pull request Apr 3, 2024
* adding arch to nodes view

* hiding arch under wide view and fix test

* fixing tests

---------

Co-authored-by: Andrei Maltsev <amaltsev@alpha-sense.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants