Skip to content

Bug 1884643 update resolving multi-attach errors procedure#27703

Merged
bmcelvee merged 1 commit intoopenshift:masterfrom
bmcelvee:BZ1884643
Nov 30, 2020
Merged

Bug 1884643 update resolving multi-attach errors procedure#27703
bmcelvee merged 1 commit intoopenshift:masterfrom
bmcelvee:BZ1884643

Conversation

@bmcelvee
Copy link
Copy Markdown
Contributor

@bmcelvee bmcelvee added this to the Next Release milestone Nov 30, 2020
@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 30, 2020
@vikram-redhat
Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 30, 2020
Copy link
Copy Markdown
Contributor

@kalexand-rh kalexand-rh left a comment

Choose a reason for hiding this comment

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

Two small suggestions, but it LGTM

Comment thread modules/storage-multi-attach-error.adoc Outdated
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.

If this is the way to fix it, I'd you can.

Suggested change
If you encounter a multi-attach error message with an RWO volume, you can force delete the pod on a shutdown or crashed node.
If you encounter a multi-attach error message with an RWO volume, force delete the pod on a shut down or crashed node.

Comment thread modules/storage-multi-attach-error.adoc Outdated
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.

Suggested change
This deletes the volumes stuck on shutdown or crashed nodes after six minutes.
This command deletes the volumes stuck on shutdown or crashed nodes after six minutes.

@kalexand-rh kalexand-rh added the peer-review-done Signifies that the peer review team has reviewed this PR label Nov 30, 2020
@openshift-ci-robot
Copy link
Copy Markdown

New changes are detected. LGTM label has been removed.

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Nov 30, 2020
@openshift-docs-preview-bot
Copy link
Copy Markdown

The preview will be available shortly at:

@bmcelvee
Copy link
Copy Markdown
Contributor Author

No QE review required.

@bmcelvee bmcelvee merged commit f9d702e into openshift:master Nov 30, 2020
@bmcelvee
Copy link
Copy Markdown
Contributor Author

bmcelvee commented Nov 30, 2020

/cherrypick enterprise-4.5

@bmcelvee
Copy link
Copy Markdown
Contributor Author

bmcelvee commented Nov 30, 2020

/cherrypick enterprise-4.6

@openshift-cherrypick-robot
Copy link
Copy Markdown

openshift-cherrypick-robot commented Nov 30, 2020

@bmcelvee: new pull request created: #27705

Details

In response to this:

/cherrypick enterprise-4.5

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.

@bmcelvee
Copy link
Copy Markdown
Contributor Author

bmcelvee commented Nov 30, 2020

/cherrypick enterprise-4.7

@openshift-cherrypick-robot
Copy link
Copy Markdown

openshift-cherrypick-robot commented Nov 30, 2020

@bmcelvee: new pull request created: #27706

Details

In response to this:

/cherrypick enterprise-4.6

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-cherrypick-robot
Copy link
Copy Markdown

openshift-cherrypick-robot commented Nov 30, 2020

@bmcelvee: new pull request created: #27707

Details

In response to this:

/cherrypick enterprise-4.7

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.

@bobfuru
Copy link
Copy Markdown
Contributor

bobfuru commented Nov 30, 2020

Thanks for fixing, @bmcelvee. I also created and merged PR 27719 to update an instance of this wording in the persistent storage docs.

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

Labels

branch/enterprise-4.5 branch/enterprise-4.6 branch/enterprise-4.7 peer-review-done Signifies that the peer review team has reviewed this PR size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants