Skip to content
This repository has been archived by the owner on Oct 29, 2021. It is now read-only.

VPP configuration #2220

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

VPP configuration #2220

wants to merge 2 commits into from

Conversation

xzfc
Copy link
Contributor

@xzfc xzfc commented Nov 20, 2020

Description

  • Set buffers-per-numa to 65535.
  • Provide an option to append additional vpp configuration to vppagent-forwarder container. The option may be used to override the default buffers-per-numa value. The option is exposed via:
    • Container environment variable:
      NSM_ADDITIONAL_VPP_CONF="buffers { buffers-per-numa 128000 }"
    • Helm chart value:
      --set vpp.additionalVppConf="buffers { buffers-per-numa 128000 }"

Motivation and Context

#2218

How Has This Been Tested?

  • Covered by existing integration testing
  • Added integration testing to cover
  • Tested locally
  • Have not tested

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@xzfc xzfc force-pushed the 2218-vpp-conf branch 2 times, most recently from 169a75f to 6ba9552 Compare November 23, 2020 05:45
* Set buffers-per-numa to 65535.
* Provide an option to append additional vpp configuration to
  vppagent-forwarder container. The option may be used to override the
  default buffers-per-numa value. The option is exposed via:
  * Container environment variable:
    NSM_ADDITIONAL_VPP_CONF="buffers { buffers-per-numa 128000 }"
  * Helm chart value:
    --set vpp.additionalVppConf="buffers { buffers-per-numa 128000 }"

Signed-off-by: Albert Safin <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant