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

Attempt to support BLSCFG #60

Closed
wants to merge 2 commits into from
Closed

Attempt to support BLSCFG #60

wants to merge 2 commits into from

Conversation

jcpunk
Copy link

@jcpunk jcpunk commented Oct 27, 2020

No description provided.

@jcpunk jcpunk marked this pull request as draft October 27, 2020 18:58
@trevor-vaughan
Copy link
Contributor

I don't quite follow. Is there something that needs to be supported that grub_menuentry doesn't currently work with?

@jcpunk
Copy link
Author

jcpunk commented Oct 27, 2020

I didn't see the BLS bits down in there.

How would you recommend I use menu_entry to remove rhgb from my kernel_opts (RHEL8)?

Related to : #58

@coveralls
Copy link

coveralls commented Oct 27, 2020

Coverage Status

Coverage decreased (-16.6%) to 78.808% when pulling d5723ed on jcpunk:blscfg into ff09ddc on hercules-team:master.

@jcpunk jcpunk marked this pull request as ready for review November 12, 2020 16:35
@jcpunk
Copy link
Author

jcpunk commented Nov 12, 2020

I've gotten this about as far as I can without someone who knows puppet providers giving a hand. In theory the tests are about right, it is the actual provider code that needs some help.....

@trevor-vaughan
Copy link
Contributor

@jcpunk Sorry it took me so long to get back to you on this.

It looks like the following still works properly:

kernel_parameter { 'rhgb': ensure => absent }

@jcpunk
Copy link
Author

jcpunk commented Dec 16, 2020

I'm not showing the existing module works with BLS config:

[riehecky@leibniz ~]$ ssh [email protected]
Activate the web console with: systemctl enable --now cockpit.socket

Last login: Wed Dec 16 08:58:32 2020 from 192.168.122.1
[root@localhost ~]# cat /proc/cmdline 
BOOT_IMAGE=(hd0,msdos1)/boot/vmlinuz-4.18.0-257.el8.x86_64 root=UUID=bed1a7ae-faff-4179-9a69-93480e0ac781 ro resume=UUID=033a1427-cacd-449b-89ae-8eea74bba365 rhgb
[root@localhost ~]# cat /etc/sysconfig/grub 
GRUB_TIMEOUT=5
GRUB_DISTRIBUTOR="$(sed 's, release .*$,,g' /etc/system-release)"
GRUB_DEFAULT=saved
GRUB_DISABLE_SUBMENU=true
GRUB_TERMINAL_OUTPUT="console"
GRUB_CMDLINE_LINUX="resume=UUID=033a1427-cacd-449b-89ae-8eea74bba365"
GRUB_DISABLE_RECOVERY="true"
GRUB_ENABLE_BLSCFG=true
[root@localhost ~]# puppet apply --verbose -e "kernel_parameter { 'rhgb': ensure => absent }"
Info: Loading facts
Info: Loading facts
Notice: Compiled catalog for localhost.localdomain in environment production in 0.03 seconds
Info: Applying configuration version '1608130737'
Notice: Applied catalog in 0.04 seconds
[root@localhost ~]# reboot
Connection to 192.168.122.171 closed by remote host.
Connection to 192.168.122.171 closed.
[riehecky@leibniz ~]$ ssh [email protected]
Activate the web console with: systemctl enable --now cockpit.socket

Last login: Wed Dec 16 08:58:39 2020 from 192.168.122.1
[root@localhost ~]# cat /proc/cmdline 
BOOT_IMAGE=(hd0,msdos1)/boot/vmlinuz-4.18.0-257.el8.x86_64 root=UUID=bed1a7ae-faff-4179-9a69-93480e0ac781 ro resume=UUID=033a1427-cacd-449b-89ae-8eea74bba365 rhgb
[root@localhost ~]# 

@trevor-vaughan
Copy link
Contributor

@jcpunk This is actually what I'm looking into right now. When I ran it manually, it seems to have:

  1. Updated /etc/default/grub
  2. Updated the BLS configuration (as evidenced by running the grub2-editenv command)

BUT, I think that grub2-mkconfig still has to be run and there is a bug #63 that I'm working on fixing right now.

@jhoblitt
Copy link
Member

Is this still needed?

@jcpunk
Copy link
Author

jcpunk commented May 13, 2022

I think the just released version 4.0.0 may do what I need. I'll need to test further to be sure

@jcpunk jcpunk closed this May 13, 2022
@jcpunk
Copy link
Author

jcpunk commented May 16, 2022

hmmm.... I'm not seeing 4.0.0 up on the forge...

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.

4 participants