Skip to content

arguments as xml file for wsmancli invoke#4

Closed
kswia wants to merge 2 commits into
jtallieu:masterfrom
kswia:master
Closed

arguments as xml file for wsmancli invoke#4
kswia wants to merge 2 commits into
jtallieu:masterfrom
kswia:master

Conversation

@kswia

@kswia kswia commented Oct 25, 2013

Copy link
Copy Markdown

Hi,

I modified invoke method to pass xml file instead of single parameters to wsman command. I found that this is the only way to pass multiple parameters with the same key, eg. VDPropNameArray and VDPropValueArray pairs or PDArray. With this modification I can pass dict with key PDArray and value as list: ['Disk.Bay.12:Enclosure.Internal.0-1:RAID.Integrated.1-1', 'Disk.Bay.13:Enclosure.Internal.0-1:RAID.Integrated.1-1']
I hope you'll find it useful.

Kamil

@kswia kswia closed this Oct 25, 2013
@jtallieu

Copy link
Copy Markdown
Owner

That functionality should already be there. If the argument is a dictionary it's used as a dictionary of argument. If its a string, it's treated as a path to an XML document with arguments.

Sent from my iPhone

On Oct 25, 2013, at 6:53 AM, Kamil Swiatkowski notifications@github.com wrote:

Hi,

I modified invoke method to pass xml file instead of single parameters to wsman command. I found that this is the only way to pass multiple parameters with the same key, eg. VDPropNameArray and VDPropValueArray pairs or PDArray. With this modification I can pass dict with key PDArray and value as list: ['Disk.Bay.12:Enclosure.Internal.0-1:RAID.Integrated.1-1', 'Disk.Bay.13:Enclosure.Internal.0-1:RAID.Integrated.1-1']
I hope you'll find it useful.

Kamil

You can merge this Pull Request by running

git pull https://github.com/kswia/dell-wsman-client-api-python master
Or view, comment on, or merge it at:

#4

Commit Summary

pass arguments to wsman as xml file to allow list arguments
Merge branch 'master' of github.com:jtallieu/dell-wsman-client-api-python
File Changes

M wsman/provider/wsmancli.py (18)
Patch Links:

https://github.com/jtallieu/dell-wsman-client-api-python/pull/4.patch
https://github.com/jtallieu/dell-wsman-client-api-python/pull/4.diff

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants