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

[qfix] Increase memory limit for VPP applications #2793

Merged

Conversation

Bolodya1997
Copy link

@Bolodya1997 Bolodya1997 commented Aug 30, 2021

Description

Upgraded VPP consumes more memory, so 400Mi limit is not enough.

Issue

networkservicemesh/cmd-forwarder-vpp#332

@Bolodya1997 Bolodya1997 marked this pull request as draft August 30, 2021 04:38
@Bolodya1997 Bolodya1997 marked this pull request as ready for review August 30, 2021 07:11
@Bolodya1997 Bolodya1997 marked this pull request as draft August 30, 2021 07:41
Signed-off-by: Vladimir Popov <vladimir.popov@xored.com>
@Bolodya1997 Bolodya1997 marked this pull request as ready for review August 30, 2021 07:44
Copy link
Member

@denis-tingaikin denis-tingaikin left a comment

Choose a reason for hiding this comment

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

Merging this to test if this fixes
networkservicemesh/cmd-forwarder-vpp#332
If it will not fix the problem then we'll profile forwarder-vpp with vpp 21.06 to catch the leak.

@denis-tingaikin denis-tingaikin merged commit 0aadeee into networkservicemesh:main Aug 30, 2021
nsmbot pushed a commit to networkservicemesh/integration-tests that referenced this pull request Aug 30, 2021
…ployments-k8s@main

PR link: networkservicemesh/deployments-k8s#2793

Commit: cc1d926
Author: Network Service Mesh Bot
Date: 2021-08-30 02:20:10 -0500
Message:
  - Update go.mod and go.sum to latest version from networkservicemesh/de…
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
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.

2 participants