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

[App Config] skip secret reference sample #19957

Merged
merged 1 commit into from
Jan 21, 2022

Conversation

HarshaNalluru
Copy link
Member

@HarshaNalluru HarshaNalluru commented Jan 20, 2022

Skipping the sample since it needs Key Vault and AAD resources as well along with the App Config resource.
It might be too costly to create the resource for just one sample.

@ghost ghost added the App Configuration Azure.ApplicationModel.Configuration label Jan 20, 2022
@HarshaNalluru
Copy link
Member Author

/azp run js - app-configuration - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@HarshaNalluru HarshaNalluru marked this pull request as ready for review January 20, 2022 23:05
@HarshaNalluru HarshaNalluru merged commit d9fe264 into Azure:main Jan 21, 2022
@HarshaNalluru HarshaNalluru deleted the harshan/issue/19617 branch January 21, 2022 19:03
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-js that referenced this pull request Aug 31, 2022
add x-ms-identifiers for hdinsight (Azure#19957)

* add x-ms-identifiers for hdinsight

* Update cluster.json
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.

2 participants