Harry Hoffman wrote:
> Hi,
>
> Just installed a new cobbler-1.2.1
>
> I have a dhcp.template and I run the following:
>
> [EMAIL PROTECTED] cobbler]# cobbler sync
>
> bootloaders
>    File "/usr/lib/python2.4/site-packages/cobbler/cobbler.py", line 77, 
> in main
>      rc = BootCLI().run(sys.argv)
>
>    File "/usr/lib/python2.4/site-packages/cobbler/cobbler.py", line 57, 
> in run
>      return self.loader.run(args)
>
>    File "/usr/lib/python2.4/site-packages/cobbler/commands.py", line 
> 113, in run
>      return fn.run()
>
>    File "/usr/lib/python2.4/site-packages/cobbler/modules/cli_misc.py", 
> line 223, in run
>      return self.api.sync()
>
>    File "/usr/lib/python2.4/site-packages/cobbler/api.py", line 381, in sync
>      return sync.run()
>
>    File "/usr/lib/python2.4/site-packages/cobbler/action_sync.py", line 
> 101, in run
>      self.dhcp.write_dhcp_file()
>
>    File 
> "/usr/lib/python2.4/site-packages/cobbler/modules/manage_isc.py", line 
> 157, in write_dhcp_file
>      elilo = os.path.basename(self.settings.bootloaders["ia64"])
>
>    File "/usr/lib/python2.4/site-packages/cobbler/settings.py", line 
> 149, in __getattr__
>      raise AttributeError, name
>
>
> Not sure what the problem is... any ideas?
>
> Cheers,
> Harry
> _______________________________________________
> cobbler mailing list
> [email protected]
> https://fedorahosted.org/mailman/listinfo/cobbler
>   

This problem will occur for new installs when someone has ISC dhcp 
management enabled.  If you have the previous (deprecated) "bootloader" 
setting in settings, it will not occur.  It has nothing to do with 
whether you have IA64 machines in your configuration.   While the 
functionality was removed, the piece of the code reading the setting was 
still there -- and my config file still had it present.

This should be the fix:

diff --git a/cobbler/modules/manage_isc.py b/cobbler/modules/manage_isc.py
index 1e9ddec..c982782 100644
--- a/cobbler/modules/manage_isc.py
+++ b/cobbler/modules/manage_isc.py
@@ -149,12 +149,7 @@ class IscManager:
         template_data = f2.read()
         f2.close()
 
-        # build each per-system definition
-        # as configured, this only works for ISC, patches accepted
-        # from those that care about Itanium.  elilo seems to be 
unmaintained
-        # so additional maintaince in other areas may be required to keep
-        # this working.
-        elilo = os.path.basename(self.settings.bootloaders["ia64"])
+        elilo = "/elilo-3.6-ia64.efi"
 
         # use a simple counter for generating generic names where a 
hostname
         # is not available



This will be applied to 1.2.3 -- which based on this and other items 
should go out Friday.

It seems that we need automated tests for clean installs that include 
DHCP and DNS management features.   Right now they do not do this (they 
mostly test the core), though I think that's definitely worth investing 
time in for future releases.

--Michael




_______________________________________________
cobbler mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/cobbler

Reply via email to