Skip to content

Add missing lsof for fasttest docker image#19751

Merged
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
azat:docker-lsof
Jan 28, 2021
Merged

Add missing lsof for fasttest docker image#19751
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
azat:docker-lsof

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Jan 28, 2021 •
edited
Loading

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

clickhouse-test uses it

ClickHouse/tests/clickhouse-test

Line 228 in 7c240d0

cmd = "lsof -i tcp:{port} -s tcp:LISTEN -Fp | awk '/^p[0-9]+$/{{print substr($0, 2)}}'".format(port=server_tcp_port)

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Jan 28, 2021
@alexey-milovidov alexey-milovidov self-assigned this Jan 28, 2021
@alesapin
Copy link
Copy Markdown
Member

alesapin commented Jan 28, 2021

:D #19760

@azat azat mentioned this pull request Jan 28, 2021
Merged
@alexey-milovidov alexey-milovidov merged commit 77e8690 into ClickHouse:master Jan 28, 2021
@alexey-milovidov
Copy link
Copy Markdown
Member

alexey-milovidov commented Jan 28, 2021

Fast test Ok.

@azat azat deleted the docker-lsof branch January 28, 2021 20:18
@azat azat mentioned this pull request Feb 18, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexey-milovidov alexey-milovidov alexey-milovidov approved these changes

Assignees

@alexey-milovidov alexey-milovidov

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@azat @alesapin @alexey-milovidov @robot-clickhouse