Skip to content

ci: Nightly fixes (2026-08-20) - #38366

Merged
def- merged 3 commits into
MaterializeInc:mainfrom
def-:pr-nightly169
Aug 20, 2026
Merged

ci: Nightly fixes (2026-08-20)#38366
def- merged 3 commits into
MaterializeInc:mainfrom
def-:pr-nightly169

Conversation

@def-

@def- def- commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Based on https://buildkite.com/materialize/nightly/builds/18140
Test run: https://buildkite.com/materialize/nightly/builds/18141

Follow-ups to #38196 and #38018

I haven't checked why the benchmarks regressed.

@def-
def- requested review from antiguru and mtabebe August 20, 2026 06:05
@def-
def- requested a review from a team as a code owner August 20, 2026 06:05

@antiguru antiguru left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks very much

antiguru and others added 2 commits August 20, 2026 09:53
…l-label test

`bin/unused-deps` runs `cargo check --workspace --lib --bins` without the
`test` feature, so it never sees `src/environmentd/src/test_util.rs`, the only
user of `tokio-stream` in `mz-environmentd`. Removing the dependency on that
report broke every job that builds the crate with all features. Restore it the
way the manifest already declares its other test-only dependencies: optional in
`[dependencies]` and named in the `test` feature, which also keeps it out of the
production binary and exempts it from the unused-dependency lint.

Comment out the metric sink test covering a null value inside the `labels` map.
The sink operator panics on such a row and aborts clusterd, which fails SLT on
both the test and nightly pipelines. The block is disabled behind a
`TODO(linear#SQL-645)` reference, so `bin/ci-closed-issues-detect` flags it once
the operator is fixed.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@def-
def- enabled auto-merge (squash) August 20, 2026 09:50
@def-

def- commented Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for fixing it up!

@def-
def- merged commit dfa4aa8 into MaterializeInc:main Aug 20, 2026
84 checks passed
@def-
def- deleted the pr-nightly169 branch August 20, 2026 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants