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

Fix RHEL-07-010491 grub superusers config #347

Merged
merged 1 commit into from
Mar 18, 2021
Merged

Fix RHEL-07-010491 grub superusers config #347

merged 1 commit into from
Mar 18, 2021

Conversation

danbarr
Copy link
Contributor

@danbarr danbarr commented Mar 18, 2021

The RHEL-07-010491 task in fix-cat1.yml attempts to check/update /boot/grub2/grub.cfg, but this file does not exist on UEFI systems and the task fails. The old version of the role performed this task on /etc/grub.d/01_users which seems more proper since direct edits to grub.cfg are overwritten by grub2-mkconfig.

Signed-off-by: Dan Barr dan@dbarr.com

Signed-off-by: Dan Barr <dan@dbarr.com>
@georgenalen georgenalen changed the base branch from devel to staging March 18, 2021 19:24
@georgenalen
Copy link
Contributor

Dan,
Thank you for addressing this issue. We have an automated testing pipeline that does not like PR's from outside of it's own role. I changed the PR to go into staging and then will perform the PR to devel from there. Thanks again, I'll do the same with your other PR

@georgenalen georgenalen merged commit aaa73ea into ansible-lockdown:staging Mar 18, 2021
@danbarr
Copy link
Contributor Author

danbarr commented Apr 23, 2021

Hi George,
I noticed this and my other fix PR #348 never made it from staging over to devel. Should I re-submit?

@georgenalen
Copy link
Contributor

Danbarr,
Sorry about that we were neck deep in other items and this somewhat fell off. I will work on getting this into devel today.

George

@georgenalen
Copy link
Contributor

Danbarr,
As an update I merged our changes into devel this morning. It was a bit out of sync with some other devel pushes so I had to get that sorted, but all is good. It will move into main with the May fixes push.

George

@danbarr
Copy link
Contributor Author

danbarr commented Apr 23, 2021

Awesome, thanks!

@danbarr danbarr deleted the patch-RHEL-07-010491 branch April 23, 2021 15:53
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.

2 participants