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

Added deviation for GetOnStateUnsupported #3833

Merged
merged 10 commits into from
Feb 27, 2025

Conversation

mnp-juniper
Copy link
Contributor

Added deviation for GetOnStateUnsupported for Juniper devices

@mnp-juniper mnp-juniper requested review from a team as code owners February 19, 2025 13:01
@OpenConfigBot
Copy link

OpenConfigBot commented Feb 19, 2025

Pull Request Functional Test Report for #3833 / af5e22e

Virtual Devices

Device Test Test Documentation Job Raw Log
Arista cEOS status
System-1.2: System g protocol test
Cisco 8000E status
System-1.2: System g protocol test
Cisco XRd status
System-1.2: System g protocol test
Juniper ncPTX status
System-1.2: System g protocol test
Nokia SR Linux status
System-1.2: System g protocol test
Openconfig Lemming status
System-1.2: System g protocol test

Hardware Devices

Device Test Test Documentation Raw Log
Arista 7808 status
System-1.2: System g protocol test
Cisco 8808 status
System-1.2: System g protocol test
Juniper PTX10008 status
System-1.2: System g protocol test
Nokia 7250 IXR-10e status
System-1.2: System g protocol test

Help

@coveralls
Copy link

coveralls commented Feb 19, 2025

Pull Request Test Coverage Report for Build 13527273797

Details

  • 7 of 16 (43.75%) changed or added relevant lines in 1 file are covered.
  • 3 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.05%) to 18.251%

Changes Missing Coverage Covered Lines Changed/Added Lines %
proto/metadata_go_proto/metadata.pb.go 7 16 43.75%
Files with Coverage Reduction New Missed Lines %
proto/metadata_go_proto/metadata.pb.go 3 5.52%
Totals Coverage Status
Change from base Build 13514438655: 0.05%
Covered Lines: 2477
Relevant Lines: 13572

💛 - Coveralls

@Swetha-haridasula Swetha-haridasula self-assigned this Feb 26, 2025
Copy link
Contributor

@Swetha-haridasula Swetha-haridasula left a comment

Choose a reason for hiding this comment

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

lgtm

@Swetha-haridasula Swetha-haridasula merged commit 2215c9d into openconfig:main Feb 27, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants