loki.source.journald: sync positions directory permissions (#3973)

Signed-off-by: Paschalis Tsilias <paschalis.tsilias@grafana.com>

CHANGELOG.md

@@ -93,6 +93,9 @@ Main (unreleased)
 
 - Fix issue in operator where any version update will restart all agent pods simultaneously. (@captncraig)
 
+- Fix an issue where `loki.source.journald` did not create the positions
+  directory with the appropriate permissions. (@tpaschalis)
+
 ### Other changes
 
 - Add metrics when clustering mode is enabled. (@rfratto)

component/loki/source/journal/journal.go

@@ -45,6 +45,11 @@ type Component struct {
 
 // New creates a new  component.
 func New(o component.Options, args Arguments) (component.Component, error) {
+	err := os.MkdirAll(o.DataPath, 0750)
+	if err != nil {
+		return nil, err
+	}
+
 	positionsFile, err := positions.New(o.Logger, positions.Config{
 		SyncPeriod:        10 * time.Second,
 		PositionsFile:     filepath.Join(o.DataPath, "positions.yml"),
@@ -54,10 +59,7 @@ func New(o component.Options, args Arguments) (component.Component, error) {
 	if err != nil {
 		return nil, err
 	}
-	err = os.MkdirAll(o.DataPath, 0644)
-	if err != nil {
-		return nil, err
-	}
+
 	c := &Component{
 		metrics:   target.NewMetrics(o.Registerer),
 		o:         o,