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

Replace download.suse.de with mirror.nue2.suse.org #184

Closed
wants to merge 1 commit into from

Conversation

dzedro
Copy link

@dzedro dzedro commented Oct 30, 2024

@Martchus
Copy link
Contributor

Shouldn't this be configurable? I suppose we need to be able to deal with tests using embargoed data at some point. So we cannot completely avoid using download.suse.de.

Copy link
Member

@okurz okurz left a comment

Choose a reason for hiding this comment

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

We must not just use a different place. This would mean to rely on a mirror running not in prg2 which is both more remote and less reliable. This would deny access to embargoed updates which still need to be tested. Also this would mean that we trigger tests before the required assets are even synced.

@dzedro
Copy link
Author

dzedro commented Oct 31, 2024

Git it, it will be more complicated and done within. https://progress.opensuse.org/issues/169129

@dzedro dzedro closed this Oct 31, 2024
@dzedro dzedro deleted the poo168097 branch October 31, 2024 09:58
@okurz
Copy link
Member

okurz commented Nov 1, 2024

No, this won't be done within https://progress.opensuse.org/issues/169129 which is only about time synchronisation to prevent starting a test before assets are available on a mirror. What should be done to use a local mirror explicitly is to look into the worker location and then select an according mirror. Alternatively we can set a mirror variable in each worker and read that with fallback to the default

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.

3 participants