Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor integration test workflows for reusability #303

Closed
wants to merge 24 commits into from

Conversation

jhamon
Copy link
Collaborator

@jhamon jhamon commented Feb 1, 2024

Problem

We want to make our integration testing reusable by the platform team, with configurations that allow:

  • Targeting staging instead of prod
  • Targeting specific environments/regions/clouds
  • Running with previously published SDK versions

Solution

  • Refactor integration testing CI configs to split into reusable workflows that individually test either serverless features or pod index features. This separation greatly improves the ergonomics of expressing the test matrix for each of these code paths.
  • Added setup steps that work when executing the tests using previously published artifacts.
  • Added a bunch more test cases that were previously missing.
  • Added cleanup step in dependency testing

Type of Change

  • Infrastructure change (CI configs, etc)

Test Plan

See tests passing.

@jhamon jhamon force-pushed the jhamon/reusable-workflow-refactor branch from 1bdc838 to 8623a55 Compare February 7, 2024 20:14
@jhamon jhamon marked this pull request as ready for review February 9, 2024 16:59
@jhamon jhamon requested a review from austin-denoble February 9, 2024 17:00
@jhamon jhamon changed the title Refactor integration test workflows Refactor integration test workflows for reusability Feb 9, 2024
@jhamon jhamon force-pushed the jhamon/reusable-workflow-refactor branch from 8623a55 to 2d11293 Compare February 12, 2024 17:41
@jhamon jhamon closed this May 13, 2024
@jhamon jhamon deleted the jhamon/reusable-workflow-refactor branch July 31, 2024 16:25
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