Skip to content

Commit

Permalink
Change default value for E2E_PARALLEL (#1124)
Browse files Browse the repository at this point in the history
The original implementation for enabling parallel running of tests was
added when ginkgo expected a numeric value for the number of nodes to
use when running in parallel. This is no longer needed as ginkgo can
detect the optimal number of nodes to use. The script for running the
e2e tests now expects "true", "y" or "Y" to enable parallel testing.
This commmit changes the default value to "false". Although the previous
value would not have resulted in parallel testing being enabled, this
change helps users understand that the value provided should be boolean.

Signed-off-by: Bridget McErlean <bmcerlean@vmware.com>
  • Loading branch information
zubron committed May 26, 2020
1 parent e1a28d2 commit 385f41e
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions cmd/sonobuoy/app/args_test.go
Expand Up @@ -38,16 +38,16 @@ func TestGetE2EConfig(t *testing.T) {
flagArgs: []string{},
expect: &ops.E2EConfig{
Focus: `\[Conformance\]`,
Parallel: "1",
Parallel: "false",
},
}, {
desc: "Flags settable",
mode: "certified-conformance",
flagArgs: []string{"--e2e-focus=foo", "--e2e-skip=bar", "--e2e-parallel=2"},
flagArgs: []string{"--e2e-focus=foo", "--e2e-skip=bar", "--e2e-parallel=true"},
expect: &ops.E2EConfig{
Focus: `foo`,
Skip: `bar`,
Parallel: "2",
Parallel: "true",
},
}, {
desc: "Focus regexp validated settable",
Expand Down
6 changes: 3 additions & 3 deletions pkg/client/mode.go
Expand Up @@ -87,7 +87,7 @@ func (m *Mode) Get() *ModeConfig {
return &ModeConfig{
E2EConfig: E2EConfig{
Focus: `\[Conformance\]`,
Parallel: "1",
Parallel: "false",
},
Selectors: []plugin.Selection{
{Name: "e2e"},
Expand All @@ -99,7 +99,7 @@ func (m *Mode) Get() *ModeConfig {
E2EConfig: E2EConfig{
Focus: `\[Conformance\]`,
Skip: nonDisruptiveSkipList,
Parallel: "1",
Parallel: "false",
},
Selectors: []plugin.Selection{
{Name: "e2e"},
Expand All @@ -110,7 +110,7 @@ func (m *Mode) Get() *ModeConfig {
return &ModeConfig{
E2EConfig: E2EConfig{
Focus: "Pods should be submitted and removed",
Parallel: "1",
Parallel: "false",
},
Selectors: []plugin.Selection{
{Name: "e2e"},
Expand Down

0 comments on commit 385f41e

Please sign in to comment.