Skip to content

zts: add spdx license tags to gang_blocks tests #17160

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

Merged
merged 1 commit into from
Mar 20, 2025

Conversation

robn
Copy link
Member

@robn robn commented Mar 20, 2025

[Sponsors: Klara, Inc., Wasabi Technology, Inc.]

Motivation and Context

checkstyle failed on unrelated files.

Looks like some missing tags in #17073, probably because that PR was branched before #17001 was landed and never rebased.

Description

Add some tags.

How Has This Been Tested?

Ran spdxcheck, then checkstyle. They did not complain.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance enhancement (non-breaking change which improves efficiency)
  • Code cleanup (non-breaking change which makes code smaller or more readable)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Library ABI change (libzfs, libzfs_core, libnvpair, libuutil and libzfsbootenv)
  • Documentation (a change to man pages or other documentation)

Checklist:

Missed in openzfs#17073, probably because that PR was branched before openzfs#17001
was landed and never rebased.

Sponsored-by: Klara, Inc.
Sponsored-by: Wasabi Technology, Inc.
Signed-off-by: Rob Norris <rob.norris@klarasystems.com>
@robn robn force-pushed the zts-gang-blocks-missing-spdx-tags branch from c98ae18 to 51b4680 Compare March 20, 2025 00:10
@robn robn changed the title zts: add spdx licents tags to gang_blocks tests zts: add spdx license tags to gang_blocks tests Mar 20, 2025
@robn robn requested a review from tonyhutter March 20, 2025 00:12
@amotin amotin added Status: Code Review Needed Ready for review and testing Status: Accepted Ready to integrate (reviewed, tested) and removed Status: Code Review Needed Ready for review and testing labels Mar 20, 2025
@github-actions github-actions bot removed the Status: Accepted Ready to integrate (reviewed, tested) label Mar 20, 2025
@amotin amotin added the Status: Accepted Ready to integrate (reviewed, tested) label Mar 20, 2025
@tonyhutter tonyhutter merged commit 5b5a514 into openzfs:master Mar 20, 2025
32 of 40 checks passed
fuporovvStack pushed a commit to fuporovvStack/zfs that referenced this pull request Apr 11, 2025
Missed in openzfs#17073, probably because that PR was branched before openzfs#17001
was landed and never rebased.

Sponsored-by: Klara, Inc.
Sponsored-by: Wasabi Technology, Inc.

Signed-off-by: Rob Norris <rob.norris@klarasystems.com>
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by: Tony Hutter <hutter2@llnl.gov>
fuporovvStack pushed a commit to fuporovvStack/zfs that referenced this pull request Apr 11, 2025
Missed in openzfs#17073, probably because that PR was branched before openzfs#17001
was landed and never rebased.

Sponsored-by: Klara, Inc.
Sponsored-by: Wasabi Technology, Inc.

Signed-off-by: Rob Norris <rob.norris@klarasystems.com>
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by: Tony Hutter <hutter2@llnl.gov>
ixhamza pushed a commit to truenas/zfs that referenced this pull request May 14, 2025
Missed in openzfs#17073, probably because that PR was branched before openzfs#17001
was landed and never rebased.

Sponsored-by: Klara, Inc.
Sponsored-by: Wasabi Technology, Inc.

Signed-off-by: Rob Norris <rob.norris@klarasystems.com>
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by: Tony Hutter <hutter2@llnl.gov>
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request Jun 13, 2025
Missed in openzfs#17073, probably because that PR was branched before openzfs#17001
was landed and never rebased.

Sponsored-by: Klara, Inc.
Sponsored-by: Wasabi Technology, Inc.

Signed-off-by: Rob Norris <rob.norris@klarasystems.com>
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by: Tony Hutter <hutter2@llnl.gov>
behlendorf pushed a commit to behlendorf/zfs that referenced this pull request Jun 16, 2025
Missed in openzfs#17073, probably because that PR was branched before openzfs#17001
was landed and never rebased.

Sponsored-by: Klara, Inc.
Sponsored-by: Wasabi Technology, Inc.

Signed-off-by: Rob Norris <rob.norris@klarasystems.com>
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by: Tony Hutter <hutter2@llnl.gov>
@behlendorf behlendorf mentioned this pull request Jun 16, 2025
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Accepted Ready to integrate (reviewed, tested)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants