On Fri, 2011-01-21 at 09:41 -0500, Konrad Scherer wrote:
On 01/19/2011 04:05 PM, Clint Byrum wrote:
So, a snapshot of the Cobbler git repository was packaged and uploaded into Ubuntu Natty-Narwhal (the future 11.04) universe archive today. This was a big effort involving multiple members of the Ubuntu Server Team.
That is great news. I looked at the package on Launchpad and there are a few things that I have noticed running cobbler on debian that don't seem to be addressed.
- The build-in tftpd server (installed as /usr/sbin/tftpd.py) requires the
package python-tornado. Would this be handled as a Suggests or Recommended package?
We should probably package that in its own binary package, Suggest it in the cobbler package, and then have it Depend on python-tornado. But before I file a bug report, AFAIK, that is not the recommended way to serve tftpd, is it?
- The apache config cobbler.conf requires that the apache module proxy_http be
loaded. It is not loaded by default on Debian squeeze. I do not know enough about the policy on apache modules to know whether the proper solution is to a) add a2enmod proxy_http to the postinst script or b) add LoadModule proxy_http to cobbler.conf. What do you think is the best solution?
That should be a conditional configuration... so the whole thing should probably be surrounded in: <IfModule proxy_http>..
Reported the bug here:
https://launchpad.net/bugs/705982
I don't think we want to enable proxy_http out of the box as it needs some consideration by the admins first to make sure they're not exposing themselves as an open proxy.
- The module sync_post_restart_services.py and action_check assume that init
scripts are located in /etc/rc.d/init.d. This of course will not work on Debian/Ubuntu. I have hacked in a call to /sbin/service <service> restart and that works but I do not know whether that is a proper cross platform solution to service restarting. Any ideas?
Right, can you file a bug for this?
https://bugs.launchpad.net/ubuntu/+source/cobbler/+filebug
Also a heads up that I submitted some patches to the mainline tree recently that will require minor changes to the package patches. For example the link from /etc/cobbler/cobbler.conf to /etc/apache2/conf.d is not necessary.
Cool! We'll hopefully be pulling from mainline going forward as much as possible, at least until we get up to the feature freeze which is just a couple of weeks away. After that we'll have to evaluate any pulls and maybe even just cherry pick bug fixes.