Skip to content

Revert PR:12186 crm_stats_ipv6_route_available counter was not decremented, as we have a fix in platform side of code#14775

Merged
yejianquan merged 1 commit intosonic-net:202405from
rajendrat:rajendrat/crmfix
Sep 27, 2024
Merged

Revert PR:12186 crm_stats_ipv6_route_available counter was not decremented, as we have a fix in platform side of code#14775
yejianquan merged 1 commit intosonic-net:202405from
rajendrat:rajendrat/crmfix

Conversation

@rajendrat
Copy link
Contributor

Description of PR

#14104
Reverting this PR as we have fix on the platform side to take care of the CRM route resource usage related issue.

This change is merged in master #14602
This is a conflict merge to 202405 branch.

Summary:
Fixes # (issue)

CRM route resource fix is committed in the platform side of code. the additional delay and checks added part of the PR/14104 is no longer needed.

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

Revert the cisco specific change added in sonic-mgmt code.

How did you do it?

How did you verify/test it?

Verified running sonic mgmt tests/test_crm.py::test_crm_route

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

…ented, as we have a fix in platform side of code
@rajendrat
Copy link
Contributor Author

@kevinwangsk : Please help merge this PR. Its already merged in master.

Copy link
Collaborator

@yejianquan yejianquan left a comment

Choose a reason for hiding this comment

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

LGTM

@yejianquan yejianquan merged commit e2229eb into sonic-net:202405 Sep 27, 2024
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.

3 participants