NAS-139300 / 26.0.0-BETA.1 / Add dataset and relative_path fields to share entries#18021
NAS-139300 / 26.0.0-BETA.1 / Add dataset and relative_path fields to share entries#18021creatorcary merged 20 commits intomasterfrom
Conversation
src/middlewared/middlewared/alembic/versions/26.04/2025-12-31_15-39_split_smb_path.py
Outdated
Show resolved
Hide resolved
|
friendly ping @anodos325 @sonicaj |
|
🤖 Claude is monitoring this PR I'll review code, respond to comments, and investigate CI failures.
|
| @@ -0,0 +1,317 @@ | |||
| # CLAUDE.md | |||
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
|
This PR has been merged and conversations have been locked. |
This represents the first change for NAS-137337. We may choose to do something similar for other services that use dataset paths.
extendandcompressSharingServiceTaskservices.pool.importandpool.dataset.unlockcall hooks to query unresolved share paths and attempt to resolve them.http://jenkins.eng.ixsystems.net:8080/job/tests/job/api_tests/7557/