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. Thanks for looking at this. -evan -evan
