Skip to content

[memory utilization] update memory threshold#19167

Merged
StormLiangMS merged 1 commit intosonic-net:masterfrom
lipxu:20250624_publicMaster_memoryThreshold
Jul 2, 2025
Merged

[memory utilization] update memory threshold#19167
StormLiangMS merged 1 commit intosonic-net:masterfrom
lipxu:20250624_publicMaster_memoryThreshold

Conversation

@lipxu
Copy link
Copy Markdown
Contributor

@lipxu lipxu commented Jun 24, 2025

Description of PR

Summary:
Fixes # (issue)
33156439

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

There are so many memory above threshold alarm in nightly test

How did you do it?

Update the FRR memory threshold and make the alarm more readable

memory_increase_threshold, FRR has it's own memory management system, not return the memory to system immediately, increase the threshold.
1: top:zebra: update from 64 to 128M
2: frr_bgp: update from 32 to 64M
3: frr_zebra: update from 16 to 64M

memory_high_threshold, frr bgp memory usage related to the count of neighbors, increase the threshold. we need to set the threshold according to the count of neighbors in the further.
1: frr_bgp: update from 128 to 256M

How did you verify/test it?

Run nightly test
https://elastictest.org/scheduler/testplan/685ac58d2461750d1f5a11c9

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@lipxu lipxu force-pushed the 20250624_publicMaster_memoryThreshold branch from a044706 to bc1cf97 Compare June 24, 2025 15:32
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Collaborator

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

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

LGTM

@StormLiangMS StormLiangMS merged commit 24916d3 into sonic-net:master Jul 2, 2025
18 checks passed
vvolam pushed a commit to vvolam/sonic-mgmt that referenced this pull request Jul 3, 2025
What is the motivation for this PR?
There are so many memory above threshold alarm in nightly test

How did you do it?
Update the FRR memory threshold and make the alarm more readable

memory_increase_threshold, FRR has it's own memory management system, not return the memory to system immediately, increase the threshold.
1: top:zebra: update from 64 to 128M
2: frr_bgp: update from 32 to 64M
3: frr_zebra: update from 16 to 64M

memory_high_threshold, frr bgp memory usage related to the count of neighbors, increase the threshold. we need to set the threshold according to the count of neighbors in the further.
1: frr_bgp: update from 128 to 256M

How did you verify/test it?
Run nightly test
https://elastictest.org/scheduler/testplan/685ac58d2461750d1f5a11c9
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jul 3, 2025
What is the motivation for this PR?
There are so many memory above threshold alarm in nightly test

How did you do it?
Update the FRR memory threshold and make the alarm more readable

memory_increase_threshold, FRR has it's own memory management system, not return the memory to system immediately, increase the threshold.
1: top:zebra: update from 64 to 128M
2: frr_bgp: update from 32 to 64M
3: frr_zebra: update from 16 to 64M

memory_high_threshold, frr bgp memory usage related to the count of neighbors, increase the threshold. we need to set the threshold according to the count of neighbors in the further.
1: frr_bgp: update from 128 to 256M

How did you verify/test it?
Run nightly test
https://elastictest.org/scheduler/testplan/685ac58d2461750d1f5a11c9
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202505: #19393

mssonicbld pushed a commit that referenced this pull request Jul 4, 2025
What is the motivation for this PR?
There are so many memory above threshold alarm in nightly test

How did you do it?
Update the FRR memory threshold and make the alarm more readable

memory_increase_threshold, FRR has it's own memory management system, not return the memory to system immediately, increase the threshold.
1: top:zebra: update from 64 to 128M
2: frr_bgp: update from 32 to 64M
3: frr_zebra: update from 16 to 64M

memory_high_threshold, frr bgp memory usage related to the count of neighbors, increase the threshold. we need to set the threshold according to the count of neighbors in the further.
1: frr_bgp: update from 128 to 256M

How did you verify/test it?
Run nightly test
https://elastictest.org/scheduler/testplan/685ac58d2461750d1f5a11c9
nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Aug 7, 2025
What is the motivation for this PR?
There are so many memory above threshold alarm in nightly test

How did you do it?
Update the FRR memory threshold and make the alarm more readable

memory_increase_threshold, FRR has it's own memory management system, not return the memory to system immediately, increase the threshold.
1: top:zebra: update from 64 to 128M
2: frr_bgp: update from 32 to 64M
3: frr_zebra: update from 16 to 64M

memory_high_threshold, frr bgp memory usage related to the count of neighbors, increase the threshold. we need to set the threshold according to the count of neighbors in the further.
1: frr_bgp: update from 128 to 256M

How did you verify/test it?
Run nightly test
https://elastictest.org/scheduler/testplan/685ac58d2461750d1f5a11c9
ashutosh-agrawal pushed a commit to ashutosh-agrawal/sonic-mgmt that referenced this pull request Aug 14, 2025
What is the motivation for this PR?
There are so many memory above threshold alarm in nightly test

How did you do it?
Update the FRR memory threshold and make the alarm more readable

memory_increase_threshold, FRR has it's own memory management system, not return the memory to system immediately, increase the threshold.
1: top:zebra: update from 64 to 128M
2: frr_bgp: update from 32 to 64M
3: frr_zebra: update from 16 to 64M

memory_high_threshold, frr bgp memory usage related to the count of neighbors, increase the threshold. we need to set the threshold according to the count of neighbors in the further.
1: frr_bgp: update from 128 to 256M

How did you verify/test it?
Run nightly test
https://elastictest.org/scheduler/testplan/685ac58d2461750d1f5a11c9
vidyac86 pushed a commit to vidyac86/sonic-mgmt that referenced this pull request Oct 23, 2025
What is the motivation for this PR?
There are so many memory above threshold alarm in nightly test

How did you do it?
Update the FRR memory threshold and make the alarm more readable

memory_increase_threshold, FRR has it's own memory management system, not return the memory to system immediately, increase the threshold.
1: top:zebra: update from 64 to 128M
2: frr_bgp: update from 32 to 64M
3: frr_zebra: update from 16 to 64M

memory_high_threshold, frr bgp memory usage related to the count of neighbors, increase the threshold. we need to set the threshold according to the count of neighbors in the further.
1: frr_bgp: update from 128 to 256M

How did you verify/test it?
Run nightly test
https://elastictest.org/scheduler/testplan/685ac58d2461750d1f5a11c9
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
What is the motivation for this PR?
There are so many memory above threshold alarm in nightly test

How did you do it?
Update the FRR memory threshold and make the alarm more readable

memory_increase_threshold, FRR has it's own memory management system, not return the memory to system immediately, increase the threshold.
1: top:zebra: update from 64 to 128M
2: frr_bgp: update from 32 to 64M
3: frr_zebra: update from 16 to 64M

memory_high_threshold, frr bgp memory usage related to the count of neighbors, increase the threshold. we need to set the threshold according to the count of neighbors in the further.
1: frr_bgp: update from 128 to 256M

How did you verify/test it?
Run nightly test
https://elastictest.org/scheduler/testplan/685ac58d2461750d1f5a11c9

Signed-off-by: opcoder0 <110003254+opcoder0@users.noreply.github.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
What is the motivation for this PR?
There are so many memory above threshold alarm in nightly test

How did you do it?
Update the FRR memory threshold and make the alarm more readable

memory_increase_threshold, FRR has it's own memory management system, not return the memory to system immediately, increase the threshold.
1: top:zebra: update from 64 to 128M
2: frr_bgp: update from 32 to 64M
3: frr_zebra: update from 16 to 64M

memory_high_threshold, frr bgp memory usage related to the count of neighbors, increase the threshold. we need to set the threshold according to the count of neighbors in the further.
1: frr_bgp: update from 128 to 256M

How did you verify/test it?
Run nightly test
https://elastictest.org/scheduler/testplan/685ac58d2461750d1f5a11c9

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
What is the motivation for this PR?
There are so many memory above threshold alarm in nightly test

How did you do it?
Update the FRR memory threshold and make the alarm more readable

memory_increase_threshold, FRR has it's own memory management system, not return the memory to system immediately, increase the threshold.
1: top:zebra: update from 64 to 128M
2: frr_bgp: update from 32 to 64M
3: frr_zebra: update from 16 to 64M

memory_high_threshold, frr bgp memory usage related to the count of neighbors, increase the threshold. we need to set the threshold according to the count of neighbors in the further.
1: frr_bgp: update from 128 to 256M

How did you verify/test it?
Run nightly test
https://elastictest.org/scheduler/testplan/685ac58d2461750d1f5a11c9

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
What is the motivation for this PR?
There are so many memory above threshold alarm in nightly test

How did you do it?
Update the FRR memory threshold and make the alarm more readable

memory_increase_threshold, FRR has it's own memory management system, not return the memory to system immediately, increase the threshold.
1: top:zebra: update from 64 to 128M
2: frr_bgp: update from 32 to 64M
3: frr_zebra: update from 16 to 64M

memory_high_threshold, frr bgp memory usage related to the count of neighbors, increase the threshold. we need to set the threshold according to the count of neighbors in the further.
1: frr_bgp: update from 128 to 256M

How did you verify/test it?
Run nightly test
https://elastictest.org/scheduler/testplan/685ac58d2461750d1f5a11c9

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
What is the motivation for this PR?
There are so many memory above threshold alarm in nightly test

How did you do it?
Update the FRR memory threshold and make the alarm more readable

memory_increase_threshold, FRR has it's own memory management system, not return the memory to system immediately, increase the threshold.
1: top:zebra: update from 64 to 128M
2: frr_bgp: update from 32 to 64M
3: frr_zebra: update from 16 to 64M

memory_high_threshold, frr bgp memory usage related to the count of neighbors, increase the threshold. we need to set the threshold according to the count of neighbors in the further.
1: frr_bgp: update from 128 to 256M

How did you verify/test it?
Run nightly test
https://elastictest.org/scheduler/testplan/685ac58d2461750d1f5a11c9
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
What is the motivation for this PR?
There are so many memory above threshold alarm in nightly test

How did you do it?
Update the FRR memory threshold and make the alarm more readable

memory_increase_threshold, FRR has it's own memory management system, not return the memory to system immediately, increase the threshold.
1: top:zebra: update from 64 to 128M
2: frr_bgp: update from 32 to 64M
3: frr_zebra: update from 16 to 64M

memory_high_threshold, frr bgp memory usage related to the count of neighbors, increase the threshold. we need to set the threshold according to the count of neighbors in the further.
1: frr_bgp: update from 128 to 256M

How did you verify/test it?
Run nightly test
https://elastictest.org/scheduler/testplan/685ac58d2461750d1f5a11c9

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
What is the motivation for this PR?
There are so many memory above threshold alarm in nightly test

How did you do it?
Update the FRR memory threshold and make the alarm more readable

memory_increase_threshold, FRR has it's own memory management system, not return the memory to system immediately, increase the threshold.
1: top:zebra: update from 64 to 128M
2: frr_bgp: update from 32 to 64M
3: frr_zebra: update from 16 to 64M

memory_high_threshold, frr bgp memory usage related to the count of neighbors, increase the threshold. we need to set the threshold according to the count of neighbors in the further.
1: frr_bgp: update from 128 to 256M

How did you verify/test it?
Run nightly test
https://elastictest.org/scheduler/testplan/685ac58d2461750d1f5a11c9

Signed-off-by: Yael Tzur <ytzur@nvidia.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
What is the motivation for this PR?
There are so many memory above threshold alarm in nightly test

How did you do it?
Update the FRR memory threshold and make the alarm more readable

memory_increase_threshold, FRR has it's own memory management system, not return the memory to system immediately, increase the threshold.
1: top:zebra: update from 64 to 128M
2: frr_bgp: update from 32 to 64M
3: frr_zebra: update from 16 to 64M

memory_high_threshold, frr bgp memory usage related to the count of neighbors, increase the threshold. we need to set the threshold according to the count of neighbors in the further.
1: frr_bgp: update from 128 to 256M

How did you verify/test it?
Run nightly test
https://elastictest.org/scheduler/testplan/685ac58d2461750d1f5a11c9
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