Skip to content

Bump krb5 to 1.22.1-final#86836

Merged
thevar1able merged 1 commit intomasterfrom
bump-krb5
Sep 10, 2025
Merged

Bump krb5 to 1.22.1-final#86836
thevar1able merged 1 commit intomasterfrom
bump-krb5

Conversation

@thevar1able
Copy link
Copy Markdown
Member

Changelog category (leave one):

  • Build/Testing/Packaging Improvement

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

Use krb5 1.22.1-final

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Sep 8, 2025

Workflow [PR], commit [02b4e4c]

Summary:

job_name test_name status info comment
Integration tests (amd_binary, 3/5) failure
test_s3_plain_rewritable/test.py::test[s3_plain_rewritable_with_metadata_cache-data_with_cache/] FAIL

@clickhouse-gh clickhouse-gh bot added pr-build Pull request with build/testing/packaging improvement submodule changed At least one submodule changed in this PR. labels Sep 8, 2025
@rschu1ze
Copy link
Copy Markdown
Member

rschu1ze commented Sep 8, 2025

This PR includes commits

  1. e4f0afd15 (suppresses TSAN)
  2. 43b8574d9 (silence build)
  3. de6abc343 (make it work with boringssl)
  4. ef0e4ec85 (suppresses MSAN)

I think we should try to upstream 1) and 4) so we can get rid of it in the long run.
3) should no longer be necessary.

@thevar1able thevar1able marked this pull request as draft September 8, 2025 16:46
@thevar1able
Copy link
Copy Markdown
Member Author

Dropped de6abc343, thanks. For other points please consider filing a separate improvement item.

@thevar1able
Copy link
Copy Markdown
Member Author

thevar1able commented Sep 9, 2025

@rschu1ze FYI ClickHouse/krb5@1279d8a, apparently DNS feature flag is broken upstream, and we don't link with resolv to enable DNS in krb5.

DNS stays disabled as before, but I removed conditionals around struct kdclist due to some forward declarations.

@thevar1able thevar1able marked this pull request as ready for review September 9, 2025 16:31
@thevar1able
Copy link
Copy Markdown
Member Author

@thevar1able thevar1able added this pull request to the merge queue Sep 10, 2025
Merged via the queue into master with commit 951941b Sep 10, 2025
119 of 122 checks passed
@thevar1able thevar1able deleted the bump-krb5 branch September 10, 2025 14:50
@robot-clickhouse robot-clickhouse added the pr-synced-to-cloud The PR is synced to the cloud repo label Sep 10, 2025
@rschu1ze
Copy link
Copy Markdown
Member

For other points please consider filing a separate improvement item.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-build Pull request with build/testing/packaging improvement pr-synced-to-cloud The PR is synced to the cloud repo submodule changed At least one submodule changed in this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants