Ubuntu
cobbler package

cobbler-web unnecessarily overrides default DocumentRoot

Bug #760012 reported by Tom Ellis
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
cobbler (Ubuntu)
Fix Released
Low
James Page
Natty
Fix Released
Low
James Page

Bug Description

Binary package hint: cobbler

Whilst trying to add additional files to the same system as cobbler web is running on, I notice that DocumentRoot is overridden by the /etc/apache2/conf.d/cobbler_web.conf by the nature of replacing the virtual host definition.

If I add the DocumentRoot back to be /var/www in /etc/apache2/conf.d/cobbler_web.conf then I can serve files from the web server and use cobbler fine too, but we need a cleaner approach.

I'm using the latest cobbler build on Natty.

Packages:
ii cobbler 2.1.0-0ubuntu5 Install server
ii cobbler-common 2.1.0-0ubuntu5 Cobbler Install server - common files
ii cobbler-web 2.1.0-0ubuntu5 Cobbler Install server - common files
ii python-cobbler 2.1.0-0ubuntu5 Install server - python libraries.

Related branches

lp:~james-page/ubuntu/natty/cobbler/fix-760012
Ubuntu Development Team: Pending requested
Diff: 102 lines (+48/-4)
6 files modified
.pc/39_cw_remove_vhost.patch/config/cobbler_web.conf (+14/-0)
.pc/applied-patches (+1/-0)
config/cobbler_web.conf (+0/-4)
debian/changelog (+7/-0)
debian/patches/39_cw_remove_vhost.patch (+25/-0)
debian/patches/series (+1/-0)
Revision history for this message
James Page (james-page) wrote :

Confirmed;

Possible fix could be to not place the cobber_web configuration in a VirtualHost definition; this would enable it for all virtual hosts (which might not be desirable) and would leave existing DocumentRoot definitions intact.

This approach worked OK for me.

Changed in cobbler (Ubuntu):
importance: Undecided → Low
status: New → Triaged
Revision history for this message
Tom Ellis (tellis) wrote :

That solution works for me too.

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Hey James, could you attach a patch, or a branch?

Revision history for this message
James Page (james-page) wrote : Re: [Bug 760012] Re: cobbler-web unnecessarily overrides default DocumentRoot

On Fri, 2011-04-15 at 12:09 +0000, Dustin Kirkland wrote:
> Hey James, could you attach a patch, or a branch?
>

I'll work on a merge proposal now.

--
James Page
Software Engineer, Ubuntu Server Team

Changed in cobbler (Ubuntu):
assignee: nobody → James Page (james-page)
Dustin Kirkland  (kirkland)
Changed in cobbler (Ubuntu Natty):
status: Triaged → Fix Committed
Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Thanks, looks good, James. I've pushed it to the branch, and uploaded to the natty queue, where it will be held for release team review. The change is trivial and straightforward, so I think it will be approved without much trouble. Thanks!

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package cobbler - 2.1.0-0ubuntu6

---------------
cobbler (2.1.0-0ubuntu6) natty; urgency=low

  * Fix DocumentRoot conflict with default vhost (LP: #760012)
    - debian/patches/39_cw_remove_vhost.patch: Remove VirtualHost definition.
 -- James Page <email address hidden> Fri, 15 Apr 2011 13:35:20 +0100

Changed in cobbler (Ubuntu Natty):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.