GitHub puppet-corosync
Sets up and manages Corosync.

Repo Checks ( 15 of 22 successfull )
Metadata Valid
No translation
passed
Correct Puppet Version Range
Supported Puppet version range is %{PUPPET_SUPPORT_RANGE}
passed
With Puppet Version Range
Puppet version range is present in requirements in metadata.json
passed
With Operatingsystem Support
No translation
passed
Operatingsystems
No translation
passed
Supports Only Current Redhat
No translation
passed
Supports Latest Redhat
No translation
failed
Supports Only Current Centos
No translation
passed
Supports Latest Centos
No translation
failed
Supports Only Current Debian
No translation
failed
Supports Latest Debian
No translation
passed
Supports Only Current Ubuntu
No translation
passed
Supports Latest Ubuntu
No translation
failed
Supports Only Current Sles
No translation
passed
Supports Latest Sles
No translation
failed
In Modulesync Repo
Is listed as a module managed using modulesync_config
passed
Synced
Has a .msync.yml file
passed
Latest Modulesync
Has been synchronized with the latest tagged version of modulesync_config
failed
Has Modulesync
Is present in voxpupuli/modulesync_config/managed_modules.yml
passed
Released
Is in modulesync_config and in forge releases.
passed
Valid Sync File
If a (optional) sync file is present, it must not contain a `.travis.yml` entry.
failed
Reference Dot Md
The repository has a REFERENCE.md. It needs to be generated / puppet-strings documentation is missing.
passed

Open Pull Requests

issue #485: remove score parameter from cs_order
merge-conflicts
tests-fail

I didn't spot an open PR on this, but there is open issue #485

It seems that the release of pacemaker 2.0.3 (in RHEL8) removes the option of supplying the superfluous score parameter.

Here is the documentation for pacemaker 2.0 which makes no mention of a score property:
https://clusterlabs.org/pacemaker/doc/en-US/Pacemaker/2.0/html/Pacemaker_Explained/s-resource-ordering.html

Pull Request (PR) description

Removes the score parameter from cs_order type as it is no longer valid and causes errors.

This Pull Request (PR) fixes the following issues

Fixes #485

Open PR in GitHub
Add support for corosync v3 on RedHat/CentOS 8
merge-conflicts
needs-rebase
tests-fail
enhancement

<!--
Thank you for contributing to this project!

-->

Pull Request (PR) description

Update the command used to check config file for corosync v3+

This Pull Request (PR) fixes the following issues

RedHat/CentOS 8 support

Open PR in GitHub
Add Puppet 8 support
enhancement
modulesync 7.5.0
modulesync

modulesync 7.5.0

Open PR in GitHub
feat: Set crm as default provider for ubuntu

Pull Request (PR) description

Sets CRM as the default provider for Ubuntu machines.

This Pull Request (PR) fixes the following issues

none (can create one if required)

I couldn't test the unit tests because of #536 It works in our environment though.

Open PR in GitHub
Pacemaker Fact & corosync.conf enhancement

<!--
Thank you for contributing to this project!

-->

Pull Request (PR) description

Two core changes:
- Adds a clustered fact called corosync to determine the version of pcs
- Includes the node name for each entry in corosync.conf to correct a bug when running pcs cluster config

Open PR in GitHub
Converge quorum member auth
enhancement

Pull Request (PR) description

The current code for authenticating to quorum members runs the auth command on every puppet run. This both updates the credentials on disk, and generates a puppet change event, which are btoh undesirable.

The proposed change checks to ensure all quorum members have an auth token in the credentials file, and updates auth for all members if any one member is missing. This results in a convergent state.

There is a caveat, in that what gets stored in the credentials file is not the original password, but an auth token. There does not seem to be a pcs command to check the tokens are still valid. So this code is only checking for presenence of auth tokens, not correctness. If the authentication token is later invalided, puppet will not correct this. It would be necessary to manually run the pcs host auth or pcs cluster auth commands to fix it.

This Pull Request (PR) fixes the following issues

Fixes #500

Open PR in GitHub
Turnoffcrypto

It is not possible to use transport: udpu and "cryptohash: none" or "cryptocipher: none" as it is not available as a parameter. I added it. UDPU doesnt support encryption and you are forced to use cryptohash or cryptocipher in corosync.conf (Version 3.x)

Open PR in GitHub
fix parameter description

Pull Request (PR) description

This comment corrects the description for the newly introduced parameter param configvalidatecmd which was introduced with #508

This Pull Request (PR) fixes the following issues

Fixes a incorrect descrption of the parameter

Open PR in GitHub
Add Debian 12 support
enhancement

includes: #541 #542

Open PR in GitHub
Add EL8 support and claim compatibility
enhancement

Pull Request (PR) description

Rework of #484 against latest master branch.

The version of corosync uses a cli parameter instead of an environment variable to point at the config file to be tested. Without this change, the env var is ignored and the config_validate_cmd command always checks the real config file location before installing the candidate file. On a new build, this will fail because the real config file does not yet exist. When being updated, the real config file is checked, instead of the candidate file, so an invalid change might be falsely reported as acceptable and lead to an outage.

This change overrides the config validate command on el8 to use the correct syntax.

It also lists el8 as a compatible OS in the module metadata, since the module is currently being used with this patch in production.

This Pull Request (PR) fixes the following issues

Fixes #526
Obsoletes #484

Open PR in GitHub