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

Model validation failed but CI job showed success exit code #4744

Closed
veronicagg opened this issue Dec 4, 2018 · 1 comment
Closed

Model validation failed but CI job showed success exit code #4744

veronicagg opened this issue Dec 4, 2018 · 1 comment
Assignees

Comments

@veronicagg
Copy link
Contributor

veronicagg commented Dec 4, 2018

Check job https://travis-ci.org/Azure/azure-rest-api-specs/jobs/457417470 Model validation seems to have thrown an exception
Number of swaggers found in this PR: 2
Files deleted in this PR are as follows:
[]
(node:4150) UnhandledPromiseRejectionWarning: #
(node:4150) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 4)
(node:4150) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

but job exited with 0 and successfully.

CI should fail due to model validation error.

@sergey-shandar
Copy link
Contributor

It's fixed after rerun.


 error : 

code: JSON_PARSING_ERROR

id: OAV104

message: 'unexpected token, token: }, line: 12, column: 7'

innerErrors:

  - kind: structure

    code: unexpected token

    position:

      line: 12

      column: 7

mccleanp pushed a commit that referenced this issue Mar 23, 2022
)

* Swagger for HybridAKS provisioned clusters (#4697)

Added swagger for HybridAKS provisioned clusters

* Swagger fix validation errors and added examples (#4744)

* added examples and fixed some validation errors

* fixed more validation errors and warnings

* Added x-ms-secret flag for passwords

* fixing liniting issues in swagger (#4748)

* Fixes for prettier, spell checks and go-sdk generation (#4767)

* Fixes for prettier, spell checks and go-sdk generation

* changed default network plugin to calico

* formatting fix

* added enums, defaults and moved nodepool config to a common definition

* changed go-sdk to track2

* Added cluster prefix to name and location
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

No branches or pull requests

2 participants