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

[Nokia] Add J2C+/H3/H4/H5 to GCU validator #3495

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dgodwin-nokia
Copy link

@dgodwin-nokia dgodwin-nokia commented Aug 23, 2024

  • Add Nokia platforms to GCU validator .json file

What I did

Nokia J2C/TH3/4/5 based platforms were not included in the GCU field validators file. This caused some of the sonic-mgmt test cases to fail related to PFCWD/BUFFER_POOL/BUFFER_PROFILE/WRED due to apply-patch validation failing. Added the platforms.

This PR is required for 202405.

How to verify it

Validated that sonic-mgmt GCU tests now pass.

Copy link

linux-foundation-easycla bot commented Aug 23, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: dgodwin-nokia / name: Dylan Godwin (5aa2d3a)

@mlok-nokia
Copy link
Contributor

@prgeor Hi Prince, Could you please review this PR to assign it to the person? Thanks

@dgodwin-nokia dgodwin-nokia changed the title [Nokia] Add H3/H4/H5 to GCU validator [Nokia] Add J2C+/H3/H4/H5 to GCU validator Sep 3, 2024
@mlok-nokia
Copy link
Contributor

@judyjoseph @prgeor @rlhui @xincunli-sonic Please help to review this PR. It is needed by 202405 branch also

@xincunli-sonic
Copy link
Contributor

Can you help provide more info, like any issue that is tracking sonic-mgmt test failure, also can you share the sonic-mgmt GCU tests passed details, Thanks!

@mlok-nokia
Copy link
Contributor

@okaravasi Can you please address the comment above?

@okaravasi
Copy link

okaravasi commented Sep 13, 2024

Can you help provide more info, like any issue that is tracking sonic-mgmt test failure, also can you share the sonic-mgmt GCU tests passed details, Thanks!

@xincunli-sonic Suggested changes are needed to pass below test suites:

  • generic_config_updater/test_pg_headroom_update.py
  • generic_config_updater/test_mmu_dynamic_threshold_config_update.py
  • generic_config_updater/test_ecn_config_update.py

That they perform changes in cfg paths /BUFFER_PROFILE, /WRED_PROFILE.

@xincunli-sonic
Copy link
Contributor

Can you help provide more info, like any issue that is tracking sonic-mgmt test failure, also can you share the sonic-mgmt GCU tests passed details, Thanks!

@xincunli-sonic Suggested changes are needed to pass below test suites:

  • generic_config_updater/test_pg_headroom_update.py
  • generic_config_updater/test_mmu_dynamic_threshold_config_update.py
  • generic_config_updater/test_ecn_config_update.py

That they perform changes in cfg paths /BUFFER_PROFILE, /WRED_PROFILE.

Were they failing in 202405 or something else, do we have github issue tracking it? And as you mentioned in top description that you did pass these tests by the new change, can you share the output where took your change, Thanks!

@dgodwin-nokia
Copy link
Author

Can you help provide more info, like any issue that is tracking sonic-mgmt test failure, also can you share the sonic-mgmt GCU tests passed details, Thanks!

@xincunli-sonic Suggested changes are needed to pass below test suites:

  • generic_config_updater/test_pg_headroom_update.py
  • generic_config_updater/test_mmu_dynamic_threshold_config_update.py
  • generic_config_updater/test_ecn_config_update.py

That they perform changes in cfg paths /BUFFER_PROFILE, /WRED_PROFILE.

Were they failing in 202405 or something else, do we have github issue tracking it? And as you mentioned in top description that you did pass these tests by the new change, can you share the output where took your change, Thanks!

@xincunli-sonic These tests have always failed for these platforms. The tests validate the ability to use "apply-patch" to modify configuration in these areas.

They will always fail for platforms that are not included in this file. "apply-patch" parses this file and checks that the current version is at least the version in the file before allowing patching of these areas.

* Add Nokia platforms to GCU validator .json file

Signed-off-by: dgodwin-nokia <dylan.godwin@nokia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

4 participants