Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pass option for test_only properly #4579

Merged
merged 3 commits into from
Mar 16, 2021
Merged

Conversation

Frawless
Copy link
Member

Signed-off-by: Jakub Stejskal xstejs24@gmail.com

Type of change

  • Bugfix

Description

test_only is not reported properly in case of failure. Instead of a proper string, it adds to rerun cmd only true or false which is useless.

Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
@Frawless Frawless added this to the 0.23.0 milestone Mar 16, 2021
@Frawless
Copy link
Member Author

@strimzi-ci run tests profile=smoke test_only

@michalxo
Copy link
Contributor

michalxo commented Mar 16, 2021

Re-run command:
@strimzi-ci run tests true profile=regression,acceptance testcase=io.strimzi.systemtest.security.SecurityS...

LGTM as fix. Thanks

@strimzi-ci
Copy link

✔️ Test Summary ✔️

TEST_PROFILE: smoke
EXCLUDED_GROUPS: loadbalancer,networkpolicies,flaky
TEST_CASE: *ST
TOTAL: 1
PASS: 1
FAIL: 0
SKIP: 0
BUILD_NUMBER: 1868
KUBE_VERSION: 1.16.0

Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
@Frawless
Copy link
Member Author

@strimzi-ci run tests profile=smoke test_only

Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
@Frawless
Copy link
Member Author

@strimzi-ci run tests profile=smoke test_only

@strimzi-ci
Copy link

✔️ Test Summary ✔️

TEST_PROFILE: smoke
EXCLUDED_GROUPS: loadbalancer,networkpolicies,flaky
TEST_CASE: *ST
TOTAL: 1
PASS: 1
FAIL: 0
SKIP: 0
BUILD_NUMBER: 1878
KUBE_VERSION: 1.16.0

@strimzi-ci
Copy link

✔️ Test Summary ✔️

TEST_PROFILE: smoke
EXCLUDED_GROUPS: loadbalancer,networkpolicies,flaky
TEST_CASE: *ST
TOTAL: 1
PASS: 1
FAIL: 0
SKIP: 0
BUILD_NUMBER: 1879
KUBE_VERSION: 1.16.0

@Frawless Frawless added the ready for merge Label for PRs which are ready for merge label Mar 16, 2021
@scholzj scholzj merged commit e5f9481 into strimzi:master Mar 16, 2021
AaruniAggarwal pushed a commit to AaruniAggarwal/strimzi-kafka-operator that referenced this pull request Mar 18, 2021
* Pass option for test_only properly

Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>

* fixup! Pass option for test_only properly

Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>

* fixup! fixup! Pass option for test_only properly

Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for merge Label for PRs which are ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants