Skip to content
This repository was archived by the owner on Dec 12, 2025. It is now read-only.

CLOUDP-155037 - ensure we are waiting for some time until all the agents have finished updating the ports #1506

Merged
nammn merged 1 commit into from
Mar 25, 2024
Merged

CLOUDP-155037 - ensure we are waiting for some time until all the agents have finished updating the ports #1506

nammn merged 1 commit into from
Mar 25, 2024

Conversation

@nammn
Copy link
Collaborator

@nammn nammn commented Mar 7, 2024 •
edited
Loading

Summary:

  • making sure we wait and retry for some time until we fail the port change test. More description is in the method which has changed.

All Submissions:

  • Have you opened an Issue before filing this PR?
  • Have you signed our CLA?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

@nammn nammn changed the title make ensuring a poll ensure we are waiting for some time until all the agents have finished updating the ports Mar 7, 2024
@nammn nammn marked this pull request as ready for review March 7, 2024 09:36
@nammn nammn changed the title ensure we are waiting for some time until all the agents have finished updating the ports CLOUDP-155037 - ensure we are waiting for some time until all the agents have finished updating the ports Mar 7, 2024
@nammn nammn merged commit 2b18764 into master Mar 25, 2024
@nammn nammn deleted the fix-flaky-test branch March 25, 2024 14:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@mircea-cosbuc mircea-cosbuc Awaiting requested review from mircea-cosbuc mircea-cosbuc is a code owner

@lsierant lsierant Awaiting requested review from lsierant lsierant is a code owner

@Julien-Ben Julien-Ben Awaiting requested review from Julien-Ben Julien-Ben is a code owner

1 more reviewer

@slaskawi slaskawi slaskawi approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nammn @slaskawi