- User Since
- Nov 14 2018, 8:11 PM (17 w, 5 d)
Fri, Mar 8
Thu, Mar 7
The channel programs look like a cool feature - I have a few questions about it:
Mon, Mar 4
I'm not sure why the history is clobbered for this updated diff revision with regards to target_prog.c being copied from be_impl.h.
Thu, Feb 28
Sun, Feb 24
A question about the added files - am I supposed to include the copyright/license information or...?
Fri, Feb 22
Feb 14 2019
Jan 4 2019
Do you use IRC or anything of the sort? Some of this might be easier to hash out in real-time, and we can summarize the results of said discussion back in this review.
Dec 27 2018
It's probably poor style to change the user interface for bectl but, it's
already recursive by default - could always ignore the '-r' flag and add a flag
for shallow creation. Upside here is the current behavior remains unchanged.
Downside being, confusion for current users (i.e. the shell game of
Dec 26 2018
I agree, the default be_create is adequate for most cases. If anything, I would lean
towards a be_create_shallow, here's my thinking. be_create is, effectively,
a recursive snapshot that gets cloned - which is consistent with the zfs
behavior of 'zfs snapshot -r'. be_create_shallow would be consistent with
a 'zfs snapshot', minus the recursive flag.
Dec 22 2018
Added tests in sbin/bectl/test and fixed styling issues.