-
Notifications
You must be signed in to change notification settings - Fork 9
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
Allow using abstract namespace for crashtracker unix socket #594
Conversation
Thanks to PrivateTmp in systemd it's pretty likely that different processes will see the same sidecar (which lives in a network namespace), but may not see the crashtracker socket in /tmp. Defaulting to the abstract unix namespace on Linux will work around any possible problems here. Signed-off-by: Bob Weinand <bob.weinand@datadoghq.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #594 +/- ##
==========================================
- Coverage 72.89% 72.85% -0.04%
==========================================
Files 243 243
Lines 34638 34676 +38
==========================================
+ Hits 25248 25262 +14
- Misses 9390 9414 +24
|
BenchmarksComparisonBenchmark execution time: 2024-08-27 08:12:18 Comparing candidate commit 7bbfeed in PR branch Found 0 performance improvements and 1 performance regressions! Performance is the same for 49 metrics, 2 unstable metrics. scenario:tags/replace_trace_tags
CandidateCandidate benchmark detailsGroup 1
Group 2
Group 3
Group 4
Group 5
Group 6
Group 7
Group 8
Group 9
Group 10
Group 11
BaselineOmitted due to size. |
8397400
to
69533a1
Compare
69533a1
to
c7a60a6
Compare
Signed-off-by: Bob Weinand <bob.weinand@datadoghq.com>
This reverts commit c7a60a6.
c06f301
to
7bbfeed
Compare
8656814
to
7bbfeed
Compare
Thanks to PrivateTmp in systemd it's pretty likely that different processes will see the same sidecar (which lives in a network namespace), but may not see the crashtracker socket in /tmp.
Defaulting to the abstract unix namespace on Linux will work around any possible problems here.