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

ST: Fix test which checks labels for cluster CMs #3524

Merged
merged 1 commit into from
Aug 19, 2020

Conversation

Frawless
Copy link
Member

@Frawless Frawless commented Aug 18, 2020

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

Type of change

  • Bugfix

Description

This PR fix KafkaST#testAppDomainLabels after merge of #3328.

Checklist

  • Make sure all tests pass

Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
@Frawless Frawless added this to the 0.20.0 milestone Aug 18, 2020
@Frawless Frawless self-assigned this Aug 18, 2020
@Frawless
Copy link
Member Author

@strimzi-ci run tests test_only testcase=KafkaST#testAppDomainLabels

@strimzi-ci
Copy link

Systemtests Failed (no tests results are present)

@Frawless
Copy link
Member Author

@strimzi-ci run tests test_only profile=regression testcase=KafkaST#testAppDomainLabels

@strimzi-ci
Copy link

✔️ Test Summary ✔️

TEST_PROFILE: regression
EXCLUDED_GROUPS: networkpolicies,flaky
TEST_CASE: KafkaST#testAppDomainLabels
TOTAL: 1
PASS: 1
FAIL: 0
SKIP: 0
BUILD_NUMBER: 1381
BUILD_ENV: oc cluster up

@Frawless Frawless marked this pull request as ready for review August 18, 2020 13:28
@Frawless Frawless requested review from see-quick and a team August 18, 2020 13:28
@Frawless Frawless added the ready for merge Label for PRs which are ready for merge label Aug 19, 2020
@Frawless Frawless merged commit 9fe4eb8 into strimzi:master Aug 19, 2020
klalafaryan pushed a commit to klalafaryan/strimzi-kafka-operator that referenced this pull request Oct 21, 2020
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 System tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants