Skip to content

Fixes for Leap 16 #294

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

Closed
wants to merge 1 commit into from
Closed

Conversation

andrii-suse
Copy link
Collaborator

  • Add MIRROR variables to openqa call.
  • Remove SLE from REPO variable name

@foursixnine

- Add MIRROR variables to openqa call.
- Remove SLE from REPO variable name
@andrii-suse andrii-suse force-pushed the leap16_fix_mirror_and_repovar branch from 9d26fcf to 52d2274 Compare May 14, 2025 13:23
@lkocman
Copy link
Contributor

lkocman commented May 14, 2025

Wow @andrii-suse. Many thanks!

REPO_0=Leap-16.0-aarch64-ppc64le-s390x-x86_64-Build1.3 \
REPO_SLE_LEAP_16_0_AARCH64_PPC64LE_S390X_X86_64=Leap-16.0-aarch64-ppc64le-s390x-x86_64-Build1.3 \
REPO_LEAP_16_0_AARCH64_PPC64LE_S390X_X86_64=Leap-16.0-aarch64-ppc64le-s390x-x86_64-Build1.3 \
Copy link
Member

Choose a reason for hiding this comment

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

In this case, we might want to have a different name for this repo (or have it duplicated to... i.e LEAP_REPO) would that be possible? See where its used at:

https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/22054/files#diff-dd7d062b30fb82649a199111da8e89224600e38f86a24a23d251e0adc6747a89R22

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

so it is basically exactly how it is written in xml, which is exactly how it is configured in OBS. Do we really need another mapping from there to some custom name? Maybe rename it in OBS if it is too ugly?

Copy link
Member

Choose a reason for hiding this comment

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

Quoting from our conversation

we should aim to have REPO_OSS or maybe REPO_LEAP_OSS (because the folder is 000productcompose:leap_oss)

@andrii-suse
Copy link
Collaborator Author

Deprecated by changes in #295 , (still needs another PR for mirrors variables)

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