search

wavefrontHQ / wavefront-cli

Wavefront CLI Tool
Apache License 2.0
5 stars 8 forks source link

INT-797: Added user permission check for telegraf config path #24

Closed gangadharaswamy closed 3 years ago

gangadharaswamy commented 3 years ago

Added user permission check to confirm whether the telegraf user has read permission to config path /etc/telegraf. If no permission then the script assigns read access to the config path.

srinivas-kandula commented 3 years ago

This fix is for the issue https://wavefront.atlassian.net/browse/INT-797

ustinov commented 3 years ago

@gangadharaswamy please don't forget to increment the package version before merging https://github.com/wavefrontHQ/wavefront-cli/blob/master/wavefront_cli/__init__.py#L2

ustinov commented 3 years ago

LGTM

gangadharaswamy commented 3 years ago

Can this be merged to master?

ustinov commented 3 years ago

Can this be merged to master?

Yes, but we need someone with the permission to do that