[illumos-Developer] Review 1116,1208, 1154

[Alexander Eremin]

On Jul 14, 2011, at 10:44 PM, Gordon Ross wrote:

> On Thu, Jul 14, 2011 at 10:49 AM, Alexander Eremin
> <alexander.eremin at nexenta.com> wrote:
>> I'd like review for next fixes:
>> 1116 beadm *_snap commands shouldn't exist
>> 1208 beadm doesn't show BE snapshots
>> webrev: http://cr.illumos.org/view/7zjkkv35/
>
> That webrev looks fine.  Could you please put some test example  
> outputs in 1208?
> (i.e. before / after the fix)
>
> Thanks,
> Gordon

Sure.

before:
# beadm list -s lx
BE/Snapshot                        Space Policy Created
lx
#

after:
# beadm list -s lx
BE/Snapshot                        Space Policy Created
lx
   lx at 2011-07-13-15:06:03          0     static 2011-07-13 19:06
   lx at 2011-07-13-16:15:23          0     static 2011-07-13 20:15
   lx at bla                          0     static 2011-07-13 20:16
   lx at test                         0     static 2011-07-13 18:39
   lx at test3                        0     static 2011-07-13 20:07
#

Thanks,
:: Alexander