Page MenuHomeFreeBSD
Feed Advanced Search

Nov 15 2021

markj closed D32521: cddl: fix missing ZFS library dependencies.
Nov 15 2021, 2:43 PM · ZFS

Nov 14 2021

greg_unrelenting.technology updated the diff for D32521: cddl: fix missing ZFS library dependencies.

Also add to Makefile.inc1 and SUBDIR_DEPEND_*. Should always work now.

Nov 14 2021, 3:19 PM · ZFS
bcr closed D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.
Nov 14 2021, 2:06 PM · ZFS
bcr added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Alright, I incorporated these last feedback items. I'm going to commit this shortly.
Thanks to all reviewers, your feedback and suggestions helped a lot in improving the text!

Nov 14 2021, 1:25 PM · ZFS

Nov 13 2021

pauamma added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

As far as I'm concerned, I think you can go ahead and commit at your discretion after this round of (very minor) suggestions. Getting to diminishing returns here.

Nov 13 2021, 9:38 PM · ZFS

Oct 30 2021

markj added a comment to D32521: cddl: fix missing ZFS library dependencies.

A buildworld fails after applying this patch to main, seems to be while linking libavl:

Oct 30 2021, 2:52 PM · ZFS

Oct 29 2021

bcr updated the diff for D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Change an instance of "alone" to "only" in the L2ARC description to not confuse people about adding multiple devices there.

Oct 29 2021, 6:58 PM · ZFS
bcr added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Reply with confirmation about the L2ARC vdevs question.

Oct 29 2021, 6:54 PM · ZFS

Oct 27 2021

pauamma added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

I'll have another look at the whole thing, but it will likely be a few days before I can.

Oct 27 2021, 8:15 PM · ZFS
bcr added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Update with some of my own replies.

Oct 27 2021, 7:31 PM · ZFS
bcr updated the diff for D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Yet another update addressing some of @pauamma_gundo.com's comments.

Oct 27 2021, 7:30 PM · ZFS
freqlabs accepted D32521: cddl: fix missing ZFS library dependencies.
Oct 27 2021, 3:06 PM · ZFS

Oct 19 2021

markj accepted D32521: cddl: fix missing ZFS library dependencies.

Looks right to me.

Oct 19 2021, 4:42 PM · ZFS

Oct 16 2021

greg_unrelenting.technology added a comment to D32521: cddl: fix missing ZFS library dependencies.

This looks good, but I'm surprised that there aren't more transitive dependencies that can be removed. For example, zfsd doesn't use libzfs_core directly, only through libzfs. Can libzfs_core be removed from its LIBADD?

Oct 16 2021, 6:58 PM · ZFS
greg_unrelenting.technology updated the diff for D32521: cddl: fix missing ZFS library dependencies.

Remove libzfs_core dep from zfsd

Oct 16 2021, 6:57 PM · ZFS
asomers added a comment to D32521: cddl: fix missing ZFS library dependencies.

This looks good, but I'm surprised that there aren't more transitive dependencies that can be removed. For example, zfsd doesn't use libzfs_core directly, only through libzfs. Can libzfs_core be removed from its LIBADD?

Oct 16 2021, 4:36 PM · ZFS
greg_unrelenting.technology updated the test plan for D32521: cddl: fix missing ZFS library dependencies.
Oct 16 2021, 4:36 PM · ZFS

Sep 27 2021

pauamma added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Aaaaaaaand done! (Split glossary to 1 sentence per line, please.)

Sep 27 2021, 2:01 AM · ZFS

Sep 26 2021

pauamma added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Mentioning it here to avoid repeating it a gazillion times: 1 sentence per line.

Sep 26 2021, 11:49 PM · ZFS

Sep 14 2021

eric_metricspace.net added a comment to D31725: Allow autounmountd to unload ZFS keys.
In D31725#720207, @rew wrote:

Sounds like a bug in the upstream module. I’d prefer to see that fixed rather than using autofs to work around the zfs_key pam module bug. Or see if it’s feasible to implement a pure autofs solution (i.e., autofs loads and unloads the key for a given dataset). If neither of the above options are possible, I’m more inclined to oppose this change until a more streamlined solution can be found.

Sep 14 2021, 12:54 AM · ZFS

Sep 12 2021

rew added a comment to D31725: Allow autounmountd to unload ZFS keys.

Sounds like a bug in the upstream module. I’d prefer to see that fixed rather than using autofs to work around the zfs_key pam module bug. Or see if it’s feasible to implement a pure autofs solution (i.e., autofs loads and unloads the key for a given dataset). If neither of the above options are possible, I’m more inclined to oppose this change until a more streamlined solution can be found.

Sep 12 2021, 8:30 PM · ZFS

Sep 10 2021

eric_metricspace.net added a comment to D31725: Allow autounmountd to unload ZFS keys.
In D31725#719491, @rew wrote:

As I explained in the description, this is intended to be used in conjunction with PAM to load a user's auth token as a key, allowing their encrypted home directory to be loaded when they log in, and unloaded once they've fully logged out. This in turn is a means to implement a common requirement on high-security systems.

According to the OpenZFS commit, the zfs_key pam module already unmounts the dataset and unloads the key when a session is closed.

For reference: https://github.com/openzfs/zfs/commit/221e67040fc47c15b3da2afb09bb48f1e9700fb9

Sep 10 2021, 12:52 AM · ZFS

Sep 9 2021

rew added a comment to D31725: Allow autounmountd to unload ZFS keys.

As I explained in the description, this is intended to be used in conjunction with PAM to load a user's auth token as a key, allowing their encrypted home directory to be loaded when they log in, and unloaded once they've fully logged out. This in turn is a means to implement a common requirement on high-security systems.

According to the OpenZFS commit, the zfs_key pam module already unmounts the dataset and unloads the key when a session is closed.

Sep 9 2021, 2:06 PM · ZFS
eric_metricspace.net added a comment to D31725: Allow autounmountd to unload ZFS keys.
In D31725#719350, @rew wrote:

Does it make sense to have autofs unload zfs keys if it doesn't even know how to load them?

Sep 9 2021, 1:09 PM · ZFS

Sep 8 2021

rew added a reviewer for D31725: Allow autounmountd to unload ZFS keys: trasz.

Does it make sense to have autofs unload zfs keys if it doesn't even know how to load them?

Sep 8 2021, 6:09 PM · ZFS

Sep 7 2021

eric_metricspace.net added a comment to D31725: Allow autounmountd to unload ZFS keys.

In general I don't think it's a good idea to add an option to opt-in unload of crypto keys: it should always be done unless explicitly opted out (or can't be opted out), so I'd recommend either removing the option and default to be safe, or make it an opt-out option (when specified, do not unload crypto key).

I'm not sure it's a good idea to abruptly change behavior in this way. I think it would be a better idea to introduce the ability as opt-in, and then change it to opt-out after announcing in advance.

I'm not very convinced that this is a behavior change (please do correct me if I was wrong). Here is my thought: the change seems to affect ZFS case only, and for ZFS the regular usage is that they are mounted by zfs mount -a, and my understanding is that this change is intended to be used with the PAM module to provide more automation to the load/unload key process.

If the assumption above was right, since there isn't current working way of loading ZFS encrypted dataset with this new workflow, we are not changing any existing behavior.

Sep 7 2021, 1:05 AM · ZFS
delphij added a comment to D31725: Allow autounmountd to unload ZFS keys.

In general I don't think it's a good idea to add an option to opt-in unload of crypto keys: it should always be done unless explicitly opted out (or can't be opted out), so I'd recommend either removing the option and default to be safe, or make it an opt-out option (when specified, do not unload crypto key).

I'm not sure it's a good idea to abruptly change behavior in this way. I think it would be a better idea to introduce the ability as opt-in, and then change it to opt-out after announcing in advance.

Sep 7 2021, 12:14 AM · ZFS

Sep 6 2021

eric_metricspace.net abandoned D31844: PAM module for loading ZFS keys on login.

Abandoning, due to the existence of the upstream module in cddl

Sep 6 2021, 11:16 PM · ZFS
eric_metricspace.net added a comment to D31725: Allow autounmountd to unload ZFS keys.

In general I don't think it's a good idea to add an option to opt-in unload of crypto keys: it should always be done unless explicitly opted out (or can't be opted out), so I'd recommend either removing the option and default to be safe, or make it an opt-out option (when specified, do not unload crypto key).

Sep 6 2021, 12:18 PM · ZFS
delphij requested changes to D31725: Allow autounmountd to unload ZFS keys.

In general I don't think it's a good idea to add an option to opt-in unload of crypto keys: it should always be done unless explicitly opted out (or can't be opted out), so I'd recommend either removing the option and default to be safe, or make it an opt-out option (when specified, do not unload crypto key).

Sep 6 2021, 12:43 AM · ZFS
delphij requested changes to D31844: PAM module for loading ZFS keys on login.

This proposed module have a lot of behavior that is very different from the Oracle and the OpenZFS implementation and since we are naming it the same way I don't think it's a good idea to diverge from the other implementations, especially the most commonly used parameters like homes= and the lack of mounting of the datasets.

Sep 6 2021, 12:27 AM · ZFS

Sep 5 2021

bcr updated the diff for D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Update including the lastest suggestions to the lower last part of the chapter.

Sep 5 2021, 5:34 PM · ZFS
bcr added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Another big update, including both @ygy's and @pauamma_gundo.com's comments.
I should have kept the diff much smaller in retrospect, sorry about that. We're getting close to the final piece now, thanks to your efforts!
The line breaks were caused by my editor trying to be smarter than it should be.

Sep 5 2021, 5:33 PM · ZFS
eric_metricspace.net updated the test plan for D31844: PAM module for loading ZFS keys on login.
Sep 5 2021, 1:49 PM · ZFS
eric_metricspace.net requested review of D31844: PAM module for loading ZFS keys on login.
Sep 5 2021, 1:41 PM · ZFS

Sep 4 2021

pauamma added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Taking a break before tackling the wall of text, but splitting lines into sentences first would make needed edits easier to spot. Pretty please?

Sep 4 2021, 1:37 AM · ZFS

Sep 3 2021

pauamma added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Hoping to finish reviewing it tonight. 4 installments is pushing it.

Sep 3 2021, 6:31 PM · ZFS
ygy added inline comments to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.
Sep 3 2021, 2:05 PM · ZFS
debdrup accepted D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

I've given it another once-over, and am pretty happy with it, so unless anyone else has any interjections, I say it's good to go.

Sep 3 2021, 11:46 AM · ZFS
bcr updated the diff for D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

New version that includes suggestions by @pauamma_gundo.com

Sep 3 2021, 11:29 AM · ZFS
bcr added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Phew, a lot of changes, most of which I concur with and changed. Thanks!
Updated patch follows.

Sep 3 2021, 11:28 AM · ZFS

Sep 1 2021

pauamma added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Taking me longer than i thought (when doesn't it for anyone?) but getting there.

Sep 1 2021, 4:49 AM · ZFS

Aug 30 2021

debdrup added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.
In D31707#715576, @bcr wrote:

That's right. My understanding is that everything is a dataset unless it was created by "zfs create -V ...", which for me is a volume (and used in this way). As you said, it may change or even blur some more with future changes. My idea is to use them consistently, for example mention datasets everywhere when it comes to ZFS features. When involving mountpoints, I'd refer to file systems, like this: "Mount the dataset as a file system into the directory tree." That way, we use it only when necessary, but keep the dataset syntax. Same when it involves volumes.

Aug 30 2021, 2:27 PM · ZFS
eric_metricspace.net requested review of D31725: Allow autounmountd to unload ZFS keys.
Aug 30 2021, 11:31 AM · ZFS
bcr added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

A few comments, mostly agreeing with the suggestion.

Aug 30 2021, 10:59 AM · ZFS
bcr updated the diff for D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Another fine set of changes, including most (if not all) suggestions by @pauamma_gundo.com

Aug 30 2021, 10:58 AM · ZFS
bcr added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.
In D31707#715576, @bcr wrote:

Maybe one more reason to use dataset more to avoid this ambiguity? ;-)

Calling them datasets introduces new ambiguities, because a dataset can both be a filesystem and a volume, and it's possible that it's definition will be expanded in future versions too.

Aug 30 2021, 9:32 AM · ZFS
pauamma added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Second installment. (Let me know if I'm too nitpicky or overwhelming.)

Aug 30 2021, 2:43 AM · ZFS

Aug 29 2021

debdrup added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.
In D31707#715576, @bcr wrote:

Maybe one more reason to use dataset more to avoid this ambiguity? ;-)

Aug 29 2021, 2:14 PM · ZFS
bcr updated the diff for D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Update diff to incorporate changes from @debdrup and @pauamma_gundo.com . Looking forward to the second half of changes. I know it's tedious and I should have kept it shorter. Will consider than for future changes like this on other chapters.

Aug 29 2021, 1:56 PM · ZFS
bcr added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Thanks for your suggestions, @debdrup and @pauamma_gundo.com . I incorporated a lot of them into the next set of changes.
I looked at our current docs for the word list on how to write "file system" versus "filesystem". It seems like it does not exist in the FDP primer anymore. It used to be there in the old one: https://www-legacy.freebsd.org/doc/en_US.ISO8859-1/books/fdp-primer/writing-style-word-list.html . Maybe one more reason to use dataset more to avoid this ambiguity? ;-)

Aug 29 2021, 1:53 PM · ZFS
pauamma added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Past 80 comments and I'm losing focus. Submitting what I have, will continue later.

Aug 29 2021, 12:47 AM · ZFS

Aug 28 2021

debdrup added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

These are a few things I spotted, but I'm not sure I caught everything and I'm on the wrong side of entirely sober. :)

Aug 28 2021, 8:30 PM · ZFS
bcr updated the diff for D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Updated diff incorporating suggestions by @ceri

Aug 28 2021, 7:43 PM · ZFS
bcr added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.
In D31707#715385, @ceri wrote:

Running out of battery so submitting what I have reviewed. Generally speaking I think the shorter text works against the intent of this being a guide, I'm afraid.

Aug 28 2021, 7:43 PM · ZFS
ceri added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Running out of battery so submitting what I have reviewed. Generally speaking I think the shorter text works against the intent of this being a guide, I'm afraid.

Aug 28 2021, 6:28 PM · ZFS
bcr updated the diff for D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

This one uses git diff --no-prefix -U1000

Aug 28 2021, 5:11 PM · ZFS
debdrup added a comment to D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.

Can you switch this diff to one with context? It makes it easier to make suggestions. :)

Aug 28 2021, 4:02 PM · ZFS
bcr requested review of D31707: Convert ZFS chapter to active voice and remove weasel/unnecessary words.
Aug 28 2021, 3:21 PM · ZFS

Jul 7 2021

mm added a reviewer for D31020: Attach zpool_influxdb to OpenZFS build: sjg.
Jul 7 2021, 6:28 AM · ZFS

Jul 6 2021

imp added a comment to D31020: Attach zpool_influxdb to OpenZFS build.

I always have to ask sjg. Maybe add him to the review?

Jul 6 2021, 3:55 PM · ZFS
freqlabs accepted D31020: Attach zpool_influxdb to OpenZFS build.

This looks good to me. I recall that Makefile.depend will be generated by whoever needs it and does not need to be committed if you aren't sure how to do it, but I don't see that in the committer's guide so I'm not sure where that idea came from. Hopefully someone else has input on that aspect.

Jul 6 2021, 3:51 PM · ZFS

Jul 4 2021

mm requested review of D31020: Attach zpool_influxdb to OpenZFS build.
Jul 4 2021, 9:13 PM · ZFS

Jun 16 2021

mm closed D30783: zfs: unbreak stable/13 clang build on i386 after b0c251b0d.
Jun 16 2021, 1:59 AM · ZFS
mm updated the diff for D30783: zfs: unbreak stable/13 clang build on i386 after b0c251b0d.

atomic.S patch by mav

Jun 16 2021, 1:55 AM · ZFS
mm added a comment to D30783: zfs: unbreak stable/13 clang build on i386 after b0c251b0d.

I am in for the assembler code patch. I will prepare an updated version.

Jun 16 2021, 1:35 AM · ZFS
mav accepted D30783: zfs: unbreak stable/13 clang build on i386 after b0c251b0d.

Actually my original version was written in assembler to be added into atomic.S without using the compiler atomics: https://people.freebsd.org/~mav/zzz3S.patch . I think it would be more consistent with original code. But since it is hopefully temporary, I am fine with any approach.

Jun 16 2021, 1:33 AM · ZFS
mm added a comment to D30783: zfs: unbreak stable/13 clang build on i386 after b0c251b0d.

I am definitely going to do that. But in the meantime we need a buildable stable/13 for i386. The solution I am proposing does (temporarily) place an additional file into cddl/lib/libspl what is outside of sys/contrib/openzfs.

Jun 16 2021, 1:27 AM · ZFS
mav added a comment to D30783: zfs: unbreak stable/13 clang build on i386 after b0c251b0d.

Could you ask him to merge the dependency instead of creating unneeded divergence, unless there is a good cause?

Jun 16 2021, 1:25 AM · ZFS
mm added a comment to D30783: zfs: unbreak stable/13 clang build on i386 after b0c251b0d.

I assume Brian just forgot to merge that commit or intentionally didn't merge that commit and did not inspect your commit for changes in that direction. The i386 build should be broken for Linux as well.

Jun 16 2021, 1:23 AM · ZFS
mav added a comment to D30783: zfs: unbreak stable/13 clang build on i386 after b0c251b0d.

Aaa... While the patch may be OK, I am surprised that my e76373de7b7384bb6e5c6fd5e04f15b54df20fb7 was merged into zfs-2.1-release before fef8bd41fc178d7212957b611c9bc81fe10cb63e. Merge in proper order would not cause this problem in first place. Do you have a clue why it was not done that way?

Jun 16 2021, 1:20 AM · ZFS
mm requested review of D30783: zfs: unbreak stable/13 clang build on i386 after b0c251b0d.
Jun 16 2021, 12:50 AM · ZFS

Jun 11 2021

heliocentric_gmail.com added a watcher for ZFS: heliocentric_gmail.com.
Jun 11 2021, 12:33 AM

May 19 2021

jhb accepted D30339: Retry OCF ENOMEM errors in ZFS.
May 19 2021, 11:14 PM · ZFS

May 18 2021

asomers accepted D30339: Retry OCF ENOMEM errors in ZFS.
May 18 2021, 8:28 PM · ZFS
mav updated the diff for D30339: Retry OCF ENOMEM errors in ZFS.
May 18 2021, 8:18 PM · ZFS
mav added inline comments to D30339: Retry OCF ENOMEM errors in ZFS.
May 18 2021, 8:15 PM · ZFS
asomers added inline comments to D30339: Retry OCF ENOMEM errors in ZFS.
May 18 2021, 7:56 PM · ZFS
markj accepted D30339: Retry OCF ENOMEM errors in ZFS.
May 18 2021, 7:54 PM · ZFS
mav requested review of D30339: Retry OCF ENOMEM errors in ZFS.
May 18 2021, 7:39 PM · ZFS

Mar 2 2021

freqlabs closed D28018: openzfs: attach pam_zfs_key to build.
Mar 2 2021, 12:27 PM · ZFS

Feb 28 2021

mm accepted D28018: openzfs: attach pam_zfs_key to build.
Feb 28 2021, 9:31 AM · ZFS
freqlabs added a reviewer for D28018: openzfs: attach pam_zfs_key to build: mm.
Feb 28 2021, 1:22 AM · ZFS

Feb 26 2021

mm closed D28935: zfs: fix handling of unsupported features.
Feb 26 2021, 11:07 PM · ZFS
freqlabs accepted D28935: zfs: fix handling of unsupported features.
Feb 26 2021, 6:22 PM · ZFS
mm requested review of D28935: zfs: fix handling of unsupported features.
Feb 26 2021, 10:59 AM · ZFS

Feb 1 2021

mm removed a reviewer for D28433: Update zfs_config.h to reflect OpenZFS gf11b09dec: mmacy.
Feb 1 2021, 9:05 PM · ZFS

Jan 31 2021

freqlabs accepted D28433: Update zfs_config.h to reflect OpenZFS gf11b09dec.

Thank you for catching this. The version strings look reasonable to me.

Jan 31 2021, 1:34 AM · ZFS
mm added a reviewer for D28433: Update zfs_config.h to reflect OpenZFS gf11b09dec: freqlabs.
Jan 31 2021, 1:26 AM · ZFS
mm requested review of D28433: Update zfs_config.h to reflect OpenZFS gf11b09dec.
Jan 31 2021, 1:24 AM · ZFS

Jan 7 2021

freqlabs accepted D28018: openzfs: attach pam_zfs_key to build.
Jan 7 2021, 10:04 PM · ZFS
greg_unrelenting.technology requested review of D28018: openzfs: attach pam_zfs_key to build.
Jan 7 2021, 11:20 AM · ZFS

Oct 22 2020

dch added a watcher for ZFS: dch.
Oct 22 2020, 8:37 AM

Aug 25 2020

emaste added a comment to D26183: ZFS: Unbreak the world build on CURRENT with the OpenZFS import.

I've been able to keep a NO_CLEAN tree going for several years; it does break occasionally but we have a standard, if ugly, way of addressing the resulting dependency issues when they occur.

Aug 25 2020, 11:45 PM · ZFS
mmacy closed D26183: ZFS: Unbreak the world build on CURRENT with the OpenZFS import.
Aug 25 2020, 11:36 PM · ZFS
mmacy added a comment to D26183: ZFS: Unbreak the world build on CURRENT with the OpenZFS import.

I will apply your patch, but it's a bit silly to call that "breaking world". -There's no guarantee that -DNO_CLEAN or -DNO_KERNELCLEAN will work across major changes.

Aug 25 2020, 11:31 PM · ZFS
mmacy added a comment to D26183: ZFS: Unbreak the world build on CURRENT with the OpenZFS import.
Aug 25 2020, 11:29 PM · ZFS
nc added a comment to D26183: ZFS: Unbreak the world build on CURRENT with the OpenZFS import.

Yes.

Aug 25 2020, 11:19 PM · ZFS
emaste added a comment to D26183: ZFS: Unbreak the world build on CURRENT with the OpenZFS import.

@neel_neelc.org was this a -DNO_CLEAN build?

Aug 25 2020, 11:17 PM · ZFS
imp accepted D26183: ZFS: Unbreak the world build on CURRENT with the OpenZFS import.

looks good to me.

Aug 25 2020, 9:43 PM · ZFS