Skip to content

Commit

Permalink
Update ci-fix.md add apiReadinessCheck (#20061)
Browse files Browse the repository at this point in the history
* Update ci-fix.md

* Update ci-fix.md

Minor change on wording.

Co-authored-by: Ray Chen <raychen@microsoft.com>
  • Loading branch information
ruowan and raych1 authored Aug 3, 2022
1 parent 02ed6d4 commit 03c410c
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions documentation/ci-fix.md
Original file line number Diff line number Diff line change
Expand Up @@ -109,6 +109,15 @@ Note: When running in Swagger PR pipeline, Avocado only report errors with file

Refer to [Avocado Readme](https://github.com/Azure/avocado/blob/master/README.md) for detailed description of validations and how-to-fix guidance.

## API Readiness Check

This CI check is to make sure service is ready before PR merge. Technically, the CI check send operationsList HTTP request to Azure Resource Provider.

To fix this CI check failure, if you haven't got ARM signed off, pls get ARM signed off first then deploy ARM manifest. After deploying ARM manifest, this operationsList HTTP request will succeed and CI pass.

NOTE: If your RP is RPaaS RP, since RPaaS requires swagger merge first. In this case, you could ignore this CI check.


## Service API Readiness Test

This CI check is to test service API readiness, by running API Scenario test to verify:
Expand Down

0 comments on commit 03c410c

Please sign in to comment.