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

oc status should let me know if all the pods for a DC are pending #6426

Closed
deads2k opened this issue Dec 19, 2015 · 7 comments
Closed

oc status should let me know if all the pods for a DC are pending #6426

deads2k opened this issue Dec 19, 2015 · 7 comments

Comments

@deads2k
Copy link
Contributor

deads2k commented Dec 19, 2015

It's annoying that I can't look at status and know if my service or pod should be accessible. If that pod isn't running, its not very useful to me.

oc status
In project mythtv-backend on server https://eads-nas:8443

dc/tmp deploys 172.30.201.81:5000/mythtv-backend/mythtv-backend-base:latest 
  #1 deployment running for 10 seconds - 1 pod

View details with 'oc describe <resource>/<name>' or list everything with 'oc get all'.
@0xmichalis
Copy link
Contributor

+1

I would expect "deployment running" to show up when at least one pod is ready. Please assign to me so I won't forget about it.

@0xmichalis
Copy link
Contributor

@deads2k "deployment running" here means the deployer. Shoule we reword it to something like "deployment process running"?

@deads2k
Copy link
Contributor Author

deads2k commented Dec 21, 2015

I'm ok with "deployment running". I'm frustrated that having "1 pod" can be in any state. I think that only Running should be considered "1 pod" Maybe just listing the state is sufficient.

@0xmichalis
Copy link
Contributor

After thinking about this more, I think the best solution would be to surface running pods in the DC status (similar to kubernetes/kubernetes#19382) and re-use that info here. See #6443 (comment)

@0xmichalis
Copy link
Contributor

Blocked on #5918

@0xmichalis
Copy link
Contributor

Blocked on #5918

Finally fixed

@0xmichalis
Copy link
Contributor

0xmichalis commented Aug 31, 2016

We should use rc.status.readyReplicas for this - coming with the next rebase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants