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.

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

yum skip broken issue #19587

Closed
tuwid opened this issue Dec 20, 2016 · 7 comments
Closed

yum skip broken issue #19587

tuwid opened this issue Dec 20, 2016 · 7 comments
Labels
affects_2.1 This issue/PR affects Ansible v2.1 feature This issue/PR relates to a feature request. module This issue/PR relates to a module. support:core This issue/PR relates to code supported by the Ansible Engineering Team.

Comments

@tuwid
Copy link

tuwid commented Dec 20, 2016

ISSUE TYPE
  • Feature Idea
COMPONENT NAME

yum module

ANSIBLE VERSION
ansible 2.1.2.0
  config file = /etc/ansible/ansible.cfg
  configured module search path = Default w/o overrides

OS / ENVIRONMENT

Mac OS Sierra 10.12.2 (16C67)

SUMMARY

yum module doesnt take --skip-broken as parameter somewhere

STEPS TO REPRODUCE

try doing an upgrade via yum on any cpanel host and you are likely to encounter that

    - name: Centos package upgrade 
      yum: name=* state=latest
      when: ansible_os_family == "RedHat"
@ansibot ansibot added affects_2.1 This issue/PR affects Ansible v2.1 bug_report module This issue/PR relates to a module. needs_triage Needs a first human triage before being processed. labels Dec 21, 2016
@LukeShortCloud
Copy link
Contributor

LukeShortCloud commented Dec 21, 2016

This would be nice to have a more feature complete Yum module.

As for your actual issue, try using cPanel's RPM repair:

# /usr/local/cpanel/scripts/check_cpanel_rpms --fix

RPM conflicts are common when using cPanel but can be corrected. I would also advise to disable any third-party repositories such as RepoForge or EPEL. If that still does not work, you should remove the RPM database entries for the conflicting packages (not the actual packages) and then run the cPanel RPM fix and/or reinstall those packages via Yum. I won't post any of the commands here because those can potentially be destructive. Contacting cPanel for assistance is always an option, too.

@jimi-c jimi-c removed the plugin label Jan 4, 2017
@abadger abadger added feature_idea and removed bug_report needs_triage Needs a first human triage before being processed. labels Jan 4, 2017
@tuwid
Copy link
Author

tuwid commented Jan 16, 2017

how about a yum clean all ?

@vmindru
Copy link
Contributor

vmindru commented Feb 11, 2017

i want to take care of this one

@tuwid
Copy link
Author

tuwid commented Feb 15, 2017

@vmindru i can do some tests for you if you want as i have mixed environments

@vmindru
Copy link
Contributor

vmindru commented Feb 15, 2017

@tuwid - #21475 some testing will help once that thread will be commented as ready to review

vmindru pushed a commit to vmindru/ansible that referenced this issue Feb 15, 2017
        things at line 646 look ugly ..
        trying to make them look a bit more human readble
        though the entire approach should be rewriten

PNTSYSOPS-1901 - internal reference

Signed-off-by: Veaceslav Mindru <vmindru@redhat.com> <mindruv@gmail.com>
vmindru pushed a commit to vmindru/ansible that referenced this issue Feb 16, 2017
abadger pushed a commit that referenced this issue Feb 21, 2017
* #19587: refactor to PEP8

* Issue: #19587: add skip_broken option to yum module
PNTSYSOPS-1901 - internal reference

Signed-off-by: Veaceslav Mindru <vmindru@redhat.com> <mindruv@gmail.com>

* Issue: #19587: fix line break, attempt to make readble

        things at line 646 look ugly ..
        trying to make them look a bit more human readble
        though the entire approach should be rewriten

PNTSYSOPS-1901 - internal reference

Signed-off-by: Veaceslav Mindru <vmindru@redhat.com> <mindruv@gmail.com>

* #19587: fixing typo, skip-broken has to be passed as list

* PR: #21475: peer review changes
            relates to: revert comment deletion #21475 (comment)
            relates to: remove irelevant var #21475 (comment)
            relates to: reformat #21475 (comment)

Signed-off-by: Veaceslav Mindru <vmindru@redhat.com> <mindruv@gmail.com>

* PR: #21475: peer review changes
            relates to: #21475 (comment)

Signed-off-by: Veaceslav Mindru <vmindru@redhat.com> <mindruv@gmail.com>

* Issue: #21475: fixing typo in the Man section

Signed-off-by: Veaceslav Mindru <vmindru@redhat.com> <mindruv@gmail.com>

* PR: #21475 - Peer review: revert variable a
        Relates to: #21475 (review)

Signed-off-by: Veaceslav Mindru <vmindru@redhat.com> <mindruv@gmail.com>

* Issue: 19587 - small correction

Signed-off-by: Veaceslav Mindru <vmindru@redhat.com> <mindruv@gmail.com>
@ansibot ansibot added the support:core This issue/PR relates to code supported by the Ansible Engineering Team. label Jun 29, 2017
@ansibot
Copy link
Contributor

ansibot commented Jul 19, 2017

@mkrizek
Copy link
Contributor

mkrizek commented Aug 17, 2017

The feature was added in 2.3.

@mkrizek mkrizek closed this as completed Aug 17, 2017
@ansibot ansibot added feature This issue/PR relates to a feature request. and removed feature_idea labels Mar 2, 2018
@ansible ansible locked and limited conversation to collaborators Apr 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.1 This issue/PR affects Ansible v2.1 feature This issue/PR relates to a feature request. module This issue/PR relates to a module. support:core This issue/PR relates to code supported by the Ansible Engineering Team.
Projects
None yet
Development

No branches or pull requests

7 participants