Skip to content

Fix more flaky integration tests#17486

Merged
alesapin merged 4 commits intomasterfrom
fix_more_flaky_tests
Nov 28, 2020
Merged

Fix more flaky integration tests#17486
alesapin merged 4 commits intomasterfrom
fix_more_flaky_tests

Conversation

@alesapin
Copy link
Copy Markdown
Member

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Nov 27, 2020

kafka_kerberos:
image: yandex/clickhouse-kerberos-kdc:${DOCKER_KERBEROS_KDC_TAG}
image: yandex/clickhouse-kerberos-kdc:${DOCKER_KERBEROS_KDC_TAG:-latest}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, but I don't understand the motivation of these changes.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just use the latest tag if don't provide anything

@alexey-milovidov alexey-milovidov self-assigned this Nov 28, 2020
@alesapin alesapin merged commit 1d179da into master Nov 28, 2020
@alesapin alesapin deleted the fix_more_flaky_tests branch November 28, 2020 07:43
alesapin added a commit that referenced this pull request Dec 5, 2020
Fix more flaky integration tests

(cherry picked from commit 1d179da)
alesapin added a commit that referenced this pull request Dec 5, 2020
Fix more flaky integration tests

(cherry picked from commit 1d179da)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants