Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(deploy): fix local storage #2089

Merged
merged 1 commit into from
Nov 12, 2022

Conversation

maxbrunet
Copy link
Member

#1989 was not enough.

Agent logs:

level=warn name=parca-agent ts=2022-11-10T22:29:57.054015758Z caller=manager.go:179 component=debuginfo buildid=4587364908de169dec62ffa538170118c1c3a078 path=/proc/6936/root/usr/lib/x86_64-linux-gnu/ld-2.31.so msg="failed to upload debug information" err="failed to upload debug information: close and receive: rpc error: code = Internal desc = failed to update metadata before uploading: failed to write metadata: mkdir /data: permission denied"

Server logs:

level=error name=parca ts=2022-11-10T22:30:00.573717776Z caller=metadata.go:211 component=debuginfo-metadata msg="failed to create metadata file" err="mkdir /data: permission denied"
level=error name=parca ts=2022-11-10T22:30:00.573778477Z caller=logger.go:38 protocol=grpc grpc.component=server grpc.service=parca.debuginfo.v1alpha1.DebugInfoService grpc.method=Upload grpc.method_type=client_stream peer.address=172.17.0.1:63240 grpc.start_time=2022-11-10T22:30:00Z grpc.code=Internal grpc.error="rpc error: code = Internal desc = failed to update metadata before uploading: failed to write metadata: mkdir /data: permission denied" grpc.time_ms=0.257 finishedcall=(MISSING)

JSON output diff:

diff --git a/before.json b/after.json
index ab798e4e..650516c3 100644
--- a/before.json
+++ b/after.json
@@ -2,7 +2,7 @@
    "parca-server-configMap": {
       "apiVersion": "v1",
       "data": {
-         "parca.yaml": "\"object_storage\":\n  \"bucket\":\n    \"config\":\n      \"directory\": \"./data\"\n    \"type\": \"FILESYSTEM\""
+         "parca.yaml": "\"object_storage\":\n  \"bucket\":\n    \"config\":\n      \"directory\": \"/var/lib/parca\"\n    \"type\": \"FILESYSTEM\""
       },
       "kind": "ConfigMap",
       "metadata": {
@@ -92,6 +92,10 @@
                         {
                            "mountPath": "/etc/parca",
                            "name": "config"
+                        },
+                        {
+                           "mountPath": "/var/lib/parca",
+                           "name": "data"
                         }
                      ]
                   }
@@ -111,6 +115,10 @@
                         "name": "parca"
                      },
                      "name": "config"
+                  },
+                  {
+                     "emptyDir": { },
+                     "name": "data"
                   }
                ]
             }

Rolled out the demo and looking good now

@maxbrunet maxbrunet requested a review from a team as a code owner November 11, 2022 18:39
Copy link
Member

@kakkoyun kakkoyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Awesome 🎉 I've been meaning to do this for some time 🔢

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants