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

test: Add CentOS 9 image mode #1920

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

martinpitt
Copy link
Member

@martinpitt martinpitt commented Dec 3, 2024

See https://gitlab.com/testing-farm/general/-/blob/main/rfd/rfd5-testing-image-mode.adoc

https://issues.redhat.com/browse/COCKPIT-1213


This doesn't work locally:

tmt run --until report provision --how virtual --image https://artifacts.dev.testing-farm.io/images/Fedora-40-image-mode-x86_64.qcow2

fails with

       Command 'rpm -q --whatprovides cockpit-ws cockpit-podman cockpit-system nginx systemd-container || rpm-ostree install --apply-live --idempotent --allow-inactive  cockpit-ws cockpit-podman cockpit-system nginx systemd-container' returned 1.
[...]
        error: Could not depsolve transaction; 3 problems detected:
         Problem 1: package nginx-2:1.26.2-1.fc40.x86_64 from updates requires nginx-core = 2:1.26.2-1.fc40, but none of the providers can be installed
          - conflicting requests
          - nothing provides nginx-mimetypes needed by nginx-core-2:1.26.2-1.fc40.x86_64 from updates
         Problem 2: package cockpit-ws-329.1-1.fc40.x86_64 from updates requires (selinux-policy >= 40.29 if selinux-policy-targeted), but none of the providers can be installed
          - cannot install both selinux-policy-40.29-2.fc40.noarch from updates and selinux-policy-40.28-1.fc40.noarch from @System
          - conflicting requests
         Problem 3: package systemd-container-255.15-1.fc40.x86_64 from updates requires libsystemd-shared-255.15-1.fc40.so()(64bit), but none of the providers can be installed
          - package systemd-container-255.15-1.fc40.x86_64 from updates requires libsystemd-shared-255.15-1.fc40.so(SD_SHARED)(64bit), but none of the providers can be installed
          - package systemd-container-255.15-1.fc40.x86_64 from updates requires systemd(x86-64) = 255.15-1.fc40, but none of the providers can be installed
          - cannot install both systemd-255.15-1.fc40.x86_64 from updates and systemd-255.13-1.fc40.x86_64 from @System
          - conflicting requests
          - nothing provides libbz2.so.1 needed by systemd-container-255.15-1.fc40.i686 from updates
          - nothing provides liblzma.so.5 needed by systemd-container-255.15-1.fc40.i686 from updates
          - nothing provides liblzma.so.5(XZ_5.0) needed by systemd-container-255.15-1.fc40.i686 from updates
          - nothing provides libacl.so.1 needed by systemd-container-255.15-1.fc40.i686 from updates
          - nothing provides libacl.so.1(ACL_1.0) needed by systemd-container-255.15-1.fc40.i686 from updates

This feels like an rpm-ostree issue together with a dnf depsolver. Reported to https://issues.redhat.com/browse/TFT-3020 . Let's see how that works in packit/TF. Also, this isn't quite the correct way, we should really build a derived container and boot into that.

@martinpitt

This comment was marked as resolved.

@martinpitt

This comment was marked as resolved.

@lbarcziova

This comment was marked as resolved.

packit.yaml Outdated Show resolved Hide resolved
@martinpitt
Copy link
Member Author

This failed even earlier, in TF install packages step -- it really can't do that in bootc mode, this needs some thinking.

- centos-stream-9-aarch64
- centos-stream-10
centos-stream-9-x86_64:
distros: ["centos-stream-9", "CentOS-Stream-9-image-mode"]

Copy link

@thrix thrix Dec 4, 2024

Choose a reason for hiding this comment

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

Suggested change
tf_extra_params:
environments:
- artifacts: []

Copy link

@thrix thrix Dec 4, 2024

Choose a reason for hiding this comment

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

@martinpitt let's try this, this should disable the artifacts installation :)

You can consume then the copr build via Packit's environment variables it injects to the tests:

https://packit.dev/docs/configuration/upstream/tests#optional-parameters

Copy link
Member Author

Choose a reason for hiding this comment

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

@thrix Thanks for the hint! I'm a bit sceptical, as in cockpit that felt additive, not replacing. But pushed, let's try!

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link

Choose a reason for hiding this comment

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

No need to report that, we have other ways to try it out, let me open a new MR so i can experiment and help

Copy link

@thrix thrix Dec 5, 2024

Choose a reason for hiding this comment

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

continuing myself in #1924 for a prototype with all the needed workarounds

@martinpitt
Copy link
Member Author

Installed build deps programmatically, so a local tmt run actually gets pretty far ⭐ But it now runs straight into cockpit-project/cockpit#21201 , so we need cockpit 330 for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants