Skip to content

Adding more explanatory note on reordering integration tests to avoid cold-start failures#1301

Open
ivanovac wants to merge 2 commits into
masterfrom
fix-composer-security-advisory-failures
Open

Adding more explanatory note on reordering integration tests to avoid cold-start failures#1301
ivanovac wants to merge 2 commits into
masterfrom
fix-composer-security-advisory-failures

Conversation

@ivanovac

@ivanovac ivanovac commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Symptom: The first test to execute consistently fails with HTTP 500 'UnknownError' from the CF API during app push. Investigation revealed this is a position-dependent issue, not specific to any particular test.

Root cause: CF environment cold-start / resource initialization issue on php.buildpacks.ci.cloudfoundry.org.

Solution: Run 'Modules' test first instead of 'Default'. This allows the first test to warm up the CF environment, and all subsequent tests (including Default) pass successfully.

Diagnosis approach:

  • Reordered tests to make 'Modules' run first
  • All 37 tests now pass reliably
  • Confirms the issue is timing/initialization, not test-specific

This is adding more clear note on reordering integration tests to avoid cold-start failures.

ivanovac added 2 commits July 7, 2026 14:56
The first test to execute consistently fails with HTTP 500 'UnknownError'
from the CF API during app push. Investigation revealed this is a
position-dependent issue, not specific to any particular test.

Root cause: CF environment cold-start / resource initialization issue
on php.buildpacks.ci.cloudfoundry.org.

Solution: Run 'Modules' test first instead of 'Default'. This allows
the first test to warm up the CF environment, and all subsequent tests
(including Default) pass successfully.

Diagnosis approach:
- Reordered tests to make 'Modules' run first
- All 37 tests now pass reliably
- Confirms the issue is timing/initialization, not test-specific

This is a workaround for an infrastructure issue. Long-term fix would
be to investigate Diego cell initialization, CC connection pools, or
security group propagation on the CI environment.
@ivanovac ivanovac changed the title Fix: Reorder integration tests to avoid cold-start failures Adding more clear note on reording integration tests to avoid cold-start failures Jul 7, 2026
@ivanovac ivanovac changed the title Adding more clear note on reording integration tests to avoid cold-start failures Adding more clear note on reordering integration tests to avoid cold-start failures Jul 7, 2026
@ivanovac ivanovac changed the title Adding more clear note on reordering integration tests to avoid cold-start failures Adding more explanatory note on reordering integration tests to avoid cold-start failures Jul 7, 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