Skip to content

Allow apt-get in the Enterprise image#691

Merged
jviotti merged 1 commit intomainfrom
apt-enterprise
Mar 3, 2026
Merged

Allow apt-get in the Enterprise image#691
jviotti merged 1 commit intomainfrom
apt-enterprise

Conversation

@jviotti
Copy link
Member

@jviotti jviotti commented Mar 3, 2026

Signed-off-by: Juan Cruz Viotti jv@jviotti.com

Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@augmentcode
Copy link

augmentcode bot commented Mar 3, 2026

🤖 Augment PR Summary

Summary: Keeps apt-get capability in the Enterprise Docker image for downstream consumers.
Changes: Stops purging apt/debian-archive-keyring/gpgv (and no longer removes bash) while continuing to purge other non-runtime packages to limit the image surface.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

RUN apt-get --yes update \
&& apt-get --yes purge --allow-remove-essential \
adduser bash debconf \
adduser debconf \
Copy link

Choose a reason for hiding this comment

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

Purging adduser may cause apt/apt-get to be removed or left in an “unmet dependency” state (since apt commonly depends on adduser for the _apt user), which could undermine the goal of keeping apt-get usable in the final image.

Severity: medium

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

@jviotti jviotti merged commit 4637fa2 into main Mar 3, 2026
6 checks passed
@jviotti jviotti deleted the apt-enterprise branch March 3, 2026 13:33
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.

1 participant