-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Pod HealthCheck #6145
Pod HealthCheck #6145
Conversation
To be honest I wouldn't want to add so many details to the pod info. There is already a lot of information there and the liveness probe can be easily checked by simply clicking edit and looking at yaml. |
I understand, but if some users can't read YAML ... What if we get a compromise. Like the healthcheck comes "closed" or "minimized" by default and one needs to click on it to open it ? Also, to make life worse, sooner or later I might provide you a patch to see the TCP ports of the containers. My users need that :/ |
We have to rethink how to better manage information display as the pod page tends to get quite long already. We can merge for now but soon we'll have to improve it. /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: floreks, marcosdiez The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Codecov Report
@@ Coverage Diff @@
## master #6145 +/- ##
==========================================
- Coverage 41.48% 41.47% -0.02%
==========================================
Files 216 216
Lines 9347 9350 +3
Branches 159 159
==========================================
Hits 3878 3878
- Misses 5217 5220 +3
Partials 252 252 |
@@ -22,4 +22,23 @@ import {Probe} from '@api/root.api'; | |||
export class ProbeComponent { | |||
@Input() initialized: boolean; | |||
@Input() probe: Probe; | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very smart from your side! And I was fighting for hours with CSS. Thank you once more!
Floresk, please feel free to change the layout as you prefer!
Also, I need CSS help removing whitespaces on the
HTTP Healthcheck URI
ps: an excellent container to test this is kube-system/coredns