Skip to content

bump openshift/api to get csi.sharedresource.openshift.io#224

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
coreydaley:bump_openshift_api
Oct 15, 2021
Merged

bump openshift/api to get csi.sharedresource.openshift.io#224
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
coreydaley:bump_openshift_api

Conversation

@coreydaley
Copy link
Copy Markdown

@coreydaley coreydaley commented Oct 15, 2021

No description provided.

@openshift-ci openshift-ci Bot requested review from dobsonj and tsmetana October 15, 2021 11:56
@coreydaley
Copy link
Copy Markdown
Author

coreydaley commented Oct 15, 2021

/assign @jsafrane
We need to pull in this update for openshift/csi-driver-shared-resource-operator#9 to pass, since it is rejecting the csi.sharedresource.openshift.io driver name.

@jsafrane
Copy link
Copy Markdown
Contributor

jsafrane commented Oct 15, 2021

/lgtm

@coreydaley coreydaley mentioned this pull request Oct 15, 2021
Merged
@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Oct 15, 2021
@jsafrane
Copy link
Copy Markdown
Contributor

jsafrane commented Oct 15, 2021

/approve
/hold
(for the e2e tests to pass, feel free to remove when it's green)

@openshift-ci openshift-ci Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 15, 2021
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 15, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: coreydaley, jsafrane

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:

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

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 15, 2021
@coreydaley
Copy link
Copy Markdown
Author

coreydaley commented Oct 15, 2021

/retest
possible vsphere flakes

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 15, 2021

@coreydaley: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-vsphere 91f1928 link false /test e2e-vsphere

Full PR test history. Your PR dashboard.

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

@coreydaley
Copy link
Copy Markdown
Author

coreydaley commented Oct 15, 2021

/retest

@jsafrane
Copy link
Copy Markdown
Contributor

jsafrane commented Oct 15, 2021

/hold cancel

@openshift-ci openshift-ci Bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 15, 2021
@openshift-merge-robot openshift-merge-robot merged commit 07cf8ba into openshift:master Oct 15, 2021
@gabemontero gabemontero mentioned this pull request Oct 21, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dobsonj dobsonj Awaiting requested review from dobsonj

@tsmetana tsmetana Awaiting requested review from tsmetana

Assignees

@jsafrane jsafrane

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@coreydaley @jsafrane @openshift-merge-robot