]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Add ipset element and hashsize tunables
authorBrian Haley <brian.haley@hp.com>
Fri, 3 Apr 2015 01:11:06 +0000 (21:11 -0400)
committerBrian Haley <brian.haley@hp.com>
Mon, 6 Apr 2015 19:33:01 +0000 (15:33 -0400)
commitb5b919a7a3569ccb93c3d7d523c1edfaeddb7cb9
tree714cd9ae7d143b8c9c2b8cffb63e812b2f64b441
parenta38d294f1e1ad515d86952e297d2b4d437d99cff
Add ipset element and hashsize tunables

Recently, these messages have been noticed in both tempest
logs, as well as reported by downstream users syslog:

  Set IPv4915d358d-2c5b-43b5-9862 is full, maxelem 65536 reached

So the default of 64K is not sufficient enough.

This change adds two config options to control both the number
of elements as well as the hashsize, since they should be
tuned together for best performance.  Slightly different
formats were required for 'ipset create' and 'ipset restore'.

The default values for these are now set to 131072 (maxelem) and
2048 (hashsize), which is an increase over their typical default values
of 65536/1024 (respectively), in order to fix the errors seen in
the tempest tests.

DocImpact

Change-Id: Ic0b5b38a840e737dc6be938230f4052974c8620f
Closes-bug: #1439817
etc/neutron.conf
neutron/agent/common/config.py
neutron/agent/linux/ipset_manager.py
neutron/tests/unit/agent/linux/test_ipset_manager.py