| Message ID | 20230227073224.15693-1-amusil@redhat.com |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [ovs-dev] tests: Decrease the number of zones and switches for interconnection | expand |
| Context | Check | Description |
|---|---|---|
| ovsrobot/apply-robot | success | apply and check: success |
| ovsrobot/github-robot-_Build_and_Test | success | github build: passed |
| ovsrobot/github-robot-_ovn-kubernetes | success | github build: passed |
Thanks Ales. Acked-by: Mark Michelson <mmichels@redhat.com> Since this was already discussed during the upstream OVN IRC meeting, I went ahead and applied this to main, branch-23.03, branch-22.12, branch-22.09, branch-22.06, and branch-22.03. On 2/27/23 02:32, Ales Musil wrote: > When we run the tests with address sanitizers enabled > the interconnection test consumes a lot of memory, > around 5 GB (on my env) per permutation. To prevent > any OOM kills on Github CI reduce the number of > zones and transit switches to 3 as this lowers > the memory consumption to around 2 GB per permutation. > > Signed-off-by: Ales Musil <amusil@redhat.com> > --- > tests/ovn.at | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/tests/ovn.at b/tests/ovn.at > index dc5c5df3f..778d2dbe0 100644 > --- a/tests/ovn.at > +++ b/tests/ovn.at > @@ -25286,8 +25286,10 @@ AT_SETUP([interconnection]) > AT_KEYWORDS([slowtest]) > > ovn_init_ic_db > -n_az=5 > -n_ts=5 > +# The number needs to stay relatively low due to high memory consumption > +# with address sanitizers enabled. > +n_az=3 > +n_ts=3 > for i in `seq 1 $n_az`; do > ovn_start az$i > done
diff --git a/tests/ovn.at b/tests/ovn.at index dc5c5df3f..778d2dbe0 100644 --- a/tests/ovn.at +++ b/tests/ovn.at @@ -25286,8 +25286,10 @@ AT_SETUP([interconnection]) AT_KEYWORDS([slowtest]) ovn_init_ic_db -n_az=5 -n_ts=5 +# The number needs to stay relatively low due to high memory consumption +# with address sanitizers enabled. +n_az=3 +n_ts=3 for i in `seq 1 $n_az`; do ovn_start az$i done
When we run the tests with address sanitizers enabled the interconnection test consumes a lot of memory, around 5 GB (on my env) per permutation. To prevent any OOM kills on Github CI reduce the number of zones and transit switches to 3 as this lowers the memory consumption to around 2 GB per permutation. Signed-off-by: Ales Musil <amusil@redhat.com> --- tests/ovn.at | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)