Skip to content

Fix submodules#89193

Merged
azat merged 2 commits intoClickHouse:masterfrom
azat:fix-submodules
Oct 29, 2025
Merged

Fix submodules#89193
azat merged 2 commits intoClickHouse:masterfrom
azat:fix-submodules

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Oct 29, 2025

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Details

Not always github allows to fetch via upstream repo, sometimes it gives "fatal: Fetched in submodule path 'contrib/zstd', but it did not contain X." error.

fatal: transport 'file' not allowed
fatal: Fetched in submodule path 'contrib/zstd', but it did not contain 437081852c396eeb61edb7d47044d8cad911885e. Direct fetching of that commit failed.

Refs:

Not always github allows to fetch via upstream repo, sometimes it gives
"fatal: Fetched in submodule path 'contrib/zstd', but it did not contain
X." error [1].

    fatal: transport 'file' not allowed
    fatal: Fetched in submodule path 'contrib/zstd', but it did not contain 437081852c396eeb61edb7d47044d8cad911885e. Direct fetching of that commit failed.

Refs:
- ClickHouse/zstd#2
- ClickHouse/thrift#2
@azat azat requested a review from al13n321 October 29, 2025 16:55
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh Bot commented Oct 29, 2025

Workflow [PR], commit [a34e59d]

Summary:

job_name test_name status info comment
Stateless tests (amd_tsan, sequential, 2/2) failure
03652_threads_count_insert FAIL cidb
AST fuzzer (amd_debug) failure
Logical error: 'Inconsistent AST formatting: the query: FAIL cidb
AST fuzzer (amd_ubsan) failure
/home/ubuntu/actions-runner/_work/ClickHouse/ClickHouse/src/Functions/tokens.cpp:32:92: runtime error: member call on null pointer of type 'DB::IColumn' FAIL cidb

@clickhouse-gh clickhouse-gh Bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Oct 29, 2025
Copy link
Copy Markdown
Member

@al13n321 al13n321 left a comment

Choose a reason for hiding this comment

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

Oooh, that's what causes that error, nice, I was assuming github is just generally flaky, good to know there's a more specific cause, thanks for fixing. I'll pay attention to this when updating submodules from now on.

@azat azat enabled auto-merge October 29, 2025 17:23
@azat azat added this pull request to the merge queue Oct 29, 2025
Merged via the queue into ClickHouse:master with commit 4109f41 Oct 29, 2025
120 of 124 checks passed
@azat azat deleted the fix-submodules branch October 29, 2025 23:05
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 added the pr-synced-to-cloud The PR is synced to the cloud repo label Oct 29, 2025
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 pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants