Prune stale Darwin fast-test skips; fix isLocalAddress for lo0 aliases by Algunenano · Pull Request #108630 · ClickHouse/ClickHouse · GitHub
Skip to content

Prune stale Darwin fast-test skips; fix isLocalAddress for lo0 aliases#108630

Merged
Algunenano merged 23 commits into
ClickHouse:masterfrom
Algunenano:darwin-skip-prune-batch8
Jun 29, 2026
Merged

Prune stale Darwin fast-test skips; fix isLocalAddress for lo0 aliases#108630
Algunenano merged 23 commits into
ClickHouse:masterfrom
Algunenano:darwin-skip-prune-batch8

Conversation

@Algunenano

@Algunenano Algunenano commented Jun 26, 2026

Copy link
Copy Markdown
Member

Prunes stale entries from the macOS/Darwin fast-test skip list (ci/defs/darwin.skip) by verifying which skipped tests now pass on Apple arm64 and un-skipping them. Each un-skipped test was run 20x consecutively before removal.

The main enabler is a fix to isLocalAddress: 127.0.0.{2..255} are now decided as non-local by value alone, never falling through to the interface scan. On macOS those addresses get assigned to lo0 as aliases to emulate distributed queries, which previously made them "local" and collapsed remote shards into local serial execution. This matches Linux behaviour and unblocks ~140 distributed/remote() tests.

Other changes:

  • Allow # comments in darwin.skip and group remaining entries into commented sections by known reason.
  • Tag 01317_no_password_in_command_line as no-darwin (relies on BSD ps/procfs).

Changelog category (leave one):

  • CI Fix or Improvement (changelog entry is not required)

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

...

Version info

  • Merged into: 26.7.1.252

@clickhouse-gh

clickhouse-gh Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

@clickhouse-gh clickhouse-gh Bot added the pr-ci label Jun 26, 2026
@clickhouse-gh

clickhouse-gh Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

LLVM Coverage Report

Metric Baseline Current Δ
Lines 85.40% 85.50% +0.10%
Functions 92.60% 92.60% +0.00%
Branches 77.60% 77.70% +0.10%

Changed lines: Changed C/C++ lines covered by tests: 30/33 (90.91%) | Lost baseline coverage: none · Uncovered code

Full report · Diff report

@Algunenano Algunenano added this pull request to the merge queue Jun 29, 2026
Merged via the queue into ClickHouse:master with commit 45cf5f6 Jun 29, 2026
342 of 344 checks passed
@Algunenano Algunenano deleted the darwin-skip-prune-batch8 branch June 29, 2026 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-ci 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.

2 participants