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

Merge main to fork #7

Merged
merged 131 commits into from
Dec 27, 2020
Merged

Merge main to fork #7

merged 131 commits into from
Dec 27, 2020

Conversation

MichalHel
Copy link
Owner

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

Changelog

Please ensure to add changelog with this PR by answering the following questions.

  1. What's the purpose of the update?
    • new service onboarding
    • new API version
    • update existing version for new feature
    • update existing version to fix swagger quality issue in s360
    • Other, please clarify
  2. When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
  3. When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
  4. If it's an update to existing version, please select SDKs of specific language and CLIs that require refresh after swagger is published.
    • SDK of .NET (need service team to ensure code readiness)
    • SDK of Python
    • SDK of Java
    • SDK of Js
    • SDK of Go
    • PowerShell
    • CLI
    • Terraform
    • No, no need to refresh for updates in this PR

Contribution checklist:

If any further question about AME onboarding or validation tools, please view the FAQ.

ARM API Review Checklist

  • Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

    • Adding new API(s)
    • Adding a new API version
    • Adding a new service
  • Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

  • If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist

If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

  • Removing API(s) in stable version
  • Removing properties in stable version
  • Removing API version(s) in stable version
  • Updating API in stable version with Breaking Change Validation errors
  • Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.

Please follow the link to find more details on PR review process.

e337 and others added 30 commits December 8, 2020 12:36
Co-authored-by: Eunice Lee <eunl@microsoft.com>
* Added support for all API in 2018-preview version

* removed extra files

* fixed UNREFERENCED_JSON_FILE

* update

* prettier fix

Co-authored-by: v-ajnava <v-ajnava@microsoft.com>
…r BYOK auto rotation (#11966)

Co-authored-by: karuna <karkrish@microsoft.com>
…11960)

* update ResourceIdentity reference

* Change IdentityTypeWithNone to IdentityType
all readme file change request code-gen team review.
#11995)

* LVA: move LVA swaggers 2.0 version from private repo to public repo for SDK generation

* fix PR failures

* remove examples as not needed anymore.

* update readme description

* mark method name as required.

* add a supression for RequiredReadOnlyProperties for methodName
Co-authored-by: Jorin Mejia <jomejia@microsoft.com>
* Update PULL_REQUEST_TEMPLATE.md

* Update comment.yml
Co-authored-by: Wan Yang <t-wayang@microsoft.com>
* Update types.json

* Update types.json

* Update types.json

* Update types.json

* Update types.json
* adding new restorable apis

* fix some typos

* fix validations

* fix more validations

* prettier

* fix name and add apitype

* fix casing

* add custom word etcd

* add mongodb

* fix prameter name

* fix validations

* make read only

* remove the word snapshot and make properties readonly

* pretty

* improved some descriptions

* update description and removed mongodb content

* fix container property name

* fix documentcollection name

* make query parameter optional

* address Meha's comments

* move restorable apis to separate file

* move restorable database apis back to cosmos-db.json

* remove point read apis

* fix avocado
* Added Access Reviews schema

* Access Reviews Schema addendum | Expose reviewer and statistics APIs
zhenglaizhang and others added 29 commits December 22, 2020 14:21
* cleanup pipeline deps

* add rest-api-specs-scripts
…iew/2020-02-01-preview (#12172)

* Adding operations for Maps Creator resource.

* Fixing typo.

* Adds suppression to readme

* Adds suppression to readme

* Adds suppression to readme

* Adds suppression to readme

* Resolving linter errors.

* Minor update.

* Updating all reference to v2 schema.

* Marked private atlas as deprecated in description.
Add ARM traffic query link. When add breakingChangeReviewRequired
* adding new properties

* updating examples

* prettier fix
)

* add AMCS config for codegen

* reorganize command group

* rename command group

* Codegen modify try to rename DataCollectionRules create parameters

* hide DataCollectionRules Create & Update commands and DataCollectionRuleAssociations Create command

* use alias instand of rename in code-gen

* use monitor-control-service as the extension name

* Update readme.python.md
* update TransparentDataEncryption Group name

* update reference of TransparentDataEncryption
* Fixes for SDK generation files.

* Fixes for SDK generation files.

* Added GO.

* Change Healthcare bot to Healthbot.

* 1. Remove unused properties of SKU.
2. Removed unused CheckNameAvailability.json
3. Remove subgroup.

* Sku is required.
…n stable/2018-09-01 (#12224)

* update to name

* update
* Initial commit as per PR review

* Actual changes to api spec for new version

* Switching to multi-api build for python

* Fix python spec to correct namespaces in multiapi mode

* Revert changes to python generation

* Add systemdata

* Fix the path for sys data reference

* Fix path once more

* remove additional properties after including systemdata

* Remove more addtional properties

* Move system data to root
…ion stable/2020-10-01 (#11407)

* Init the next version so diffs can work better.

* Updates readme

* Updates API version in new specs and examples

* Updates to StorageTargetProperties and added examples of cmk, mtu

Add 202 to the storage cache and storage target create/update and update examples

Fix ST no junction example's 202 response.

add properties for nfs extended groups

fix issues with extended groups properties found with autorest

add properties and objects for Active Directory username download

undo unintended change

changes from comments in pull request

AccessPolicy support.

Example fixes for Access Policies.

Fix attribute names to match latest RP.

update to credential properties for LDAP and Active Directory

marking password properties with x-ms-secret tag

minor changes on extended groups and add examples

Added blob NFS and some other validation fixes.

Update required property for domainName from dnsName

Updated blobNfs examples and some kpi fixes.

Correct validation errors in examples.

Added systemdata to resources.

Remove x-ms-secret in keyvault reference due to linter error and common types not using it.

Remove blobNfs from this version.

Remove blobNfs from spec file.

Remove x-ms-secret due to linter errors.

Fix certificate spelling.

Updating prettier and spell check errors.

Used prettier on main spec file.

Readded x-ms-secret that open api hub failed on but the PR pipeline allows.

* Add prettier fix after rebase

* Remove 202 bodies and add systemData to examples.

* Update spec with prettier.

* Address comments on spec for descriptions, readmes, and extendedGroupsEnabled.

* Updating to address addition ldap and pattern comments.

* Update version tag to include 01

* Restore changes that would be considered breaking changes matching 2020-03-01

* Address a few other comments.

* Update netbios field names and some descriptions.

* Fix s360 for missing debugInfo operation.

* Fix credscan error in example file.

* Update required fields, new validator failure on debugInfo return codes, and bindPassword example value.

* Update debug info example with new return codes.

* Update other credscan password errors.
@MichalHel MichalHel merged commit c0c9145 into MichalHel:master Dec 27, 2020
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.