Skip to content

Integration test enhancements#461

Merged
TimHess merged 1 commit into
mainfrom
integration_tests
May 13, 2026
Merged

Integration test enhancements#461
TimHess merged 1 commit into
mainfrom
integration_tests

Conversation

@TimHess
Copy link
Copy Markdown
Member

@TimHess TimHess commented May 13, 2026

  • Split Linux/Windows jobs by integration environment and decouple from each other
  • Add service plan alternatives and fix auto-generated space cleanup
  • Use unique service instance names per sample
  • Capture recent CF logs on app crash or task failure
  • Add (obscured) warning logging
  • Increase Windows container memory
  • Support additional app and SI wait-for and failed states
  • Use windows2022 stack
  • Use a GHA var instead of the repo name to determine if tests actually run
  • Only write non-empty vars to user.ini

- Split Linux/Windows jobs by integration environment and decouple from each other
- Add service plan alternatives and fix auto-generated space cleanup
- Use unique service instance names per sample
- Capture recent CF logs on app crash or task failure
- Add warning logging
- Increase Windows container memory
- Support additional wait-for or failed states
- Use windows2022 stack
- Use a GHA var instead of the repo name to determine if tests actually run
- Only write non-empty vars to user.ini
@TimHess TimHess marked this pull request as ready for review May 13, 2026 15:56
@TimHess TimHess requested a review from bart-vmware May 13, 2026 15:56
Copy link
Copy Markdown
Member

@bart-vmware bart-vmware left a comment

Choose a reason for hiding this comment

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

Thanks, this is really great!

@TimHess TimHess merged commit d9c294b into main May 13, 2026
20 checks passed
@TimHess TimHess deleted the integration_tests branch May 13, 2026 17:10
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