Hopefully this will be the last self-reply...
1) I have more time than I thought for code review, so take until Friday
7/9 COB if needed.
2) This code review fixes
14223 BEADM_ERR_BE_DOES_NOT_EXIST has an extra space
as well as the other bugs listed.
Thanks,
Jack
On 07/ 1/10 03:02 PM, Jack Schwartz wrote:
Hi everyone.
Apparently the original link looked OK but some had trouble with it.
Try this one instead please:
http://cr.opensolaris.org/~schwartz/100701.1/webrev/
Thanks,
Jack
On 07/ 1/10 11:26 AM, Jack Schwartz wrote:
Hi everyone.
Here is a webrev for the following beadm or libbe library bugfixes:
6495 <http://defect.opensolaris.org/bz/show_bug.cgi?id=6495> libbe
should capture and give useful error when installgrub, or ict.py fails.
15375 <http://defect.opensolaris.org/bz/show_bug.cgi?id=15375> libbe
changes to utilize new versioning features of installboot and installgrub
15404 <http://defect.opensolaris.org/bz/show_bug.cgi?id=15404> beadm
activate should run installboot on SPARC
16409 <http://defect.opensolaris.org/bz/show_bug.cgi?id=16409> libbe
error messages need internationalization
16429 <http://defect.opensolaris.org/bz/show_bug.cgi?id=16429> Remove
be_get_last_zone_be_callback
http://cr.opensolaris.org/~schwartz/100701.1/webrev/
Most of these changes are needed as part of the Snap -> ON project.
Note that 15375 and 15404 depend on installboot and installgrub
changes being made by another team (6944352). I have been using test
versions of installboot and installgrub to verify my changes.
Tested on SPARC and X86 by creating and then activating new BEs, then
running installboot or installgrub on the same BEs and looking for
messages that show the right thing was done by beadm. This done with
and without BE_PRINT_ERR set to true.
Please send your feedback by Tuesday 7/6 COB.
Thanks,
Jack
P.S. Note: even though these changes are being reviewed against
slim_source for clarity, they will be going back to Glenn's ON-based
gate for putback to ON.
P.P.S. Thanks in advance to Evan for agreeing to review these changes.
_______________________________________________
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
