Skip to content

Preserve apt automatic upgrade Release fields - #1525

Open
hstct wants to merge 1 commit into
pulp:mainfrom
ATIX-AG:apt-pinning-expand
Open

hstct wants to merge 1 commit into
pulp:mainfrom
ATIX-AG:apt-pinning-expand

Conversation

@hstct

@hstct hstct commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Preserving NotAutomatic and ButAutomaticUpgrades fields. These fields are used by official Debian backport repositories to indicate to APT that backports should use a default priority.

@quba42 quba42 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

See the comments below. I am not sure if we should implement the suggestions. Let's discuss it and then upgrade the threads accordingly.

Comment thread pulp_deb/app/serializers/content_serializers.py
Comment thread pulp_deb/app/models/content/metadata.py Outdated
Comment thread CHANGES/+preserve-release-metadata-fields.bugfix Outdated
@hstct
hstct force-pushed the apt-pinning-expand branch from e3f6cd0 to e203b16 Compare September 15, 2026 14:29
@hstct
hstct force-pushed the apt-pinning-expand branch from e203b16 to f670e2d Compare September 15, 2026 14:30
@@ -0,0 +1 @@
Added support to sync and publish the `NotAutomatic` and eButAutomaticUpgrades`` Release file fields.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nitpick:

Suggested change
Added support to sync and publish the `NotAutomatic` and eButAutomaticUpgrades`` Release file fields.
Added support to sync and publish the `NotAutomatic` and `ButAutomaticUpgrades` Release file fields.

@quba42 quba42 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

With the exception of the typo in the changelog message, this all looks good to me.

Tested against Debian bookworm-backports and experimental, as well as a repo that does not sport either flag. => Everything works as expected.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants