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

improve microcode handling #274

Closed
stbuehler opened this issue May 28, 2023 · 3 comments · Fixed by #290
Closed

improve microcode handling #274

stbuehler opened this issue May 28, 2023 · 3 comments · Fixed by #290
Assignees
Milestone

Comments

@stbuehler
Copy link
Contributor

Opening yet another issue for this, and linking various other issues similar to this.

I'm using needrestart as part of my icinga monitoring, and checks with an unknown result are rather... sad.
I can (and did) disable the microcode check completely, but if now firmware updates actually would become available I won't see them in the monitoring.

I think the unknown result (in nagios/icinga language) should be reserved for "missing tools / missing data" and potentially certain timeouts / too failures.

To achieve this I suggest adding an "unavailable" status (or whatever name to use), indicating that you don't need to install/setup additional stuff (which wouldn't count as error in icinga), and you simply aren't expected to do anything about it.

Such status could be used in these situations:

  • Known platform without firmware in the first place (microcode check is currently skipped for vms/containers)
  • Known platform with firmware installed, but none matching your processor
  • Maybe simply ignore unknown platforms?

Especially the Intel uCode check shouldn't run on AMD platforms - is there any reason not to check for GenuineIntel the way the AMD code checks for AuthenticAMD?

Related issues:

@liske liske self-assigned this Aug 6, 2023
@liske liske added this to the v3.7 milestone Aug 6, 2023
This was referenced Oct 29, 2023
@fritz-fritz
Copy link
Contributor

@stbuehler would you mind looking over the pull requests i've submitted?

@anarcat
Copy link
Contributor

anarcat commented Nov 21, 2023

@stbuehler i submitted #290 to avoid warning when there's no available microcode file and #288 to fix detection in the AMD case, let me know if that works for you, the former PR is marked as fixing this issue.

@fritz-fritz
Copy link
Contributor

Bumped #284 with the suggested changes. It prevents Intel checks from being run on non-Intel platforms.

I think between this and #290 we will have addressed all outstanding related issues.

@liske liske closed this as completed in #290 Aug 9, 2024
@liske liske closed this as completed in 0e1ffe8 Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants