community.general icon indicating copy to clipboard operation
community.general copied to clipboard

Fix #3320: lvreduce should be called without pvs

Open eszense opened this issue 1 year ago • 4 comments

SUMMARY

Fix #3320

lvreduce does not require pv as input, and will produce error like "Command does not accept argument /dev/sdb1" if pv given. This fix omit pv for calls to lvreduce but retains pv for calls to lvextend

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

lvol

eszense avatar Jul 28 '22 11:07 eszense

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

ansibullbot avatar Jul 28 '22 11:07 ansibullbot

Docs Build 📝

This PR is closed and any previously published docsite has been unpublished.

github-actions[bot] avatar Jul 30 '22 00:07 github-actions[bot]

@eszense ping

needs_info

felixfontein avatar Sep 03 '22 10:09 felixfontein

@eszense This pullrequest is waiting for your response. Please respond or the pullrequest will be closed.

click here for bot help

ansibullbot avatar Oct 05 '22 11:10 ansibullbot

Please note that in #5461 the collection repository was restructured to remove the directory tree in plugins/modules/, and the corresponding tree in tests/unit/plugins/modules/. Your PR modifies files in this directory structure, and unfortunately now has some conflicts, potentially because of this. Please rebase with the current main branch to remove the conflicts.

felixfontein avatar Nov 03 '22 05:11 felixfontein

@eszense You have not responded to information requests in this pullrequest so we will assume it no longer affects you. If you are still interested in this, please create a new pullrequest with the requested information.

click here for bot help

ansibullbot avatar Nov 07 '22 05:11 ansibullbot