Skip to content

Commit

Permalink
Merge pull request #2 from Azure/master
Browse files Browse the repository at this point in the history
Merge Master
  • Loading branch information
david-msft authored Jan 6, 2021
2 parents a5c1e62 + a5dcb30 commit 4871a17
Show file tree
Hide file tree
Showing 972 changed files with 120,122 additions and 1,046 deletions.
11 changes: 9 additions & 2 deletions .github/comment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
label: BreakingChangeReviewRequired
variables:
openapiHub: https://portal.azure-devex-tools.com
onLabeledComments: "Hi @${PRAuthor}, one or multiple breaking change(s) is detected in your PR. Please check out the breaking change(s), and provide business justification in the PR comment and @ PR assignee why you must have these change(s), and how external customer impact can be mitigated. Please ensure to follow [breaking change policy](https://aka.ms/AzBreakingChangesPolicy) to request breaking change review and approval before proceeding swagger PR review. </br>**Action**: To initiate an evaluation of the breaking change, create a new intake using the [template for breaking changes](https://aka.ms/Breakingchangetemplate). Addition details on the process and office hours are on the [Breaking change Wiki](https://dev.azure.com/msazure/AzureWiki/_wiki/wikis/AzureWiki.wiki/37684/Breaking-Changes).</br> If you want to know the production traffic statistic, please see [ARM Traffic statistic](${openapiHub}/api/pr/query-traffic?pr=${owner}/${repo}/${PRNumber}&days=5). </br> If you think it is false positive breaking change, please provide the reasons in the PR comment, report to Swagger Tooling Team via https://aka.ma/swaggerfeedback."
onLabeledComments: "Hi @${PRAuthor}, one or multiple breaking change(s) is detected in your PR. Please check out the breaking change(s), and provide business justification in the PR comment and @ PR assignee why you must have these change(s), and how external customer impact can be mitigated. Please ensure to follow [breaking change policy](https://aka.ms/AzBreakingChangesPolicy) to request breaking change review and approval before proceeding swagger PR review. </br>**Action**: To initiate an evaluation of the breaking change, create a new intake using the [template for breaking changes](https://aka.ms/Breakingchangetemplate). Addition details on the process and office hours are on the [Breaking change Wiki](https://dev.azure.com/msazure/AzureWiki/_wiki/wikis/AzureWiki.wiki/37684/Breaking-Changes).</br> If you want to know the production traffic statistic, please see [ARM Traffic statistic](${openapiHub}/api/pr/query-traffic?pr=${owner}/${repo}/${PRNumber}&days=5). </br> If you think it is false positive breaking change, please provide the reasons in the PR comment, report to Swagger Tooling Team via https://aka.ms/swaggerfeedback."

- rule:
type: label
Expand All @@ -33,7 +33,14 @@
- rule:
type: label
label: CI-FixRequiredOnFailure
onLabeledComments: "Hi @${PRAuthor}, Your PR has some issues. Please fix the CI sequentially by following the order of `Avocado, semantic validation, model validation, breaking change, lintDiff`. <table><tr><th>Task</th><th>How to fix</th><th>Priority</th><th>Support</th></tr><tr><td>Avocado</td><td>[Fix-Avocado](https://github.com/Azure/avocado)</td><td>High</td><td>ruowan@microsoft.com</td></tr><tr><td>LintDiff</td><td>[Fix-LintDiff](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/ci-fix.md#linter-validation)</td><td>high</td><td>Jianye.Xi@microsoft.com, ruoxuan@microsoft.com</td></tr><tr><td>Semantic validation</td><td>[Fix-SenmanticValidation-Error](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/ci-fix.md#semantic-validation)</td><td>High</td><td>raychen@microsoft.com, Jianye.xi@microsoft.com</td></tr><tr><td>Model validation</td><td>[Fix-ModelValidation-Error](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/ci-fix.md#model-validation)</td><td>High</td><td>raychen@microsoft.com,Jianye.xi@microsoft.com</td></tr></table> If you need further help, please feedback via [swagger feedback](https://aka.ms/swaggerfeedback)."
onLabeledComments: >-
Hi @${PRAuthor}, Your PR has some issues. Please fix the CI sequentially by following the order of `Avocado, semantic validation, model validation, breaking change, lintDiff`.
<table><tr><th>Task</th><th>How to fix</th><th>Priority</th><th>Support (Microsoft alias)</th></tr>
<tr><td>Avocado</td><td>[Fix-Avocado](https://github.com/Azure/avocado)</td><td>High</td><td>ruowan</td></tr>
<tr><td>Semantic validation</td><td>[Fix-SemanticValidation-Error](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/ci-fix.md#semantic-validation)</td><td>High</td><td>raychen, jianyxi</td></tr>
<tr><td>Model validation</td><td>[Fix-ModelValidation-Error](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/ci-fix.md#model-validation)</td><td>High</td><td>raychen,jianyxi</td></tr>
<tr><td>LintDiff</td><td>[Fix-LintDiff](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/ci-fix.md#linter-validation)</td><td>high</td><td>jianyxi, ruoxuan</td></tr></table>
If you need further help, please feedback via [swagger feedback](https://aka.ms/swaggerfeedback)."
- rule:
type: PROpen
Expand Down
4 changes: 4 additions & 0 deletions custom-words.txt
Original file line number Diff line number Diff line change
Expand Up @@ -696,6 +696,7 @@ getcertificate
getconsumergroup
GETCONTENTSUMMARY
GETFILESTATUS
getikesas
getissuers
getitems
getjob
Expand Down Expand Up @@ -1063,6 +1064,7 @@ MSAZR
MSAZRDE
MSAZRUSGO
MSCONCAT
mscv
MSGETACLSTATUS
MSGETFILESTATUS
msix
Expand Down Expand Up @@ -1417,6 +1419,8 @@ remediate
remediated
Remediations
remoteloginsettings
remoterendering
remoterenderingvm
remoteuser
REMOVEACL
REMOVEACLENTRIES
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,12 @@
"schema": {
"$ref": "#/definitions/ARMErrorResponseBody"
}
},
"default": {
"description": "Error response describing why the operation failed.",
"schema": {
"$ref": "#/definitions/ArmErrorResponse"
}
}
},
"x-ms-examples": {
Expand Down Expand Up @@ -74,6 +80,12 @@
"schema": {
"$ref": "#/definitions/MetadataEntityListResult"
}
},
"default": {
"description": "Error response describing why the operation failed.",
"schema": {
"$ref": "#/definitions/ArmErrorResponse"
}
}
},
"x-ms-examples": {
Expand Down Expand Up @@ -330,6 +342,12 @@
"type": "string"
}
}
},
"default": {
"description": "Error response describing why the operation failed.",
"schema": {
"$ref": "#/definitions/ArmErrorResponse"
}
}
},
"deprecated": false,
Expand Down Expand Up @@ -369,6 +387,12 @@
},
"204": {
"description": "NoContent. Recommendation generation has been completed."
},
"default": {
"description": "Error response describing why the operation failed.",
"schema": {
"$ref": "#/definitions/ArmErrorResponse"
}
}
},
"deprecated": false,
Expand Down Expand Up @@ -405,7 +429,8 @@
"in": "query",
"description": "The number of recommendations per page if a paged version of this API is being used.",
"required": false,
"type": "integer"
"type": "integer",
"format": "int32"
},
{
"name": "$skipToken",
Expand All @@ -421,6 +446,12 @@
"schema": {
"$ref": "#/definitions/ResourceRecommendationBaseListResult"
}
},
"default": {
"description": "Error response describing why the operation failed.",
"schema": {
"$ref": "#/definitions/ArmErrorResponse"
}
}
},
"deprecated": false,
Expand Down Expand Up @@ -492,6 +523,12 @@
"schema": {
"$ref": "#/definitions/ResourceRecommendationBase"
}
},
"default": {
"description": "Error response describing why the operation failed.",
"schema": {
"$ref": "#/definitions/ArmErrorResponse"
}
}
},
"deprecated": false,
Expand Down Expand Up @@ -547,6 +584,12 @@
"schema": {
"$ref": "#/definitions/ArmErrorResponse"
}
},
"default": {
"description": "Error response describing why the operation failed.",
"schema": {
"$ref": "#/definitions/ArmErrorResponse"
}
}
},
"deprecated": false,
Expand Down Expand Up @@ -609,6 +652,12 @@
"schema": {
"$ref": "#/definitions/ArmErrorResponse"
}
},
"default": {
"description": "Error response describing why the operation failed.",
"schema": {
"$ref": "#/definitions/ArmErrorResponse"
}
}
},
"deprecated": false,
Expand Down Expand Up @@ -653,6 +702,12 @@
"responses": {
"204": {
"description": "NoContent. The recommendation has been activated."
},
"default": {
"description": "Error response describing why the operation failed.",
"schema": {
"$ref": "#/definitions/ArmErrorResponse"
}
}
},
"deprecated": false,
Expand Down Expand Up @@ -699,6 +754,12 @@
"schema": {
"$ref": "#/definitions/SuppressionContractListResult"
}
},
"default": {
"description": "Error response describing why the operation failed.",
"schema": {
"$ref": "#/definitions/ArmErrorResponse"
}
}
},
"deprecated": false,
Expand Down
Loading

0 comments on commit 4871a17

Please sign in to comment.