Skip to content

sdk for stats API#1614

Merged
jianghaolu merged 29 commits intoAzure:masterfrom
dihan0604:master
Dec 4, 2015
Merged

sdk for stats API#1614
jianghaolu merged 29 commits intoAzure:masterfrom
dihan0604:master

Conversation

@dihan0604
Copy link
Copy Markdown
Contributor

No description provided.

@azuresdkci
Copy link
Copy Markdown
Contributor

Can one of the admins verify this patch?

@azurecla
Copy link
Copy Markdown

Hi @dihan0604, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!


It looks like you're working at Microsoft (dihan). If you're full-time, we DON'T require a contribution license agreement.



If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@stankovski
Copy link
Copy Markdown
Member

Please fix the tests and add coverage for the new operations.

@dihan0604
Copy link
Copy Markdown
Contributor Author

Hi Denis,
I have added recorded test as your suggested. Could you take a look and merge it?

Thanks
Ding

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why is this version changed? Did you make any changes to the Network libraries?

@dihan0604
Copy link
Copy Markdown
Contributor Author

We added some new API in Network libraries.

@jianghaolu
Copy link
Copy Markdown
Contributor

Hi @dihan0604 I don't see any network library changes here, is there anything I'm missing?

@nupurkot
Copy link
Copy Markdown
Contributor

nupurkot commented Dec 3, 2015

LGTM

@dihan0604
Copy link
Copy Markdown
Contributor Author

Have addressed the comment, please merge it. I updated the express route version and revert the one for network

@jianghaolu
Copy link
Copy Markdown
Contributor

@azuresdkci retest this please

@jianghaolu
Copy link
Copy Markdown
Contributor

The master branch is failing due to another merged bad pull request. Should be fixed in a bit.

@yugangw-msft
Copy link
Copy Markdown
Contributor

@dihan0604, please fetch latest master and merge, so to trigger CI run. We have reverted problematic PRs to clean up the current master branch

jianghaolu added a commit that referenced this pull request Dec 4, 2015
@jianghaolu jianghaolu merged commit fd76a1c into Azure:master Dec 4, 2015
gucalder added a commit to AuxMon/azure-sdk-for-net that referenced this pull request Aug 31, 2017
…erations do not throw standard exceptions, but return ErrorResponses with more information (PR Azure#1614).
shahabhijeet pushed a commit that referenced this pull request Sep 7, 2017
…Action Groups API, and fixing some documentation typos (from Swagger) (#3646)

* Adding ActivityLogAlerts and ActionGroups API calls, PATH operations, and the corresponding unit tests

* Generating code from the 'current' branch of the Swagger specs repo

* Adding scenario tests

* Improving scenario tests and re-recording them

* Making sure the scenario tests point to the more recent version of the Resource Manager dll. Enabling tests for netcoreapp1.1

* Introducing multidimensional metrics API and removing the previous one.

* Reverting netcoreapp1.1 test target since it fails to find the records.

* Returning generate.md to its Azure\psSdkJson6 state

* Re-enabling tests for netcoreapp1.1

* [Monitor] Adding multi-dim metrics API and making sure ActionGroupsOperations do not throw standard exceptions, but return ErrorResponses with more information (PR #1614).

* [Monitor] Fixing documentation typos (from Swagger). Improving the error response for Action Groups API.
JasonYang-MSFT pushed a commit to JasonYang-MSFT/azure-sdk-for-net that referenced this pull request Dec 8, 2017
…Action Groups API, and fixing some documentation typos (from Swagger) (Azure#3646)

* Adding ActivityLogAlerts and ActionGroups API calls, PATH operations, and the corresponding unit tests

* Generating code from the 'current' branch of the Swagger specs repo

* Adding scenario tests

* Improving scenario tests and re-recording them

* Making sure the scenario tests point to the more recent version of the Resource Manager dll. Enabling tests for netcoreapp1.1

* Introducing multidimensional metrics API and removing the previous one.

* Reverting netcoreapp1.1 test target since it fails to find the records.

* Returning generate.md to its Azure\psSdkJson6 state

* Re-enabling tests for netcoreapp1.1

* [Monitor] Adding multi-dim metrics API and making sure ActionGroupsOperations do not throw standard exceptions, but return ErrorResponses with more information (PR Azure#1614).

* [Monitor] Fixing documentation typos (from Swagger). Improving the error response for Action Groups API.
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.

7 participants