Skip to content

[air] Suppress noisy missing requirements.yaml log - #6715

Open
caroline-db wants to merge 1 commit into
mainfrom
air/suppress-requirements-warning
Open

caroline-db wants to merge 1 commit into
mainfrom
air/suppress-requirements-warning

Conversation

@caroline-db

@caroline-db caroline-db commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Changes

  • Suppress the backend-derived log that requirements.yaml is absent when streaming Databricks Air logs.
  • Preserve other streamed logs and raw log downloads.
  • Preserve behavior for Python CLI side, rather than removing the message from the backend

Why

requirements.yaml is unsupported by Databricks Air and rejected earlier. Its absence is therefore expected, making the backend notice non-actionable noise.

Tests

  • go test ./experimental/air/cmd -run 'TestEmitLogLine|TestSuppressLogLine'
  • ./task checks
  • ./task test (passed after an unrelated acceptance-test flake succeeded on retry)

Confirmed E2E no longer prints out this log line:
Screenshot 2026-09-16 at 1 22 28 PM

This PR was written with Codex.

@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: d64f20b

Run: 35153078345

Env ✅​pass 🙈​skip Time
✅​ aws linux 276 15 4:08
✅​ aws windows 278 13 3:41
✅​ azure linux 275 15 4:04
✅​ azure windows 277 13 3:51
✅​ gcp linux 276 15 4:15
✅​ gcp windows 278 13 4:18
Top 3 slowest tests (at least 2 minutes):
duration env testname
4:16 gcp windows TestAccept
3:48 azure windows TestAccept
3:39 aws windows TestAccept

@caroline-db
caroline-db force-pushed the air/suppress-requirements-warning branch from c6ddef6 to d64f20b Compare September 16, 2026 21:31
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.

3 participants