Skip to content

Add AlmaLinux 10 configs#1586

Merged
Conan-Kudo merged 1 commit intorpm-software-management:mainfrom
jonathanspw:almalinux10
Jun 6, 2025
Merged

Add AlmaLinux 10 configs#1586
Conan-Kudo merged 1 commit intorpm-software-management:mainfrom
jonathanspw:almalinux10

Conversation

@jonathanspw
Copy link
Contributor

No description provided.

@jonathanspw jonathanspw marked this pull request as draft May 27, 2025 19:19
@jonathanspw jonathanspw marked this pull request as ready for review May 27, 2025 20:18
@scaronni
Copy link
Contributor

Also, running:

$ mock -r alma+epel-10-x86_64_v2 --init

Fails with:

No match for argument: epel-rpm-macros
Error: Unable to find a match: epel-rpm-macros

I guess it's because it's not pulling the x86_64_v2 image from dockerhub (is it even available?). Maybe for that use case would be beneficial to drop the usage of the bootstrap image.

@jonathanspw
Copy link
Contributor Author

Also, running:

$ mock -r alma+epel-10-x86_64_v2 --init

Fails with:

No match for argument: epel-rpm-macros
Error: Unable to find a match: epel-rpm-macros

I guess it's because it's not pulling the x86_64_v2 image from dockerhub (is it even available?). Maybe for that use case would be beneficial to drop the usage of the bootstrap image.

epel-rpm-macros comes from the EPEL v2 repo for 10.0, which is currently still empty. The mass rebuild for this is ongoing since 10.0 just hit GA yesterday.

The v2 arch is set to not use docker/quay base images because there's not a way (that I'm aware of) in mock to properly point it at the x86_64_v2 variant so on v3 hosts it would just pull the x86_64 images (v3) which would produce unexpected results (ie, not proper v2 builds).

@jonathanspw
Copy link
Contributor Author

The CI failure looks unrelated to this PR.

@Conan-Kudo
Copy link
Member

A new release of distribution-gpg-keys needs to be made and mock-core-configs needs to be bumped to require it.

plaimbock added a commit to plaimbock/mock that referenced this pull request May 31, 2025
@Conan-Kudo
Copy link
Member

Conan-Kudo commented Jun 2, 2025

Can you squash this into one commit? Then the only thing missing is @xsuchy making a distribution-gpg-keys release and shipping it to Fedora so the dependency in the mock-core-configs spec can be bumped.

@jonathanspw
Copy link
Contributor Author

Can you squash this into one commit? Then the only thing missing is @xsuchy making a distribution-gpg-keys release and shipping it to Fedora so the dependency in the mock-core-configs spec can be bumped.

I figured it would get squashed on merge, but I can.

@Conan-Kudo
Copy link
Member

We do rebase merging in this project exclusively, so you need to do your own squashing.

@jonathanspw
Copy link
Contributor Author

We do rebase merging in this project exclusively, so you need to do your own squashing.

All squashed.

@praiskup
Copy link
Member

praiskup commented Jun 4, 2025

Testing farm is failing for x86_64_v2, can you double-check this is OK?

...
PASSED: alma+epel-10-x86_64.cfg (tmpfs)
PASSED: alma+epel-10-x86_64.cfg
FAILED: alma+epel-10-x86_64_v2.cfg (tmpfs)
FAILED: alma+epel-10-x86_64_v2.cfg
...
PASSED: alma-kitten+epel-10-x86_64.cfg (tmpfs)
PASSED: alma-kitten+epel-10-x86_64.cfg
PASSED: almalinux-10-x86_64.cfg (tmpfs)
PASSED: almalinux-10-x86_64.cfg
PASSED: almalinux-10-x86_64_v2.cfg (tmpfs)
PASSED: almalinux-10-x86_64_v2.cfg
...

@praiskup
Copy link
Member

praiskup commented Jun 5, 2025

We need to let this one finish, and then re-trigger CI:
https://copr.fedorainfracloud.org/coprs/build/9137018

@praiskup
Copy link
Member

praiskup commented Jun 5, 2025

/packit build

1 similar comment
@Conan-Kudo
Copy link
Member

/packit build

@praiskup
Copy link
Member

praiskup commented Jun 5, 2025

Argh, sorry! I meant to submit the bulid here: https://copr.fedorainfracloud.org/coprs/build/9137091
And plus we need #1590 or ignore docker test for now.

@praiskup
Copy link
Member

praiskup commented Jun 5, 2025

/packit build

@praiskup
Copy link
Member

praiskup commented Jun 6, 2025

Would you mind rebasing one last time, against current main?

@Conan-Kudo Conan-Kudo merged commit a2720d3 into rpm-software-management:main Jun 6, 2025
23 checks passed
@jonathanspw jonathanspw deleted the almalinux10 branch June 6, 2025 20:38
praiskup added a commit to praiskup/mock that referenced this pull request Jun 17, 2025
The rpm-software-management#1586 introduced broken config files:
$ mock -r alma-kitten+epel-10-x86_64_v2 --scrub=all
ERROR: Could not find included config file: /etc/mock/almalinux-kitten-10-x86_64_v2.cfg

It doesn't make sense to skip the testing in this situation.
praiskup added a commit that referenced this pull request Jun 17, 2025
The #1586 introduced broken config files:
$ mock -r alma-kitten+epel-10-x86_64_v2 --scrub=all
ERROR: Could not find included config file: /etc/mock/almalinux-kitten-10-x86_64_v2.cfg

It doesn't make sense to skip the testing in this situation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants