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

Health 1.1 system_generic_health_check #3693

Merged
merged 3 commits into from
Feb 26, 2025
Merged

Conversation

lvaish05
Copy link
Contributor

removing deviations that are not required and adding code for components presence

Includes changes that were done in pull requests 2820 and 2936

@lvaish05 lvaish05 requested a review from a team as a code owner January 14, 2025 05:59
@OpenConfigBot
Copy link

OpenConfigBot commented Jan 14, 2025

Pull Request Functional Test Report for #3693 / 8317fe8

Virtual Devices

Device Test Test Documentation Job Raw Log
Arista cEOS status
Health-1.1: Generic Health Check
Cisco 8000E status
Health-1.1: Generic Health Check
Cisco XRd status
Health-1.1: Generic Health Check
Juniper ncPTX status
Health-1.1: Generic Health Check
Nokia SR Linux status
Health-1.1: Generic Health Check
Openconfig Lemming status
Health-1.1: Generic Health Check

Hardware Devices

Device Test Test Documentation Raw Log
Arista 7808 status
Health-1.1: Generic Health Check
Cisco 8808 status
Health-1.1: Generic Health Check
Juniper PTX10008 status
Health-1.1: Generic Health Check
Nokia 7250 IXR-10e status
Health-1.1: Generic Health Check

Help

@coveralls
Copy link

coveralls commented Jan 14, 2025

Pull Request Test Coverage Report for Build 13522231002

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 18.205%

Totals Coverage Status
Change from base Build 13514438655: 0.0%
Covered Lines: 2479
Relevant Lines: 13617

💛 - Coveralls

@AmrNJ AmrNJ self-assigned this Jan 15, 2025
Copy link
Contributor

@AmrNJ AmrNJ left a comment

Choose a reason for hiding this comment

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

Linecard memory utilization path not returning any values.

@AmrNJ
Copy link
Contributor

AmrNJ commented Feb 25, 2025

LGTM. Works in the latest image.

@AmrNJ AmrNJ merged commit 300f44f into openconfig:main Feb 26, 2025
13 of 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.

4 participants