Add Fusion support to snapshot module #702
Conversation
plugins/modules/purefa_snap.py
Outdated
| res = array.post_remote_volume_snapshots( | ||
| source_names=[module.params["name"]], | ||
| on=module.params["offload"], | ||
| remote_volume_snapshot=RemoteVolumeSnapshotPost( |
There was a problem hiding this comment.
getting unknown keyword error, remote_volume_snapshot not required here.
There was a problem hiding this comment.
remote_volume_snapshot is a required field. I'll check what is going on.
There was a problem hiding this comment.
This is the correct syntax, I just tried it manually, and remote_volume_snapshot is required.
If this errors again can you send the full traceback
There was a problem hiding this comment.
@sdodsley , might be backward compatibility issue, when we ran in purity version - 6.3.4, it errors out
There was a problem hiding this comment.
@skeerthivasan I think I figured it out. Please retry
| - Name of fleet member on which to perform the volume operation. | ||
| - This requires the array receiving the request is a member of a fleet | ||
| and the context name to be a member of the same fleet. | ||
| type: str |
There was a problem hiding this comment.
pls add default="" for handling fleet array without context
plugins/modules/purefa_snap.py
Outdated
| @@ -562,6 +780,7 @@ def main(): | |||
| default="present", | |||
| choices=["absent", "copy", "present", "rename"], | |||
| ), | |||
| context=dict(type="str"), | |||
There was a problem hiding this comment.
pls add default="" for handling fleet array without context
SUMMARY
Add Fusion support with
contextparameter.Complete REST v2 conversion
ISSUE TYPE
COMPONENT NAME
purefa_snap.py