Bug 1021099

Summary: /etc/httpd/conf.d/puppet.conf should have pulp prefix to avoid puppetmaster conflict
Product: [Retired] Pulp Reporter: Dominic Cleal <dcleal>
Component: puppet-supportAssignee: Barnaby Court <bcourt>
Status: CLOSED CURRENTRELEASE QA Contact: Preethi Thomas <pthomas>
Severity: medium Docs Contact:
Priority: high    
Version: MasterCC: bcourt, pthomas, skarmark
Target Milestone: ---Keywords: Triaged
Target Release: 2.3.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-12-09 14:30:01 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Dominic Cleal 2013-10-19 11:47:56 UTC 
Description of problem:
pulp-puppet-plugins installs /etc/httpd/conf.d/puppet.conf to provide the Puppet Forge compatible API, however this path is often used for the puppetmaster vhost itself when running under Passenger.

Currently when using Katello, the puppetmaster installation via node-installer is causing this file to be overwritten.

I'd suggest prefixing the file with "pulp_" to be clearer as to what the service is.

Version-Release number of selected component (if applicable):
pulp-puppet-plugins-2.3.0-0.22.beta.el6sat.noarch

How reproducible:
Always

Steps to Reproduce:
1. rpm -ql pulp-puppet-plugins | grep /etc/httpd

Actual results:
/etc/httpd/conf.d/puppet.conf

Expected results:
/etc/httpd/conf.d/pulp_puppet.conf

Additional info:
https://github.com/theforeman/puppet-puppet/blob/master/manifests/server/passenger.pp#L42
Comment 2 Barnaby Court 2013-10-25 18:56:51 UTC 
Bugzilla: https://github.com/pulp/pulp_puppet/pull/81
Comment 3 Jeff Ortel 2013-11-01 22:28:59 UTC 
build: 2.3.0-0.27.beta
Comment 4 Preethi Thomas 2013-11-05 18:51:54 UTC 
verified
[root@pulp-v2-server ~]# rpm -qa pulp-server
pulp-server-2.3.0-0.27.beta.el6.noarch

root@qe-blade-08 ~]#  rpm -ql pulp-puppet-plugins | grep /etc/httpd
/etc/httpd/conf.d/pulp_puppet.conf
Comment 5 Preethi Thomas 2013-12-09 14:30:01 UTC 
Pulp 2.3 released.