-
Notifications
You must be signed in to change notification settings - Fork 100
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
Feature: Install Keepalived from Source #293
Conversation
This indeed has more value than dropping the molecule testing, which starts to annoy me in the long run. Would you be okay to help maintain the build from source ? Else I am quite reluctant to add the build from source, as the PPA option (or other repos in other distros) is generally good enough. I need to dig deeper into this. @zerwes do you have an opinion on the build from source? |
Thank your for responding! On a side note: The modifications I did for getting the Molecule tests up and running are mostly independent of the "compile from source" feature. Would you like/accept a PR for that only? I'm not sure what issues you've had with Molecule testing and systemd in the past. The roles I'm maintaining almost all rely on systemd services as well, and once the test is set up correctly it works reliably (or if it doesn't it's not systemd's fault). |
I was trying to install and configure keepalived with this role. However, I ran into issues because my package manager only serves a pretty old version of keepalived (2.2.7), which caused troubles e.g. during configuration verification.
So I extended the role to allow for compiling and installing keepalived from source.
The role now supports the following scenarios:
The follwing role variables control installation from source:
The installation prefix is generated dynamically using
keepalived_source_tag
.In addition, the following variable control the build dependencies on a distribution level:
In addition to that, I also revamped the molecule tests. The ones that already existed are working now, and I also added a couple of my own for testing source installation.
The tests now run successfully on the following platforms:
RockyLinux 9 also works, but I had to disable the automated tests due to this issue.
Please let me know if you don't have any interest in merging these features. Then I'll publish the fork myself (of course while adhering to the license restrictions).