Skip to content

Conversation

@jouvin
Copy link
Contributor

@jouvin jouvin commented Nov 12, 2024

No description provided.

@jouvin jouvin added this to the 24.10 milestone Nov 12, 2024
@jouvin jouvin requested a review from jrha November 12, 2024 15:16
@jouvin
Copy link
Contributor Author

jouvin commented Nov 12, 2024

@jrha I realized I did a big mess with an unfortunate push to EL9 branch. And I misconfigured the target branch in this PR... I'll sort out everything later today (tonight) as I am unavailable for the rest of the afternoon...

@jrha
Copy link
Member

jrha commented Nov 12, 2024

I should be able to reset the branches.

@jrha jrha changed the base branch from el8.x-x86_64 to el9.x-x86_64 November 12, 2024 15:47
@jouvin jouvin force-pushed the el9_mnstate_selection branch from d3861d4 to fe8aea8 Compare November 12, 2024 19:35
@jouvin jouvin force-pushed the el9_mnstate_selection branch 8 times, most recently from 30c2e4a to 728b5f1 Compare November 12, 2024 21:24
@jouvin
Copy link
Contributor Author

jouvin commented Nov 12, 2024

Looks good!

@jouvin jouvin force-pushed the el9_mnstate_selection branch from 728b5f1 to 67ec849 Compare November 13, 2024 07:47
@jouvin
Copy link
Contributor Author

jouvin commented Nov 13, 2024

@jrha as for your points on chkconfig and initscripts, if you can wait until this evening before making a release, I will try do reinstall an EL9 machine without them and make them at least optional and disabled by default.

jrha
jrha previously approved these changes Nov 13, 2024
@jouvin jouvin force-pushed the el9_mnstate_selection branch from f45fbbf to 4c5c4fd Compare November 13, 2024 16:04
@jouvin
Copy link
Contributor Author

jouvin commented Nov 13, 2024

I just pushed modifications removing initscripts by default (with a variable to help the transition if somebody really needs it but probably something we can remove later) and added a variable to disable installation of chkconfig and ncm-chkconfig that must be kept to true until ncm-named and ncm-systemd no longer require it.

@jrha jrha merged commit 7d94baf into quattor:el9.x-x86_64 Nov 13, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants