Skip to content

Add Kylin 10, Azure Linux 2 and 3 configuration files#1536

Merged
Conan-Kudo merged 3 commits intorpm-software-management:mainfrom
scaronni:main
Feb 7, 2025
Merged

Add Kylin 10, Azure Linux 2 and 3 configuration files#1536
Conan-Kudo merged 3 commits intorpm-software-management:mainfrom
scaronni:main

Conversation

@scaronni
Copy link
Contributor

@scaronni scaronni commented Feb 6, 2025

As mentioned in the openSUSE 15.6 merge request, we are moving everything to mock for building packages in NVIDIA. These are more configurations for additional distributions I had the "pleasure" to work on for supporting NVIDIA drivers + CUDA:

  • Kylin 10
    • No OCI image used, as there's no official image available.
  • Azure Linux 2
    • Microsoft decided to rename it from CBL Mariner 2.0 (following Common Base Linux 1.0) mid lifecycle, so the repositories are called mariner and there's cm2 as a dist tag.
    • No OCI image used, it uses "tiny dandified dnf" as the package manager.
  • Azure Linux 3
    • No OCI image used, it uses "tiny dandified dnf" as the package manager.

@xsuchy
Copy link
Member

xsuchy commented Feb 6, 2025

LGTM
Are you willing to add yourself to https://rpm-software-management.github.io/mock/Mock-Core-Configs as a maintainer for these new configs?

@scaronni
Copy link
Contributor Author

scaronni commented Feb 6, 2025

LGTM Are you willing to add yourself to https://rpm-software-management.github.io/mock/Mock-Core-Configs as a maintainer for these new configs?

Yes, sure. Also for openSUSE Leap (previous merge request), @Conan-Kudo said it was fine. Thanks.

@scaronni
Copy link
Contributor Author

scaronni commented Feb 6, 2025

@Conan-Kudo @xsuchy i rebased every commit, including the dependency on distribution-gpg-keys and the use of the local GPG key for Kylin.

@scaronni
Copy link
Contributor Author

scaronni commented Feb 6, 2025

Just tested changes for Kylin, they work fine with distribution-gpg-keys-1.109 .

@scaronni
Copy link
Contributor Author

scaronni commented Feb 6, 2025

@xsuchy shall I add myself to https://github.com/rpm-software-management/mock/blob/main/docs/Mock-Core-Configs.md in this same merge request?

@Conan-Kudo
Copy link
Member

Put it per each commit for each distro you're adding.

@xsuchy
Copy link
Member

xsuchy commented Feb 6, 2025

... and in this PR.

Thank you.

@Conan-Kudo
Copy link
Member

You can see an example of this in #1533.

@scaronni
Copy link
Contributor Author

scaronni commented Feb 6, 2025

Done. Added the Azure Linux maintainer line in the Azure Linux 2 commit.

@praiskup
Copy link
Member

praiskup commented Feb 7, 2025

Thank you for the contribution! Awesome to meet all those various RPM distros here on Mock, welcome! ❤️

LGTM, just let's wait for the CI to see if the configuration works.

@praiskup
Copy link
Member

praiskup commented Feb 7, 2025

The TF run has not finished yet, azure-linux-2 seems to fail:

07:24:02                 out: FAILED: azure-linux-2-x86_64.cfg (tmpfs)
07:24:02                 out: FAILED: azure-linux-2-x86_64.cfg
07:24:02                 out: PASSED: azure-linux-3-x86_64.cfg (tmpfs)
07:24:02                 out: PASSED: azure-linux-3-x86_64.cfg

@praiskup
Copy link
Member

praiskup commented Feb 7, 2025

@scaronni
Copy link
Contributor Author

scaronni commented Feb 7, 2025

The TF run has not finished yet, azure-linux-2 seems to fail:

07:24:02                 out: FAILED: azure-linux-2-x86_64.cfg (tmpfs)
07:24:02                 out: FAILED: azure-linux-2-x86_64.cfg
07:24:02                 out: PASSED: azure-linux-3-x86_64.cfg (tmpfs)
07:24:02                 out: PASSED: azure-linux-3-x86_64.cfg

Fixed, when switching to local keyes for the Azure Linux 2 repos, the metadata key was missing. On Azure Linux 3 there is only one key.

Copy link
Member

@Conan-Kudo Conan-Kudo left a comment

Choose a reason for hiding this comment

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

Everything looks good now!

@Conan-Kudo Conan-Kudo merged commit 025084b into rpm-software-management:main Feb 7, 2025
24 checks passed
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