On Fri, Jun 8, 2012 at 3:18 PM, Greg Swift
<gregswift(a)gmail.com> wrote:
> On Fri, Jun 8, 2012 at 1:42 PM, Dan Rao <dhananja(a)cs.wisc.edu> wrote:
>> Hi all,
>>
>> I upgraded my Cobbler install to 2.2.3-1.el6 (from epel-testing) last
>> night.
>>
>> Interestingly, when I run cobbler sync, only machines that have netboot
>> enabled get entries in my dhcpd.conf.
>>
>> The offending line(s) appear to be in
>> /usr/lib/python2.6/site-packages/cobbler/modules/manage_isc.py - lines
>> 164-165
>>
>> ----
>> if not interface["netboot_enabled"]:
>> continue
>> ----
>>
>> (This is in the section that generates the ISC dhcpd config file).
>>
>>
>> I commented those lines out to get my server back, but I was curious as
>> to why that was there in the first place.
>>
>> Any insight would be appreciated!
>
> It appears that this was added to prevent ppc64 reboot loops.
>
https://github.com/cobbler/cobbler/commit/bc76631ea158c76b1cdc8d3af28c9dc...
>
> I know that we had the potential for reboot loops even on regular x86
> hardware, but we dealt with it without code change. Our systems boot
> harddrive then netboot. So on initial install they netboot, once
> there is an OS they regular boot. If you need to re-install you
> either use koan or function key to select network boot at boot time.
>
> That being said, our systems also turn static post-install, so this
> change would not affect us. However, if we ran off DHCP period we
> would have the same issue you are experiencing with this.
>
> Maybe the better check would be:
>
> if not interface["netboot_enabled"] and interface['static']:
> continue
>
> ?
also, might be good to file it as a bug on github?
Will do.
I've been meaning to get started contributing to the project anyways (we
use IPv6 heavily and have some enhancements to Cobbler in that
area)...this is a good push to get me off my rear!
Best,
Dan