Skip to content

[ML][Pipelines] Test: update unstable tests in ci#28157

Merged
wangchao1230 merged 3 commits intoAzure:mainfrom
elliotzh:zhangxingzhi/fix-unstable-tests
Jan 6, 2023
Merged

[ML][Pipelines] Test: update unstable tests in ci#28157
wangchao1230 merged 3 commits intoAzure:mainfrom
elliotzh:zhangxingzhi/fix-unstable-tests

Conversation

@elliotzh
Copy link
Contributor

@elliotzh elliotzh commented Jan 4, 2023

Description

For some reason, tests involving in-repo file operations are unstable:
image

Do those file operations in temp folder in this PR to make them more stable.

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

@ghost ghost added the Machine Learning label Jan 4, 2023
@elliotzh elliotzh added the ML-Pipelines AreaPath label Jan 4, 2023
@elliotzh elliotzh changed the title fix: use a temp folder for each asset utils test fix: update unstable tests in ci Jan 4, 2023
@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

@elliotzh elliotzh force-pushed the zhangxingzhi/fix-unstable-tests branch from 2826235 to fc8c5e5 Compare January 5, 2023 02:39
@elliotzh elliotzh changed the title fix: update unstable tests in ci [ML][Pipelines] Test: update unstable tests in ci Jan 6, 2023
@wangchao1230 wangchao1230 merged commit c5e1e85 into Azure:main Jan 6, 2023
iscai-msft added a commit that referenced this pull request Apr 26, 2023
…into version_tolerant_models

* 'main' of https://github.com/Azure/azure-sdk-for-python: (1604 commits)
  [Storage] Matching `_shared` folder contents across all packages (#28031)
  [ml] Rename MLException back to MlException (#28210)
  [EventHubs] update readme to include uamqp backcompat section (#28198)
  [Monitor][Ingestion] Adjust upload error handling (#27556)
  Add 3.11 to QA (#28169)
  p1 component deployment exisitng job definition (#28194)
  Typing for identity (#28159)
  [ServiceBus] Allow clearing of session state (#27726)
  [ML][Pipelines] Fix parallel-for validation rule: allow empty dict as loop config item (#28178)
  [ML][Pipelines] feat: enable concurrent component registration (#28118)
  [ML][Pipelines] Test: update invalid reuse test (#28180)
  [AutoRelease] t2-loadtesting-2022-12-23-48532(can only be merged by SDK owner) (#28042)
  [ML][Pipelines] Test: update unstable tests in ci (#28157)
  [ML]: Adding experimental decorator to JobDefinition (#28200)
  [Identity] Fix 'az' existence check in CliCred (#27991)
  [EventHubs] fix tracing bug (#28149)
  ShortForMigrateFrom (#28192)
  Address the API review comments (#28188)
  [SchemaRegistryAvro] update recordings (#28158)
  [EventHub] update uamqp_transport docstring (#28186)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants