Bug 1004805 - pulp-dev.py script doesn't handle F19
Summary: pulp-dev.py script doesn't handle F19
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Pulp
Classification: Retired
Component: z_other
Version: Master
Hardware: Unspecified
OS: Unspecified
medium
unspecified
Target Milestone: ---
: 2.3.0
Assignee: Michael Hrivnak
QA Contact: Preethi Thomas
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-09-05 14:08 UTC by Jay Dobies
Modified: 2013-12-09 14:31 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-12-09 14:31:04 UTC
Embargoed:


Attachments (Terms of Use)

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.


Note You need to log in before you can comment on or make changes to this bug.