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

[AppConfiguration] fix tests #17917

Merged
8 commits merged into from
Apr 13, 2021
Merged

Conversation

seankane-msft
Copy link
Member

No description provided.

@seankane-msft seankane-msft added the App Configuration Azure.ApplicationModel.Configuration label Apr 8, 2021
@seankane-msft seankane-msft self-assigned this Apr 8, 2021
@seankane-msft seankane-msft removed the request for review from xiangyan99 April 8, 2021 20:46
@seankane-msft
Copy link
Member Author

/azp run python - appconfiguration - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@seankane-msft
Copy link
Member Author

Removes skipped tests

@ghost
Copy link

ghost commented Apr 8, 2021

Hello @seankane-msft!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@seankane-msft
Copy link
Member Author

/azp run python - appconfiguration - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@seankane-msft seankane-msft linked an issue Apr 8, 2021 that may be closed by this pull request
@seankane-msft
Copy link
Member Author

/check-enforcer evaluate

@ghost ghost merged commit 80382ed into Azure:master Apr 13, 2021
@seankane-msft seankane-msft deleted the appconfig-fix-tests branch April 13, 2021 17:01
iscai-msft added a commit that referenced this pull request Apr 15, 2021
…into protocol_base

* 'master' of https://github.com/Azure/azure-sdk-for-python: (337 commits)
  [Container Registry] adding blurb about env vars to set (#17976)
  shared_requirements msrest verison update (#17938)
  Deprecate Azure-Synapse (#18027)
  add perf tests (#17519)
  Increment package version after release of azure-search-documents (#18017)
  [Communication]: Migrated python identity tests to use ARM template (#17890)
  [Container Registry] fixing up the test-resources-post script (#17883)
  [Communication]: Removed error code assertion from unauthorized phone number test (#17955)
  Add Conda Publishing (#17889)
  fix search fields (#18014)
  Sync eng/common directory with azure-sdk-tools for PR 1552 (#18011)
  [AppConfiguration] fix tests (#17917)
  updating generated code (#17874)
  Update document translation sample slug (#17989)
  update release date (#17988)
  [Communication] - SDK - Synced SMS and Phone Numbers shared folders (#17985)
  Remove unnecessary attribute access (#18007)
  [AutoRelease] t2-healthcareapis-2021-04-12-56802 (#17968)
  Revert retain. (#18004)
  Sync eng/common directory with azure-sdk-tools for PR 1553 (#18002)
  ...
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-python that referenced this pull request Feb 24, 2022
Update the label property of request body of find target resource api (Azure#17917)

* Update the label property of request body of find target resource api

It should be a map with the key of "label".

Right now the request body is defined as:
{
  "key":"value"
}

But it should be changed to:
{
  "labels":{
    "key":"value"
   }
}

* Update a description

* Fix validation check of example json file for find target resource group api
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App Configuration Azure.ApplicationModel.Configuration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[AppConfig] Label skipped issues with known issues or unskip
2 participants