Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

apt_*: add more info message while updating apt cache #83230

Merged
merged 1 commit into from
May 23, 2024
Merged

apt_*: add more info message while updating apt cache #83230

merged 1 commit into from
May 23, 2024

Conversation

Akasurde
Copy link
Member

@Akasurde Akasurde commented May 9, 2024

SUMMARY

Fixes: #77941

Signed-off-by: Abhijeet Kasurde akasurde@redhat.com

ISSUE TYPE
  • Bugfix Pull Request

@Akasurde Akasurde requested a review from s-hertel May 9, 2024 23:42
@ansibot ansibot added bug This issue/PR relates to a bug. needs_triage Needs a first human triage before being processed. module This issue/PR relates to a module. has_issue needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels May 9, 2024
@ansibot
Copy link
Contributor

ansibot commented May 9, 2024

The test ansible-test sanity --test pep8 [explain] failed with 2 errors:

lib/ansible/modules/apt.py:1397:78: E226: missing whitespace around arithmetic operator
lib/ansible/modules/apt_repository.py:753:74: E226: missing whitespace around arithmetic operator

click here for bot help

@Akasurde Akasurde force-pushed the i77941 branch 3 times, most recently from 3d3f18f to e9f8bf2 Compare May 10, 2024 04:15
@ansibot ansibot removed the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. label May 10, 2024
@mkrizek mkrizek removed the needs_triage Needs a first human triage before being processed. label May 14, 2024
@ansibot ansibot added the needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html label May 14, 2024
@mkrizek
Copy link
Contributor

mkrizek commented May 14, 2024

Possibly related #81743

bcoca
bcoca reviewed May 16, 2024
View reviewed changes
lib/ansible/modules/apt.py Outdated Show resolved Hide resolved
@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 May 23, 2024
@Akasurde
apt_*: add more info message while updating apt cache
0a7fe12 
Fixes: ansible#77941

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
@ansibot ansibot removed needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. labels May 23, 2024
@Akasurde Akasurde requested review from s-hertel and bcoca May 23, 2024 21:12
@Akasurde Akasurde merged commit 3af9d0f into ansible:devel May 23, 2024
66 checks passed
@Akasurde Akasurde deleted the i77941 branch May 23, 2024 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bcoca bcoca bcoca approved these changes

@s-hertel s-hertel Awaiting requested review from s-hertel

Assignees
No one assigned
Labels
bug This issue/PR relates to a bug. has_issue module This issue/PR relates to a module.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Log the exception when the update cache command fails
5 participants
@Akasurde @ansibot @mkrizek @bcoca @s-hertel