Skip to content

[2026.3] Remove API-test container log collection added in #555 - #558

Merged
kingjia90 merged 1 commit into
2026.3from
remove-api-test-log-collection
Sep 30, 2026
Merged

kingjia90 merged 1 commit into
2026.3from
remove-api-test-log-collection

Conversation

@kingjia90

Copy link
Copy Markdown
Contributor

Follow-up to #555. The Mercure v0.24 pin (plus the supervisord restart) restores the API tests. The failure-time container log collection was only a diagnostic aid, so this removes it:

  • .github/scripts/07-collect-logs.sh
  • the "Collect container logs" / "Upload container logs" steps in api-tests.yml
  • the /container-logs/ .gitignore entry and the docs line

All four files return to their pre-#555 state. The Mercure pin and supervisord restart are unchanged.

🤖 Generated with Claude Code

@kingjia90 kingjia90 added this to the 2026.3 milestone Sep 30, 2026
Only the Mercure v0.24 pin is needed to restore the API tests; the
failure-time log collection was a diagnostic aid and is dropped.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kingjia90
kingjia90 force-pushed the remove-api-test-log-collection branch from 9a63dde to bcf00d3 Compare September 30, 2026 09:09
@kingjia90
kingjia90 merged commit 7c4b5a6 into 2026.3 Sep 30, 2026
13 checks passed
@kingjia90
kingjia90 deleted the remove-api-test-log-collection branch September 30, 2026 09:12
@kingjia90 kingjia90 self-assigned this Sep 30, 2026
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.

1 participant