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

NO-JIRA: Bump fedora-coreos-config #1644

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

coreosbot-releng
Copy link

@coreosbot-releng coreosbot-releng commented Oct 29, 2024

Created by GitHub workflow (source).

Adam Piasecki (1):
      denylist: add multipath.partition

Dusty Mabe (7):
      tests/kola: add SELinux systemd /var/cache exceptions in upgrade test
      Move to Fedora Linux 41
      denylist: drop kdump.crash.nfs and ext.config.boot.bootupd-validate
      overrides: drop graduated grub2-2.12-10.fc41 override
      07fix-selinux-labels: fix service ordering
      tests/kola: fix potential unbound variable error
      ci: use OSbuild to build metal/metal4k in one shot

HuijingHei (5):
      test: extend bootupd test to include testing adoption and updates
      denylist: deny `ext.config.boot.bootupd-validate` until we rollout F41
      tests/kola: remove dtb exceptions in upgrade test SELinux checks
      tests/upgrade: drop the rollback deployment on aarch64
      tests/bootupd-validate: skip checking BIOS on older bootupd

Jonathan Lebon (17):
      workflows/add-overrides: install python3-dnf
      Inherit from fedora-bootc's tier-x on Fedora 42+
      manifests: conditionalize duplicate manifests
      manifests: split system-configuration.yaml
      manifests: split networking-tools.yaml
      manifests: split user-experience.yaml
      manifests: hoist up bootable-rpm-ostree.yaml include
      manifests: move veritysetup request to ignition-and-ostree.yaml
      manifests: move bootc to bootable-rpm-ostree.yaml
      manifests: conditionalize bootable-rpm-ostree.yaml
      manifests: just disable systemd-firstboot, don't nuke
      manifests: split ignition-and-ostree.yaml
      manifests: split tier-x dupes into separate manifest
      workflows: add `bump-fedora-bootc` workflow
      manifests/tier-x: use readonly in repo config only
      workflows/remove-graduated-overrides: clone with submodule
      manifests/fedora-coreos: move from `rojig` to `metadata`

Joseph Marrero Corchado (1):
      manifest: Add bootc

Michael Armijo (1):
      denylist: remove fcos.internet and podman.workflow

Nikita Dubrovskii (3):
      dnf5: use 'setopt' to enable 'updates-testing' repo
      dnf5 builddep: drop '--spec' as it no longer exists
      coreos-secex-ignition-prepare: remount /usr rw if needed

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Oct 29, 2024
@openshift-ci-robot
Copy link

@coreosbot-releng: This pull request explicitly references no jira issue.

In response to this:

Created by GitHub workflow (source).

Dusty Mabe (1):
     tests/kola: add SELinux systemd /var/cache exceptions in upgrade test

HuijingHei (2):
     test: extend bootupd test to include testing adoption and updates
     denylist: deny `ext.config.boot.bootupd-validate` until we rollout F41

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 openshift-eng/jira-lifecycle-plugin repository.

Copy link
Member

@ravanelli ravanelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci bot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Nov 1, 2024
@ravanelli
Copy link
Member

/retest

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 2822e38 and 2 for PR HEAD f8ac2a5 in total

@openshift-ci-robot
Copy link

@coreosbot-releng: This pull request explicitly references no jira issue.

In response to this:

Created by GitHub workflow (source).

Dusty Mabe (5):
     tests/kola: add SELinux systemd /var/cache exceptions in upgrade test
     Move to Fedora Linux 41
     denylist: drop kdump.crash.nfs and ext.config.boot.bootupd-validate
     overrides: drop graduated grub2-2.12-10.fc41 override
     07fix-selinux-labels: fix service ordering

HuijingHei (3):
     test: extend bootupd test to include testing adoption and updates
     denylist: deny `ext.config.boot.bootupd-validate` until we rollout F41
     tests/kola: remove dtb exceptions in upgrade test SELinux checks

Nikita Dubrovskii (2):
     dnf5: use 'setopt' to enable 'updates-testing' repo
     dnf5 builddep: drop '--spec' as it no longer exists

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Nov 2, 2024
@c4rt0
Copy link
Contributor

c4rt0 commented Nov 2, 2024

/retest

@c4rt0
Copy link
Contributor

c4rt0 commented Nov 2, 2024

/lgtm

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

openshift-ci bot commented Nov 2, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: c4rt0, coreosbot-releng, ravanelli

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-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 2822e38 and 2 for PR HEAD 1121db2 in total

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

@coreosbot-releng: This pull request explicitly references no jira issue.

In response to this:

Created by GitHub workflow (source).

Dusty Mabe (6):
     tests/kola: add SELinux systemd /var/cache exceptions in upgrade test
     Move to Fedora Linux 41
     denylist: drop kdump.crash.nfs and ext.config.boot.bootupd-validate
     overrides: drop graduated grub2-2.12-10.fc41 override
     07fix-selinux-labels: fix service ordering
     tests/kola: fix potential unbound variable error

HuijingHei (3):
     test: extend bootupd test to include testing adoption and updates
     denylist: deny `ext.config.boot.bootupd-validate` until we rollout F41
     tests/kola: remove dtb exceptions in upgrade test SELinux checks

Nikita Dubrovskii (2):
     dnf5: use 'setopt' to enable 'updates-testing' repo
     dnf5 builddep: drop '--spec' as it no longer exists

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 openshift-eng/jira-lifecycle-plugin repository.

Copy link
Contributor

openshift-ci bot commented Nov 3, 2024

New changes are detected. LGTM label has been removed.

@c4rt0
Copy link
Contributor

c4rt0 commented Nov 3, 2024

/retest-required

@c4rt0
Copy link
Contributor

c4rt0 commented Nov 3, 2024

/retest

1 similar comment
@c4rt0
Copy link
Contributor

c4rt0 commented Nov 3, 2024

/retest

@travier
Copy link
Member

travier commented Nov 19, 2024

Looks like RHCOS builds are failing on this right now:

Executing `postprocess` inline script '8'
+ /usr/bin/bootupctl backend generate-update-metadata
Generated update layout for BIOS: grub2-tools-1:2.06-86.el9_4.1.x86_64
rpm: no arguments given for query
error: generating metadata failed: Failed to invoke rpm -qf
error: Installing packages: bwrap(/usr/bin/rpmostree-postprocess-inline-8): Child process killed by signal 1
failed to execute cmd-build: exit status 1

@jlebon
Copy link
Member

jlebon commented Nov 19, 2024

Yeah, I'll add patches to this PR to adapt it for tier-x.

@jlebon
Copy link
Member

jlebon commented Nov 19, 2024

Pushed an update, but actually though as per coreos/fedora-coreos-config#3177 (comment), I think we need to wait until branching to land this. Is there anything we actually need in 4.18 from this PR?

Hmm OK, "07fix-selinux-labels: fix service ordering" would be good at least to squash the associated systemd warning. So basically:

@jlebon
Copy link
Member

jlebon commented Nov 28, 2024

Moved this to #1658 and added a commit there.

Adam Piasecki (1):
      denylist: add multipath.partition

Dusty Mabe (7):
      tests/kola: add SELinux systemd /var/cache exceptions in upgrade test
      Move to Fedora Linux 41
      denylist: drop kdump.crash.nfs and ext.config.boot.bootupd-validate
      overrides: drop graduated grub2-2.12-10.fc41 override
      07fix-selinux-labels: fix service ordering
      tests/kola: fix potential unbound variable error
      ci: use OSbuild to build metal/metal4k in one shot

HuijingHei (5):
      test: extend bootupd test to include testing adoption and updates
      denylist: deny `ext.config.boot.bootupd-validate` until we rollout F41
      tests/kola: remove dtb exceptions in upgrade test SELinux checks
      tests/upgrade: drop the rollback deployment on aarch64
      tests/bootupd-validate: skip checking BIOS on older bootupd

Jonathan Lebon (17):
      workflows/add-overrides: install python3-dnf
      Inherit from fedora-bootc's tier-x on Fedora 42+
      manifests: conditionalize duplicate manifests
      manifests: split system-configuration.yaml
      manifests: split networking-tools.yaml
      manifests: split user-experience.yaml
      manifests: hoist up bootable-rpm-ostree.yaml include
      manifests: move veritysetup request to ignition-and-ostree.yaml
      manifests: move bootc to bootable-rpm-ostree.yaml
      manifests: conditionalize bootable-rpm-ostree.yaml
      manifests: just disable systemd-firstboot, don't nuke
      manifests: split ignition-and-ostree.yaml
      manifests: split tier-x dupes into separate manifest
      workflows: add `bump-fedora-bootc` workflow
      manifests/tier-x: use readonly in repo config only
      workflows/remove-graduated-overrides: clone with submodule
      manifests/fedora-coreos: move from `rojig` to `metadata`

Joseph Marrero Corchado (1):
      manifest: Add bootc

Michael Armijo (1):
      denylist: remove fcos.internet and podman.workflow

Nikita Dubrovskii (3):
      dnf5: use 'setopt' to enable 'updates-testing' repo
      dnf5 builddep: drop '--spec' as it no longer exists
      coreos-secex-ignition-prepare: remount /usr rw if needed
Copy link
Contributor

openshift-ci bot commented Nov 29, 2024

@coreosbot-releng: 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/scos-9-build-test-metal 7609cee link true /test scos-9-build-test-metal
ci/prow/scos-9-build-test-qemu 7609cee link true /test scos-9-build-test-qemu
ci/prow/rhcos-9-build-test-metal 7609cee link true /test rhcos-9-build-test-metal
ci/prow/rhcos-9-build-test-qemu 7609cee link true /test rhcos-9-build-test-qemu

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.

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. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants