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: Ensure we have enough values to split #56802

Open
wants to merge 1 commit into
base: devel
from

Conversation

Projects
None yet
3 participants
@adrianlzt
Copy link
Contributor

commented May 22, 2019

Avoid raising an exception if pkgstr does not complains with expected
value.

SUMMARY

Fixes #56800

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

lib/ansible/modules/packaging/os/yum.py

ADDITIONAL INFORMATION

Ensure we have enough values to split
Avoid raising an exception if pkgstr does not complains with expected
value.
@ansibot

This comment has been minimized.

@Akasurde Akasurde changed the title Ensure we have enough values to split YUM: Ensure we have enough values to split May 23, 2019

@mkrizek
Copy link
Contributor

left a comment

Commented in the issue this is trying to fix. I am not sure this is the correct fix. We should probably prevent 'incorrect data' getting into pkg_to_dict in the first place.

@ansibot ansibot removed the needs_triage label May 24, 2019

@mkrizek

This comment has been minimized.

Copy link
Contributor

commented May 27, 2019

Hrm, so we get Plugin "subscription-manager" can't be imported from repoquery even with --quiet. Not sure I have a better idea so LGTM.

cc @maxamillion

@mkrizek mkrizek requested a review from maxamillion May 27, 2019

@ansibot ansibot added the stale_ci label Jun 4, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.