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

[bitnami/valkey-sentinel] Update tests to check the master to primary change in the configuration #74090

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jotamartos
Copy link
Contributor

@jotamartos jotamartos commented Oct 31, 2024

Description of the change

Add test to validate we are not adding both parameters to the configuration

Benefits

Will find errors like this one earlier

#74017

Possible drawbacks

None

Additional information

This test simply checks that we do not accidentally add both conf parameters to the conf file

… change in the configuration

Signed-off-by: Jota Martos <jotamartos@vmware.com>
@bitnami-bot bitnami-bot added the verify Execute verification workflow for these changes label Oct 31, 2024
carrodher
carrodher previously approved these changes Oct 31, 2024
Signed-off-by: Jota Martos <jotamartos@vmware.com>
# Remove if sentinel gets updated: https://github.com/valkey-io/valkey/blob/7.2/sentinel.conf
check-primary-master-change:
exec: if [ $(grep "SENTINEL \(master-reboot-down-after-period\|primary-reboot-down-after-period\)" /tmp/sentinel.conf | wc -l) -gt 1 ]; then exit 1; fi
exit-status: 0
check-valkey-server:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test shouldn't be working. Checking why it succeeded

valkey-sentinel-1  | valkey-sentinel 15:58:19.88 INFO  ==> ** Starting Valkey Sentinel **
valkey-sentinel-1  | 
valkey-sentinel-1  | *** FATAL CONFIG FILE ERROR (Version 7.2.7) ***
valkey-sentinel-1  | Reading the configuration file, at line 358
valkey-sentinel-1  | >>> 'SENTINEL master-reboot-down-after-period mymaster 0'
valkey-sentinel-1  | No such master with specified name.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bitnami verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants