Evan Layton wrote: > Sundar Yamunachari wrote: >> Evan Layton wrote: >>> I need to get a review for this very simple (two line) fix for a >>> libzfs interface change that went into build 125. >>> >>> The bug: >>> http://defect.opensolaris.org/bz/show_bug.cgi?id=11827 >>> >>> The webrev: >>> http://cr.opensolaris.org/~evanl/11827 >>> >>> This bug will cause beadm activate to fail if there is an updated >>> grub version and installadm needs to be run. this is because >>> be_activate() uses the zpool_vdev_name() interface to find the >>> devices in the pool that need to have installgrub run on them. >>> >>> Thanks, >>> -evan >>> _______________________________________________ >>> caiman-discuss mailing list >>> caiman-discuss at opensolaris.org >>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >> Evan, >> >> The fix looks good. Are these only two places in libbe, the >> interface zpool_vdev_name() used? >> >> Thanks, >> Sundar > > Yes, It's only used here to get the devices fromt he pool > configuration so we know what to run installgrub on. Okay. Thanks.
- Sundar > > Thanks for looking at this. > > -evan
