diff options
author | Valerij Fredriksen <valerijf@yahooinc.com> | 2022-04-20 10:29:04 +0200 |
---|---|---|
committer | Valerij Fredriksen <valerijf@yahooinc.com> | 2022-04-20 10:29:04 +0200 |
commit | 2d111efed5a1c2a41fa5a047bdaae98d1b4ac74e (patch) | |
tree | 95622c209c935cc6bb599011fa9073438b4d48db /node-admin | |
parent | 4aa2d5bdecc22607b66ba5988a9812e2a3028e1f (diff) |
Chmod all files in container storage
Diffstat (limited to 'node-admin')
-rw-r--r-- | node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java index 454dafb2384..75977da369c 100644 --- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java +++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java @@ -29,6 +29,7 @@ import com.yahoo.vespa.hosted.node.admin.maintenance.servicedump.VespaServiceDum import com.yahoo.vespa.hosted.node.admin.nodeadmin.ConvergenceException; import com.yahoo.vespa.hosted.node.admin.task.util.file.FileFinder; import com.yahoo.vespa.hosted.node.admin.task.util.file.UnixPath; +import com.yahoo.vespa.hosted.node.admin.task.util.fs.ContainerPath; import java.time.Clock; import java.time.Duration; @@ -141,7 +142,8 @@ public class NodeAgentImpl implements NodeAgent { // TODO: Remove after this has rolled out everywhere int[] stats = new int[]{0, 0, 0}; - FileFinder.files(initialContext.paths().underVespaHome("")).forEachPath(path -> { + ContainerPath vespaHome = initialContext.paths().underVespaHome(""); + FileFinder.files(initialContext.paths().of("/")).forEachPath(path -> { UnixPath unixPath = new UnixPath(path); String permissions = unixPath.getPermissions(); @@ -150,12 +152,12 @@ public class NodeAgentImpl implements NodeAgent { stats[0]++; } - if (unixPath.getOwnerId() != initialContext.users().vespa().uid()) { + if (path.startsWith(vespaHome) && unixPath.getOwnerId() != initialContext.users().vespa().uid()) { unixPath.setOwnerId(initialContext.users().vespa().uid()); stats[1]++; } - if (unixPath.getGroupId() != initialContext.users().vespa().gid()) { + if (path.startsWith(vespaHome) && unixPath.getGroupId() != initialContext.users().vespa().gid()) { unixPath.setGroupId(initialContext.users().vespa().gid()); stats[2]++; } |