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

Update error message when gather bootstrap fails from create cluster. #3256

Closed
wants to merge 1 commit into from

Conversation

patrickdillon
Copy link
Contributor

@patrickdillon patrickdillon commented Mar 6, 2020

The gather bootstrap error message specifies to use the --key command to specify a key. When that error message is shown after running create cluster, users may think the --key flag is available in the create cluster command.

Resolves: #3188

Error messages are hard... so suggestions are definitely encouraged.

The gather bootstrap error message specifies to use the --key command to specify a key. When that error message is shown after running create cluster, users may think the --key flag is available in the create cluster command.

Addresses: openshift#3188
@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Mar 6, 2020
@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign abhinavdahiya
You can assign the PR to them by writing /assign @abhinavdahiya in a comment when ready.

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

Needs approval from an approver in each of these files:

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

@patrickdillon
Copy link
Contributor Author

/test e2e-steps

@jstuever
Copy link
Contributor

/retest

@jstuever
Copy link
Contributor

/lgtm
A good way to clarify.

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Mar 13, 2020
@jstuever jstuever removed their assignment Mar 19, 2020
@jstuever
Copy link
Contributor

/assign @abhinavdahiya

@patrickdillon
Copy link
Contributor Author

/test e2e-vsphere
Testing this on a completely unrelated PR to see if master vsphere has apiserver failures.

@openshift-ci-robot
Copy link
Contributor

@patrickdillon: The following tests failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/prow/e2e-libvirt 9d4c8e8 link /test e2e-libvirt
ci/prow/e2e-vsphere 9d4c8e8 link /test e2e-vsphere

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

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. I understand the commands that are listed here.

@@ -103,7 +103,7 @@ var (
logrus.Error("Attempted to gather ClusterOperator status after installation failure: ", err2)
}
if err2 := runGatherBootstrapCmd(rootOpts.dir); err2 != nil {
logrus.Error("Attempted to gather debug logs after installation failure: ", err2)
logrus.Error("Attempted to gather debug logs after installation failure by running openshift-install gather bootstrap:", err2)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
logrus.Error("Attempted to gather debug logs after installation failure by running openshift-install gather bootstrap:", err2)
logrus.Error("Failed to gather debug logs using openshift-install gather bootstrap: ", err2)

@abhinavdahiya abhinavdahiya added the version/4.6 Tracking changes that should end up in 4.6 release label May 8, 2020
@patrickdillon
Copy link
Contributor Author

/close
#3615 is handling this & more

@openshift-ci-robot
Copy link
Contributor

@patrickdillon: Closed this PR.

In response to this:

/close
#3615 is handling this & more

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. version/4.6 Tracking changes that should end up in 4.6 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

failed ssh connect on install advises me to use --key option which doesn't exist
4 participants