Skip to content

Add cancel and rename subscription API#5209

Merged
jhendrixMSFT merged 17 commits intoAzure:masterfrom
navysingla:nasingla/cancelRenameSpecs
May 2, 2019
Merged

Add cancel and rename subscription API#5209
jhendrixMSFT merged 17 commits intoAzure:masterfrom
navysingla:nasingla/cancelRenameSpecs

Conversation

@navysingla
Copy link
Contributor

Adding swagger

Latest improvements:

MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.

Contribution checklist:

  • I have reviewed the documentation for the workflow.
  • Validation tools were run on swagger spec(s) and have all been fixed in this PR.
  • The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

  • Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
  • adding/removing APIs.
  • adding/removing properties.
  • adding/removing API-version.
  • adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

  • If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
    Please follow the link to find more details on API review process.

Adding swagger
@msftclas
Copy link

msftclas commented Feb 15, 2019

CLA assistant check
All CLA requirements met.

@AutorestCI
Copy link

AutorestCI commented Feb 15, 2019

Automation for azure-sdk-for-js

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-js#2648

@AutorestCI
Copy link

AutorestCI commented Feb 15, 2019

Automation for azure-sdk-for-python

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-python#5040

@AutorestCI
Copy link

AutorestCI commented Feb 15, 2019

Automation for azure-sdk-for-ruby

Nothing to generate for azure-sdk-for-ruby

@AutorestCI
Copy link

AutorestCI commented Feb 15, 2019

Automation for azure-sdk-for-java

Nothing to generate for azure-sdk-for-java

@AutorestCI
Copy link

AutorestCI commented Feb 15, 2019

Automation for azure-sdk-for-node

A PR has been created for you:
Azure/azure-sdk-for-node#4925

@AutorestCI
Copy link

AutorestCI commented Feb 15, 2019

Automation for azure-sdk-for-go

Nothing to generate for azure-sdk-for-go

@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@jianghaolu
Copy link
Contributor

@azuresdkci add to whitelist

@jianghaolu
Copy link
Contributor

The spec you created is not parsable - please fix. Also, can you change your title to "Add xxx version of billing.json". The title will become the commit message when merged.

@navysingla navysingla changed the title Create billing.json Add 2018-11-01-preview version billing.json Feb 15, 2019
@navysingla
Copy link
Contributor Author

navysingla commented Feb 15, 2019 via email

@navysingla
Copy link
Contributor Author

The spec you created is not parsable - please fix. Also, can you change your title to "Add xxx version of billing.json". The title will become the commit message when merged.

@navysingla navysingla closed this Feb 15, 2019
@navysingla navysingla reopened this Feb 15, 2019
@majastrz majastrz added the WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required label Feb 16, 2019
@jhendrixMSFT
Copy link
Member

Your new swagger isn't referenced in the README file, can you please fix?
Also have these APIs been deployed yet? We can't merge this PR until then.

@jhendrixMSFT
Copy link
Member

@navysingla any update?

@jhendrixMSFT
Copy link
Member

Closing due to inactivity. Please re-open when ready to engage.

Changes in readme file
updated default package
@jhendrixMSFT
Copy link
Member

Your example files aren't wired up to their operations, please see here for an example how to do this. It' causing some of the linter failures, also missing is the operations API. Does your service have it?

Fixed example links
adxsdknet added a commit to adxsdknet/azure-sdk-for-net that referenced this pull request May 1, 2019
REST Spec PR 'Azure/azure-rest-api-specs#5209'
REST Spec PR Author 'navysingla'
REST Spec PR Last commit
adxsdknet added a commit to adxsdknet/azure-sdk-for-net that referenced this pull request May 1, 2019
REST Spec PR 'Azure/azure-rest-api-specs#5209'
REST Spec PR Author 'navysingla'
REST Spec PR Last commit
@jhendrixMSFT
Copy link
Member

Is there an issue tracking adding the missing operations API? Also can you please confirm these APIs have been deployed?

@navysingla
Copy link
Contributor Author

navysingla commented May 1, 2019

APIs have been deployed, we already have operations API. Added in the swagger too

adxsdknet added a commit to adxsdknet/azure-sdk-for-net that referenced this pull request May 2, 2019
REST Spec PR 'Azure/azure-rest-api-specs#5209'
REST Spec PR Author 'navysingla'
REST Spec PR Last commit
@jhendrixMSFT jhendrixMSFT merged commit 4d7bf5b into Azure:master May 2, 2019
dsgouda pushed a commit to Azure/azure-sdk-for-net that referenced this pull request May 2, 2019
* .NET SDK Resource Provider:'Subscription'
REST Spec PR 'Azure/azure-rest-api-specs#5209'
REST Spec PR Author 'navysingla'
REST Spec PR Last commit

* .NET SDK Resource Provider:'Subscription'
REST Spec PR 'Azure/azure-rest-api-specs#5209'
REST Spec PR Author 'navysingla'
REST Spec PR Last commit
mentat9 pushed a commit to mentat9/azure-sdk-for-net that referenced this pull request Jun 10, 2019
* .NET SDK Resource Provider:'Subscription'
REST Spec PR 'Azure/azure-rest-api-specs#5209'
REST Spec PR Author 'navysingla'
REST Spec PR Last commit

* .NET SDK Resource Provider:'Subscription'
REST Spec PR 'Azure/azure-rest-api-specs#5209'
REST Spec PR Author 'navysingla'
REST Spec PR Last commit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants

Comments