Github-Mirror/seaweedfs
1
0
Fork 0
mirror of https://github.com/seaweedfs/seaweedfs.git synced 2026-09-14 02:20:41 +02:00
Code Issues Packages Projects Releases Wiki Activity

shell: reset noLock in the admin script dispatcher too

Browse Source 
Three dispatchers reuse one CommandEnv: the interactive shell, the master's
maintenance script runner, and the plugin worker's admin script handler. The
third is exempt anyway because it calls ForceNoLock, so this changes nothing
today -- it keeps the rule the same everywhere rather than resting on that
exemption staying in place.
This commit is contained in:
Chris Lu
2026-08-31 11:39:09 -07:00
parent 9ca9dd2379
commit 1cb659019e
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
weed/plugin/worker/admin_script_handler.go
+5
View File
@@ -290,6 +290,11 @@ func (h *AdminScriptHandler) Execute(ctx context.Context, request *plugin_pb.Exe
continue
}
found = true
// The env is reused for every command in the script, and noLock
// belongs to the invocation that set it. ForceNoLock already exempts
// this path, so this changes nothing today -- it keeps the rule the
// same in all three dispatchers rather than resting on that.
commandEnv.SetNoLock(false)
if err := command.Do(cmd.Args, commandEnv, output); err != nil {
msg := fmt.Sprintf("%s: %v", cmd.Name, err)
errorMessages = append(errorMessages, msg)