-
-
Notifications
You must be signed in to change notification settings - Fork 162
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
test_corosync_config fails for pacemaker 3.x #508
Comments
AndreasPfaffeneder
changed the title
test_corosync_config fails for 3.x
test_corosync_config fails for pacemaker 3.x
Jun 10, 2021
config_validate_cmd exists |
need to get the parameter to the signature of the main class. |
With #509 its possible to provide a custom check command which solves that issue. For reference, the check-command for pacemaker 3x.:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Affected Puppet, Ruby, OS and module versions/distributions
How to reproduce (e.g Puppet code you use)
Multiple steps required:
1.) Have a corosync.conf which is not working, something like:
/etc/corosync/corosync.conf
[...]
totem{
crypto_hash: sha1
crypto_cipher: aes256
transport: udpu
}
[...]
This configuration will not validate, because crypto_hash & crpto_cipher must not be set when using udpu.
Now, trying to deploy a configuration will persistently fail, because test_corosync_config defaults to true, but the actual command does not use the temporarly written configuration file from puppet in /etc/corosync/corosync.conf<puppet-name-extension> via
/usr/bin/env COROSYNC_MAIN_CONFIG_FILE=/etc/corosync/corosync.conf<puppet-name-extension> /usr/sbin/corosync -t but the currently deployed configuration file
So the check command should be changed to something like
/usr/sbin/corosync -t -c %
What are you seeing
Check command validates current configuration, not future configuration.
What behaviour did you expect instead
Check command checks the file deployed by puppet.
Output log
Any additional information you'd like to impart
The text was updated successfully, but these errors were encountered: