Skip to content

doc oc adm must-gather#14436

Closed
sanchezl wants to merge 1 commit intoopenshift:enterprise-4.1from
sanchezl:add_oc_adm_must-gather
Closed

doc oc adm must-gather#14436
sanchezl wants to merge 1 commit intoopenshift:enterprise-4.1from
sanchezl:add_oc_adm_must-gather

Conversation

@sanchezl
Copy link
Copy Markdown
Contributor

@sanchezl sanchezl commented Apr 12, 2019

Add new oc adm must-gather administrative command.

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Apr 12, 2019
@vikram-redhat
Copy link
Copy Markdown
Contributor

@bergerhoffer - FYI

@bergerhoffer
Copy link
Copy Markdown
Contributor

Hi @sanchezl, is the must-gather command provided by default with the CLI? I'm testing it now using the CLI build today, and it doesn't look to be available or shown in the help. Has it not officially been added yet? Or is there something that users must do to enable it?

I am planning on adding a section on using must-gather, but am waiting on @deads2k for some initial information.

@sanchezl
Copy link
Copy Markdown
Contributor Author

sanchezl commented Apr 15, 2019

is the must-gather command provided by default with the CLI? I'm testing it now using the CLI build today, and it doesn't look to be available or shown in the help. Has it not officially been added yet? Or is there something that users must do to enable it?

Hi @bergerhoffer, the command will be provided by default. It has not been merged as of this moment (openshift/origin#22430).

I am planning on adding a section on using must-gather, but am waiting on @deads2k for some initial information.

If you can point me to a template and/or location for this section, I would be happy to provide an initial draft in this PR.

@bergerhoffer
Copy link
Copy Markdown
Contributor

That would be great @sanchezl! I'll forward you the email that had some questions I had on it.

And I'll watch that PR and will merge this PR once that gets in. Thanks for adding this!


Gather debugging information.

.Example: Gather debugging information
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Contributor

@bergerhoffer bergerhoffer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sanchezl So now that I see the PR was merged and can test, I see that must-gather is listed in the help under cluster management (not other commands). Can you move this to the cli-administrator-cluster-management.adoc file [1] instead (alphabetically)? Thanks!

[1] https://github.com/openshift/openshift-docs/blob/enterprise-4.0/modules/cli-administrator-cluster-management.adoc

@kalexand-rh kalexand-rh changed the base branch from enterprise-4.0 to enterprise-4.1 April 17, 2019 15:52
@openshift-ci-robot
Copy link
Copy Markdown

@sanchezl: PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 24, 2019
@bergerhoffer
Copy link
Copy Markdown
Contributor

Closing this PR in favor of #14811.

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

Labels

needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants