DMTF / YANG-to-Redfish-Converter

This tool converts a YANG model file to the corresponding Redfish schema, specified in OData CSDL, in accordance with the YANG-to-CSDL Mapping Specification.
Other
4 stars 5 forks source link

Travis - Unknown element name Collection #39

Closed jcleung5549 closed 5 years ago

jcleung5549 commented 5 years ago

First Travis run openconfig_acl_v1.xml.txt

metadata/openconfig_acl_v1.xml ✗ is valid syntax Error: Unknown element name Collection at Schema.defaultElementParse (C:\Users\jcleung\Documents\github external\branch-networktf\Redfish\node_modules\CSDLParser\lib\ParserCommon.js:87:11) at parseEntity (C:\Users\jcleung\Documents\github external\branch-networktf\Redfish\node_modules\CSDLParser\lib\ParserCommon.js:26:11) at Schema.initEntity (C:\Users\jcleung\Documents\github external\branch-networktf\Redfish\node_modules\CSDLParser\lib\ParserCommon.js:14:5) at new Schema (C:\Users\jcleung\Documents\github external\branch-networktf\Redfish\node_modules\CSDLParser\lib\Schema.js:23:3) at Metadata.parseDataServiceElement (C:\Users\jcleung\Documents\github external\branch-networktf\Redfish\node_modules\CSDLParser\lib\Metadata.js:129:25) at Object.parseEntity (C:\Users\jcleung\Documents\github external\branch-networktf\Redfish\node_modules\CSDLParser\lib\ParserCommon.js:26:11) at Metadata.parseDataServices (C:\Users\jcleung\Documents\github external\branch-networktf\Redfish\node_modules\CSDLParser\lib\Metadata.js:116:18) at Metadata.parseElement (C:\Users\jcleung\Documents\github external\branch-networktf\Redfish\node_modules\CSDLParser\lib\Metadata.js:105:12) at Object.parseEntity (C:\Users\jcleung\Documents\github external\branch-networktf\Redfish\node_modules\CSDLParser\lib\ParserCommon.js:26:11) at Metadata.parse (C:\Users\jcleung\Documents\github external\branch-networktf\Redfish\node_modules\CSDLParser\lib\Metadata.js:57:18)

jcleung5549 commented 5 years ago

Travis install messages (in case it is helpful)

$ npm install npm WARN deprecated nomnom@1.8.1: Package no longer supported. Contact support@npmjs.com for more info. npm WARN deprecated cryptiles@2.0.5: This version is no longer maintained. Please upgrade to the latest version. npm WARN deprecated hoek@2.16.3: This version is no longer maintained. Please upgrade to the latest version. npm WARN deprecated boom@2.10.1: This version is no longer maintained. Please upgrade to the latest version.

libxmljs@0.19.5 install C:\Users\jcleung\Documents\github external\branch-networktf\Redfish\node_modules\libxmljs node-pre-gyp install --fallback-to-build --loglevel http

node-pre-gyp WARN Using request for node-pre-gyp https download node-pre-gyp http GET https://github.com/libxmljs/libxmljs/releases/download/v0.19.5/node-v48-win32-x64.tar.gz node-pre-gyp http 200 https://github.com/libxmljs/libxmljs/releases/download/v0.19.5/node-v48-win32-x64.tar.gz [libxmljs] Success: "C:\Users\jcleung\Documents\github external\branch-networktf\Redfish\node_modules\libxmljs\build\Release\xmljs.node" is installed via remote

libxmljs-mt@0.18.3 install C:\Users\jcleung\Documents\github external\branch-networktf\Redfish\node_modules\libxmljs-mt node-pre-gyp install --fallback-to-build --loglevel http . . .

jcleung5549 commented 5 years ago

No longer seeing with YANG-2019. Waiting for a clean Travis run, before closing this issue.

tomasg2012 commented 5 years ago

The current version, however doesn't follow the changes to the other issue which asks for non-repeat Annotations. I would close this anyway, at least to observe the changes in the other thread.

On Fri, Aug 2, 2019, 10:32 AM John Leung notifications@github.com wrote:

No longer seeing with YANG-2019. Waiting for a clean Travis run, before closing this issue.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/DMTF/YANG-to-Redfish-Converter/issues/39?email_source=notifications&email_token=ABKQIXATE5PRPO3Y6DHGLLLQCRHQ5A5CNFSM4HCMWAZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3OCV2Q#issuecomment-517745386, or mute the thread https://github.com/notifications/unsubscribe-auth/ABKQIXCTNYTC4ZJDHMB22J3QCRHQ5ANCNFSM4HCMWAZQ .

jcleung5549 commented 5 years ago

Aug 2 meeting - agreed to close.