Skip to content

Add debug_output_enabled parameter to macOS CI workflow#3542

Merged
rnro merged 3 commits intoapple:mainfrom
rnro:macos_ci_debug_output
Mar 9, 2026
Merged

Add debug_output_enabled parameter to macOS CI workflow#3542
rnro merged 3 commits intoapple:mainfrom
rnro:macos_ci_debug_output

Conversation

@rnro
Copy link
Copy Markdown
Contributor

@rnro rnro commented Mar 9, 2026

Motivation

  • xcodebuild output is suppressed with -quiet in the macOS tests workflow, making it hard to diagnose CI failures on the main branch.

Modifications

  • Add a debug_output_enabled boolean input to macos_tests.yml (defaults to false), which controls a QUIET_ARG env var used by all xcodebuild invocations.
  • When debug_output_enabled is true, -quiet is elided and full xcodebuild output is emitted.
  • Enable debug_output_enabled in main.yml so nightly and main-branch push builds produce verbose output.

Result

  • PR builds remain concise (default behaviour unchanged).
  • Main-branch builds now include full xcodebuild output for easier failure triage.

rnro added 2 commits March 9, 2026 15:28
Motivation

* xcodebuild output is suppressed with `-quiet` in the macOS tests
  workflow, making it hard to diagnose CI failures on the main branch.

Modifications

* Add a `debug_output_enabled` boolean input to `macos_tests.yml`
  (defaults to `false`), which controls a `QUIET_ARG` env var used by
  all xcodebuild invocations.
* When `debug_output_enabled` is `true`, `-quiet` is elided and full
  xcodebuild output is emitted.
* Enable `debug_output_enabled` in `main.yml` so nightly and
  main-branch push builds produce verbose output.

Result

* PR builds remain concise (default behaviour unchanged).
* Main-branch builds now include full xcodebuild output for easier
  failure triage.
Copy link
Copy Markdown
Member

@fabianfett fabianfett left a comment

Choose a reason for hiding this comment

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

LGTM

@rnro rnro added the semver/none No version bump required. label Mar 9, 2026
@rnro rnro enabled auto-merge (squash) March 9, 2026 15:48
@rnro rnro merged commit 2fdfe86 into apple:main Mar 9, 2026
51 of 55 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver/none No version bump required.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants