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

configs: Build Mageia bootstrap chroots with target arch #1305

Closed

Conversation

wally-mageia
Copy link
Contributor

@wally-mageia wally-mageia commented Feb 4, 2024

Fixes #1304, cross-arch Mageia builds.

@Conan-Kudo
Copy link
Member

Can you amend your commit message so that the bug reference is on its own line?

For example:

configs: Build Mageia bootstrap chroots with target arch

Fixes: https://github.com/rpm-software-management/mock/issues/1304

@Conan-Kudo Conan-Kudo self-assigned this Feb 4, 2024
@wally-mageia
Copy link
Contributor Author

Looks like I messed something by adding another fix into the same branch. I'm not so familiar with github workflow. :(

@wally-mageia wally-mageia force-pushed the mageia-configs branch 2 times, most recently from f85dcd5 to f6d087c Compare February 4, 2024 10:09
@praiskup
Copy link
Member

praiskup commented Feb 5, 2024

@wally-mageia would you mind rebasing against the latest HEAD?

@@ -63,9 +64,9 @@ skip_if_unavailable=False

[updates-debuginfo]
name=Mageia $releasever - {{ target_arch }} - Updates - Debug
#baseurl=http://mirrors.kernel.org/mageia/distrib/$releasever/{{ target_arch }}/media/debug/core/updates/
baseurl=https://distrib-coffee.ipsl.jussieu.fr/pub/linux/Mageia-archive/distrib/$releasever/{{ target_arch }}/media/debug/core/updates/
Copy link
Member

Choose a reason for hiding this comment

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

This deserves inline comment :) using *.fr baseurl instead the mirror list.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should be fixed with 807eb40.

@wally-mageia wally-mageia force-pushed the mageia-configs branch 2 times, most recently from 2651255 to e6dff0a Compare February 5, 2024 16:30
@praiskup
Copy link
Member

praiskup commented Feb 6, 2024

@wally-mageia do you have ETA for fixing this PR? It would be nice to have it in #1307.

@@ -30,9 +31,10 @@ user_agent={{ user_agent }}

[mageia]
name=Mageia $releasever - {{ target_arch }}
#baseurl=http://mirrors.kernel.org/mageia/distrib/$releasever/{{ target_arch }}/media/core/release/
# Mirrorlist doesn't work ATM as old Mageia repos are only available in distrib-coffee's Mageia-archive
baseurl=https://distrib-coffee.ipsl.jussieu.fr/pub/linux/Mageia-archive/distrib/$releasever/{{ target_arch }}/media/core/release/
Copy link
Member

Choose a reason for hiding this comment

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

They at least appear to be working :) what am I missing?

curl 'https://www.mageia.org/mirrorlist/?release=9&arch=x86_64&section=core&repo=release'
ftp://ftp.fi.muni.cz/pub/linux/mageia/distrib/9/x86_64/media/core/release/
ftp://vodka.sublink.org/mageia/distrib/9/x86_64/media/core/release/
http://ftp.cc.uoc.gr/pub/linux/mageia/distrib/9/x86_64/media/core/release/
http://fr2.rpmfind.net/linux/mageia/distrib/9/x86_64/media/core/release/
ftp://mirror.datacenter.by/pub/mirrors/mageia.org/distrib/9/x86_64/media/core/release/
ftp://mirrors.netix.net/mageia/distrib/9/x86_64/media/core/release/
http://quantum-mirror.hu/mirrors/pub/mageia/distrib/9/x86_64/media/core/release/
http://ftp.klid.dk/ftp/mageia/distrib/9/x86_64/media/core/release/
ftp://mirror2.tuxinator.org/mageia/distrib/9/x86_64/media/core/release/
ftp://ftp.snt.utwente.nl/pub/os/linux/mageia/distrib/9/x86_64/media/core/release/

Copy link
Member

Choose a reason for hiding this comment

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

Note that using a single mirror in default mock-core-configs is a bad idea; it will both cause DDoS on the mirror and make the builds slow for far distances.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ATM mirrorlist doesn't handle correctly archived Mageia releases. Mageia 1-7 releases are only available in this one certain mirror in France. It's not the best option, but the only working one if one wants to build pkgs for old releases.

From mirror.readme available in the root of the mirror tree.

The expected size of the mirror is around 1.8-2TB.
The mirror only contains the last 2 stable releases + the development
branch called cauldron.

For older unsupported releases, you can find them at the archive
provided by distrib-coffee mirror:
https://distrib-coffee.ipsl.jussieu.fr/pub/linux/Mageia-archive/

Copy link
Member

Choose a reason for hiding this comment

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

Then we should have the mirrorlist service return just that one URL for archived releases.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ATM mirrorlist doesn't handle correctly archived Mageia releases.

curl 'https://www.mageia.org/mirrorlist/?release=7&arch=x86_64&section=core&repo=release'
returns only non-working entries.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've now fixed the mirrorlist service in Mageia side to only return distrib-coffee's Mageia-archive URL for Mageia 7. I have also rebased to not include Mageia 7 EOL commits. I'll create a new pull request for it.

@@ -4,6 +4,7 @@ config_opts['dist'] = 'mga{{ releasever }}' # only useful for --resultdir varia
config_opts['extra_chroot_dirs'] = [ '/run/lock', ]
config_opts['macros']['%distro_section'] = 'core'
config_opts['package_manager'] = 'dnf'
config_opts['bootstrap_forcearch'] = '{{ target_arch }}'
Copy link
Member

Choose a reason for hiding this comment

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

I'm almost sure this is not correct :-( we need a precise reason why this is needed and document it.

Copy link
Member

Choose a reason for hiding this comment

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

Can you check #1317 whether it helps?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I did tests earlier with no success. See #1304 (comment).

Copy link
Member

Choose a reason for hiding this comment

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

Ah, I see your comment in the original issue. Lemme think about it a bit more.

Copy link
Member

Choose a reason for hiding this comment

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

Can you please take one more look at #1317?

@praiskup
Copy link
Member

I believe this is resolved by #1317 - but thank you very much for driving this effort.

@praiskup praiskup closed this Feb 12, 2024
@wally-mageia wally-mageia deleted the mageia-configs branch April 12, 2024 19:15
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.

Mageia 8, 9 and cauldron configuration files for aarch64 and armv7hl need to be updated
3 participants