Skip to content

Commit

Permalink
connectivity: perf deployment improved
Browse files Browse the repository at this point in the history
Deploy perf test actors only in the first test namespace in
case of tests concurrent run to avoid resource wasting.
Perf tests always run at the beginning, sequentially and in
the first test namespace.

Signed-off-by: viktor-kurchenko <[email protected]>
  • Loading branch information
viktor-kurchenko authored and michi-covalent committed Jul 15, 2024
1 parent 6b5e89e commit 69dc217
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions connectivity/check/deployment.go
Original file line number Diff line number Diff line change
Expand Up @@ -425,7 +425,9 @@ func (ct *ConnectivityTest) deploy(ctx context.Context) error {
}
}

if ct.params.Perf {
// Deploy perf actors (only in the first test namespace
// in case of tests concurrent run)
if ct.params.Perf && ct.params.TestNamespaceIndex == 0 {
return ct.deployPerf(ctx)
}

Expand Down Expand Up @@ -1143,7 +1145,7 @@ func (ct *ConnectivityTest) deploymentList() (srcList []string, dstList []string
if ct.params.MultiCluster == "" && !ct.params.SingleNode {
srcList = append(srcList, client3DeploymentName)
}
} else {
} else if ct.params.TestNamespaceIndex == 0 {
srcList = []string{}
if ct.params.PerfPodNet {
srcList = append(srcList, perfClientDeploymentName)
Expand Down

0 comments on commit 69dc217

Please sign in to comment.