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

rabbitmq_ct_helpers: Always give $RABBITMQ_FEATURE_FLAGS a value #13104

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dumbbell
Copy link
Member

@dumbbell dumbbell commented Jan 20, 2025

… even if skip_metadata_store_configuration is given.

Why

If skip_metadata_store_configuration is set to true, we let the node boot with the stable feature flags all enabled. This may prevent clustering in mixed-version testing if the secondary umbrella misses a stable feature flag.

How

If skip_metadata_store_configuration is set, we restore the previous behavior of starting the node with the required feature flags only. I.e. all stable ones are disabled at first.

This is a follow-up to #13077.

... even if `skip_metadata_store_configuration` is given.

[Why]
If `skip_metadata_store_configuration` is set to true, we let the node
boot with the stable feature flags all enabled. This may prevent
clustering in mixed-version testing if the secondary umbrella misses a
stable feature flag.

[How]
If `skip_metadata_store_configuration` is set, we restore the previous
behavior of starting the node with the required feature flags only. I.e.
all stable ones are disabled at first.
@dumbbell dumbbell requested a review from ansd January 20, 2025 14:17
@dumbbell dumbbell self-assigned this Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant