<div dir="ltr">Ok, a first stab at this is at <a href="https://salsa.debian.org/zfsonlinux-team/zfs/-/merge_requests/25">https://salsa.debian.org/zfsonlinux-team/zfs/-/merge_requests/25</a><div><br></div><div>Note: I'm using a zpool property ("inhibitupgrade") instead of a dataset user property, because getting to dataset properties (especially user properties) from the zpool data structure seems to be remarkably tedious. If there's a straightforward idiomatic way to do so, that might still be a better approach. But given that this property changes the behavior of the zpool command, perhaps pool properties are the right place for this flag.</div><div><br></div><div>This quick test.sh script attached seems to do the right thing - see text.txt output.</div><div><br></div><div>Thoughts and comments welcome,</div><div><br></div><div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div>Colm</div></blockquote></div><div><br></div><div><br></div><div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 18 Aug 2020 at 08:38, Richard Laager <<a href="mailto:rlaager@wiktel.com">rlaager@wiktel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 8/14/20 10:09 AM, Colm Buckley wrote:<br>
> I see what Ubuntu were trying to achieve with<br>
> 4100-disable-bpool-upgrade.patch, but I really don't like the hardcoding<br>
> of the "bpool" name into the tools like this.<br>
> <br>
> Would it be reasonable to adopt this patch (changing the behavior of<br>
> `zpool status` and `zpool upgrade`) if instead of checking for<br>
> name='bpool', it checked for a property; eg: if<br>
> org.debian:grub-workaround = true<br>
Yes, a property would be better. We can bikeshed the name and value. ;)<br>
<br>
I'd rather avoid words like "workaround".<br>
<br>
Conventionally, ZFS properties tend to be on/off rather than true/false.<br>
<br>
Maybe org.debian:grub-features=on?<br>
<br>
The long-term upstream plan was to have a "features mask" function, such<br>
that you could do something like `zpool set features=grub bpool`, which<br>
would mask the features (for `zpool upgrade` and the warning in `zpool<br>
status`) to the subset supported by grub. Nobody is currently working on<br>
that.<br>
<br>
Maybe org.debian:features=grub?<br>
<br>
-- <br>
Richard<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><span style="color:rgb(153,153,153);font-size:x-small">Colm Buckley | </span><a href="mailto:colm@tuatha.org" style="font-size:x-small" target="_blank">colm@tuatha.org</a><div><br></div></div></div></div></div>