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