Update permission for read in /sys/fs/cgroup/system.slice/- path#18987
Conversation
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
There was a problem hiding this comment.
Approving as its not possible to use syntax like /sys/fs/cgroup/system.slice/*/system.slice/opensearch.service/-
Edit: FilePermission in JSM never supported wildcards within the path..only at the end. See https://docs.oracle.com/en/java/javase/12/docs//api/java.base/java/io/FilePermission.html
In JDK21 and prior I think the JDK would allow implicit access.
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
|
Added a TODO so we will change this later when wildcard in path is supported later. |
|
❕ Gradle check result for 3b2f47d: UNSTABLE Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## 3.2 #18987 +/- ##
=========================================
Coverage 72.93% 72.93%
- Complexity 69352 69368 +16
=========================================
Files 5643 5643
Lines 318713 318713
Branches 46112 46112
=========================================
+ Hits 232441 232468 +27
+ Misses 67488 67446 -42
- Partials 18784 18799 +15 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…8987) * Update permission for read in /sys/fs/cgroup/system.slice/- path Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> * Update TODO comment Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> --------- Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> (cherry picked from commit 42a7209) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…ensearch-project#18987) (opensearch-project#18992) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…ensearch-project#18987) (opensearch-project#18992) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…ensearch-project#18987) (opensearch-project#18992) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Description
Update permission for read in /sys/fs/cgroup/system.slice/- path.
If running in docker the folder location changed compares to #18975:
Related Issues
Follow up #18975
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.