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

EL9 support : remove initscripts support and chkconfig requirement #1

Merged
merged 1 commit into from
Apr 17, 2024

Conversation

jouvin
Copy link

@jouvin jouvin commented Apr 2, 2024

No description provided.

@stdweird
Copy link
Owner

stdweird commented Apr 3, 2024

@jouvin this does a lot more then onlyh removing teh dependency. it now requires systemd.

@jouvin
Copy link
Author

jouvin commented Apr 3, 2024

@stdweird it is the goal! I'll update the title. You cannot remove the dependency without relying on systemd. Is it a problem for you?

@jouvin jouvin changed the title EL9 support : remove chkconfig dependency EL9 support : remove initscripts support and chkconfig requirement Apr 3, 2024
@stdweird
Copy link
Owner

stdweird commented Apr 4, 2024

i don't care, we have 0 hosts that don't have systemd. but can't speak for the community. maybe you need to add a config_chkconfig.pan that locks the version to the latest release that still has chkconfig, so people can easily lock if they don't have systemd yet.

@jouvin
Copy link
Author

jouvin commented Apr 4, 2024

I sent an email to the Quattor list, don't know if you saw it, asking if everybody had a problem with dropping EL6 compatibility in next Quattor release. The only response was from Jerome who said he was ok. I suspect nobody requires the next Quattor version to support an OS version as obsolete as EL6. It doesn't prevent managing EL6 systems with 23.6.0 and should not be a problem as an EL6 box probably has a frozen configuration...

My choice in the PRs I made for ncm-cdispd and cdp-listend is really to drop the support of non systemd OS versions... Create too much complexity and a nightmare in recent OS versions like EL9...

@@ -7,11 +7,6 @@
#
# ${developer-info}
# ${author-info}
#
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jouvin remove this file entirely

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stdweird Done!

- chkconfig dependency is a source of problems in EL9
- Breaks EL6 support
@stdweird stdweird merged commit bade726 into stdweird:unitfile Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants