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

Revert "Alibaba: removal of master and 4.16 jobs" #46795

Merged
merged 1 commit into from
Dec 14, 2023
Merged

Revert "Alibaba: removal of master and 4.16 jobs" #46795

merged 1 commit into from
Dec 14, 2023

Conversation

jcpowermac
Copy link
Contributor

This reverts commit 8037af5.

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 14, 2023
@JoelSpeed
Copy link
Contributor

/lgtm
/pj-rehearse ack

@openshift-ci-robot openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Dec 14, 2023
@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Dec 14, 2023
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Dec 14, 2023
@JoelSpeed
Copy link
Contributor

/lgtm

1 similar comment
@jmguzik
Copy link
Contributor

jmguzik commented Dec 14, 2023

/lgtm

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Dec 14, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jcpowermac, jmguzik, JoelSpeed

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 lgtm Indicates that a PR is ready to be merged. label Dec 14, 2023
@jcpowermac
Copy link
Contributor Author

/retest

@jcpowermac
Copy link
Contributor Author

/override ci/prow/build-clusters

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Dec 14, 2023

@jcpowermac: jcpowermac unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:openshift: openshift-release-oversight.

In response to this:

/override ci/prow/build-clusters

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.

@jmguzik
Copy link
Contributor

jmguzik commented Dec 14, 2023

/override ci/prow/build-clusters

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Dec 14, 2023

@jmguzik: Overrode contexts on behalf of jmguzik: ci/prow/build-clusters

In response to this:

/override ci/prow/build-clusters

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-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Dec 14, 2023

@jcpowermac: all tests passed!

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

@stbenjam
Copy link
Member

/hold

I don't think the error you're reporting from the release controller is actually fatal. You just need a corresponding PR to ocp-build-data to remove the ART side of things.

@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 Dec 14, 2023
@stbenjam
Copy link
Member

CI payloads are broken

https://amd64.ocp.releases.ci.openshift.org/releasestream/4.16.0-0.ci/release/4.16.0-0.ci-2023-12-14-105222

I guess we can revert this until we get all the neccessary PR's staged to merge in one go

/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 Dec 14, 2023
@openshift-merge-bot openshift-merge-bot bot merged commit 209cf30 into openshift:master Dec 14, 2023
18 checks passed
@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Dec 14, 2023

@jcpowermac: Updated the image-mirror-mappings configmap in namespace ci at cluster app.ci using the following files:

  • key mapping_origin_4_16 using file core-services/image-mirroring/openshift/mapping_origin_4_16

In response to this:

This reverts commit 8037af5.

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.

@Miciah Miciah mentioned this pull request Dec 19, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmguzik jmguzik jmguzik approved these changes

@bertinatto bertinatto Awaiting requested review from bertinatto

@damdo damdo Awaiting requested review from damdo

Assignees

@JoelSpeed JoelSpeed

@jmguzik jmguzik

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. rehearsals-ack Signifies that rehearsal jobs have been acknowledged
Projects
None yet
Milestone
No milestone
5 participants
@jcpowermac @JoelSpeed @jmguzik @stbenjam @openshift-ci-robot