Skip to content

feat: Add validation task for NGINX modules and installing NGINX Plus HA keepalived module #764

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

Merged
merged 9 commits into from
Aug 22, 2024

Conversation

alessfg
Copy link
Member

@alessfg alessfg commented Aug 13, 2024

Proposed changes

  • Add support for installing the NGINX Plus HA keepalived module.
  • Add validation tasks to check the Ansible version, the Jinja2 version, whether the required Ansible collections for this role are installed, and whether you are trying to install a valid NGINX module.

Checklist

Before creating a PR, run through this checklist and mark each as complete:

@alessfg alessfg self-assigned this Aug 13, 2024
@alessfg alessfg added this to the 0.25.0 milestone Aug 13, 2024
@alessfg alessfg force-pushed the nginx-ha-keepalived branch 2 times, most recently from 605b077 to 82afc51 Compare August 20, 2024 11:48
@alessfg alessfg force-pushed the nginx-ha-keepalived branch 2 times, most recently from a6446e2 to 7edb02b Compare August 20, 2024 12:16
@alessfg alessfg force-pushed the nginx-ha-keepalived branch from 7edb02b to e5bacfe Compare August 20, 2024 13:04
@alessfg alessfg force-pushed the nginx-ha-keepalived branch from b97357b to fb2cd30 Compare August 20, 2024 21:10
@alessfg alessfg enabled auto-merge (squash) August 21, 2024 12:34
@alessfg alessfg disabled auto-merge August 21, 2024 12:34
@alessfg alessfg enabled auto-merge (squash) August 21, 2024 12:35
@alessfg alessfg disabled auto-merge August 21, 2024 12:38
@alessfg alessfg merged commit 536e5d3 into main Aug 22, 2024
18 checks passed
@alessfg alessfg deleted the nginx-ha-keepalived branch August 22, 2024 09:25
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.

1 participant