-
Notifications
You must be signed in to change notification settings - Fork 43
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
Document the Jsonnet example #655
Conversation
// This is a Jsonnet file. Please, check https://jsonnet.org/ for more | ||
// information about the language. | ||
|
||
// The hw.libsonnet file contains hardware information from the "lshw" tool. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On Leap 15.4 at least, lshw.rpm or jsonnet.rpm do not contain a hw.libsonnet file. Where do I find it or how do I know what API it provides?
Co-authored-by: Martin Vidner <mvidner@suse.cz>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
lshw
makes sense now, it contains the data about the current system. Somehow I tricked myself into thinking that it contained jsonnet code.
// https://github.com/openSUSE/agama/blob/master/rust/agama-lib/share/profile.schema.json | ||
|
||
// The "hw.libsonnet" file contains hardware information of the storage devices | ||
// from the "lshw" tool. It is generated by running as root: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the most important information is that it is explicitly run by Agama itself, there is no magic for that.
https://github.com/openSUSE/agama/blob/b4bf25fdffd46735800a8396f1715fffe181e1db/rust/agama-lib/src/profile.rs#L128
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I merged too soon. I opened #655.
Add some comments to the Jsonnet-based example.