the-distro/infra
9
0
Fork 5
Code Issues 35 Pull requests 6 Packages Projects 1 Releases Wiki Activity

fix(gerrit): tmpdir-related failure #119

Open
kfearsoff wants to merge 1 commit from kfearsoff/infra:fix-alloy-gerrit into main
pull from: kfearsoff/infra:fix-alloy-gerrit
merge into: the-distro:main
Conversation 1 Commits 1 Files changed 1 +1
kfearsoff commented 2024-09-28 13:42:23 +00:00
First-time contributor
Copy link

Closes #108

Closes #108
kfearsoff added 1 commit 2024-09-28 13:42:26 +00:00
janik reviewed 2024-09-28 17:19:09 +00:00
services/gerrit/default.nix
@ -112,6 +112,7 @@ in
pyroscope.java "java" {
targets = discovery.relabel.java.output
forward_to = [pyroscope.write.production.receiver]
tmp_dir = "/var/lib/gerrit" // The default "/tmp" can be cleaned up and provoke failure, see #108
janik commented 2024-09-28 17:19:09 +00:00
Owner
Copy link

I don't see how this change closes #108 it just moves the occasional unpacking problem from /tmp to /var/lib/gerrit. Btw I'm pretty sure that it's a bad idea to change the tmp_dir setting to a non-temporary directory.

I don't see how this change closes #108 it just moves the occasional unpacking problem from `/tmp` to `/var/lib/gerrit`. Btw I'm pretty sure that it's a bad idea to change the `tmp_dir` setting to a non-temporary directory.
kfearsoff commented 2024-09-28 17:31:58 +00:00
Author
First-time contributor
Copy link

My understanding is that unpacking fails because the /tmp path got cleaned up by the system, hence the "directory doesn't exist" error message. The error message is accurate - the tmpdir is deleted, so when Alloy tries to access it, the OS says that the directory doesn't exist. The tmp_dir seems to only be used for unpacking the asprof, so it should be fine? I hope?

My understanding is that unpacking fails because the `/tmp` path got cleaned up by the system, hence the "directory doesn't exist" error message. The error message is accurate - the tmpdir is deleted, so when Alloy tries to access it, the OS says that the directory doesn't exist. The `tmp_dir` seems to only be used for unpacking the asprof, so it should be fine? I hope?
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u fix-alloy-gerrit:kfearsoff-fix-alloy-gerrit
git checkout kfearsoff-fix-alloy-gerrit

Merge

Merge the changes and update on Forgejo.
git checkout main
git merge --no-ff kfearsoff-fix-alloy-gerrit
git checkout main
git merge --ff-only kfearsoff-fix-alloy-gerrit
git checkout kfearsoff-fix-alloy-gerrit
git rebase main
git checkout main
git merge --no-ff kfearsoff-fix-alloy-gerrit
git checkout main
git merge --squash kfearsoff-fix-alloy-gerrit
git checkout main
git merge --ff-only kfearsoff-fix-alloy-gerrit
git checkout main
git merge kfearsoff-fix-alloy-gerrit
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: the-distro/infra#119
No description provided.
Delete branch "kfearsoff/infra:fix-alloy-gerrit"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?