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

Multi-poller script sync - permissions #1597

Closed
eschoeller opened this issue May 2, 2018 · 4 comments
Closed

Multi-poller script sync - permissions #1597

eschoeller opened this issue May 2, 2018 · 4 comments

Comments

@eschoeller
Copy link

I've found that scripts synchronized to a remote poller from the main poller lose the executable bit that was set. In some cases these scripts are called directly from within Cacti, without an interpreter prepended to the data input command. So, the remote poller is unable to get any output from them, since the O/S returns a "permission denied" when the script is called directly (aka without a /bin/bash or /bin/perl). Is this by design? What's the best fix? I added a +x to the first script I had trouble with, but I suspect that the automagic sync will reset that back to -x at some point?

@eschoeller
Copy link
Author

eschoeller commented May 2, 2018

As an example .. on the Main poller:

-rwxr-xr-x  1 root     root   2486 Apr 17  2017 tomcat-jvm-snmp.pl
-rwxr-xr-x  1 root     root   2274 Apr 17  2017 tomcat-jvm-snmp.pl.20131118
-rwxr-xr-x  1 root     root   2022 Apr 17  2017 tomcatstats-test.pl
-rwxr-xr-x  1 root     root   2493 Apr 17  2017 tomcatstats.pl
-rw-r--r--  1 root     root    198 Apr 17  2017 unix_processes.pl
-rw-r--r--  1 root     root    405 Apr 17  2017 unix_tcp_connections.pl
-rw-r--r--  1 root     root    386 Apr 17  2017 unix_users.pl
-rw-r--r--  1 root     root    849 Apr 17  2017 weatherbug.pl
-rw-r--r--  1 root     root    231 Apr 17  2017 webhits.pl
-rwxr-xr-x  1 root     root   3566 Apr 17  2017 ws_apachestats.local.pl
-rwxr-xr-x  1 root     root   3315 Apr 17  2017 ws_apachestats.pl

And on the remote poller:

-rw-r--r--  1 www-data www-data   2486 Apr 13 00:46 tomcat-jvm-snmp.pl
-rw-r--r--  1 www-data www-data   2274 Apr 13 00:46 tomcat-jvm-snmp.pl.20131118
-rw-r--r--  1 www-data www-data   2022 Apr 13 00:46 tomcatstats-test.pl
-rw-r--r--  1 www-data www-data   2493 Apr 13 00:46 tomcatstats.pl
-rw-rw-r--  1 www-data www-data    198 Feb 25 18:12 unix_processes.pl
-rw-rw-r--  1 www-data www-data    405 Feb 25 18:12 unix_tcp_connections.pl
-rw-rw-r--  1 www-data www-data    386 Feb 25 18:12 unix_users.pl
-rw-rw-r--  1 www-data www-data    849 Feb 25 18:12 weatherbug.pl
-rw-rw-r--  1 www-data www-data    231 Feb 25 18:12 webhits.pl
-rw-r--r--  1 www-data www-data   3566 Apr 13 00:46 ws_apachestats.local.pl
-rw-r--r--  1 www-data www-data   3315 Apr 13 00:46 ws_apachestats.pl

@netniV
Copy link
Member

netniV commented May 2, 2018

I have created a feature patch for 1.2 to include this. Are you able to test out the change? It is not currently merged with the base development branch yet.

@eschoeller
Copy link
Author

I'm still back at 1.1.36 right now

@netniV
Copy link
Member

netniV commented May 2, 2018

In that case, I'll close this for now. If you do get a change to try it out in a test environment that would be great.

@netniV netniV closed this as completed May 2, 2018
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants