diff mbox series

[ovs-dev,1/3] perf-northd.at: Don't start ovn-controllers.

Message ID 170083922268.2385884.18093194128580487008.stgit@dceara.remote.csb
State Accepted
Headers show
Series Improve in-tree ovn-northd perf testing. | expand

Checks

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 fail github build: failed

Commit Message

Dumitru Ceara Nov. 24, 2023, 3:20 p.m. UTC
The goal is to test northd performance so there's no point to start
hypervisors.

Signed-off-by: Dumitru Ceara <dceara@redhat.com>
---
 tests/perf-northd.at |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Mark Michelson Dec. 4, 2023, 7:11 p.m. UTC | #1
Thanks Dumitru,

Acked-by: Mark Michelson <mmichels@redhat.com>

On 11/24/23 10:20, Dumitru Ceara wrote:
> The goal is to test northd performance so there's no point to start
> hypervisors.
> 
> Signed-off-by: Dumitru Ceara <dceara@redhat.com>
> ---
>   tests/perf-northd.at |    4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/perf-northd.at b/tests/perf-northd.at
> index ca115dadc2..eb579bf19b 100644
> --- a/tests/perf-northd.at
> +++ b/tests/perf-northd.at
> @@ -169,7 +169,7 @@ m4_define([OVN_BASIC_SCALE_CONFIG], [
>       done
>   ])
>   
> -OVN_FOR_EACH_NORTHD([
> +OVN_FOR_EACH_NORTHD_NO_HV([
>   AT_SETUP([ovn-northd basic scale test -- 200 Hypervisors, 200 Logical Ports/Hypervisor])
>   PERF_RECORD_START()
>   
> @@ -181,7 +181,7 @@ PERF_RECORD_STOP()
>   AT_CLEANUP
>   ])
>   
> -OVN_FOR_EACH_NORTHD([
> +OVN_FOR_EACH_NORTHD_NO_HV([
>   AT_SETUP([ovn-northd basic scale test -- 500 Hypervisors, 50 Logical Ports/Hypervisor])
>   PERF_RECORD_START()
>   
> 
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
Numan Siddique Dec. 6, 2023, 9:31 p.m. UTC | #2
On Mon, Dec 4, 2023 at 2:12 PM Mark Michelson <mmichels@redhat.com> wrote:
>
> Thanks Dumitru,
>
> Acked-by: Mark Michelson <mmichels@redhat.com>

Thanks.

Acked-by: Numan Siddique <numans@ovn.org>

Numan

>
> On 11/24/23 10:20, Dumitru Ceara wrote:
> > The goal is to test northd performance so there's no point to start
> > hypervisors.
> >
> > Signed-off-by: Dumitru Ceara <dceara@redhat.com>
> > ---
> >   tests/perf-northd.at |    4 ++--
> >   1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/tests/perf-northd.at b/tests/perf-northd.at
> > index ca115dadc2..eb579bf19b 100644
> > --- a/tests/perf-northd.at
> > +++ b/tests/perf-northd.at
> > @@ -169,7 +169,7 @@ m4_define([OVN_BASIC_SCALE_CONFIG], [
> >       done
> >   ])
> >
> > -OVN_FOR_EACH_NORTHD([
> > +OVN_FOR_EACH_NORTHD_NO_HV([
> >   AT_SETUP([ovn-northd basic scale test -- 200 Hypervisors, 200 Logical Ports/Hypervisor])
> >   PERF_RECORD_START()
> >
> > @@ -181,7 +181,7 @@ PERF_RECORD_STOP()
> >   AT_CLEANUP
> >   ])
> >
> > -OVN_FOR_EACH_NORTHD([
> > +OVN_FOR_EACH_NORTHD_NO_HV([
> >   AT_SETUP([ovn-northd basic scale test -- 500 Hypervisors, 50 Logical Ports/Hypervisor])
> >   PERF_RECORD_START()
> >
> >
> > _______________________________________________
> > dev mailing list
> > dev@openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> >
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
diff mbox series

Patch

diff --git a/tests/perf-northd.at b/tests/perf-northd.at
index ca115dadc2..eb579bf19b 100644
--- a/tests/perf-northd.at
+++ b/tests/perf-northd.at
@@ -169,7 +169,7 @@  m4_define([OVN_BASIC_SCALE_CONFIG], [
     done
 ])
 
-OVN_FOR_EACH_NORTHD([
+OVN_FOR_EACH_NORTHD_NO_HV([
 AT_SETUP([ovn-northd basic scale test -- 200 Hypervisors, 200 Logical Ports/Hypervisor])
 PERF_RECORD_START()
 
@@ -181,7 +181,7 @@  PERF_RECORD_STOP()
 AT_CLEANUP
 ])
 
-OVN_FOR_EACH_NORTHD([
+OVN_FOR_EACH_NORTHD_NO_HV([
 AT_SETUP([ovn-northd basic scale test -- 500 Hypervisors, 50 Logical Ports/Hypervisor])
 PERF_RECORD_START()