diff options
author | Valerij Fredriksen <freva@users.noreply.github.com> | 2022-05-06 12:51:33 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-05-06 12:51:33 +0200 |
commit | 79d3d7d9af534d12bb9445f22245f9e48b4a2269 (patch) | |
tree | 26f72d4b3cd24ae21825c87aaeb1beefe9217646 /node-admin | |
parent | 4196e5235ba1df3cd5cdd198073f193f4cbc6c4f (diff) |
Revert "Set correct user on container crash path"
Diffstat (limited to 'node-admin')
-rw-r--r-- | node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/coredump/CoredumpHandler.java | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/coredump/CoredumpHandler.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/coredump/CoredumpHandler.java index ab4c993bab9..2271991fa15 100644 --- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/coredump/CoredumpHandler.java +++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/coredump/CoredumpHandler.java @@ -9,7 +9,6 @@ import com.yahoo.vespa.hosted.node.admin.nodeadmin.ConvergenceException; import com.yahoo.vespa.hosted.node.admin.nodeagent.NodeAgentContext; 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.file.UnixUser; import com.yahoo.vespa.hosted.node.admin.task.util.fs.ContainerPath; import com.yahoo.vespa.hosted.node.admin.task.util.process.Terminal; @@ -85,15 +84,9 @@ public class CoredumpHandler { public void converge(NodeAgentContext context, Supplier<Map<String, Object>> nodeAttributesSupplier, boolean throwIfCoreBeingWritten) { - ContainerPath containerCrashPath = context.paths().of(crashPatchInContainer, context.users().vespa()); + ContainerPath containerCrashPath = context.paths().of(crashPatchInContainer); ContainerPath containerProcessingPath = containerCrashPath.resolve(PROCESSING_DIRECTORY_NAME); - // TODO (freva): Remove after 7.584 - UnixUser vespaUser = context.users().vespa(); - UnixPath processingPath = new UnixPath(containerProcessingPath); - if (processingPath.getOwnerId() != vespaUser.uid()) processingPath.setOwnerId(vespaUser.uid()); - if (processingPath.getGroupId() != vespaUser.gid()) processingPath.setGroupId(vespaUser.gid()); - updateMetrics(context, containerCrashPath); if (throwIfCoreBeingWritten) { |