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

[Communication] Add variables necessary for using new azure subscription for SMS live tests #15213

Merged
merged 5 commits into from
Nov 14, 2020

Conversation

turalf
Copy link
Contributor

@turalf turalf commented Nov 10, 2020

We are using the new subscription for our testing.
The new subscription id is added to the keyvault and this PR changes the tests.yml to access that subscriptionId.
This PR also enabled the SMS tests based on the static resources which live in a new subscription.

@@ -15,7 +15,7 @@
ResponseReplacerProcessor
)

SKIP_PHONE_NUMBER_TESTS = True
SKIP_PHONE_NUMBER_TESTS = False
PHONE_NUMBER_TEST_SKIP_REASON= "Phone Number infra for live tests not ready yet"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: remove this comment?

ankitarorabit
ankitarorabit previously approved these changes Nov 12, 2020
@turalf
Copy link
Contributor Author

turalf commented Nov 12, 2020

/azp run python - communication - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

ankitarorabit
ankitarorabit previously approved these changes Nov 12, 2020
@turalf
Copy link
Contributor Author

turalf commented Nov 12, 2020

/azp run python - communication - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@turalf
Copy link
Contributor Author

turalf commented Nov 13, 2020

/azp run python - communication - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@turalf
Copy link
Contributor Author

turalf commented Nov 13, 2020

/azp run python - communication - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@turalf turalf merged commit 402533e into Azure:master Nov 14, 2020
@turalf turalf deleted the add-env-vars branch November 14, 2020 00:02
iscai-msft added a commit to iscai-msft/azure-sdk-for-python that referenced this pull request Nov 14, 2020
…into new_polling

* 'master' of https://github.com/Azure/azure-sdk-for-python: (281 commits)
  [Communication] Add variable groups necessary for using new azure subscription (Azure#15213)
  Sync eng/common directory with azure-sdk-tools for PR 1153 (Azure#15322)
  [Communication] Update version for Preview 3 release (Azure#15317)
  Increment package version after release of azure_data_tables (Azure#15291)
  [monitor] README + samples _ ci.yml (Azure#15288)
  refdoc aka.ms link was broken and silently redirecting, have replaced with azuresdkdocs link as is used in other SDKs. (Azure#15169)
  Sync eng/common directory with azure-sdk-tools for PR 1202 (Azure#15297)
  Updating OT exporter to consume latest version of OpenTelemetry API/SDK (Azure#15289)
  Change live test resource DeleteAfterHours tag to 8 hours (Azure#15294)
  Add Invoke-DevOpsAPI.ps1, Add functions for Canceling and Listing Builds (Azure#15013)
  The original one is removed. Point to the new one (Azure#15287)
  Sync eng/common directory with azure-sdk-tools for PR 1170 (Azure#15087)
  Adding OpenTelemetry exporter code (Azure#14784)
  update external link (Azure#15273)
  Remove invalid characters in basename sourced from username (Azure#15268)
  Sync eng/common directory with azure-sdk-tools for PR 1188 (Azure#15267)
  Increment version for core releases (Azure#15269)
  Increment package version after release of azure_identity (Azure#15266)
  Fix CI (Azure#15265)
  Increment package version after release of azure_eventgrid (Azure#15262)
  ...
@turalf turalf changed the title [Communication] Add variable groups necessary for using new azure subscription [Communication] Add variable groups necessary for using new azure subscription for SMS tests Nov 16, 2020
@turalf turalf changed the title [Communication] Add variable groups necessary for using new azure subscription for SMS tests [Communication] Add variables necessary for using new azure subscription for SMS live tests Nov 16, 2020
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-python that referenced this pull request Aug 27, 2021
[NetAppFiles] s360 fixes to 2021-01 (Azure#15213)

* Apply s360 fixes

* Fix typo

* Rename op, add example

* Remove snapshotpolicy listVolumes false positive, fix description for snapshotNamne property all versions

* ReplicationSceheduled is no longer required in ReplicationObject

Co-authored-by: Audunn Baldvinsson <audunn.baldvinsson@netapp.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants