Skip to content

[v18] Reduce flakiness of TestIntegrations/TrustedCluster tests#61731

Merged
rosstimothy merged 1 commit intobranch/v18from
tross/backport-61700/v18
Nov 24, 2025
Merged

[v18] Reduce flakiness of TestIntegrations/TrustedCluster tests#61731
rosstimothy merged 1 commit intobranch/v18from
tross/backport-61700/v18

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

Backport #61700 to branch/v18

The tests establish trust between two clusters, connect via SSH
to a host in the leaf, tear down trust, reestablish trust, and
then attempt another SSH connection. The flakiness stemmed from
tearing down and and reestablishing trust too quickly without
first properly verifying that the leaf cluster disappeared from
the root before reestablishing trust. While the test did make
use of `helpers.WaitForClusters`, it is satisfied if tunnels
for clusters were healthy within the last 10s. But this could
be true of a stale cluster that was not yet cleaned up. To
remedy this the test was updated to first wait for the reverse
tunnel for the leaf cluster to be completely gone before
reestablishing trust and proceeding.

Closes #60836.
Closes #48823.
@rosstimothy rosstimothy added backport no-changelog Indicates that a PR does not require a changelog entry labels Nov 24, 2025
@rosstimothy rosstimothy marked this pull request as ready for review November 24, 2025 14:46
@rosstimothy rosstimothy added this pull request to the merge queue Nov 24, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 24, 2025
@rosstimothy rosstimothy added this pull request to the merge queue Nov 24, 2025
Merged via the queue into branch/v18 with commit 4665d57 Nov 24, 2025
46 checks passed
@rosstimothy rosstimothy deleted the tross/backport-61700/v18 branch November 24, 2025 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport no-changelog Indicates that a PR does not require a changelog entry size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants