fix: update filestorage mount

[obs-gh-alexlew]

No description provided.

[orca-security-us]

Orca Security Scan Summary

Status	Check	Issues by priority
Passed	Infrastructure as Code	0   0   0   0	View in Orca
Passed	SAST	0   0   0   0	View in Orca
Passed	Secrets	0   0   0   0	View in Orca
Passed	Vulnerabilities	0   0   0   0	View in Orca

[github-actions]

The following chart updates are necessary:

diff --git a/charts/agent/Chart.yaml b/charts/agent/Chart.yaml
index beb24e4..b398b11 100644
--- a/charts/agent/Chart.yaml
+++ b/charts/agent/Chart.yaml
@@ -2,7 +2,7 @@ apiVersion: v2
 name: agent
 description: Chart to install K8s collection stack based on Observe Agent
 type: application
-version: 0.56.0
+version: 0.55.1
 appVersion: "2.3.0"
 dependencies:
   - name: opentelemetry-collector

[obs-gh-alexlew]

So this one is weird, it seems to work right now anyways? and if I go into the container and ls i get

/ # ls /var/lib
docker         misc           observe-agent  otelcol

but filestorage is only defined in observe-agent.

It almost seems like this shouldn't actually work so I'm kind of confused why the current state is working. The agent also expects the path to be /var/lib/observe-agent/filestorage

[obs-gh-mattcotter]

So this one is weird, it seems to work right now anyways? and if I go into the container and ls i get

/ # ls /var/lib
docker         misc           observe-agent  otelcol

but filestorage is only defined in observe-agent.

It almost seems like this shouldn't actually work so I'm kind of confused why the current state is working. The agent also expects the path to be /var/lib/observe-agent/filestorage

How did you end up running into this? Instead of changing the mounts, can we override the dir in the extension configuration:

helm-charts/charts/agent/templates/_config-extensions.tpl

Lines 1 to 4 in a69f837

	{{- define "config.extensions.file_storage" -}}
	file_storage:
	create_directory: true
	{{- end -}}

I think that would better preserve backwards compatibility.

[obs-gh-alexlew]

just forwarded you the customer email

[obs-gh-mattcotter]

I still think changing the bundled config instead of the mount is better for backwards compatibility.