Skip to content

Rebase to RHEL 8.6#815

Merged
cgwalters merged 6 commits intoopenshift:masterfrom
travier:rhel86
Jun 6, 2022
Merged

Rebase to RHEL 8.6#815
cgwalters merged 6 commits intoopenshift:masterfrom
travier:rhel86

Conversation

@travier
Copy link
Copy Markdown
Member

@travier travier commented Jun 3, 2022

@openshift-ci openshift-ci Bot requested review from ashcrow and saqibali-2k June 3, 2022 16:02
@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 3, 2022
@miabbott
Copy link
Copy Markdown
Member

miabbott commented Jun 3, 2022

May want to pull in #800 too

travier and others added 3 commits June 3, 2022 18:03
Manual revert of the last revert to RHEL 8.5 to keep other changes in
place.
The WALinuxAgent-udev package exists now.

This reverts commit 5cd6441.
The directory is no longer needed because the 25rhcos-azure-udev-rules
overlay got removed in the previous commit.
dustymabe and others added 2 commits June 3, 2022 18:35
This migrates the glue code that installs the udev rules into the
initramfs to an overlay module in the RHCOS repo. It was no longer
needed in FCOS and removed from the overlay there in [1].

[1] coreos/fedora-coreos-config#1686
We've worked with the Containers team to enable builds of certain
container-tools RPMs from the RHAOS repo. This allows us to land OCP
specific changes to these packages and potentially ship newer versions
of these packages than what would available in the module.
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 3, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cgwalters, saqibali-2k, 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 [cgwalters,saqibali-2k,travier]

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

@HuijingHei
Copy link
Copy Markdown
Contributor

/retest

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 6, 2022

@travier: 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/test-in-cluster d4ce1ac link true /test test-in-cluster
ci/prow/test-qemu-kola d4ce1ac link true /test test-qemu-kola

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.

@cgwalters
Copy link
Copy Markdown
Member

cgwalters commented Jun 6, 2022

EDIT: I think the real problem is I was using coreos/rpm-ostree#3727 which is broken

@cgwalters
Copy link
Copy Markdown
Member

OK, we're still hitting the default 4 hour cap on CI runs - the build is still ~3h, and ~1h is not enough to get through all of the kola qemu runs.

Bigger picture, we are going to need to get away from "run all tests on all PRs" - we need to split the kola tests into tiers; xref coreos/coreos-assembler#2237 (comment)

I run kola locally using this PR and all passed. So let's get this in and continue to iterate on CI.

@cgwalters cgwalters merged commit 939d8c8 into openshift:master Jun 6, 2022
@cgwalters
Copy link
Copy Markdown
Member

/cherrypick release-4.11

@openshift-cherrypick-robot
Copy link
Copy Markdown

@cgwalters: new pull request created: #817

Details

In response to this:

/cherrypick release-4.11

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.

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants