-
-
Notifications
You must be signed in to change notification settings - Fork 77
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
Install latest rabbitmq on all OSes #302
Conversation
Blocked until have GHA - see #297. As not happy to move until have the CI runs ran on all 3 OSes. |
@@ -0,0 +1,59 @@ | |||
--- | |||
|
|||
- name: RabbitMQ on RHEL8 requires module(perl:5.26) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Considering we moved to official RabbitMQ insructions (https://www.rabbitmq.com/install-rpm.html#package-cloud), is this perl requirement still a thing?
I see it's from the previous Ansible role.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@armab I took it out and install was fine, so no-longer required.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR! 👍
Just left one minor comment to address.
gpgcheck: no | ||
enabled: yes | ||
sslverify: yes | ||
when: ansible_facts.os_family == 'RedHat' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Considering the file is named rabbitmq_redhat.yml
which is included conditionally depending on OS,
I think this line could be removed.
when: ansible_facts.os_family == 'RedHat' |
gpgcheck: no | ||
enabled: yes | ||
sslverify: yes | ||
when: ansible_facts.os_family == 'RedHat' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when: ansible_facts.os_family == 'RedHat' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@armab Good spot - will remove!
Update the ansible installer to install latest rabbitmq from PackageCloud, rather than version bundled in OS.