Skip to content

Conversation

@mpurg
Copy link
Contributor

@mpurg mpurg commented Nov 21, 2025

Description:

  • Add 60s lock timeout to apt-get commands

Rationale:

  • This largely mitigates the issue of packages failing to install due to other apt processes (unattended-upgrades) running in the background at the same time.
  • By default, apt-get fails immediately if another apt process holds the lock, and there's no good way to acquire the lock from a bash script.

This largely mitigates the issue of packages failing
to install due to other apt processes (unattended-upgrades)
running in the background at the same time.

By default, apt-get fails immediately if another apt process
holds the lock, and there's no good way to acquire the lock
from a bash script.
@openshift-ci
Copy link

openshift-ci bot commented Nov 21, 2025

Hi @mpurg. Thanks for your PR.

I'm waiting for a github.com member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-ci openshift-ci bot added the needs-ok-to-test Used by openshift-ci bot. label Nov 21, 2025
@Mab879 Mab879 added this to the 0.1.80 milestone Nov 21, 2025
@Mab879 Mab879 requested a review from a team November 21, 2025 21:11
@jan-cerny jan-cerny changed the title Add 60s timeout for acquiring the APT lock. Add 60s timeout for acquiring the APT lock Nov 24, 2025
Copy link
Contributor

@dodys dodys left a comment

Choose a reason for hiding this comment

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

lgtm, thanks!

@dodys dodys added Debian Debian product related. Ubuntu Ubuntu product related. labels Nov 24, 2025
@dodys dodys merged commit ac96d5d into ComplianceAsCode:master Nov 24, 2025
123 of 130 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Debian Debian product related. needs-ok-to-test Used by openshift-ci bot. Ubuntu Ubuntu product related.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants