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

lvol: check return message in both stderr and stdout #7601

Merged
merged 2 commits into from
Dec 3, 2023
Merged

lvol: check return message in both stderr and stdout #7601

merged 2 commits into from
Dec 3, 2023

Conversation

russoz
Copy link
Collaborator

@russoz russoz commented Nov 25, 2023

SUMMARY

Fixes #7182

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

lvol

@ansibullbot
Copy link
Collaborator

cc @abulimov @jhoekx @unkaputtbar112 @zigaSRC
click here for bot help

@ansibullbot ansibullbot added bug This issue/PR relates to a bug module module plugins plugin (any type) labels Nov 25, 2023
@felixfontein felixfontein added check-before-release PR will be looked at again shortly before release and merged if possible. backport-7 backport-8 Automatically create a backport for the stable-8 branch labels Nov 26, 2023
@felixfontein
Copy link
Collaborator

Thanks for your contribution! This needs a changelog fragment.

@felixfontein
Copy link
Collaborator

Will merge this in a few days if nobody objects.

@felixfontein felixfontein removed the check-before-release PR will be looked at again shortly before release and merged if possible. label Dec 3, 2023
@felixfontein felixfontein merged commit 3255855 into ansible-collections:main Dec 3, 2023
@patchback Patchback
Copy link

patchback bot commented Dec 3, 2023 •
edited
Loading

Backport to stable-7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-7/32558558c00ee6c59db3a4f7e4decbf70a19ba8c/pr-7601

Backported as #7673

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

@felixfontein
Copy link
Collaborator

@russoz thanks for your contribution!

patchback bot pushed a commit that referenced this pull request Dec 3, 2023
@russoz @patchback
lvol: check return message in both stderr and stdout (#7601)
2050a82 
* lvol: check return message in both stderr and stdout

* add changelog frag

(cherry picked from commit 3255855)
@patchback patchback bot mentioned this pull request Dec 3, 2023
Merged
@patchback Patchback
Copy link

patchback bot commented Dec 3, 2023 •
edited
Loading

Backport to stable-8: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-8/32558558c00ee6c59db3a4f7e4decbf70a19ba8c/pr-7601

Backported as #7674

🤖 @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 3, 2023
@russoz @patchback
lvol: check return message in both stderr and stdout (#7601)
7d69c4e 
* lvol: check return message in both stderr and stdout

* add changelog frag

(cherry picked from commit 3255855)
@patchback patchback bot mentioned this pull request Dec 3, 2023
Merged
felixfontein pushed a commit that referenced this pull request Dec 3, 2023
@patchback @russoz
[PR #7601/32558558 backport][stable-7] lvol: check return message in …
b3d79d7 
…both stderr and stdout (#7673)

lvol: check return message in both stderr and stdout (#7601)

* lvol: check return message in both stderr and stdout

* add changelog frag

(cherry picked from commit 3255855)

Co-authored-by: Alexei Znamensky <103110+russoz@users.noreply.github.com>
felixfontein pushed a commit that referenced this pull request Dec 3, 2023
@patchback @russoz
[PR #7601/32558558 backport][stable-8] lvol: check return message in …
c329192 
…both stderr and stdout (#7674)

lvol: check return message in both stderr and stdout (#7601)

* lvol: check return message in both stderr and stdout

* add changelog frag

(cherry picked from commit 3255855)

Co-authored-by: Alexei Znamensky <103110+russoz@users.noreply.github.com>
@russoz russoz deleted the 7182-lvol-fix branch December 3, 2023 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
backport-8 Automatically create a backport for the stable-8 branch bug This issue/PR relates to a bug module module plugins plugin (any type)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

lvol: Idempotency issue due to LVM2-2.03.17 std message
3 participants
@russoz @ansibullbot @felixfontein