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.

Sign up for GitHub

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

Jump to bottom

Replace <yandex> to <clickhouse> in configs and more. #29182

Merged
merged 51 commits into from
Oct 1, 2021
Merged

Replace <yandex> to <clickhouse> in configs and more. #29182

merged 51 commits into from
Oct 1, 2021

Conversation

alexey-milovidov
Copy link
Member

@alexey-milovidov alexey-milovidov commented Sep 19, 2021 •
edited

Changelog category (leave one):

  • Documentation (changelog entry is not required)

See https://clickhouse.com/blog/en/2021/clickhouse-inc/

@robot-clickhouse robot-clickhouse added the pr-documentation Documentation PRs for the specific code PR label Sep 19, 2021
@alexey-milovidov alexey-milovidov marked this pull request as draft September 20, 2021 01:17
@alexey-milovidov
Copy link
Member Author

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Sep 21, 2021

Command update: success

Branch has been successfully updated

@alexey-milovidov
Copy link
Member Author

Integration tests flaky check (asan) — Failed to read...

Timeout due to too many changed tests - Ok.

@alexey-milovidov
Copy link
Member Author

Performance — Failed to parse the report.

Failed because of trying to run old version with new config.d files. Ok.

@alexey-milovidov
Copy link
Member Author

AST fuzzer:

CalledProcessError: Command 'docker pull clickhouse/fuzzer:29182-4ade6cc5da6ee881a2a92c59f55e3a98e2e87d12' returned non-zero exit status 1.

@alexey-milovidov alexey-milovidov marked this pull request as ready for review September 22, 2021 00:25
@alexey-milovidov alexey-milovidov requested a review from a team as a code owner September 22, 2021 00:25
@alexey-milovidov
Copy link
Member Author

@Mergifyio update

@alexey-milovidov alexey-milovidov added the force tests The label does nothing, NOOP, None, nil label Sep 22, 2021
@mergify
Copy link
Contributor

mergify bot commented Sep 22, 2021

Command update: success

Branch has been successfully updated

@alexey-milovidov
Copy link
Member Author

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Sep 26, 2021

Command update: success

Branch has been successfully updated

@alexey-milovidov alexey-milovidov self-assigned this Sep 27, 2021
@CLAassistant
Copy link

CLAassistant commented Sep 28, 2021 •
edited

CLA assistant check
All committers have signed the CLA.

@alexey-milovidov
Copy link
Member Author

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Sep 29, 2021

Command update: success

Branch has been successfully updated

@alexey-milovidov alexey-milovidov mentioned this pull request Sep 29, 2021
Closed
@alexey-milovidov
Copy link
Member Author

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Sep 29, 2021

Command update: success

Branch has been successfully updated

@alexey-milovidov
Copy link
Member Author

One more step to get tests passing, please be patient...

@alexey-milovidov alexey-milovidov merged commit b29e877 into master Oct 1, 2021
@alexey-milovidov alexey-milovidov deleted the yandex-to-clickhouse-in-configs branch October 1, 2021 09:24
@azat azat mentioned this pull request Oct 25, 2021
Merged
@soeunj soeunj mentioned this pull request Jul 28, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@alexey-milovidov alexey-milovidov

Labels
force tests The label does nothing, NOOP, None, nil pr-documentation Documentation PRs for the specific code PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@alexey-milovidov @alesapin @CLAassistant @robot-clickhouse