Re: [et-mgmt-tools] koan 0.2.9 parsing problem (safe_load function)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Works like a charm w/ centos-5!!! Thanks, Michael!  :)

On 4/23/07, Michael DeHaan <mdehaan@xxxxxxxxxx> wrote:
Adam Rosenwald wrote:
> New issue with cobbler 0.4.7 and koan 0.2.9.
>
> After installing cobbler 0.4.7 (from RPM), I made `cobbler check`
> happy, and then proceeded with provisioning `cobbler system add
> --name=00:16:3E:35:00:71 <--additional items>`.  No problem.  The
> system profile was allocated properly.
>
> Then, I ensure cobblerd was running.  No problem with XMLRPC port.
>
> I had a dom0 server on which I installed koan 0.2.9 (from RPM).
>
> Then...
>
> ====BEGIN OUTPUT=====
>
> ... SNIP ....
>
> =======END OUTPUT==========
>
> It appears that 'profile' is beting passed as a hash to safe_load(),
> yet there is something wrong with the hash.
>
> This problem does not occur when running koan with
> --profile="" rather it only occurs with
> --system=<system_name>.
>
> Any thoughts?  Thanks. --A.
> ------------------------------------------------------------------------
>
> _______________________________________________
> et-mgmt-tools mailing list
> et-mgmt-tools@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/et-mgmt-tools

Fix:

diff --git a/koan/app.py b/koan/app.py
index 1a9205a..08d4ee6 100755
--- a/koan/app.py
+++ b/koan/app.py
@@ -524,7 +524,7 @@ class Koan:
         system_data = None
         self.debug("fetching configuration for system: %s" %
old_system_name)
         try:
-            data = "">+            system_data =
self.xmlrpc_server.get_system_for_koan(system_name)
         except:
             self.connect_fail()
         profile_data =
self.get_profile_xmlrpc(self.safe_load(system_data,'profile'))


I'll push a koan update shortly to include this (probably tomorrow) ...
I want to see if any other sparks fly first.

--Michael



_______________________________________________
et-mgmt-tools mailing list
et-mgmt-tools@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/et-mgmt-tools


[Index of Archives]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux