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.

Sign up for GitHub

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

Jump to bottom

logstash_plugin - invoke run_command passing list #3808

Merged
merged 3 commits into from
Dec 1, 2021
Merged

logstash_plugin - invoke run_command passing list #3808

merged 3 commits into from
Dec 1, 2021

Conversation

russoz
Copy link
Collaborator

@russoz russoz commented Nov 29, 2021

SUMMARY

Passing command as list instead of as string.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/modules/monitoring/logstash_plugin.py

@ansibullbot
Copy link
Collaborator

cc @nerzhul
click here for bot help

@ansibullbot ansibullbot added feature This issue/PR relates to a feature request module module monitoring plugins plugin (any type) labels Nov 29, 2021
@felixfontein felixfontein added backport-4 check-before-release PR will be looked at again shortly before release and merged if possible. labels Nov 30, 2021
@felixfontein felixfontein merged commit c587d21 into ansible-collections:main Dec 1, 2021
@patchback
Copy link

patchback bot commented Dec 1, 2021 •
edited
Loading

Backport to stable-4: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4/c587d21ba047e8d7ef571f21be5e0883c59f7411/pr-3808

Backported as #3827

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Dec 1, 2021
@russoz @patchback
logstash_plugin - invoke run_command passing list (#3808)
a700402 
* logstash_plugin - invoke run_command passing list

* added changelog fragment

* rogue chglog frag escaped its caged and was seen running around into a different PR

(cherry picked from commit c587d21)
@patchback patchback bot mentioned this pull request Dec 1, 2021
Merged
@felixfontein
Copy link
Collaborator

@RUSSO thanks!

@russoz russoz deleted the logstash_plugin-run_command-list branch December 1, 2021 06:08
felixfontein pushed a commit that referenced this pull request Dec 1, 2021
@patchback @russoz
logstash_plugin - invoke run_command passing list (#3808) (#3827)
6b99d48 
* logstash_plugin - invoke run_command passing list

* added changelog fragment

* rogue chglog frag escaped its caged and was seen running around into a different PR

(cherry picked from commit c587d21)

Co-authored-by: Alexei Znamensky <103110+russoz@users.noreply.github.com>
@felixfontein felixfontein removed the check-before-release PR will be looked at again shortly before release and merged if possible. label Dec 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@felixfontein felixfontein felixfontein approved these changes

Assignees
No one assigned
Labels
feature This issue/PR relates to a feature request module module monitoring plugins plugin (any type)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@russoz @ansibullbot @felixfontein