Skip to content

action: add option to disable apt-cacher - #14

Merged
zyga merged 2 commits into
zyga:mainfrom
bboozzoo:bboozzoo/option-disable-apt-cacher
Jan 23, 2026
Merged

action: add option to disable apt-cacher#14
zyga merged 2 commits into
zyga:mainfrom
bboozzoo:bboozzoo/option-disable-apt-cacher

Conversation

@bboozzoo

Copy link
Copy Markdown
Contributor

Add an option to disable (stop actually) apt-cacher-ng before running the test. Make sure the socket is cleaned up so that image-garden would not pick it up and try to setup proxing.

Add an option to disable (stop actually) apt-cacher-ng before running
the test. Make sure the socket is cleaned up so that image-garden would
not pick it up and try to setup proxing.

Signed-off-by: Maciek Borzecki <maciek.borzecki@gmail.com>

@zyga zyga left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks good. It'd be nice to update README.md as well.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds a new input option cache-deb-packages to control whether apt-cacher-ng should be used for caching Debian packages during image preparation. When disabled, the action stops the apt-cacher-ng service and removes its socket file to prevent image-garden from attempting to use it for proxying.

Changes:

  • Added new boolean input cache-deb-packages with default value of true to maintain backward compatibility
  • Added a new step to disable apt-cacher-ng service when caching is disabled
  • Implemented socket cleanup to ensure image-garden doesn't try to use a disabled service

Signed-off-by: Maciek Borzecki <maciek.borzecki@gmail.com>

@zyga zyga left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM!

@zyga
zyga merged commit 3a6aa90 into zyga:main Jan 23, 2026
1 check passed
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