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.

Sign up for GitHub

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

Jump to bottom

oc debug on a scaled to zero deployment doesn't work #14286

Closed
smarterclayton opened this issue May 22, 2017 · 0 comments
Closed

oc debug on a scaled to zero deployment doesn't work #14286

smarterclayton opened this issue May 22, 2017 · 0 comments
Assignees
@fabianofranz @juanvallejo
Labels
area/usability component/cli kind/bug Categorizes issue or PR as related to a bug. priority/P2

Comments

@smarterclayton
Copy link
Contributor 

$ oc debug --loglevel=8 -c prometheus --one-container -o yaml deploy/prometheus

Should have generated yaml. Instead it blocks waiting for a pod to show up. But no pod will ever show up. It should instead use the pod template in the deployment.
@pweil- pweil- added component/cli kind/bug Categorizes issue or PR as related to a bug. priority/P2 labels May 24, 2017
@juanvallejo juanvallejo mentioned this issue Sep 15, 2017
Merged
openshift-merge-robot added a commit that referenced this issue Oct 25, 2017
@openshift-merge-robot
Merge pull request #16379 from juanvallejo/jvallejo/fix-debug-on-depl…
598a4ac 
…oyment-w-0-replicas

Automatic merge from submit-queue.

UPSTREAM: 53606: use deployment pod template if 0 replicas

Fixes #14286
Fixes #14915

Prevent `oc debug` from hanging indefinitely when dealing
with a deployment that has been scaled down to 0.

cc @openshift/cli-review @smarterclayton
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fabianofranz fabianofranz

@juanvallejo juanvallejo

Labels
area/usability component/cli kind/bug Categorizes issue or PR as related to a bug. priority/P2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@fabianofranz @smarterclayton @juanvallejo @pweil-