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.

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

Use ex rebuild to pull from COPR #447

Merged
merged 1 commit into from
Sep 24, 2022

Conversation

cgwalters
Copy link
Member

This works around coreos/rpm-ostree#4037

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Sep 23, 2022
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Sep 23, 2022
@cgwalters
Copy link
Member Author

OK I rebased 🏄 this on master

@vrutkovs
Copy link
Member

/lgtm
/retest

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Sep 24, 2022
@vrutkovs
Copy link
Member

/approve

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 24, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cgwalters, vrutkovs

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 Sep 24, 2022
@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD aebad56 and 2 for PR HEAD 4c03e8e in total

@cgwalters
Copy link
Member Author

I think upgrades will be broken until this merges
/override e2e-gcp-upgrade

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 24, 2022

@cgwalters: cgwalters 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:

I think upgrades will be broken until this merges
/override e2e-gcp-upgrade

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.

@vrutkovs
Copy link
Member

/override ci/prow/e2e-gcp-upgrade

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 24, 2022

@vrutkovs: Overrode contexts on behalf of vrutkovs: ci/prow/e2e-gcp-upgrade

In response to this:

/override ci/prow/e2e-gcp-upgrade

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.

@LorbusChris
Copy link
Member

What's the reason for removing ostree container commit. Is it not needed?

@cgwalters
Copy link
Member Author

What's the reason for removing ostree container commit. Is it not needed?

It's a bug, introduced by ostreedev/ostree-rs-ext#367 and fixed by ostreedev/ostree-rs-ext#377 and waiting on someone to review ostreedev/ostree-rs-ext#379 so I can bump rpm-ostree, which will get the fixed version into the COPR.

@vrutkovs
Copy link
Member

/test e2e-aws-ovn

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 24, 2022

@cgwalters: 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.

@openshift-merge-robot openshift-merge-robot merged commit 2ae7c63 into openshift:master Sep 24, 2022
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. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants