Skip to content

Move apt sources lists aside more comprehensively - #86050

Open
cjwatson wants to merge 1 commit into
ansible:develfrom
cjwatson:integration-test-apt-sources-list
Open

Move apt sources lists aside more comprehensively#86050
cjwatson wants to merge 1 commit into
ansible:develfrom
cjwatson:integration-test-apt-sources-list

Conversation

@cjwatson

@cjwatson cjwatson commented Oct 21, 2025

Copy link
Copy Markdown
Contributor
SUMMARY

apt sources may be defined in any of /etc/apt/sources.list, /etc/apt/sources.list.d/*.list, or /etc/apt/sources.list.d/*.sources. When disabling system repositories, move all of these aside rather than only /etc/apt/sources.list.

This fixes integration test failures in certain corner cases in Debian's CI environment: when testing reverse-dependencies, we use a testbed with most packages from the testing suite but some packages pinned from unstable. The latter are currently set up using /etc/apt/sources.list.d/*.list, and as a result the apt integration tests accidentally upgraded the testbed to unstable in the middle of the test run, causing confusion in later tests.

ISSUE TYPE
  • Test Pull Request

apt sources may be defined in any of `/etc/apt/sources.list`,
`/etc/apt/sources.list.d/*.list`, or
`/etc/apt/sources.list.d/*.sources`.  When disabling system
repositories, move all of these aside rather than only
`/etc/apt/sources.list`.

This fixes integration test failures in certain corner cases in Debian's
CI environment: when testing reverse-dependencies, we use a testbed with
most packages from the testing suite but some packages pinned from
unstable.  The latter are currently set up using
`/etc/apt/sources.list.d/*.list`, and as a result the `apt` integration
tests accidentally upgraded the testbed to unstable in the middle of the
test run, causing confusion in later tests.
@ansibot ansibot added test This PR relates to tests. needs_triage Needs a first human triage before being processed. labels Oct 21, 2025
@s-hertel s-hertel added needs_verified This issue needs to be verified/reproduced by maintainer and removed needs_triage Needs a first human triage before being processed. labels Nov 4, 2025
@ansibot ansibot added the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label Nov 4, 2025
@leegarrett

Copy link
Copy Markdown
Contributor

@cjwatson Can you rebase your patch on devel?

Can we get this patch merged? It helps us in Debian downstream, and is also useful for other Debian-based distros running the tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs_verified This issue needs to be verified/reproduced by maintainer stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. test This PR relates to tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants