-
Notifications
You must be signed in to change notification settings - Fork 1
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
Problem with generated specification #1
Comments
Hi @bartoszm checking in to see if you had a chance to look at this? |
Hi @bartoszm any update on this issue? |
@bartoszm the issue may be an ordering issue. If I add additional yang into the directory, the order in which some of the refs get written seems to get changed. @@ -30429,9 +30475,9 @@ definitions:
tapi.eth.OamServicePointAugmentation3:
type: "object"
properties:
eth-oam-service:eth-oam-mep-service-point:
description: "none"
$ref: "#/definitions/tapi.eth.EthOamService""#/definitions/tapi.eth.EthOamMepServicePoint"
x-augmentation:
prefix: "tapi-eth"
namespace: "urn:onf:otcc:yang:tapi-eth" |
If you include this yang file in your TAPI/YANG directory it will generate the issue. |
Hi @damianoneill for your detailed information. |
Hi @bartoszm I am using the latest version of y2s-tapi-cli to generate a single swagger specification for the TAPI-2.1.1 YANG.
y2s.log
I then use go-swagger to generate a go data model.
I used this approach previously with an older version of the tapi-cli jar (yang2swagger-tapi-cli-1.0-cli.jar) and an older version of the TAPI YANG (2.1.0) with no issues.
When running with the output of tapi-cli-1.4 against TAPI-2.1.1 I now get the following errors.
Running the same sequence with the latest TAPI-YANG on the develop branch results in the same issue.
Running the latest tools against the TAPI-2.1.0 YANG works fine.
So it looks like something has changed in the YANG generation from 2.1.0 to 2.1.0+ for the connectivity service generation that has introduced the problem.
Not sure how to progress, if theres anything I can help with, please just ask.
Thanks,
Damian.
The text was updated successfully, but these errors were encountered: