diff options
author | Valerij Fredriksen <valerijf@yahooinc.com> | 2022-05-06 13:01:19 +0200 |
---|---|---|
committer | Valerij Fredriksen <valerijf@yahooinc.com> | 2022-05-06 13:01:19 +0200 |
commit | 01b97a4e16a6dad1bb21c5752c1201bdcebe456b (patch) | |
tree | 16f44134e8a22f14c82509d954a55e4ea91d4260 /node-admin | |
parent | 95880e2aec2acd6ffe78b7e43d7ed58cda5c79b3 (diff) |
Only chown if exists
Diffstat (limited to 'node-admin')
-rw-r--r-- | node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/maintenance/coredump/CoredumpHandler.java | 6 |
1 files changed, 4 insertions, 2 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..0d0b8bd335e 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 @@ -91,8 +91,10 @@ public class CoredumpHandler { // 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()); + processingPath.getAttributesIfExists().ifPresent(attr -> { + if (attr.ownerId() != vespaUser.uid()) processingPath.setOwnerId(vespaUser.uid()); + if (attr.groupId() != vespaUser.gid()) processingPath.setGroupId(vespaUser.gid()); + }); updateMetrics(context, containerCrashPath); |