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

[2.9] puppet: Fix command line construction #70463

Merged
merged 1 commit into from Jul 17, 2020

Conversation

Akasurde
Copy link
Member

@Akasurde Akasurde commented Jul 4, 2020

SUMMARY

Fix command line construction in the puppet module
related to check mode and using manifests directly.

Also, fixes 69ead0b which
introduced another if-statement in the middle of a if/elif pair,
which causes the elif to execute together with the original if
which created '--noop --no-noop' commands.

Fixes: #60576

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

changelogs/fragments/114_puppet_quote.yml
lib/ansible/modules/system/puppet.py

Fix command line construction in the puppet module
related to check mode and using manifests directly.

Also, fixes 69ead0b which
introduced another if-statement in the middle of a if/elif pair,
which causes the elif to execute together with the original if
which created '--noop --no-noop' commands.

Fixes: ansible#60576
@ansibot ansibot added affects_2.9 This issue/PR affects Ansible v2.9 backport This PR does not target the devel branch. bug This issue/PR relates to a bug. collection Related to Ansible Collections work collection:community.general community_review In order to be merged, this PR must follow the community review workflow. has_issue module This issue/PR relates to a module. needs_triage Needs a first human triage before being processed. support:community This issue/PR relates to code supported by the Ansible community. system System category labels Jul 6, 2020
@ansibot ansibot added the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label Jul 14, 2020
@Akasurde Akasurde changed the title puppet: Fix command line construction [2.9] puppet: Fix command line construction Jul 15, 2020
@nitzmahone nitzmahone merged commit 6789fa7 into ansible:stable-2.9 Jul 17, 2020
@sivel sivel removed the needs_triage Needs a first human triage before being processed. label Jul 21, 2020
@ansible ansible locked and limited conversation to collaborators Aug 14, 2020
@Akasurde Akasurde deleted the backport/2.9/114 branch February 8, 2021 08:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.9 This issue/PR affects Ansible v2.9 backport This PR does not target the devel branch. bug This issue/PR relates to a bug. collection:community.general collection Related to Ansible Collections work community_review In order to be merged, this PR must follow the community review workflow. has_issue module This issue/PR relates to a module. stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. support:community This issue/PR relates to code supported by the Ansible community. system System category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants