Simplify worker node troubleshooting steps

[ironcladlou]

Use simpler/more copy-pastable commands for worker node troubleshooting.

[cgwalters]

Great minds think alike! #1049

[ironcladlou]

@cgwalters 😄

[cgwalters]

Yours seems better though a minor bit - I like "topic prefixes" in commit messages (git log docs in this repo shows some of that). Combining yours and mine: docs/troubleshooting: Simplify worker node debugging commands?

[ironcladlou]

@cgwalters

Yours seems better though a minor bit - I like "topic prefixes" in commit messages (git log docs in this repo shows some of that). Combining yours and mine: docs/troubleshooting: Simplify worker node debugging commands?

Good idea... done

[crawford]

Can you keep the equal sign for consistency with the rest of the doc?

[ironcladlou]

Done

…

On Fri, Jan 11, 2019 at 10:53 AM Alex Crawford ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In docs/user/troubleshooting.md <#1050 (comment)>: > @@ -11,23 +11,24 @@ The installer doesn't provision worker nodes directly, like it does with master The status of the Machine API Operator can be checked by running the following command from the machine used to install the cluster: ```sh -oc --config=${INSTALL_DIR}/auth/kubeconfig --namespace=openshift-cluster-api get pods +oc --config=${INSTALL_DIR}/auth/kubeconfig --namespace openshift-cluster-api get deployments Can you keep the equal sign for consistency with the rest of the doc? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#1050 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AASNO25UxpudrRHOCD_20TXsAKCml-Ejks5vCLOJgaJpZM4Z7mzc> .

-- Dan Mace Principal Software Engineer, OpenShift Red Hat [email protected]

[crawford]

This text also needs to be updated.

[ironcladlou]

Done

[DanyC97]

@ironcladlou nit: double the

[ironcladlou]

Thanks, fixed

[abhinavdahiya]

/lgtm
/retest

/hold

@crawford doc change only, okay post freeze?

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abhinavdahiya, ironcladlou

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [abhinavdahiya]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[crawford]

@abhinavdahiya yes, but have me sign off on all of them anyway.

And yes, this one is fine to merge.

[abhinavdahiya]

@abhinavdahiya yes, but have me sign off on all of them anyway.

that's why i added the /hold to get it signed-off 😇

And yes, this one is fine to merge.

/hold cancel