Skip to content
This repository has been archived by the owner on Oct 22, 2019. It is now read-only.

Additions to param.pp for Fedora #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Utanith
Copy link

@Utanith Utanith commented Jun 28, 2013

No description provided.

@ghost ghost assigned torian Jul 1, 2013
@torian
Copy link
Owner

torian commented Jul 1, 2013

Thanks for pull request. The only thing I'm not very confortable with is that it could fit under Redhat|CentOS. I don't see the point in duplicating the block if it is the same. Could you try to extend the Redhat|CentOS case, and send a pull request again ?

Thank you again.

@torian
Copy link
Owner

torian commented Jul 11, 2013

A patch has been submitted and merged to move from $::operatingsystem to $::osfamily. I'd be most gratefull to you if you can pull the latest version and test it on fedora (as it's a redhat derivative).

Thanks

@torian
Copy link
Owner

torian commented Jul 18, 2013

Any news ?

@Utanith
Copy link
Author

Utanith commented Jul 18, 2013

I've been assigned other tasks at work; I passed the email along to my
coworker who is in a position to actually check in on this. I'll see if
he's got anything to say about it when I go into the office tomorrow.

On 7/18/2013 6:04 AM, Emiliano Castagnari wrote:

Any news ?


Reply to this email directly or view it on GitHub
#11 (comment).

@nealian
Copy link

nealian commented Jul 19, 2013

(I'm the coworker)
I can confirm that the client works with SSL/TLS, and the basic master server (with the foobar config), after your patches for Issue #14.
I'm not really in a position to test the other configuration options, but hopefully this is enough to get started.

@nealian
Copy link

nealian commented Jul 19, 2013

On a slight sidenote, though, all versions of RHEL and Fedora still prefer Certificate Authority certs in /etc/openldap/cacerts, not /etc/openldap/certs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants