Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't count '@' for dataset namelen if not a snapshot #5456

Merged
merged 2 commits into from
Dec 9, 2016

Conversation

tuxoko
Copy link
Contributor

@tuxoko tuxoko commented Dec 5, 2016

This fix pool unimportable when dataset namelen is 255



Signed-off-by: Chunwei Chen [email protected]

Fix #5432

This fix pool unimportable when dataset namelen is 255

Signed-off-by: Chunwei Chen <[email protected]>
Fix openzfs#5432
@mention-bot
Copy link

@tuxoko, thanks for your PR! By analyzing the history of the files in this pull request, we identified @ikozhukhov, @ahrens and @behlendorf to be potential reviewers.

Copy link
Contributor

@dinatale2 dinatale2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! The code looks good to me. Would you mind adding a simple test case to test for this? Thank you!

Copy link
Contributor

@behlendorf behlendorf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding your original test case to the suite suite would be ideal.

@tuxoko
Copy link
Contributor Author

tuxoko commented Dec 6, 2016

http://build.zfsonlinux.org/builders/CentOS%207.1%20x86_64%20%28TEST%29/builds/3758/steps/shell_11/logs/stdio
Warning: Test 'zfs_create_014_pos' removed from TestGroup '/usr/share/zfs/zfs-tests/tests/functional/cli_root/zfs_create' because it failed verification.?

@behlendorf
Copy link
Contributor

Usually this is due to the test case mode bits. Make sure it's executable.

@tuxoko
Copy link
Contributor Author

tuxoko commented Dec 6, 2016

Oh, I forgot add it to Makefile.am

@behlendorf behlendorf merged commit a806cb6 into openzfs:master Dec 9, 2016
ironMann pushed a commit to ironMann/zfs that referenced this pull request Dec 13, 2016
Don't count '@' for dataset namelen if not a snapshot.  This
fixes making a pool unimportable when the  dataset namelen
is 255.

Add test file for zfs create name length 255.

Reviewed-by: Giuseppe Di Natale <[email protected]>
Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Chunwei Chen <[email protected]>
Closes openzfs#5432 
Closes openzfs#5456
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request Feb 2, 2017
Don't count '@' for dataset namelen if not a snapshot.  This
fixes making a pool unimportable when the  dataset namelen
is 255.

Add test file for zfs create name length 255.

Reviewed-by: Giuseppe Di Natale <[email protected]>
Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Chunwei Chen <[email protected]>
Closes openzfs#5432 
Closes openzfs#5456
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request Feb 2, 2017
Don't count '@' for dataset namelen if not a snapshot.  This
fixes making a pool unimportable when the  dataset namelen
is 255.

Add test file for zfs create name length 255.

Reviewed-by: Giuseppe Di Natale <[email protected]>
Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Chunwei Chen <[email protected]>
Closes openzfs#5432
Closes openzfs#5456
Requires-builders: style
behlendorf pushed a commit that referenced this pull request Feb 3, 2017
Don't count '@' for dataset namelen if not a snapshot.  This
fixes making a pool unimportable when the  dataset namelen
is 255.

Add test file for zfs create name length 255.

Reviewed-by: Giuseppe Di Natale <[email protected]>
Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Chunwei Chen <[email protected]>
Closes #5432
Closes #5456
wli5 pushed a commit to wli5/zfs that referenced this pull request Feb 28, 2017
Don't count '@' for dataset namelen if not a snapshot.  This
fixes making a pool unimportable when the  dataset namelen
is 255.

Add test file for zfs create name length 255.

Reviewed-by: Giuseppe Di Natale <[email protected]>
Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Chunwei Chen <[email protected]>
Closes openzfs#5432 
Closes openzfs#5456
wli5 pushed a commit to wli5/zfs that referenced this pull request Feb 28, 2017
Don't count '@' for dataset namelen if not a snapshot.  This
fixes making a pool unimportable when the  dataset namelen
is 255.

Add test file for zfs create name length 255.

Reviewed-by: Giuseppe Di Natale <[email protected]>
Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Chunwei Chen <[email protected]>
Closes openzfs#5432
Closes openzfs#5456
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a dataset with name length 255 cause pool unimportable
4 participants