Skip to content
This repository has been archived by the owner on Oct 30, 2018. It is now read-only.

LVG module fix when using multiple devices (string to list) #56

Merged
merged 1 commit into from Nov 6, 2014
Merged

LVG module fix when using multiple devices (string to list) #56

merged 1 commit into from Nov 6, 2014

Conversation

filipenf
Copy link
Contributor

@filipenf filipenf commented Oct 6, 2014

I've found out that when running the pvcreate command we were passing all devices as if they were one parameter like:

['/sbin/vgcreate', '-s', '4', 'vg0', '/dev/xvdb /dev/xvdc']

while the correct way would be:

['/sbin/vgcreate', '-s', '4', 'vg0', '/dev/xvdb', '/dev/xvdc']

@filipenf filipenf mentioned this pull request Oct 6, 2014
Closed
@bcoca bcoca added P2 and removed P3 labels Nov 6, 2014
@bcoca bcoca changed the title Replace dev_string to dev_list LVG module fix when using multiple devices (string to list) Nov 6, 2014
bcoca added a commit that referenced this pull request Nov 6, 2014
@bcoca
Merge pull request #56 from filipenf/fix-lvg
664858e 
LVG module fix when using multiple devices (string to list)
@bcoca bcoca merged commit 664858e into ansible:devel Nov 6, 2014
@echo-devnull echo-devnull mentioned this pull request Jan 21, 2015
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
bugfix_pull_request P2
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@filipenf @jimi-c @bcoca