Skip to content

NAS-139300 / 26.0.0-BETA.1 / Add dataset and relative_path fields to share entries#18021

Merged
creatorcary merged 20 commits intomasterfrom
smb-path
Feb 13, 2026
Merged

NAS-139300 / 26.0.0-BETA.1 / Add dataset and relative_path fields to share entries#18021
creatorcary merged 20 commits intomasterfrom
smb-path

Conversation

@creatorcary
Copy link
Contributor

@creatorcary creatorcary commented Jan 13, 2026

This represents the first change for NAS-137337. We may choose to do something similar for other services that use dataset paths.

  • Add "dataset" field which is calculated on create and update based on "path"
  • Add "relative_path" field which is calculated on create and update based on "path"
  • Keep "path" field to avoid concatenating the fields and prepending "/mnt/" when the full path is required
  • No extra mess in extend and compress
  1. Alembic migration adds columns to all SharingServiceTask services.
  2. migration/0018 attempts to populate the columns based on the existing path fields.
  3. pool.import and pool.dataset.unlock call hooks to query unresolved share paths and attempt to resolve them.

http://jenkins.eng.ixsystems.net:8080/job/tests/job/api_tests/7557/

@bugclerk
Copy link
Contributor

@bugclerk bugclerk changed the title Add dataset and relative_path fields to SMB share entries NAS-139300 / 26.04 / Add dataset and relative_path fields to SMB share entries Jan 13, 2026
@bugclerk bugclerk changed the title NAS-139300 / 26.04 / Add dataset and relative_path fields to SMB share entries NAS-139300 / 26.0.0-BETA.1 / Add dataset and relative_path fields to SMB share entries Jan 26, 2026
@creatorcary creatorcary changed the title NAS-139300 / 26.0.0-BETA.1 / Add dataset and relative_path fields to SMB share entries NAS-139300 / 26.0.0-BETA.1 / Add dataset and relative_path fields to share entries Feb 5, 2026
@creatorcary
Copy link
Contributor Author

friendly ping @anodos325 @sonicaj

@claude
Copy link

claude bot commented Feb 11, 2026

🤖 Claude is monitoring this PR

I'll review code, respond to comments, and investigate CI failures.
Start any comment with (skip) to have me ignore it.

  • Push fix commits (uncheck to only post review comments)

@truenas truenas deleted a comment from claude bot Feb 12, 2026
@@ -0,0 +1,317 @@
# CLAUDE.md
Copy link
Contributor

Choose a reason for hiding this comment

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

We should also probably give directive to use the etc plugin framework when generating config files. That means creating files in etc_files dir (should have one per file generated -- .py scripts should return bytes of file). Once etc files have been created then it should be inserted into appropriate etc plugin group (plugins/etc.py) for the particular area of code.

Also note to claude that atomic_write() should be used whenever there is a risk of data race on a file that would result in undefined application behavior.

Copy link
Contributor

Choose a reason for hiding this comment

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

I feel like this file is a little bit too long. In my experience, even 50-line CLAUDE.md gets blatantly ignored.

Do you use makefiles in your workflow for example?

etc files-related things should definitely go to CLAUDE.md inside etc files directory.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I created the file with claude init and then prompted claude to make any additions or amendments. I try not to edit memory files manually, figuring the agent knows what format it prefers to read from. The headings and subheadings should help it find the relevant info it needs, even for larger files.

@creatorcary creatorcary merged commit f121409 into master Feb 13, 2026
3 checks passed
@creatorcary creatorcary deleted the smb-path branch February 13, 2026 20:21
@bugclerk
Copy link
Contributor

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Feb 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants