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.

Sign up for GitHub

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

Jump to bottom

Master submodule update 10 28 #2501

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Master submodule update 10 28 #2501

wants to merge 3 commits into from

Conversation

jrvaldes
Copy link
Contributor

Ran:

 ./hack/update_submodules.sh master

@jrvaldes
[submodule][kubelet] Update to 7ca9eb1e9e5
753c1c0 
Update to openshift/kubernetes@7ca9eb1

This commit was generated using hack/update_submodules.sh
@jrvaldes
[build] Update kubelet version to v1.31.2+7ca9eb1
4c4684a 
This commit was generated using hack/update_submodules.sh
@jrvaldes
[vendor] Use v1.31.2+7ca9eb1 k8s and OpenShift libraries
35af9b3 
This commit was generated using hack/update_submodules.sh
@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Oct 28, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jrvaldes, mansikulkarni96

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

The pull request process is described 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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 28, 2024
@jrvaldes
Copy link
Contributor Author

/test remaining-required

@jrvaldes
Copy link
Contributor Author

/retest-required

2 similar comments
@jrvaldes
Copy link
Contributor Author

/retest-required

@jrvaldes
Copy link
Contributor Author

/retest-required

@jrvaldes
Copy link
Contributor Author

jrvaldes commented Nov 1, 2024

vsphere-proxy-e2e-operator test passed must-gather failed

@mansikulkarni96
Copy link
Member

/test azure-e2e-operator

@mansikulkarni96
Copy link
Member

/override ci/prow/vsphere-proxy-e2e-operator

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Nov 1, 2024

@mansikulkarni96: Overrode contexts on behalf of mansikulkarni96: ci/prow/vsphere-proxy-e2e-operator

In response to this:

/override ci/prow/vsphere-proxy-e2e-operator

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-sigs/prow repository.

@mansikulkarni96
Copy link
Member

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 1, 2024
@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 9dc53bb and 2 for PR HEAD 35af9b3 in total

@sebsoto
Copy link
Contributor

sebsoto commented Nov 4, 2024

/override "Red Hat Konflux / windows-machine-config-operator-bundle-master-on-pull-request" "Red Hat Konflux / windows-machine-config-operator-master-on-pull-request"

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Nov 4, 2024

@sebsoto: Overrode contexts on behalf of sebsoto: Red Hat Konflux / windows-machine-config-operator-bundle-master-on-pull-request, Red Hat Konflux / windows-machine-config-operator-master-on-pull-request

In response to this:

/override "Red Hat Konflux / windows-machine-config-operator-bundle-master-on-pull-request" "Red Hat Konflux / windows-machine-config-operator-master-on-pull-request"

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-sigs/prow repository.

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 0951f6f and 2 for PR HEAD 35af9b3 in total

4 similar comments
@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 0951f6f and 2 for PR HEAD 35af9b3 in total

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 0951f6f and 2 for PR HEAD 35af9b3 in total

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 0951f6f and 2 for PR HEAD 35af9b3 in total

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 0951f6f and 2 for PR HEAD 35af9b3 in total

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 05e38eb and 1 for PR HEAD 35af9b3 in total

@jrvaldes
Copy link
Contributor Author

jrvaldes commented Nov 6, 2024

@sebsoto @mansikulkarni96 could you override the azure-e2e-operator job

it passed in the pass https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_windows-machine-config-operator/2501/pull-ci-openshift-windows-machine-config-operator-master-azure-e2e-operator/1852419348442910720

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 4312fa7 and 0 for PR HEAD 35af9b3 in total

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Nov 7, 2024

@jrvaldes: 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/azure-e2e-operator 35af9b3 link true /test azure-e2e-operator

Full PR test history. Your PR dashboard.

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

@openshift-ci-robot
Copy link

/hold

Revision 35af9b3 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 Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mansikulkarni96 mansikulkarni96 mansikulkarni96 approved these changes

@sebsoto sebsoto Awaiting requested review from sebsoto

Assignees

@mansikulkarni96 mansikulkarni96

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. 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.
4 participants
@jrvaldes @mansikulkarni96 @openshift-ci-robot @sebsoto