Re: [developer] Re: zpool create proposal summary

2019-03-25 Thread Rich
I think we're agreeing loudly - as I said, I have concerns about the implications of allowing it to be configurable, but if we did, we should probably blacklist overriding any hardcoded defines. Not allowing that also resolves that problem. - Rich On Mon, Mar 25, 2019 at 10:08 PM Josh Paetzel w

Re: [developer] Re: zpool create proposal summary

2019-03-25 Thread Josh Paetzel
On Mon, Mar 25, 2019, at 6:32 PM, Rich wrote: > I'm not usually in favor of limiting configurability, but in this > case, I agree with Joshua - letting people add their own custom > defines for e.g. debian-buster or epel6 or ubuntu1604 might be fine > (though at that point I'd still like somethi

Re: [developer] Re: zpool create proposal summary

2019-03-25 Thread Rich
I'm not usually in favor of limiting configurability, but in this case, I agree with Joshua - letting people add their own custom defines for e.g. debian-buster or epel6 or ubuntu1604 might be fine (though at that point I'd still like something like a constraint solver to say "I want this to work o

Re: [developer] Re: zpool create proposal summary

2019-03-25 Thread Joshua M. Clulow
On Mon, 25 Mar 2019 at 10:53, Gregor Kopka wrote: > Looks good. > > Only thing: please make this feature lookup the features for a set from a > plain text file (like in /etc/zfs/), just containing > features_name = comma, separated, list, of, feature, names > and not something compiled in. > It s

Re: [developer] zpool create proposal summary

2019-03-25 Thread Richard Laager
On 3/25/19 3:52 PM, Matthew Ahrens wrote: > The distros can lag arbitrarily far behind, and we have no control over > them. They can lag arbitrarily far behind in theory, but in practice the lag will be bounded. If one of the "tier 1" distros stops updating their ZFS packaging in a timely manner,

Re: [developer] zpool create proposal summary

2019-03-25 Thread Matthew Ahrens
On Mon, Mar 25, 2019 at 1:43 PM Richard Laager wrote: > On 3/25/19 8:15 AM, Josh Paetzel wrote: > > "-o features=portable" means that features that are currently supported > by all tier-1 platforms. This definition will change over time. > Specifically, as of the current date, the following all

Re: [developer] zpool create proposal summary

2019-03-25 Thread Richard Laager
On 3/25/19 8:15 AM, Josh Paetzel wrote: > Portability of boot pools is currently not a consideration. Dealing with this for boot pools is a serious and timely concern of mine. I use Ubuntu, which uses GRUB. It is clear that, in practice, GRUB is not going to be kept up-to-date with new OpenZFS fea

Re: [developer] zpool create proposal summary

2019-03-25 Thread Richard Laager
On 3/25/19 8:15 AM, Josh Paetzel wrote: > "-o features=portable" means that features that are currently supported by > all tier-1 platforms. This definition will change over time. Specifically, > as of the current date, the following all supported the features: >- a numbered release of Free

Re: [developer] zpool create proposal summary

2019-03-25 Thread Rich
First, let me say I'm so pleased someone actually drove this forward, thank you. :) I think my only lament about this would be the lack of zpool create defaults, but that's a conversation for once everyone is confident this doesn't cook your pool somehow. - Rich On Mon, Mar 25, 2019 at 9:16 AM J

[developer] Re: zpool create proposal summary

2019-03-25 Thread Gregor Kopka
Looks good. Only thing: *please* make this feature lookup the features for a set *from a plain text file* (like in /etc/zfs/), just containing features_name = comma, separated, list, of, feature, names and *not* something compiled in. It should be easy to define custom feature sets, this would

[developer] zpool create proposal summary

2019-03-25 Thread Josh Paetzel
This email is a summarization of the current proposal for easing the task of creating OS portable pools with zpool create. There's been a ton of discussion around this feature and we wanted to summarize the current discussion in preparation for tomorrow's meeting: Add new options to "zpool cre