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

ignore version when deciding callback loading #38281

Merged
merged 2 commits into from Apr 10, 2018

Conversation

bcoca
Copy link
Member

@bcoca bcoca commented Apr 4, 2018

SUMMARY

The code already defaulted to load the callback if the properties are not present
there was no need for us to also check the version

fixes #38270

related #38266, fixes the specific plugin by updating the version, but this PR makes it so that it isn't required anymore

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

tqm

ANSIBLE VERSION
2.6

The code already defaulted to load the callback if the properties are not present
there was no need for us to also check the version

fixes ansible#38270
@ansibot ansibot added bug This issue/PR relates to a bug. 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 Apr 4, 2018
@bcoca bcoca removed the needs_triage Needs a first human triage before being processed. label Apr 4, 2018
@ansibot ansibot added the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. label Apr 4, 2018
@bcoca bcoca added this to To Do in 2.5.x blocker list via automation Apr 4, 2018
@bcoca bcoca removed the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. label Apr 10, 2018
@bcoca bcoca merged commit 1850bb7 into ansible:devel Apr 10, 2018
@bcoca bcoca deleted the ignore_cb_version branch April 10, 2018 21:15
@nitzmahone nitzmahone moved this from To Do to 2.5.2 holding in 2.5.x blocker list Apr 18, 2018
bcoca added a commit to bcoca/ansible that referenced this pull request Apr 27, 2018
* ignore version when deciding callback loading

The code already defaulted to load the callback if the properties are not present
there was no need for us to also check the version

fixes ansible#38270

* fix error msg on set optoins to use correct name

(cherry picked from commit 1850bb7)
@bcoca bcoca moved this from 2.5.2 holding to To Do in 2.5.x blocker list Apr 27, 2018
@bcoca bcoca moved this from To Do to Done in 2.5.x blocker list Apr 30, 2018
bcoca added a commit that referenced this pull request Apr 30, 2018
* ignore version when deciding callback loading

The code already defaulted to load the callback if the properties are not present
there was no need for us to also check the version

fixes #38270

* fix error msg on set optoins to use correct name

(cherry picked from commit 1850bb7)
ilicmilan pushed a commit to ilicmilan/ansible that referenced this pull request Nov 7, 2018
* ignore version when deciding callback loading

The code already defaulted to load the callback if the properties are not present
there was no need for us to also check the version

fixes ansible#38270

* fix error msg on set optoins to use correct name
@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.
Labels
bug This issue/PR relates to a bug. support:core This issue/PR relates to code supported by the Ansible Engineering Team.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

set_options on a callback gets called even when not whitelisted
2 participants