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.

Sign up for GitHub

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

Jump to bottom

Revert "[Yang model] add Restapi yang file" #12374

Merged
merged 1 commit into from
Oct 12, 2022
Merged

Revert "[Yang model] add Restapi yang file" #12374

merged 1 commit into from
Oct 12, 2022

Conversation

saiarcot895
Copy link
Contributor

Reverts #12287

This is causing a build failure for all builds. The PR build was incorrectly marked as passing due to a different build issue.

libyang[0]: Regular expression "(/[a-zA-Z0-9_-.]+)*/([a-zA-Z0-9_-.]+)./[a-z]{3}" is not valid (".]+)*/([a-zA-Z0-9_-.]+)./[a-z]{3})$": range out of order in character class).
libyang[0]: Module "sonic-restapi" parsing failed.
ERROR:YANG-TEST: Exception >Module "sonic-restapi" parsing failed.< in /sonic/src/sonic-yang-models/tests/yang_model_tests/test_yang_model.py:114
ERROR:YANG-TEST: Exception >Module "sonic-restapi" parsing failed.< in /sonic/src/sonic-yang-models/tests/yang_model_tests/test_yang_model.py:258

@saiarcot895
Copy link
Contributor Author

FYI @prsunny

@lguohan lguohan merged commit 7087763 into master Oct 12, 2022
@qiluo-msft
Copy link
Collaborator

@ganglyu for tracking yang model readiness.

prsunny added a commit that referenced this pull request Oct 12, 2022
@prsunny
Revert "Revert "[Yang model] add Restapi yang file (#12287)" (#12374)"
7d180b7 
This reverts commit 7087763.
This was referenced Oct 12, 2022
Closed
Merged
qiluo-msft pushed a commit that referenced this pull request Oct 15, 2022
@prsunny
add Restapi Yang model (#12379)
1f2ee7b 
#### Why I did it

Reverting PR #12374

Fix build issues

This PR fixed #10548
@lguohan lguohan deleted the revert-12287-restapi_yang branch December 20, 2022 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@qiluo-msft qiluo-msft Awaiting requested review from qiluo-msft qiluo-msft is a code owner

@lguohan lguohan Awaiting requested review from lguohan lguohan is a code owner

@yxieca yxieca Awaiting requested review from yxieca

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@saiarcot895 @qiluo-msft @lguohan