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

Volume Access Group improvements #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ssgelm
Copy link
Contributor

@ssgelm ssgelm commented May 2, 2019

This PR does two things to improve the ability to use volume access groups:

  • Sort the list of VAG initiators and volumes when checking if in sync

    This prevents a permanent diff when the list of either initiators or
    volumes is sorted differently from what the API returns

  • Improve readability of changes in VAG initiators and volumes

    Currently the puppet output when initiators or volumes change in a
    volume access group is a very hard to read output of an array. This
    changes it to only show what's different, making it much more readable.

ssgelm added 2 commits May 1, 2019 23:26
This prevents a permanent diff when the list of either initiators or
volumes is sorted differently from what the API returns
Currently the puppet output when initiators or volumes change in a
volume access group is a very hard to read output of an array.  This
changes it to only show what's different, making it much more readable.
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.

1 participant