Add serverSecurityAlertPolicies APIs for postgresql servers#2947
Add serverSecurityAlertPolicies APIs for postgresql servers#2947lmazuel merged 11 commits intoAzure:masterfrom ayeletshpigelman:master
Conversation
Automation for azure-libraries-for-javaA PR has been created for you based on this PR content. Once this PR will be merged, content will be added to your service PR: |
Automation for azure-sdk-for-pythonThe initial PR has been merged into your service PR: |
AutoRest linter results for SDK Related Validation Errors/WarningsThese errors are reported by the SDK team's validation tools, reachout to ADX Swagger Reviewers directly for any questions or concerns. File: specification/postgresql/resource-manager/readme.md
|
AutoRest linter results for ARM Related Validation Errors/WarningsThese errors are reported by the ARM team's validation tools, reachout to ARM RP API Review directly for any questions or concerns. File: specification/postgresql/resource-manager/readme.md
|
Automation for azure-sdk-for-nodeA PR has been created for you: |
Automation for azure-sdk-for-goA PR has been created for you: |
AutoRest linter results for SDK Related Validation Errors/WarningsThese errors are reported by the SDK team's validation tools, reachout to ADX Swagger Reviewers directly for any questions or concerns. File: specification/postgresql/resource-manager/readme.md
|
AutoRest linter results for ARM Related Validation Errors/WarningsThese errors are reported by the ARM team's validation tools, reachout to ARM RP API Review directly for any questions or concerns. File: specification/postgresql/resource-manager/readme.md
|
AutoRest linter results for SDK Related Validation Errors/WarningsThese errors are reported by the SDK team's validation tools, reachout to ADX Swagger Reviewers directly for any questions or concerns. File: specification/postgresql/resource-manager/readme.md
|
AutoRest linter results for ARM Related Validation Errors/WarningsThese errors are reported by the ARM team's validation tools, reachout to ARM RP API Review directly for any questions or concerns. File: specification/postgresql/resource-manager/readme.md
|
| Code | Id | Source | Message |
|---|---|---|---|
| TrackedResourceListByImmediateParent | R3010 | Link | The child tracked resource, 'securityAlertPolicies' with immediate parent 'Server', must have a list by immediate parent operation. |
| EnumInsteadOfBoolean | R3018 | Link | Booleans are not descriptive and make them hard to use. Consider using string enums with allowed set of values defined. Property: emailAccountAdmins |
❌0 new Errors.(0 total)
AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback
Thanks for your co-operation.
There was a problem hiding this comment.
We only maintain one file for APIs. Please merge your policies API into PostgreSQL.json file.
AutoRest linter results for ARM Related Validation Errors/WarningsThese errors are reported by the ARM team's validation tools, reachout to ARM RP API Review directly for any questions or concerns. File: specification/postgresql/resource-manager/readme.md
|
| Code | Id | Source | Message |
|---|---|---|---|
| TrackedResourceListByImmediateParent | R3010 | Link | The child tracked resource, 'securityAlertPolicies' with immediate parent 'Server', must have a list by immediate parent operation. |
| EnumInsteadOfBoolean | R3018 | Link | Booleans are not descriptive and make them hard to use. Consider using string enums with allowed set of values defined. Property: emailAccountAdmins |
❌0 new Errors.(0 total)
AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback
Thanks for your co-operation.
AutoRest linter results for SDK Related Validation Errors/WarningsThese errors are reported by the SDK team's validation tools, reachout to ADX Swagger Reviewers directly for any questions or concerns. File: specification/postgresql/resource-manager/readme.md
|
There was a problem hiding this comment.
Guess not expected description right? :)
There was a problem hiding this comment.
only in case of error.
do I need to do a change?
There was a problem hiding this comment.
This is the official documentation to explain why people see that. This should be something simple, example:
"default": {
"description": "CDN error response describing why the operation failed.",
}creates this:
https://docs.microsoft.com/en-us/rest/api/cdn/profiles/get#responses
I let you imagine what it would give on docs.microsoft.com if you keep the same string :)
There was a problem hiding this comment.
fixed.
please merge the pull request
|
Looks good. |
AutoRest linter results for SDK Related Validation Errors/WarningsThese errors are reported by the SDK team's validation tools, reachout to ADX Swagger Reviewers directly for any questions or concerns. File: specification/postgresql/resource-manager/readme.md
|
AutoRest linter results for ARM Related Validation Errors/WarningsThese errors are reported by the ARM team's validation tools, reachout to ARM RP API Review directly for any questions or concerns. File: specification/postgresql/resource-manager/readme.md
|
AutoRest linter results for SDK Related Validation Errors/WarningsThese errors are reported by the SDK team's validation tools, reachout to ADX Swagger Reviewers directly for any questions or concerns. File: specification/postgresql/resource-manager/readme.md
|
AutoRest linter results for ARM Related Validation Errors/WarningsThese errors are reported by the ARM team's validation tools, reachout to ARM RP API Review directly for any questions or concerns. File: specification/postgresql/resource-manager/readme.md
|
| Code | Id | Source | Message |
|---|---|---|---|
| TrackedResourceListByImmediateParent | R3010 | Link | The child tracked resource, 'securityAlertPolicies' with immediate parent 'Server', must have a list by immediate parent operation. |
| EnumInsteadOfBoolean | R3018 | Link | Booleans are not descriptive and make them hard to use. Consider using string enums with allowed set of values defined. Property: emailAccountAdmins |
❌0 new Errors.(0 total)
AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback
Thanks for your co-operation.
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information
api-versionin the path should match theapi-versionin the spec).Quality of Swagger