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

systemd_units plugin reports errors for "not-found" units #7990

Closed
squidpickles opened this issue Aug 16, 2020 · 3 comments
Closed

systemd_units plugin reports errors for "not-found" units #7990

squidpickles opened this issue Aug 16, 2020 · 3 comments
Labels
bug unexpected problem or unintended behavior

Comments

@squidpickles
Copy link
Contributor

SystemD's output when listing units changed in v245. When units are in not-found state, they include a ● glyph in front. Previously, using --no-legend disabled output of this glyph, but now it's included regardless of that setting.

Related bug report: systemd/systemd#15077

It appears the fix was merged, but I have encountered several systems running v246 (latest release) that still have this problem.

Adding --plain to the command invocation works across the old (pre-v245) and newer versions.
squidpickles added a commit to squidpickles/telegraf that referenced this issue Aug 16, 2020
@squidpickles
systemd_units: add --plain to command invocation (influxdata#7990)
e05a4aa
@squidpickles squidpickles mentioned this issue Aug 16, 2020
Merged
3 tasks
@sjwang90 sjwang90 added the bug unexpected problem or unintended behavior label Aug 21, 2020
@srebhan
Copy link
Member

srebhan commented Nov 12, 2020 •
edited
Loading

Similar fix for the problem is in PR #8381.

@srebhan srebhan mentioned this issue Nov 12, 2020
Closed
3 tasks
ssoroka pushed a commit that referenced this issue Nov 12, 2020
@squidpickles
systemd_units: add --plain to command invocation (#7990) (#7991)
97fb465
@srebhan
Copy link
Member

srebhan commented Nov 12, 2020

@squidpickles can you please test with current master after PR #7991 was merged!?

ssoroka pushed a commit that referenced this issue Nov 13, 2020
@squidpickles @ssoroka
systemd_units: add --plain to command invocation (#7990) (#7991)
5046a61 
(cherry picked from commit 97fb465)
@squidpickles
Copy link
Contributor Author

@srebhan It works! Fantastic, thanks.

arstercz pushed a commit to arstercz/telegraf that referenced this issue Mar 5, 2023
@squidpickles
systemd_units: add --plain to command invocation (influxdata#7990) (i…
9a2de4c 
…nfluxdata#7991)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug unexpected problem or unintended behavior
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@squidpickles @sjwang90 @srebhan