Bug 2103053: ci: Rework to merge all scripts into one by travier · Pull Request #869 · openshift/os

travier · 2022-06-29T10:21:18Z

Manual backports of changes from master:

8efbd98 ci: Merge all Prow CI "build" scripts into one
787d5da ci: Do not fail on rpm-ostree missing
5e9b2b2 ci: Regroup tests in qemu & metal buckets
eb3a605 ci: Prepare and clean-up test names
0a70681 ci: Remove now unused scripts
3d4cf57 ci: Remove Dockerfiles (moved to release)
a0e0483 ci/prow-entrypoint: No-op cosa init if already initialized
8cbb015 ci: Add an explicitly named test entry for COSA CI
a193baf ci: Remove now unused test names

openshift-ci · 2022-06-29T10:21:22Z

@travier: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh.

Details

In response to this:

ci: Rework to merge all scripts into one

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.

travier · 2022-06-29T10:23:53Z

Goes with openshift/release#29949

miabbott · 2022-06-30T12:51:52Z

Looks like a chicken + egg problem here, so let's merge this first then openshift/release#29949

/lgtm

miabbott · 2022-06-30T12:52:11Z

/override ci/prow/build-test-qemu

travier · 2022-06-30T12:53:36Z

/hold

travier · 2022-06-30T12:53:50Z

Let's try a retest first
/retest

travier · 2022-06-30T12:55:53Z

Sorry @miabbott, I was looking at other PRs. We should be able to merge this one over green CI.

openshift-ci · 2022-06-30T13:05:49Z

@miabbott: Overrode contexts on behalf of miabbott: ci/prow/build-test-qemu

Details

In response to this:

/override ci/prow/build-test-qemu

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.

Manual backports of changes from master: - 8efbd98 ci: Merge all Prow CI "build" scripts into one - 787d5da ci: Do not fail on rpm-ostree missing - 5e9b2b2 ci: Regroup tests in qemu & metal buckets - eb3a605 ci: Prepare and clean-up test names - 0a70681 ci: Remove now unused scripts - 3d4cf57 ci: Remove Dockerfiles (moved to release) - a0e0483 ci/prow-entrypoint: No-op `cosa init` if already initialized - 8cbb015 ci: Add an explicitly named test entry for COSA CI - a193baf ci: Remove now unused test names We can then remove Dockerfile, build-test-qemu.sh & validate.sh once the release changes are merged.

openshift-ci · 2022-06-30T16:01:29Z

@travier: all tests passed!

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.

HuijingHei · 2022-07-01T02:50:13Z

/LGTM

travier · 2022-07-01T10:10:47Z

/unhold

openshift-ci · 2022-07-01T10:37:27Z

@travier: This pull request references Bugzilla bug 2103053, which is invalid:

expected Bugzilla bug 2103053 to depend on a bug targeting a release in 4.12.0 and in one of the following states: MODIFIED, ON_QA, VERIFIED, but no dependents were found

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

Details

In response to this:

Bug 2103053: ci: Rework to merge all scripts into one

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.

travier · 2022-07-01T10:43:31Z

/bugzilla refresh

openshift-ci · 2022-07-01T10:43:34Z

@travier: This pull request references Bugzilla bug 2103053, which is invalid:

expected Bugzilla bug 2103053 to depend on a bug targeting a release in 4.12.0 and in one of the following states: MODIFIED, ON_QA, VERIFIED, but no dependents were found

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

Details

In response to this:

/bugzilla refresh

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.

travier · 2022-07-01T10:50:06Z

/bugzilla refresh

openshift-ci · 2022-07-01T10:50:16Z

@travier: This pull request references Bugzilla bug 2103053, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker.

6 validation(s) were run on this bug

bug is open, matching expected state (open)
bug target release (4.11.0) matches configured target release for branch (4.11.0)
bug is in the state ASSIGNED, which is one of the valid states (NEW, ASSIGNED, ON_DEV, POST, POST)
dependent bug Bugzilla bug 2103061 is in the state VERIFIED, which is one of the valid states (MODIFIED, ON_QA, VERIFIED)
dependent Bugzilla bug 2103061 targets the "4.12.0" release, which is one of the valid target releases: 4.12.0
bug has dependents

Requesting review from QA contact:
/cc @mike-nguyen

Details

In response to this:

/bugzilla refresh

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.

mike-nguyen · 2022-07-01T15:02:02Z

/lgtm

mike-nguyen · 2022-07-01T15:05:05Z

/approve

openshift-ci · 2022-07-01T15:10:17Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: HuijingHei, miabbott, mike-nguyen, travier

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:

~~OWNERS~~ [HuijingHei,miabbott,mike-nguyen,travier]

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

miabbott · 2022-07-05T14:38:56Z

@aravindhp looks like you are the staff engineer for the CoreOS team; could you provide the staff-eng-approved label?

We got ourselves in a mess with the release-4.11 branch here because the repo wasn't properly enrolled in the Prow branch management.

This is just trying to get the CI jobs on release-4.11 branch back to something functional and reliable. It does not change any code that is going into RHCOS 4.11 or any code already part of RHCOS 4.11.

openshift-ci · 2022-07-05T17:09:36Z

@travier: All pull requests linked via external trackers have merged:

openshift/os#869

Bugzilla bug 2103053 has been moved to the MODIFIED state.

Details

In response to this:

Bug 2103053: ci: Rework to merge all scripts into one

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 Bot requested review from dustymabe and ravanelli June 29, 2022 10:22

openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 29, 2022

travier mentioned this pull request Jun 29, 2022

ci-operator/openshift/os: Backport CI fixes to 4.11 openshift/release#29949

Merged

travier force-pushed the 4.11-ci-backport branch from 1f03a01 to 7371fbb Compare June 29, 2022 10:33

openshift-ci Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jun 30, 2022

openshift-ci Bot assigned miabbott Jun 30, 2022

openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jun 30, 2022

travier force-pushed the 4.11-ci-backport branch from 7371fbb to b129284 Compare June 30, 2022 13:41

openshift-ci Bot removed the lgtm Indicates that a PR is ready to be merged. label Jun 30, 2022

openshift-ci Bot assigned HuijingHei Jul 1, 2022

openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jul 1, 2022

openshift-ci Bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 1, 2022

travier changed the title ~~ci: Rework to merge all scripts into one~~ Bug 2103053: ci: Rework to merge all scripts into one Jul 1, 2022

openshift-ci Bot added bugzilla/severity-low Referenced Bugzilla bug's severity is low for the branch this PR is targeting. bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. labels Jul 1, 2022

openshift-ci Bot added bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. and removed bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. labels Jul 1, 2022

openshift-ci Bot requested a review from mike-nguyen July 1, 2022 10:50

mike-nguyen approved these changes Jul 1, 2022

View reviewed changes

mrunalp added the staff-eng-approved Indicates a release branch PR has been approved by a staff engineer (formerly group/pillar lead). label Jul 5, 2022

openshift-ci Bot merged commit 208aa42 into openshift:release-4.11 Jul 5, 2022

travier deleted the 4.11-ci-backport branch July 5, 2022 17:14

Conversation

travier commented Jun 29, 2022

Uh oh!

openshift-ci Bot commented Jun 29, 2022

Uh oh!

travier commented Jun 29, 2022

Uh oh!

miabbott commented Jun 30, 2022

Uh oh!

miabbott commented Jun 30, 2022

Uh oh!

travier commented Jun 30, 2022

Uh oh!

travier commented Jun 30, 2022

Uh oh!

travier commented Jun 30, 2022

Uh oh!

openshift-ci Bot commented Jun 30, 2022

Uh oh!

openshift-ci Bot commented Jun 30, 2022

Uh oh!

HuijingHei commented Jul 1, 2022

Uh oh!

travier commented Jul 1, 2022

Uh oh!

openshift-ci Bot commented Jul 1, 2022

Uh oh!

travier commented Jul 1, 2022

Uh oh!

openshift-ci Bot commented Jul 1, 2022

Uh oh!

travier commented Jul 1, 2022

Uh oh!

openshift-ci Bot commented Jul 1, 2022

Uh oh!

mike-nguyen commented Jul 1, 2022

Uh oh!

mike-nguyen commented Jul 1, 2022

Uh oh!

openshift-ci Bot commented Jul 1, 2022

Uh oh!

miabbott commented Jul 5, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

openshift-ci Bot commented Jul 5, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

miabbott commented Jul 5, 2022 •

edited

Loading