-
Notifications
You must be signed in to change notification settings - Fork 87
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
train should be using /etc/os-release for all Linux os detection #685
Comments
tas50
changed the title
inspec should be using /etc/os-release for all Linux os detection
train should be using /etc/os-release for all Linux os detection
Jul 27, 2021
Here's a list of all the files and commands we're running to check the OS. This is pointless and slow
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Linux long ago moved to using /etc/os-release for all operating system information. We moved Chef to this several years ago with Ohai and it's been a great success. We no longer have to add every new operating system as they're released and instead we just use the OS data that comes from /etc/os-release automatically. For our legacy platform names we do a simple remap and for everything new we just use the value out of /etc/os-release. Train should be using this same logic so we can settle on a single detection path and so we don't have to add every new distro to train:
https://github.com/chef/ohai/blob/master/lib/ohai/plugins/linux/platform.rb
The text was updated successfully, but these errors were encountered: