Skip to content

squid: qa/workunits/mon: ensure election strategy is "connectivity" for stretch mode#61496

Merged
kamoltat merged 1 commit intoceph:squidfrom
ljflores:wip-69168-squid
Apr 8, 2025
Merged

squid: qa/workunits/mon: ensure election strategy is "connectivity" for stretch mode#61496
kamoltat merged 1 commit intoceph:squidfrom
ljflores:wip-69168-squid

Conversation

@ljflores
Copy link
Copy Markdown
Member

backport tracker: https://tracker.ceph.com/issues/69168


backport of #60927
parent tracker: https://tracker.ceph.com/issues/69107

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

…tch mode

The election strategy is randomly chosen for this type of test. Sometimes,
the test passes if the "connectivity" election strategy happens to be picked.
But if a different strategy, i.e. "classic", is picked, then the test will fail.

We can ensure that the election strategy is "connectivity" by setting it in the
workunit with the ceph CLI command. Although connectivity was specified in
stretch-mode-5-mons-8-osds.yaml, that config ultimately gets overridden by
the "qa/mon_config" yaml.

Fixes: https://tracker.ceph.com/issues/69107
Signed-off-by: Laura Flores <lflores@ibm.com>
(cherry picked from commit de2d918)
@ljflores ljflores added this to the squid milestone Jan 23, 2025
@ljflores ljflores requested a review from a team as a code owner January 23, 2025 21:30
@ljflores ljflores added the tests label Jan 23, 2025
@ljflores ljflores requested a review from kamoltat January 23, 2025 21:30
@ronen-fr
Copy link
Copy Markdown
Contributor

@kamoltat ?

@ljflores
Copy link
Copy Markdown
Member Author

ljflores commented Mar 5, 2025

jenkins test api

Copy link
Copy Markdown
Member

@kamoltat kamoltat left a comment

Choose a reason for hiding this comment

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

LGTM

@kamoltat kamoltat added the backport: no-conflicts Backport without conflicts label Apr 8, 2025
@kamoltat
Copy link
Copy Markdown
Member

kamoltat commented Apr 8, 2025

This should be fine to merge as it is an isolated test change within a standalone test file.

@kamoltat kamoltat merged commit 3814fc2 into ceph:squid Apr 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants