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

Added support for global GRUB configuration #18

Merged
merged 1 commit into from
Mar 2, 2016

Conversation

trevor-vaughan
Copy link
Contributor

This patch adds support for grub_menuentry which provides the ability to
manage individual Menu entries for both GRUB Legacy and GRUB2.

Testing has been focused on Linux-based operating systems and it is
possible that features may be missing for chainloaded operating systems.

Support for managing GRUB2 users has also been added. This compares
against the running configuration, not what is saved in the
compilation files in grub.d.

The following custom types were created:

  • grub_config => Manages global GRUB settings
  • grub_menuentry => Manages GRUB menuentries
  • grub_user => Manages GRUB2 users

Fixes #10

This patch adds support for grub_menuentry which provides the ability to
manage individual Menu entries for both GRUB Legacy and GRUB2.

Testing has been focused on Linux-based operating systems and it is
possible that features may be missing for chainloaded operating systems.

Support for managing GRUB2 users has also been added. This compares
against the *running* configuration, not what is saved in the
compilation files in grub.d.

The following custom types were created:
  * grub_config => Manages global GRUB settings
  * grub_menuentry => Manages GRUB menuentries
  * grub_user => Manages GRUB2 users

Fixes voxpupuli#10
@raphink
Copy link
Member

raphink commented Mar 2, 2016

That is quite a PR! Would you mind splitting the providers in separate PRs please?

Also, if you provide the format, I don't mind providing the missing lenses.

@trevor-vaughan
Copy link
Contributor Author

Hi @raphink. I don't mind, but I don't know that I'll have time for a while as we're trying to push out a new build and free time is at an all time low. Would you prefer one PR per provider?

My GRUB2 menuentry parser starts at https://github.com/trevor-vaughan/augeasproviders_grub/blob/86fca6f5b9dbdf16a3f714270b36137fd8b4bb58/lib/puppet/provider/grub_menuentry/grub2.rb#L22

@raphink
Copy link
Member

raphink commented Mar 2, 2016

@trevor-vaughan Actually, since it's not your first large contribution, how about I give you commit rights on augeasproviders projects and you can add this yourself to the project?

@trevor-vaughan
Copy link
Contributor Author

@raphink That would be awesome but...are you sure? I'll make sure to have issue discussions before doing anything drastic. Thanks!

@raphink
Copy link
Member

raphink commented Mar 2, 2016

I trust you. And I'll try to have a look anyway before merging, in case I have suggestions.

@raphink raphink closed this Mar 2, 2016
@raphink raphink reopened this Mar 2, 2016
@raphink
Copy link
Member

raphink commented Mar 2, 2016

Closed by mistake (wrong button)

trevor-vaughan added a commit that referenced this pull request Mar 2, 2016
Added support for global GRUB configuration
@trevor-vaughan trevor-vaughan merged commit d8302b5 into voxpupuli:master Mar 2, 2016
@raphink
Copy link
Member

raphink commented Mar 3, 2016

OK. Could you update the change log and tag 2.4.0 please?

@trevor-vaughan
Copy link
Contributor Author

Sorry about this. I just realized that I bumped to 2.3.1.

We didn't add any breaking changes, just new features. Should this be 2.4.0 or 2.3.1?

@raphink
Copy link
Member

raphink commented Mar 4, 2016

New features means new minor, hence 2.4.0. Breaking changes would mean new major (3.0.0). Version 2.3.1 would mean bugfix.

For easy reference: https://twitter.com/auroraeosrose/status/585209388857700352

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants