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.

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

added capture and return of information about changed volume #53839

Merged
merged 1 commit into from Mar 17, 2019

Conversation

Projects
None yet
4 participants
@bblanham
Copy link
Contributor

bblanham commented Mar 15, 2019

SUMMARY

Added additional return information for all actions by capturing the dictionary returned by the purestorage rest api and passing that to json_exit as an additional parameter called 'volume'.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

purefa_volume

ADDITIONAL INFORMATION

Returned information will contain at minimum the 'name' attribute and in some cases these additional attributes: 'created', 'serial', 'size', 'source'. The reason for this change was so that information could be captured and used later in a playbook without the need to make additional calls to the storage array.


@ansibot

This comment has been minimized.

@ansibot ansibot removed the needs_triage label Mar 15, 2019

@sdodsley

This comment has been minimized.

Copy link
Contributor

sdodsley commented Mar 15, 2019

shipit

@bblanham

This comment has been minimized.

Copy link
Contributor Author

bblanham commented Mar 17, 2019

bot_status

@ansibot

This comment has been minimized.

Copy link
Contributor

ansibot commented Mar 17, 2019

Components

lib/ansible/modules/storage/purestorage/purefa_volume.py
support: community
maintainers: bannaych genegr lionmax opslounge raekins sdodsley sile16

Metadata

waiting_on: maintainer
changes_requested_by: null
needs_info: False
needs_revision: False
needs_rebase: False
merge_commits: []
too many files or commits: False
mergeable_state: clean
shippable_status: success
maintainer_shipits (module maintainers): 1
community_shipits (namespace maintainers): 0
ansible_shipits (core team members): 0
shipit_actors (maintainers or core team members): sdodsley
shipit_actors_other: []
automerge: automerge shipit test failed

click here for bot help

@opslounge
Copy link

opslounge left a comment

shipit

@ansibot ansibot merged commit ec0babe into ansible:devel Mar 17, 2019

1 check passed

Shippable Run 114062 status is SUCCESS.
Details

@bblanham bblanham deleted the bblanham:purefa_volume_add_return_info branch Mar 18, 2019

catcombo added a commit to catcombo/ansible that referenced this pull request Mar 21, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.