-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
SMART input works in 1.15.3 and fails in 1.16.0 with exact same config #8313
Comments
Missing NVMe attributes were added to I suspect what caused your problem but lets clarify few things:
Moreover, can you run |
@p-zak Thank you for your reply!
Got it! I was just trying anything I could think of to get it to work. Finally I tried downgrading which worked.
No inconsistency, that log is to show that it worked with 1.15.3. If you observe the very last code block in the GHI body you can see I upgraded, ran the failing test (which shows the 1.16.0 version) then downgrading with dpkg to 1.15.3 and ran the test again, which worked. All with no configuration change.
I will note however, that my configuration specified the devices array, with by-id devs since the letter identifiers are not stable. It is my understading from the documentation that when devices is specified |
I just thought so, just wanted to have everything clear :) We know what the problem is (accidentally recognized Till that time I see these options:
|
@p-zak Thank you very much for your help in this issue! Since there are no security fixes (that I could see from the release notes anyway) in 1.16.0 I think I will stay on 1.15.3 until a patch can be released. I am more than happy to run a dev build / RC on my host to verify the fix when the time comes! Please reply here with the build to use and I will test 🙂 If I do encounter a need to upgrade before the patch can be released I will use one of the workarounds you suggested. Again, I appreciate your swift help in finding the issue! |
This is also an issue when '+' is present in device string (such lines seem to be ignored by Telegraf altogether):
|
@chrishoage @Feliksas Should work right now. Can you check using nightly build https://github.com/influxdata/telegraf#nightly-builds? |
I can confirm the nightly fixes the issue. |
Relevant telegraf.conf:
System info:
Docker
Steps to reproduce:
telegraf --test
telegraf --test
Expected behavior:
Config to work after upgrade
Actual behavior:
Config fails.
Additional info:
I initially saw this error. After installing nvme-cli the error went away, but the SMART input would still not output anything
Here is the sample commands showing that downgrading works
The text was updated successfully, but these errors were encountered: