Skip to content

docker: avoid chown of .#22102

Merged
alesapin merged 2 commits intomasterfrom
filimonov-docker-empty-dirname
Mar 26, 2021
Merged

docker: avoid chown of .#22102
alesapin merged 2 commits intomasterfrom
filimonov-docker-empty-dirname

Conversation

@filimonov
Copy link
Copy Markdown
Contributor

@filimonov filimonov commented Mar 24, 2021

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

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Docker entrypoint: avoid chown of . in case when LOG_PATH is empty. Closes #22100

Detailed description / Documentation draft:

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Mar 24, 2021
@alesapin alesapin self-assigned this Mar 25, 2021
@alesapin
Copy link
Copy Markdown
Member

Cannot affect any failed tests.

@alesapin alesapin merged commit 4c67e25 into master Mar 26, 2021
@alesapin alesapin deleted the filimonov-docker-empty-dirname branch March 26, 2021 19:19
alesapin added a commit that referenced this pull request Mar 29, 2021
Backport #22102 to 21.2: docker: avoid chown of .
alesapin added a commit that referenced this pull request Mar 29, 2021
Backport #22102 to 21.3: docker: avoid chown of .
alesapin added a commit that referenced this pull request Mar 30, 2021
Backport #22102 to 21.1: docker: avoid chown of .
alesapin added a commit that referenced this pull request Mar 30, 2021
Backport #22102 to 20.8: docker: avoid chown of .
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docker entrypoint script try to chown '.' if log path is not set in config

3 participants