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

Add new colocation influence #30

Merged
merged 1 commit into from
Jul 2, 2023

Conversation

@OndrejHome
Copy link
Owner

Hello @Obihoernchen ,

Thank you for opening this PR!
As mentioned in the required PR from pcs-modules-2 I will merge this one once we are done with pcs-modules-2.

One small note: For completeness it would be nice to add the influence: optional also in here - https://github.com/OndrejHome/ansible.ha-cluster-pacemaker/blob/master/README.md?plain=1#L286-L291

Thank you!

@OndrejHome OndrejHome self-assigned this Jun 17, 2023
@OndrejHome
Copy link
Owner

As the OndrejHome/ansible.pcs-modules-2#41 is now merged I have also checked changes here and they work for me on Debian 11.7 (causing error if I try influence as expected) and on Fedora 38 where they configure it as expected.
Therefore let merge also this one in.

@OndrejHome OndrejHome merged commit 917bf09 into OndrejHome:master Jul 2, 2023
@OndrejHome
Copy link
Owner

Once again, thank you @Obihoernchen for contributions!
The new influence option is available now in master branch or from tag 39.0.0 of this role.

@Obihoernchen
Copy link
Author

@OndrejHome Thank you very much for your fast responses!

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