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

Apply GNOISubcomponentPath deviation to the related tests #853

Merged
merged 2 commits into from
Jan 5, 2023

Conversation

mojiiba
Copy link
Contributor

@mojiiba mojiiba commented Dec 7, 2022

This patch applies GNOISubcomponentPath to the related tests that use gnoi system APIs. Motivation for GNOISubcomponentPath is discussed in the previous pull #717.

@mojiiba mojiiba requested a review from a team as a code owner December 7, 2022 06:20
@mojiiba mojiiba requested a review from sezhang2 December 7, 2022 06:20
@sezhang2
Copy link
Contributor

sezhang2 commented Dec 7, 2022

/gcbrun

@coveralls
Copy link

coveralls commented Dec 7, 2022

Pull Request Test Coverage Report for Build 3842519988

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 46.471%

Totals Coverage Status
Change from base Build 3841712965: 0.0%
Covered Lines: 586
Relevant Lines: 1261

💛 - Coveralls

@mojiiba
Copy link
Contributor Author

mojiiba commented Dec 9, 2022

@sezhang2 could u please review this pr?

sezhang2
sezhang2 previously approved these changes Dec 9, 2022
sezhang2
sezhang2 previously approved these changes Dec 14, 2022
@sezhang2
Copy link
Contributor

/gcbrun

@mojiiba
Copy link
Contributor Author

mojiiba commented Dec 19, 2022

@sezhang2 can u please merge this pr?

@mojiiba mojiiba requested a review from vosipchu December 19, 2022 05:13
@sezhang2
Copy link
Contributor

Looks like you requested the review from vosipchu. I can help merge after vosipchu approves this PR.

@mojiiba mojiiba requested review from vosipchu and removed request for vosipchu December 19, 2022 05:51
@mojiiba
Copy link
Contributor Author

mojiiba commented Dec 19, 2022

I intended to add him as assignee, but added as reviewer by mistake. anyway we can wait for his approval. @vosipchu can u please let us know if u are ok merging this pr?

@thesrinath
Copy link
Contributor

/gcbrun

@mojiiba mojiiba force-pushed the moji/gnoi_component_path branch from 7dcb6ac to 5f37898 Compare January 4, 2023 23:42
@thesrinath
Copy link
Contributor

/gcbrun

thesrinath
thesrinath previously approved these changes Jan 5, 2023
@thesrinath thesrinath dismissed their stale review January 5, 2023 03:14

Tests still failing. The deviation logic seems to be backwards.

@thesrinath thesrinath merged commit a30f186 into openconfig:main Jan 5, 2023
singh-prem pushed a commit to singh-prem/featureprofiles that referenced this pull request Jan 17, 2023
…#853)

* apply GNOISubcomponentPath deviation to related tests

* gofmt
prinikasn pushed a commit that referenced this pull request Jun 30, 2023
* apply GNOISubcomponentPath deviation to related tests

* gofmt
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

Successfully merging this pull request may close these issues.

5 participants