Run component test in parallel #1797
Conversation
|
/test integration |
amitkrout
force-pushed
the
componentParallelTestRun
branch
from
717ec5d to
300d765
Compare
|
@kadel @mohammedzee1000 Pr related to parallel run keeps growing. Can you guys start reviewing from this component/sub-component pr cc_ @cdrage |
| @@ -114,12 +114,12 @@ test-json-format-output: | |||
| # Run component e2e tests | |||
| .PHONY: test-cmp-e2e | |||
| test-cmp-e2e: | |||
| go test -v github.com/openshift/odo/tests/integration --ginkgo.focus="odoCmpE2e" -ginkgo.slowSpecThreshold=$(SLOW_SPEC_THRESHOLD) -ginkgo.v -timeout $(TIMEOUT) | |||
| ginkgo -v -nodes=$(TEST_EXEC_NODES) -focus="odoCmpE2e" slowSpecThreshold=$(SLOW_SPEC_THRESHOLD) -randomizeAllSpecs tests/integration/ -timeout $(TIMEOUT) | |||
There was a problem hiding this comment.
why are we now randomizing? you need to document this in your PR description / commit.
There was a problem hiding this comment.
|
|
||
| * For the entire integration suite use: | ||
| ---- | ||
| $ make test-e2e |
There was a problem hiding this comment.
@amitkrout again, you're removing documentation and not adding it again... no documentation on make test-integration
There was a problem hiding this comment.
Ahh...my bad. Will update it
There was a problem hiding this comment.
amitkrout
force-pushed
the
componentParallelTestRun
branch
from
71c789b to
488921a
Compare
openshift-ci-robot
added
the
lgtm
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cdrage The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci-robot
added
the
approved
rm3l
added
the
estimated-size/L (20-40)
What is the purpose of this change? What does it change?
To reduce test time both locally and on CI too.
Was the change discussed in an issue?
fixes - part of #1473
How to test changes?
make test-cmp-e2e && make test-cmp-sub-e2e