Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1004805

Summary: pulp-dev.py script doesn't handle F19
Product: [Retired] Pulp Reporter: Jay Dobies <jason.dobies>
Component: z_otherAssignee: Michael Hrivnak <mhrivnak>
Status: CLOSED CURRENTRELEASE QA Contact: Preethi Thomas <pthomas>
Severity: unspecified Docs Contact:
Priority: medium    
Version: MasterCC: mhrivnak, 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:31:04 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:
Embargoed:

Description Jay Dobies 2013-09-05 14:08:18 UTC
With F18, the pulp-dev.py script was modified to symlink a specific HTTP conf file. This check is for F18 explicitly and not a greater than check, so F19 dev installs are broken.

Below is the relevant snippet from pulp-dev.py:

    # Get links for httpd conf files according to distro
    pre_f18_apache_conf = ('server/etc/httpd/conf.d/pulp.conf', '/etc/httpd/conf.d/pulp.conf')
    f18_apache_conf = ('server/etc/httpd/conf.d/pulp_f18.conf', '/etc/httpd/conf.d/pulp.conf')
    s, n, r, v, m = os.uname()
    if 'fc18' in r:
        src, dst = f18_apache_conf
    else:
        src, dst = pre_f18_apache_conf
    links.append((src, dst))

The RPM spec is correct, it checks for version >= 18, so this is only an issue for developer installs.

Comment 1 Michael Hrivnak 2013-09-11 19:03:08 UTC
need to now only check for whether this is fedora, any version.

Comment 2 Sayli Karmarkar 2013-09-17 19:58:30 UTC
This is only a problem in dev environment, so not sure why this was aligned to 2.2 and on high priority. Moving it to 2.3 and medium priority.

Comment 3 Michael Hrivnak 2013-10-02 20:27:02 UTC
https://github.com/pulp/pulp/pull/649

Comment 4 Jeff Ortel 2013-10-11 15:25:47 UTC
build: 2.3.0-0.18.beta

Comment 5 Sayli Karmarkar 2013-10-17 20:00:33 UTC
Verified the PR and also the output of pulp-dev.py -

creating link: /etc/httpd/conf.d/pulp.conf pointing to /home/skarmark/git/pulp/server/etc/httpd/conf.d/pulp_apache_24.conf

Comment 6 Preethi Thomas 2013-12-09 14:31:04 UTC
Pulp 2.3 released.