Add AIX support for LVs and Filesystems #43
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I love it! I hate to even ask but do you think you'll have time to work on any spec tests for it? If you say no I'm going to merge it anyway as it's an improvement, I was just hoping you were feeling charitable. :) |
|
I was trying to find time to write spec tests :-) - Will try and bash some out this week but feel free to merge this, I can put specs in a separate PR if you like. |
apenney
pushed a commit
that referenced
this pull request
Nov 25, 2013
Add AIX support for LVs and Filesystems
cegeka-jenkins
pushed a commit
to cegeka/puppet-lvm
that referenced
this pull request
Oct 23, 2017
Add AIX support for LVs and Filesystems
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds support for AIX logical volumes and creation and resizing of filesystems on AIX. The filesystem type has many new attributes currently only supported on AIX, the idea is that in time this model can be used for Linux platforms too instead of just passing a whole bunch of OS specific data in the options attribute. Until these attributes are handled by the lvm provider they are AIX only.
Also re-indented the filesystem type to conventional 2 space tabs.