-
Notifications
You must be signed in to change notification settings - Fork 16
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
St 1 ephemeral previously formatted #28
Conversation
default['ephemeral_lvm']['filesystem'] = "ext4" | ||
# RHEL 7 and CentOS 7 uses XFS as their default file system. | ||
default['ephemeral_lvm']['filesystem'] = | ||
node['platform'] =~ /redhat|centos/ && node['platform_version'] =~ /^7\./ ? 'xfs' : 'ext4' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's use value_for_platform_family
for this, I think you will be able to just use 'rhel'
as the family name.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kitchen testing shows using value_for_platform_family
works fine. However, testing in RS returns the following error: Option filesystem must be a kind of String! You passed {["7.0", "7.1"]=>"xfs", "default"=>"ext4"}.
To continue moving forward, I will use 'node["platform_family"]
and node['platform_version']
.
…d based on OS. Remove setting of ephemeral_lvm:filesystem to use default values.
…eady exists which determines that lvm already is setup so need to run wipefs.
…in attributes/default.rb.
…ult due to RS not interpreting value_for_platform_family correctly.
…e/bats/verify_ephemeral.bats.
…ously_formatted St 1 ephemeral previously formatted
No description provided.