Upstream `zfs send -V` -- replication status reporting in process title.
We have this code for years, but in context of ZoL integration would be good to upstream it, if possible.
- Status changed from Screened to Unscreened
- Assignee changed from Alexander Motin to Sean Fagan
- Target version changed from Backlog to 12.0
Sean, could you look on this code. IIRC it is custom to our tree for years. We need to revisit it, possibly polish and try to push upstream (ZoL and/or FreeBSD). Otherwise, if it is bad, we will have to cary it along, that is inconvenient.
That was added in commit 31bfc932ac26582a7a493b980bab31969404fcbe (by Xin); this also added the property filtering code. I don't think we want to do all of that, since I seem to recall filtering is either in ZoL from someone else, or there's a PR for it. The setproctitle part is easily separated, however.
Thoughts on that?
Right, the "redacted send/receive" -- that's what I was thinking of. As I recall, Matt expressed a desire to have it pushed back until after the crypto code got in, since both made significant changes to the send/receive code, and it'd probably be better to do the redaction after the crypto code.
That commit is incomplete -- it doesn't have the code to call setproctitle(), for example. But the one it was cherry-picked from -- 13cc5237d6dbed102722cf4d3655783ee7e32fa5 -- does. And doesn't have as much as the one I found via git blame.
So I think that's the one to use.