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

(rke role) add EL9 support #705

Merged
merged 9 commits into from
Nov 23, 2022
Merged

(rke role) add EL9 support #705

merged 9 commits into from
Nov 23, 2022

Conversation

@jhoblitt jhoblitt marked this pull request as draft November 21, 2022 20:54
@jhoblitt jhoblitt changed the title (rke role) add EL9 support (WIP) (rke role) add EL9 support Nov 21, 2022
@jhoblitt jhoblitt force-pushed the IT-3057/el9 branch 6 times, most recently from f2e4953 to 5c4b049 Compare November 21, 2022 23:11
@jhoblitt jhoblitt force-pushed the IT-3057/el9 branch 10 times, most recently from 14e695e to 837a710 Compare November 22, 2022 17:39
Copy link
Contributor

@cbarria cbarria left a comment

Choose a reason for hiding this comment

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

Looks Great 👍

@jhoblitt jhoblitt force-pushed the IT-3057/el9 branch 8 times, most recently from 7608396 to 6d3bad1 Compare November 23, 2022 19:08
@jhoblitt jhoblitt changed the title (WIP) (rke role) add EL9 support (rke role) add EL9 support Nov 23, 2022
@jhoblitt jhoblitt force-pushed the IT-3057/el9 branch 5 times, most recently from 5caa9bf to f9b0701 Compare November 23, 2022 19:45
To allow for not including the network class.
As /etc/sysconfig/network-scripts backwards compatibility on EL9 is
(currently) completely broken for vlan interfaces.
This is a kludge to allow this class to be included on EL9 nodes when no
dispatch script is being created.
@jhoblitt jhoblitt marked this pull request as ready for review November 23, 2022 20:04
@jhoblitt jhoblitt merged commit 37e18af into master Nov 23, 2022
@jhoblitt jhoblitt deleted the IT-3057/el9 branch November 23, 2022 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants