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.

Sign up for GitHub

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

Jump to bottom

Change the workflow on how we set the right permissions for perf-plugin #16558

Merged
merged 8 commits into from Dec 11, 2023
Merged

Change the workflow on how we set the right permissions for perf-plugin #16558

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
4256f3d
Change the workflow on how we set the right permissions
tkatsoulas Dec 7, 2023
5b75b69
Add a failsafe in case fail to set the cap_perfmon
tkatsoulas Dec 7, 2023
091a479
add EOF new line
tkatsoulas Dec 7, 2023
7793b9e
Fix workflow
tkatsoulas Dec 7, 2023
d10b961
minor spelling
tkatsoulas Dec 7, 2023
015d840
we made it odyssey
tkatsoulas Dec 7, 2023
63f3037
Apply suggestion from code review
tkatsoulas Dec 7, 2023
4be0d6d
Tidy-up postinst script.
Ferroin Dec 11, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion contrib/debian/netdata-plugin-perf.postinst
View file
Open in desktop
Expand Up @@ -7,7 +7,7 @@ case "$1" in
chown root:netdata /usr/libexec/netdata/plugins.d/perf.plugin
chmod 0750 /usr/libexec/netdata/plugins.d/perf.plugin

if ! setcap cap_perfmon+ep /usr/libexec/netdata/plugins.d/perf.plugin; then
if setcap cap_perfmon+ep /usr/libexec/netdata/plugins.d/perf.plugin; then
tkatsoulas marked this conversation as resolved.
Show resolved Hide resolved
echo "Set cap_perfmon capabilties for perf.plugin"
else
if ! setcap cap_sys_admin+ep /usr/libexec/netdata/plugins.d/perf.plugin; then
Expand Down