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: use --auto-remove with dist-upgrade #37594

Merged
merged 1 commit into from
Mar 28, 2018
Merged

apt: use --auto-remove with dist-upgrade #37594

merged 1 commit into from
Mar 28, 2018

Conversation

mkrizek
Copy link
Contributor

@mkrizek mkrizek commented Mar 19, 2018

SUMMARY

This seems to have been forgotten in #30747.

Fixes #35900

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

apt

ANSIBLE VERSION
devel
ADDITIONAL INFORMATION

@mkrizek mkrizek requested a review from mscherer March 19, 2018 11:33
@mkrizek
Copy link
Contributor Author

mkrizek commented Mar 19, 2018

cc @dj4ngo

@ansibot
Copy link
Contributor

ansibot commented Mar 19, 2018

cc @mgwilliams
click here for bot help

@ansibot ansibot added bug This issue/PR relates to a bug. module This issue/PR relates to a module. needs_info This issue requires further information. Please answer any outstanding questions. needs_triage Needs a first human triage before being processed. support:core This issue/PR relates to code supported by the Ansible Engineering Team. labels Mar 19, 2018
@mkrizek mkrizek added this to READY TO REVIEW in CEM Mar 19, 2018
@mkrizek mkrizek requested a review from Akasurde March 19, 2018 12:57
@mkrizek mkrizek mentioned this pull request Mar 19, 2018
Merged
@Akasurde Akasurde removed the needs_triage Needs a first human triage before being processed. label Mar 19, 2018
@mkrizek mkrizek mentioned this pull request Mar 19, 2018
Closed
@mkrizek mkrizek removed the needs_info This issue requires further information. Please answer any outstanding questions. label Mar 19, 2018
@ansibot ansibot added the core_review In order to be merged, this PR must follow the core review workflow. label Mar 19, 2018
@mkrizek mkrizek requested a review from resmo March 23, 2018 09:21
@mscherer
Copy link
Contributor

shipit

@ansibot ansibot added shipit This PR is ready to be merged by Core and removed core_review In order to be merged, this PR must follow the core review workflow. labels Mar 28, 2018
@mkrizek mkrizek merged commit 447b342 into ansible:devel Mar 28, 2018
@mkrizek mkrizek deleted the 35900-autoremove-fix branch March 28, 2018 10:18
@mkrizek mkrizek moved this from READY TO REVIEW to DONE in CEM Mar 28, 2018
ryancurrah pushed a commit to ryancurrah/ansible that referenced this pull request Apr 4, 2018
@mkrizek @ryancurrah
apt: use --auto-remove with dist-upgrade (ansible#37594)
8852e77
@dagwieers dagwieers added the packaging Packaging category label Mar 3, 2019
@ansible ansible locked and limited conversation to collaborators Apr 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mscherer mscherer mscherer approved these changes

@Akasurde Akasurde Awaiting requested review from Akasurde

@resmo resmo Awaiting requested review from resmo

Assignees
No one assigned
Labels
bug This issue/PR relates to a bug. module This issue/PR relates to a module. packaging Packaging category shipit This PR is ready to be merged by Core support:core This issue/PR relates to code supported by the Ansible Engineering Team.
Projects
No open projects
CEM
DONE
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

In the apt module the upgrade parameter cannot be used together with the autoremove in the same task
5 participants
@mkrizek @ansibot @mscherer @dagwieers @Akasurde