-
Notifications
You must be signed in to change notification settings - Fork 92
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
Refactor out grub config to profile file #76
Refactor out grub config to profile file #76
Conversation
57335eb
to
d85bcb4
Compare
b2da0ed
to
69ec3e7
Compare
@chris-rock those minor whitespace issues are fixed on the #77 pr. not sure how they got into master 🤷♂️ |
@markdchurchill Can you rebase on latest master? |
class GrubConf < Inspec.resource(1) | ||
name 'grub_conf' | ||
|
||
def locations |
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.
🎉 Thank you for moving the config to the resource!
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.
Done :)
Signed-off-by: Mark Churchill <markdchurchill@outlook.com>
69ec3e7
to
2a917dc
Compare
Ooops, I mean done here. |
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.
Great work @markdchurchill
See #17, #74, etc.
This pulls the canonical locations for the grub.cfg file into one spot so we aren't chasing our tail in multiple controls. Not sure if the pattern is appropriate, I'm not an ops/inspec guy.