[Elastic-Agent] Do not take ownership of Endpoint log path (elastic#2…

…3444)

* Do not take ownership when custom log path is defined in spec.

* Add changelog.

* Empty commit.

(cherry picked from commit 71e406b)

x-pack/elastic-agent/CHANGELOG.next.asciidoc

@@ -30,6 +30,7 @@
 - Fix shell wrapper for deb/rpm packaging {pull}23038[23038]
 - Fixed parsing of npipe URI {pull}22978[22978]
 - Remove artifacts on transient download errors {pull}23235[23235]
+- Do not take ownership of Endpoint log path {pull}23444[23444]
 
 ==== New features
 

x-pack/elastic-agent/pkg/core/monitoring/beats/beats_monitor.go

@@ -85,7 +85,12 @@ func (b *Monitor) generateLoggingFile(spec program.Spec, pipelineID string) stri
 
 func (b *Monitor) generateLoggingPath(spec program.Spec, pipelineID string) string {
 	return filepath.Dir(b.generateLoggingFile(spec, pipelineID))
+}
 
+func (b *Monitor) ownLoggingPath(spec program.Spec) bool {
+	// if the spec file defines a custom log path then agent will not take ownership of the logging path
+	_, ok := spec.LogPaths[b.operatingSystem]
+	return !ok
 }
 
 // EnrichArgs enriches arguments provided to application, in order to enable
@@ -139,6 +144,7 @@ func (b *Monitor) Cleanup(spec program.Spec, pipelineID string) error {
 
 // Prepare executes steps in order for monitoring to work correctly
 func (b *Monitor) Prepare(spec program.Spec, pipelineID string, uid, gid int) error {
+	takeOwnership := b.ownLoggingPath(spec)
 	drops := []string{b.generateLoggingPath(spec, pipelineID)}
 	if drop := b.monitoringDrop(spec, pipelineID); drop != "" {
 		drops = append(drops, drop)
@@ -161,8 +167,10 @@ func (b *Monitor) Prepare(spec program.Spec, pipelineID string, uid, gid int) er
 			}
 		}
 
-		if err := changeOwner(drop, uid, gid); err != nil {
-			return err
+		if takeOwnership {
+			if err := changeOwner(drop, uid, gid); err != nil {
+				return err
+			}
 		}
 	}