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

ovirt_network: Fix update_check for cluster network role #53318

Merged
merged 1 commit into from
Mar 5, 2019

Conversation

nijinashok
Copy link
Contributor

SUMMARY

Currently if we didn't specify the "role" in the task which is already
available for the network, the update_check will return FALSE. The
patch fixes the same and update_check will only return FALSE if there
is any additonal role specified for the network that are not currently
available for the network.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ovirt_network

ADDITIONAL INFORMATION

Fixes bug https://bugzilla.redhat.com/show_bug.cgi?id=1673225

Currently if we didn't specify the "role" in the task which is already
available for the network, the update_check will return FALSE. The
patch fixes the same and update_check will only return FALSE if there
is any additonal role specified for the network that are not currently
available for the network.
@ansibot
Copy link
Contributor

ansibot commented Mar 5, 2019

@ansibot ansibot added affects_2.8 This issue/PR affects Ansible v2.8 bug This issue/PR relates to a bug. cloud community_review In order to be merged, this PR must follow the community review workflow. module This issue/PR relates to a module. needs_triage Needs a first human triage before being processed. ovirt oVirt and RHV community support:community This issue/PR relates to code supported by the Ansible community. labels Mar 5, 2019
Copy link
Contributor

@machacekondra machacekondra left a comment

Choose a reason for hiding this comment

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

thanks! shipit

@ansibot ansibot removed the needs_triage Needs a first human triage before being processed. label Mar 5, 2019
Copy link
Contributor

@mwperina mwperina left a comment

Choose a reason for hiding this comment

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

shipit

@ansibot ansibot added automerge This PR was automatically merged by ansibot. shipit This PR is ready to be merged by Core and removed community_review In order to be merged, this PR must follow the community review workflow. labels Mar 5, 2019
@ansibot ansibot merged commit ff0285c into ansible:devel Mar 5, 2019
@nijinashok nijinashok deleted the fix_bug_1673225 branch March 6, 2019 01:45
@nijinashok nijinashok restored the fix_bug_1673225 branch March 6, 2019 12:42
@nijinashok nijinashok deleted the fix_bug_1673225 branch March 6, 2019 12:50
@ansible ansible locked and limited conversation to collaborators Jul 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.8 This issue/PR affects Ansible v2.8 automerge This PR was automatically merged by ansibot. bug This issue/PR relates to a bug. cloud module This issue/PR relates to a module. ovirt oVirt and RHV community shipit This PR is ready to be merged by Core support:community This issue/PR relates to code supported by the Ansible community.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants