Fix for 766983 - Oz ignores <persisted>No</persisted> syntax in system templates

James Laska jlaska at redhat.com
Tue Dec 13 16:11:44 UTC 2011


Greetings,

Long story short, oz was expecting <persistent>, while the tdl.rng specifies
<persisted>.  I've corrected the problem, and added some additional changes
noted below.

> GIT: [PATCH 1/5] Include error message in exception

Minor unittest cleanup.  Sometimes it's not oz that failed to import, but
guestfs, numpy or m2crypto etc...  Displaying the error makes resolving missing
libraries sane.

> GIT: [PATCH 2/5] Use proper element name for persisted

Simple fix for bug#766983

> GIT: [PATCH 3/5] Raise exception instead of returning None

Minor change based on inspection.  Just some additional belts & suspenders.
This seems like an appropriate change, and causes no unittest regressions.

> GIT: [PATCH 4/5] Fix bug#766983 and add a unittest.

Add a new tdl unittest to test acceptable values for <persisted>.

> GIT: [PATCH 5/5] Add a log.debug call prior to _remove_repos

Minor change to assist debugging.  I found this handy while debugging this on a
running conductor system.

Thanks,
James




More information about the aeolus-devel mailing list