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

try to get the os name form the pod assigned node #2282

Merged
merged 4 commits into from
Nov 12, 2023

Conversation

wjiec
Copy link
Contributor

@wjiec wjiec commented Nov 10, 2023

this PR fixes #2281.

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.

@wjiec Nice work! (I was looking at this last nite)
Thank you so much for this quick update as we should repair ASAP!!

internal/view/pod.go Outdated Show resolved Hide resolved
internal/view/pod.go Outdated Show resolved Hide resolved
internal/view/pod.go Outdated Show resolved Hide resolved
@wjiec
Copy link
Contributor Author

wjiec commented Nov 10, 2023

Hi @derailed, thanks for the quick feedback, what do you think of the changes now?

BTW, the k9s is a great project and it has helped me very much at work, so I hope it gets a little better!

@wjiec wjiec requested a review from derailed November 10, 2023 16:07
internal/view/pod.go Outdated Show resolved Hide resolved
@derailed
Copy link
Owner

@wjiec Thank you for your kind words! You are a stand up guy!!
You've made my day! am so humbled by this comment as this is exactly why we're still here...

@wjiec
Copy link
Contributor Author

wjiec commented Nov 10, 2023

I'm a bad talker, so let's make k9s even better! 😊

@wjiec
Copy link
Contributor Author

wjiec commented Nov 10, 2023

any nitpicks are okay, I'm a perfectionist too, and I look forward to hearing from you.

@wjiec wjiec requested a review from derailed November 10, 2023 17:02
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.

@wjiec Thank you for the quick updates and turn around Jayson!
No worries. I got you.

@derailed derailed merged commit eee1862 into derailed:master Nov 12, 2023
3 checks passed
derailed added a commit that referenced this pull request Nov 12, 2023
* get the os name from the node when needed

* code quality

* remove err sentinel

---------

Co-authored-by: Fernand Galiana <fernand.galiana@gmail.com>
@wjiec wjiec deleted the bugfix/node-shell branch November 12, 2023 06:00
thejoeejoee pushed a commit to thejoeejoee/k9s that referenced this pull request Feb 23, 2024
* get the os name from the node when needed

* code quality

* remove err sentinel

---------

Co-authored-by: Fernand Galiana <fernand.galiana@gmail.com>
placintaalexandru pushed a commit to placintaalexandru/k9s that referenced this pull request Apr 3, 2024
* get the os name from the node when needed

* code quality

* remove err sentinel

---------

Co-authored-by: Fernand Galiana <fernand.galiana@gmail.com>
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.

[BUG] Can't run Node shell
2 participants