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.

Sign up for GitHub

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

Jump to bottom

AZAffinity Flaky Test #2841

Closed
wants to merge 8 commits into from
Closed

AZAffinity Flaky Test #2841

wants to merge 8 commits into from

Conversation

Muhammad-awawdi-amazon
Copy link
Collaborator

@Muhammad-awawdi-amazon Muhammad-awawdi-amazon commented Dec 19, 2024 •
edited
Loading

Issue link

This Pull Request is linked to issue (URL): [#2837]

Checklist

Before submitting the PR make sure the following are checked:

  • This Pull Request is related to one issue.
  • Commit message has a detailed description of what changed and why.
  • Tests are added or updated.
  • CHANGELOG.md and documentation files are updated.
  • Destination branch is correct - main or release
  • Commits will be squashed upon merging.

@Muhammad-awawdi-amazon
added timeout between tests
7533f26 
Signed-off-by: Muhammad Awawdi <[email protected]>
@Muhammad-awawdi-amazon Muhammad-awawdi-amazon requested a review from a team as a code owner December 19, 2024 12:09
@Muhammad-awawdi-amazon Muhammad-awawdi-amazon added node Node.js wrapper testing Everything about testing Flaky-tests Flaky-tests labels Dec 19, 2024
@Muhammad-awawdi-amazon Muhammad-awawdi-amazon marked this pull request as draft December 19, 2024 15:02
@Muhammad-awawdi-amazon Muhammad-awawdi-amazon changed the title AZAffinity Flaky Test - Add Timeout AZAffinity Flaky Test Dec 19, 2024
@Muhammad-awawdi-amazon
re-try mechanism
50fcea2 
Signed-off-by: Muhammad Awawdi <[email protected]>
@Muhammad-awawdi-amazon
prettier
a71bfa7 
Signed-off-by: Muhammad Awawdi <[email protected]>
@Muhammad-awawdi-amazon
fix
00c3209 
Signed-off-by: Muhammad Awawdi <[email protected]>
@Muhammad-awawdi-amazon
retry mechanism for second test
968a9c0 
Signed-off-by: Muhammad Awawdi <[email protected]>
@Muhammad-awawdi-amazon
final modification
8d4b973 
Signed-off-by: Muhammad Awawdi <[email protected]>
@Muhammad-awawdi-amazon Muhammad-awawdi-amazon marked this pull request as ready for review December 23, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adarovadya adarovadya Awaiting requested review from adarovadya

Assignees

@Muhammad-awawdi-amazon Muhammad-awawdi-amazon

Labels
Flaky-tests Flaky-tests node Node.js wrapper testing Everything about testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Muhammad-awawdi-amazon