Page MenuHomeFreeBSD
Differential D51022

Document verbatim import mode for zpool import.
AbandonedPublic
Actions

Authored by felix.the.red_gmail.com on Jun 24 2025, 9:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Sep 5, 3:55 AM
Unknown Object (File)
Fri, Sep 4, 9:35 PM
Unknown Object (File)
Fri, Sep 4, 6:22 PM
Unknown Object (File)
Thu, Sep 3, 9:29 PM
Unknown Object (File)
Thu, Sep 3, 2:15 AM
Unknown Object (File)
Wed, Sep 2, 12:33 PM
Unknown Object (File)
Wed, Sep 2, 8:43 AM
Unknown Object (File)
Tue, Sep 1, 7:21 PM
View All 80 Files
Subscribers
crest_freebsd_rlwinm.de
delphij
imp
rob.norris_klarasystems.com
ziaee

Details

Reviewers
None
Group Reviewers
docs
Test Plan

PR:162775

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

felix.the.red_gmail.com created this revision.Jun 24 2025, 9:53 PM
Herald added subscribers: ziaee, delphij, imp. · View Herald TranscriptJun 24 2025, 9:53 PM
felix.the.red_gmail.com requested review of this revision.Jun 24 2025, 9:53 PM
rob.norris_klarasystems.com added a subscriber: rob.norris_klarasystems.com.Jun 24 2025, 10:48 PM
ziaee added a comment.EditedJun 25 2025, 4:58 AM

I think this review needs to be closed and submitted upstream at https://github.com/openzfs/zfs

ziaee added a comment.Jun 25 2025, 9:15 PM

Excuse me, I was going through my emails too fast. This was from a previous implementation of ZFS, before we switched to OpenZFS.

felix.the.red_gmail.com added a comment.Jun 25 2025, 10:25 PM

What's the path forward with this? Should I submit it to upstream? Will you close the PR?

ziaee added a comment.Jun 25 2025, 10:44 PM

Yes, I will close the PR, and you should use the option to abandon this revision. Thank you, and sorry for the wasted time.

crest_freebsd_rlwinm.de added a subscriber: crest_freebsd_rlwinm.de.Jun 26 2025, 12:24 AM

Creating a review for a patch from 2011 in a PR that didn't get resolved in time doesn't make sense. The patch is against the old FreeBSD ZFS fork. FreeBSD has since replaced this branch of ZFS with OpenZFS.

felix.the.red_gmail.com added a comment.Jun 26 2025, 12:30 AM

I did not know that history when I started.
I often only have what is in the PR to work with.
I will abandon this revision.

felix.the.red_gmail.com abandoned this revision.Jun 26 2025, 12:30 AM

Overtaken by events

Revision Contents
Changeset List

PathSize
sys/
contrib/
openzfs/
man/
man8/
17 lines

Diff 157547

View Options

sys/contrib/openzfs/man/man8/zpool-import.8

Loading...