Skip to content

Add additional comments with findings from k8s 1.26 bump#27728

Merged
openshift-merge-robot merged 1 commit into
openshift:masterfrom
soltysh:add_comments
Mar 2, 2023
Merged

Add additional comments with findings from k8s 1.26 bump#27728
openshift-merge-robot merged 1 commit into
openshift:masterfrom
soltysh:add_comments

Conversation

@soltysh
Copy link
Copy Markdown
Contributor

@soltysh soltysh commented Feb 15, 2023

/hold
/assign @xueqzhan

until after #27694 merges

@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 Feb 15, 2023
@openshift-ci openshift-ci Bot requested review from bparees and csrwng February 15, 2023 15:12
@openshift-ci openshift-ci Bot added e2e-images-update Related to images used by e2e tests vendor-update Touching vendor dir or related files approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Feb 15, 2023
@soltysh
Copy link
Copy Markdown
Contributor Author

soltysh commented Feb 16, 2023

/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 Feb 16, 2023
@xueqzhan
Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Feb 16, 2023
@openshift-ci-robot
Copy link
Copy Markdown

/retest-required

Remaining retests: 0 against base HEAD 7f6fb95 and 2 for PR HEAD 090dbc4 in total

@openshift-ci-robot
Copy link
Copy Markdown

/retest-required

Remaining retests: 0 against base HEAD 55ca147 and 1 for PR HEAD 090dbc4 in total

@openshift-ci-robot
Copy link
Copy Markdown

/retest-required

Remaining retests: 0 against base HEAD 6834f50 and 0 for PR HEAD 090dbc4 in total

@openshift-ci-robot
Copy link
Copy Markdown

/hold

Revision 090dbc4 was retested 3 times: holding

@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 Feb 17, 2023
@openshift-merge-robot openshift-merge-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Feb 20, 2023
@openshift-ci openshift-ci Bot removed the lgtm Indicates that a PR is ready to be merged. label Mar 2, 2023
@soltysh
Copy link
Copy Markdown
Contributor Author

soltysh commented Mar 2, 2023

/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 Mar 2, 2023
@soltysh
Copy link
Copy Markdown
Contributor Author

soltysh commented Mar 2, 2023

/retest-required

Copy link
Copy Markdown
Contributor

@sanchezl sanchezl left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Mar 2, 2023
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Mar 2, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: sanchezl, soltysh, xueqzhan

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

/retest-required

Remaining retests: 0 against base HEAD e8b3293 and 2 for PR HEAD 2511c8d in total

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Mar 2, 2023

@soltysh: The following tests 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-aws-ovn-cgroupsv2 2511c8d link false /test e2e-aws-ovn-cgroupsv2
ci/prow/e2e-metal-ipi-sdn 2511c8d link false /test e2e-metal-ipi-sdn
ci/prow/e2e-openstack-ovn 2511c8d link false /test e2e-openstack-ovn
ci/prow/e2e-aws-ovn-single-node-serial 2511c8d link false /test e2e-aws-ovn-single-node-serial
ci/prow/e2e-aws-ovn-single-node 2511c8d link false /test e2e-aws-ovn-single-node
ci/prow/e2e-azure-ovn-etcd-scaling 2511c8d link false /test e2e-azure-ovn-etcd-scaling
ci/prow/e2e-vsphere-ovn-etcd-scaling 2511c8d link false /test e2e-vsphere-ovn-etcd-scaling
ci/prow/e2e-aws-ovn-upgrade 2511c8d link false /test e2e-aws-ovn-upgrade
ci/prow/e2e-aws-ovn-single-node-upgrade 2511c8d link false /test e2e-aws-ovn-single-node-upgrade
ci/prow/e2e-aws-ovn-etcd-scaling 2511c8d link false /test e2e-aws-ovn-etcd-scaling

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.

@openshift-merge-robot openshift-merge-robot merged commit 349d5bc into openshift:master Mar 2, 2023
@soltysh soltysh deleted the add_comments branch March 3, 2023 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. e2e-images-update Related to images used by e2e tests lgtm Indicates that a PR is ready to be merged. vendor-update Touching vendor dir or related files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants