Thanks for reviewing, Ethan. I appreciate it.
Jack
On 06/28/12 02:20 PM, Ethan Quach wrote:
Jack,
This looks ok to me. Thanks for filing that new bug.
-ethan
On 06/24/12 14:38, Jack Schwartz wrote:
Hi everyone.
Here is a code review for:
7178811 installadm create-manifest fails when mac criteria is specified
http://monaco.us.oracle.com/detail.jsf?cr=7178811
Please review:
https://cr.opensolaris.org/action/browse/caiman/schwartz/7178811_1
Testing is listed in the bug report. Also checked pep8 and pylint.
This bug manifests a larger problem, for which there is not enough
time to solve correctly right now, but restores previously working
functionality. Read on for the gory details...
=====================
AI manifest criteria validation pandoras box was opened when I pushed
a change for:
7091202 installadm create-profile needs better messages for invalid
criteria
Currently criteria is validated against the only RelaxNG schema we have in our
gate. It is then massaged into the form it is stored in an AI database. This
is redundant; perhaps data can be validated as it is massaged. It is
inconsistent with the rest of the install tools, which do not use RelaxNG
schemas. Many of the values just pass through the schema as strings with a
kind of no-op validation. IMO this can and should be cleaned up.
... but not now. We are in the last non-stopper build. There is lots of
testing needed, since we need to account for criteria manifests as well as
criteria specified on the command-line. There isn't enough time to do it
right, right now.
I have opened bug:
7179350 Cleanup AI manifest criteria validation
to do this in update 2.
7178811 was caused by the fix for 7091202. The breakage was due to a switch in
the order of processing criteria: I switched massaging the data before doing
validation. (My testing missed a case which 7178811 brought out.) For now,
I'll recommend restoring the original order as it is low risk. Messages will
largely be as they have been (memory still gets more checking), but all else
should work as it did.
Thanks,
Jack
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
