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

Update Debian CI environments #462

Closed
wants to merge 1 commit into from

Conversation

nmeyerhans
Copy link
Contributor

This PR refreshes the Debian environments used in the CI configuration, as they've been out of date with respect to actual Debian releases for some time.

Use static release names corresponding to specific releases, rather than moving targets. In other words, replace names like "stable" and "testing" with specific distros, "bullseye", "bookworm", etc.

Remove configuration that assumed that "oldstable" referred to Debian 9, as this hasn't been the case since the Debian 11 release in mid-2021 and Debian 9 is fully EOL at this point anyway.

Use static release names corresponding to specific releases, rather
than moving targets.

Remove configuration that assumed that "oldstable" referred to Debian
9, as this hasn't been the case since the Debian 11 release in
mid-2021 and Debian 9 is fully EOL at this point anyway.

Use debian:bullseye in place of debian:testing, which is the
soon-to-be-released Debian 12.

Signed-off-by: Noah Meyerhans <noahm@debian.org>
@nmeyerhans
Copy link
Contributor Author

Note that this will address the test failure seen in #459

@nmeyerhans nmeyerhans requested a review from pevik May 5, 2023 17:57
@pevik
Copy link
Contributor

pevik commented May 5, 2023

Well, I intended it to be moving target, so that I didn't have to keep updating it (I'd easily forget). Yes, sometimes Debian release requires adjustments, but very rarely. I'd actually prefer have way to have moving targets for other distros, but they have IMHO only "unstable" equivalent. Would you mind to amend your commit?

@pevik
Copy link
Contributor

pevik commented May 16, 2023

@nmeyerhans would you please update the PR to keep the generic names? Or shell I do it on your behalf? I need to fix CI for #465 fix.

pevik pushed a commit to pevik/iputils that referenced this pull request May 16, 2023
Remove configuration that assumed that "oldstable" referred to Debian 9,
as this hasn't been the case since the Debian 11 release in
mid-2021 and Debian 9 is fully EOL at this point anyway.

That fixes error in CI:
E: The repository 'http://http.debian.net/debian stretch-backports Release' does not have a Release file.

Closes: iputils#462
Reviewed-by: Petr Vorel <pvorel@suse.cz>
Signed-off-by: Noah Meyerhans <noahm@debian.org>
[ pvorel: keep generic distro names ]
Signed-off-by: Petr Vorel <pvorel@suse.cz>
@pevik
Copy link
Contributor

pevik commented May 16, 2023

@nmeyerhans If you don't mind, I'd prefer to merge just this change:
pevik@c9ad9e1

@pevik pevik added this to the next release milestone May 17, 2023
@pevik pevik closed this in c9ad9e1 May 17, 2023
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.

None yet

2 participants