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

Glusterfs support for RHEL7,CentOS7,Fedora20 #262

Closed
wants to merge 1 commit into from
Closed

Glusterfs support for RHEL7,CentOS7,Fedora20 #262

wants to merge 1 commit into from

Conversation

gildub
Copy link
Member

@gildub gildub commented Jun 11, 2014

Requires Puppet module-data which is already in Openstack-Puppet-Modules.

Hiera is also required since this is relying on yaml datastores

Hiera is included by default but a symbolic link to the default /etc/hiera.yaml
configuration file must be created in /etc/puppet for puppet-hiera to
kick in properly.

https://bugzilla.redhat.com/show_bug.cgi?id=1091777

Requires Puppet module-data which is already in Openstack-Puppet-Modules.

Hiera is also required since this is relying on yaml datastores

Hiera is included by default but a symbolic link to the default /etc/hiera.yaml
configuration file must be created in /etc/puppet for puppet-hiera to
kick in properly.

https://bugzilla.redhat.com/show_bug.cgi?id=1091777
@jguiditta
Copy link
Member

Unfortunately, it turns out that module-data breaks foreman (version of hiera is newer than foreman can work with, so import will break everything on foreman side) according to mhulan, so we cannot use this at the current time, sorry :(

@gildub
Copy link
Member Author

gildub commented Jun 13, 2014

@jguiditta,
My test worked but I also ended up re-aligning hiera versions:
https://bugzilla.redhat.com/show_bug.cgi?id=1095985

@gildub
Copy link
Member Author

gildub commented Jul 8, 2014

@jguiditta, now that hiera versions (and puppet) have been bumped up, we can merge this one.
I've tested with latest on RHEL7.

@gildub
Copy link
Member Author

gildub commented Sep 22, 2014

@jguiditta,
Can you please provide an update for targeting this one?
Thanks.

@@ -29,6 +29,7 @@
$volume3_path = $quickstack::params::gluster_volume3_path,
$volume3_uid = $quickstack::params::gluster_volume3_uid,
) {
include gluster::params
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see this referenced file in the gluster module in current build of opm, openstack-puppet-modules-2014.1-21.8.el6ost.noarch. Once it is there, this looks pretty non-invasive

@gildub
Copy link
Member Author

gildub commented Sep 29, 2014

@jguiditta, params.pp somehow in not the commit used by OPM (2014.1-21.8).
Pull Requested OPM to get gluster latest:
redhat-openstack/openstack-puppet-modules#115
Thanks.

@jguiditta jguiditta closed this Jul 20, 2016
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.

2 participants