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

DP-1.14: Updating WRED config as per test requirements #3710

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

charantejag504
Copy link
Contributor

@charantejag504 charantejag504 commented Jan 22, 2025

DP-1.14: Updating WRED config as per test requirements

Config updated from minThreshold (0), maxThreshold (55) to minThresholdPercent (1), maxThresholdPercent (2) to adhere more closely to the test requirements.

@charantejag504 charantejag504 requested a review from a team as a code owner January 22, 2025 20:36
@OpenConfigBot
Copy link

OpenConfigBot commented Jan 22, 2025

Pull Request Functional Test Report for #3710 / b7829d2

Virtual Devices

Device Test Test Documentation Job Raw Log
Arista cEOS status
DP-1.14: QoS basic test
f9ddbf06 Log
Cisco 8000E status
DP-1.14: QoS basic test
e2a286ae Log
Cisco XRd status
DP-1.14: QoS basic test
e60dd09b Log
Juniper ncPTX status
DP-1.14: QoS basic test
49736300 Log
Nokia SR Linux status
DP-1.14: QoS basic test
5ad47b95 Log
Openconfig Lemming status
DP-1.14: QoS basic test
ec398ed4 Log

Hardware Devices

Device Test Test Documentation Raw Log
Arista 7808 status
DP-1.14: QoS basic test
Log
Cisco 8808 status
DP-1.14: QoS basic test
Log
Juniper PTX10008 status
DP-1.14: QoS basic test
Log
Nokia 7250 IXR-10e status
DP-1.14: QoS basic test
Log

Help

@charantejag504 charantejag504 changed the title Updating WRED config as per test requirements DP-1.14: Updating WRED config as per test requirements Jan 23, 2025
@Swetha-haridasula Swetha-haridasula self-assigned this Feb 27, 2025
@Swetha-haridasula
Copy link
Contributor

/fptest all

@coveralls
Copy link

Pull Request Test Coverage Report for Build 13544931862

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

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

💛 - Coveralls

@Swetha-haridasula
Copy link
Contributor

/fptest physical

@Swetha-haridasula
Copy link
Contributor

/fptest cisco-8808

@Swetha-haridasula
Copy link
Contributor

/fptest physical

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