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.

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

Add Testing of Conda Artifacts #18478

Merged
merged 130 commits into from
May 18, 2021
Merged

Add Testing of Conda Artifacts #18478

merged 130 commits into from
May 18, 2021

Conversation

scbedd
Copy link
Member

@scbedd scbedd commented May 2, 2021

azure-devops and conda ...0/10 would not recommend.

Note that while I'll take feedback on the step Evaluate OS Specific PATH and Parameters in test-conda.yml, if it looks hacky, there is probably a reason for it.

@benbp I have extended archetype-sdk-tests-generate.yml in this PR because I wanted a nice way to filter out running tests on certain python versions (we only need to support >=3.6 for conda). Rather than doing wonky conditions on every step, I merely extended our usage of archetype-sdk-tests-generate so that we invoke it twice. Once for when we do have conda artifacts to test, once for our regular tests..

Reason being, even if we used parameter sets functionality in Create-Job-Matrix.ps1, we still run into the issue where we still have to have a condition attached to every step within test-conda (wherever the steps live). This is due to the fact that the variables are essentially only available at runtime, which will means we can't use any compile-time ${{ if to keep our logs clean.

scbedd and others added 30 commits April 26, 2021 14:41
@scbedd scbedd closed this May 17, 2021
@scbedd scbedd reopened this May 18, 2021
@scbedd
Copy link
Member Author

scbedd commented May 18, 2021

Now azure-storage has issues. I don't want to spam the build system and eat more minutes than I already have. Closing again temporarily.

@scbedd scbedd closed this May 18, 2021
@scbedd scbedd reopened this May 18, 2021
@scbedd scbedd merged commit 4398c15 into master May 18, 2021
@scbedd scbedd deleted the feature/add-conda-testing branch May 18, 2021 17:45
iscai-msft added a commit to iscai-msft/azure-sdk-for-python that referenced this pull request May 18, 2021
…into switch_to_westus_2

* 'master' of https://github.com/Azure/azure-sdk-for-python: (28 commits)
  Add Testing of Conda Artifacts (Azure#18478)
  reset continuation token (Azure#18772)
  pipline prepare (Azure#18755)
  [AutoRelease] t2-machinelearningcompute-2021-05-14-14426(wave4) (Azure#18750)
  [AutoRelease] t2-iothubprovisioningservices-2021-05-14-29542(wave4) (Azure#18748)
  [AutoRelease] t2-agfood-2021-05-17-53436 (Azure#18768)
  unskip custom form tests - unlabeled table bounding box regression fixed (Azure#18776)
  [Tables] test cleanup (Azure#18676)
  adding mindependency to the default set of PR tests (Azure#18775)
  updating sample for beta2 api (Azure#18711)
  useless folder del (Azure#18771)
  [AutoRelease] t2-recoveryservicesbackup-2021-05-14-03506(wave4) (Azure#18749)
  [AutoRelease] t2-network-2021-05-14-21408(Do not merge) (Azure#18746)
  [AutoRelease] t2-iothub-2021-05-14-97452 (Azure#18753)
  Rename to AZURE_PHONE_NUMBER (Azure#18760)
  [Communication]: Removed tests.yml file for single pipeline (Azure#18684)
  [Storage][ChangeLog]STG77 Changelog (Azure#18678)
  [Communication]: Added unit tests to check if idempotence parameters are being set (Azure#18739)
  allow multiple invocations of archetype-sdk-tests-generate (Azure#18759)
  [Media] Ava sdk update (Azure#18359)
  ...
iscai-msft added a commit to agniveshadhikari/azure-sdk-for-python that referenced this pull request May 18, 2021
…into agrifood/test-infra

* 'master' of https://github.com/Azure/azure-sdk-for-python: (33 commits)
  Increment version for textanalytics releases (Azure#18797)
  remove PHI with multiple actions due to known service error (Azure#18794)
  [textanalyitcs] switch to westus2 region (Azure#18791)
  Remove implicit dependency on successful completion of verify-links (Azure#18277)
  Add Testing of Conda Artifacts (Azure#18478)
  reset continuation token (Azure#18772)
  pipline prepare (Azure#18755)
  [AutoRelease] t2-machinelearningcompute-2021-05-14-14426(wave4) (Azure#18750)
  [AutoRelease] t2-iothubprovisioningservices-2021-05-14-29542(wave4) (Azure#18748)
  [AutoRelease] t2-agfood-2021-05-17-53436 (Azure#18768)
  unskip custom form tests - unlabeled table bounding box regression fixed (Azure#18776)
  [Tables] test cleanup (Azure#18676)
  adding mindependency to the default set of PR tests (Azure#18775)
  updating sample for beta2 api (Azure#18711)
  useless folder del (Azure#18771)
  [AutoRelease] t2-recoveryservicesbackup-2021-05-14-03506(wave4) (Azure#18749)
  [AutoRelease] t2-network-2021-05-14-21408(Do not merge) (Azure#18746)
  [AutoRelease] t2-iothub-2021-05-14-97452 (Azure#18753)
  Rename to AZURE_PHONE_NUMBER (Azure#18760)
  [Communication]: Removed tests.yml file for single pipeline (Azure#18684)
  ...
iscai-msft added a commit to iscai-msft/azure-sdk-for-python that referenced this pull request May 28, 2021
…into protocol_base

* 'master' of https://github.com/Azure/azure-sdk-for-python: (365 commits)
  Agrifood/test infra (Azure#18779)
  [formrecognizer] update samples to train a model where a model_id is needed (Azure#18789)
  replace response urls too (Azure#18758)
  [AutoRelease] t2-resource-2021-05-19-78417 (Azure#18807)
  [pipeline]update autorest version for pipeline (Azure#18782)
  [AutoRelease] t2-digitaltwins-2021-05-18-25594(wave4) (Azure#18786)
  [AutoRelease] t2-hybridkubernetes-2021-05-19-49364(wave4) (Azure#18802)
  Added Async APIs and Samples for attestation SDK (Azure#18735)
  [AVA] Updating links in package ReadMe (Azure#18800)
  Sync eng/common directory with azure-sdk-tools for PR 1585 (Azure#18793)
  Increment version for textanalytics releases (Azure#18797)
  remove PHI with multiple actions due to known service error (Azure#18794)
  [textanalyitcs] switch to westus2 region (Azure#18791)
  Remove implicit dependency on successful completion of verify-links (Azure#18277)
  Add Testing of Conda Artifacts (Azure#18478)
  reset continuation token (Azure#18772)
  pipline prepare (Azure#18755)
  [AutoRelease] t2-machinelearningcompute-2021-05-14-14426(wave4) (Azure#18750)
  [AutoRelease] t2-iothubprovisioningservices-2021-05-14-29542(wave4) (Azure#18748)
  [AutoRelease] t2-agfood-2021-05-17-53436 (Azure#18768)
  ...
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-python that referenced this pull request Apr 8, 2022
New Api-Version onboarding 2021-12-01-preview (Azure#18478)

* Adding 2021-12-01-preview as a copy of 2021-08-01

* Readme update

* Updating version in the 2021-12-01-preview folder

* update description for keyvault contract (Azure#17150)

Co-authored-by: Kacie Kang <jikang@microsoft.com>

* fix for the last cherry pick (06d2de8)

* fixing readme again

* APIM Policy fragments (Azure#16403)

* Adding the policy fragments spec

* Formatting

* Updating descriptions

* PolicyFragmentIdParameter as a string

* ID validation

* Adding fragment description

* Adding policy fragment format

* Adding raw xml format

* Addinga 202 response for creating/updating

* Generalising the resource collection

* formatting

* Moving everything to 2021-12-01

* version fix

* Adding pagination params. Generalising the IdParam

* Adding missing fields, fixing id param name

* Removing link formats. Fixing format param name

* POST list references

* name filter now supports "contains"

* PUT is always async

* Fixing header location url

* PUT can now return 200 and 201

* Adding more OData query support. Fixing examples.

* Resolving the cross breaking change

Co-authored-by: Milan Zolota <mizolota@microsoft.com>

* Added CORS and CSP configuration resources (Azure#16822)

* Added CORS and CSP settings resources.

* Added missing definitions.

* Added suppression of R4009 for apimportalsettings.json.

* Ran Prettier.

* Renamed definitions.

* Fixed references.

* Updated examples.

* Reversed change.

* Created new resource for portal settings.

* Added list example.

* Added missing collection contract.

* Added missing example. Cleaned up old ones.

* Fixed references.

* Fixed examples

* Fixed version.

* Added custom words. Fixed linting.

* Fixes.

* Fixes.

* Fixes.

* Fixes.

* Fixes.

* Fixes.

* Fixes.

* Fixes.

* Fixes.

* Fixes.

* Fixes

* Fixes.

* Fixes.

* Fixes

* Fixes

* Removed conflicting constraint.

* Updated readme.go.md.

* Reverted removing the restriction.

* Revert "Reverted removing the restriction."

This reverts commit 72d070f6ccc8231e4d4c9f22ac3a9c3090a545aa.

* Removed duplicate.

* Addressing PR comments.

* Adjusted reportUri property type in CSP config contract. (Azure#17947)

* Added missing body parameter description for content type resource. (Azure#18036)

* Igo/msal client version (Azure#17866)

* Added clientLibrary property to IdentityProvider

* fixed length for clientLibrary property

* Model validation fixes

* moved changes for clientLibrary property to the right spec file

* Update readme.md

Co-authored-by: Milan Zolota <mizolota@microsoft.com>
Co-authored-by: Kacie <35987382+KacieKK@users.noreply.github.com>
Co-authored-by: Kacie Kang <jikang@microsoft.com>
Co-authored-by: Alexander Zaslonov <zaslonov.alexander@gmail.com>
Co-authored-by: Igor O <ygrik@hotmail.com>
Co-authored-by: Yuchao Yan <yuchaoyan@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Azure.Core Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants