Skip to content

Commit

Permalink
Update Logstash stack mon tests (#6741)
Browse files Browse the repository at this point in the history
This adjusts the number of volumes expected from Beats sidecars in the Logstash
Stack Monitoring unit tests.

Why? Because we don't test PRs with an automatic merge of the main branch (πŸ›πŸž), we missed that the tests in #6732 had to be updated to take into account the changes made by #6703, which adds a new temp volume to the Beats sidecars.
  • Loading branch information
thbkrkr committed Apr 28, 2023
1 parent d9b4774 commit e679b5f
Showing 1 changed file with 10 additions and 10 deletions.
20 changes: 10 additions & 10 deletions pkg/controller/logstash/stackmon/sidecar_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -99,8 +99,8 @@ func TestWithMonitoring(t *testing.T) {
},
containersLength: 2,
esEnvVarsLength: 0,
podVolumesLength: 2,
metricsVolumeMountsLength: 2,
podVolumesLength: 3,
metricsVolumeMountsLength: 3,
},
{
name: "with logs monitoring",
Expand All @@ -112,8 +112,8 @@ func TestWithMonitoring(t *testing.T) {
},
containersLength: 2,
esEnvVarsLength: 1,
podVolumesLength: 3,
logVolumeMountsLength: 3,
podVolumesLength: 4,
logVolumeMountsLength: 4,
},
{
name: "with metrics and logs monitoring",
Expand All @@ -126,9 +126,9 @@ func TestWithMonitoring(t *testing.T) {
},
containersLength: 3,
esEnvVarsLength: 1,
podVolumesLength: 4,
metricsVolumeMountsLength: 2,
logVolumeMountsLength: 3,
podVolumesLength: 6,
metricsVolumeMountsLength: 3,
logVolumeMountsLength: 4,
},
{
name: "with metrics and logs monitoring with different es ref",
Expand All @@ -141,9 +141,9 @@ func TestWithMonitoring(t *testing.T) {
},
containersLength: 3,
esEnvVarsLength: 1,
podVolumesLength: 5,
metricsVolumeMountsLength: 2,
logVolumeMountsLength: 3,
podVolumesLength: 7,
metricsVolumeMountsLength: 3,
logVolumeMountsLength: 4,
},
}

Expand Down

0 comments on commit e679b5f

Please sign in to comment.