Skip to content

Fix for Arista and remove unnecessary and invalid deviation from Nokia #4293

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

Merged
merged 3 commits into from
Jun 16, 2025

Conversation

rohit-rp
Copy link
Contributor

This is a deviation for cisco, Arista and Nokia follow the standard. Fixing the test

@rohit-rp rohit-rp requested a review from a team as a code owner June 16, 2025 03:56
@OpenConfigBot
Copy link

OpenConfigBot commented Jun 16, 2025

@coveralls
Copy link

Pull Request Test Coverage Report for Build 15671449963

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 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 17.719%

Totals Coverage Status
Change from base Build 15657461120: 0.0%
Covered Lines: 2478
Relevant Lines: 13985

💛 - Coveralls

@ram-mac
Copy link
Contributor

ram-mac commented Jun 16, 2025

/fptest cisco-8808

Copy link
Contributor

@ram-mac ram-mac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nikki-blr nikki-blr merged commit f7a702f into openconfig:main Jun 16, 2025
13 of 14 checks passed
@rohit-rp rohit-rp deleted the fixtests branch June 16, 2025 08:20
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