Skip to content

test: increased timeout and added retries to flaky tests#7491

Merged
averikitsch merged 7 commits into
mainfrom
fix-asset
Dec 12, 2022
Merged

test: increased timeout and added retries to flaky tests#7491
averikitsch merged 7 commits into
mainfrom
fix-asset

Conversation

@kweinmeister
Copy link
Copy Markdown
Contributor

@kweinmeister kweinmeister commented Dec 10, 2022 •
edited
Loading

  • Asset/AutoML: Adds retries to failing tests
  • Asset:
    • Updates timeout from 5->10 min (later we could apply a filter on asset type to speed up test)
    • Refactors dataset creation to occur in helper method
      • Dataset creation fails sometimes and should be included when retrying

Fixes #7463, Fixes #7456, Fixes #7446

@product-auto-label product-auto-label Bot added the samples Issues that are directly related to samples. label Dec 10, 2022
@kweinmeister kweinmeister changed the title test: add retries to flaky Asset and AutoML tests test: increased timeout and added retries to flaky tests Dec 10, 2022
@kweinmeister kweinmeister marked this pull request as ready for review December 10, 2022 18:38
@kweinmeister kweinmeister requested review from a team and yoshi-approver as code owners December 10, 2022 18:38
@averikitsch averikitsch merged commit 1ee87d5 into main Dec 12, 2022
@averikitsch averikitsch deleted the fix-asset branch December 12, 2022 17:53
@minherz minherz mentioned this pull request Jun 27, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@averikitsch averikitsch averikitsch approved these changes

@yoshi-approver yoshi-approver Awaiting requested review from yoshi-approver yoshi-approver is a code owner

Assignees

@kurtisvg kurtisvg

Labels

samples Issues that are directly related to samples.

Projects

None yet

Milestone

No milestone

3 participants

@kweinmeister @averikitsch @kurtisvg