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

Running Job is shown in red, like the failed ones in the Jobs view #2686

Open
gdubicki opened this issue Apr 30, 2024 · 7 comments
Open

Running Job is shown in red, like the failed ones in the Jobs view #2686

gdubicki opened this issue Apr 30, 2024 · 7 comments

Comments

@gdubicki
Copy link
Sponsor




Describe the bug

On the Jobs view for a CronJob, the jobs that are failed (the ones on the bottom of the below screenshot) look the same as the running one (on the top on the below screenshot).

This is erroneous in my opinion, as the color red should not be used for non-failed resources.

To Reproduce

Steps to reproduce the behavior:

  1. Have a CronJob with some failed and some successful and a running job
  2. Go to this CronJob and "enter" it to show its list of Jobs

Expected behavior

I expect the running, non-failed Job to be shown in a different color than red - blue or green perhaps?

Screenshots

Screenshot 2024-04-30 at 14 05 43

Versions (please complete the following information):

  • OS: macOS Sonoma 14.4.1 (23E224)
  • K9s: v0.32.4
  • K8s: v1.29.4-gke.1043000
@gdubicki
Copy link
Sponsor Author

Slightly related to #725?

@KasperHeyndrickx
Copy link
Contributor

Not exactly a regression, but I did some testing and up until v0.31.8 the job would stay green even when it failed.
image

Behavior changed here in internal/render/job.go

After this all pending, running and failing jobs are displayed in red

KasperHeyndrickx added a commit to KasperHeyndrickx/k9s that referenced this issue May 11, 2024
@gdubicki
Copy link
Sponsor Author

Thanks for the PR #2698, @KasperHeyndrickx!

@KasperHeyndrickx
Copy link
Contributor

Do you know if I should ping someone to get an approval for the github workflow?

@gdubicki
Copy link
Sponsor Author

Do you know if I should ping someone to get an approval for the github workflow?

I don’t know. I can’t find easy way to see the repo’s maintainers, but here are core members listed: https://github.com/derailed/k9s?tab=readme-ov-file#meet-the-core-team

@gdubicki
Copy link
Sponsor Author

The PR helped fix my problem. Using the build from it:

Screenshot 2024-05-12 at 11 39 55

@KasperHeyndrickx
Copy link
Contributor

Thanks for confirming!

derailed pushed a commit that referenced this issue May 18, 2024
* fix: job color based on failures (#2686)

* fix: don't show error when job succeeds in next attempt
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

No branches or pull requests

2 participants