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

CI: split load shedder integration tests #3675

Merged
merged 1 commit into from
Feb 12, 2024
Merged

Conversation

NachoSoto
Copy link
Contributor

Follow up to #3673.

Changes:

  • Split loadshedder-integration-tests into integration-tests-all and loadshedder-integration-tests-v3
  • Removed loadshedder-integration-tests from build_test: this reverts Run load shedder integration tests on release branches #3673. We already run loadshedder tests as part of SK1 and SK2 test suites.
  • Add BackendIntegrationTests-Shedder test plane. This isn't used by CI (see above point), but it can be useful to run all of the load shedder tests.

Follow up to #3673.

### Changes:
- Split `loadshedder-integration-tests` into `integration-tests-all` and `loadshedder-integration-tests-v3`
- Removed `loadshedder-integration-tests` from `build_test`: this reverts #3673. We already run loadshedder tests as part of `SK1` and `SK2` test suites.
- Add `BackendIntegrationTests-Shedder` test plane. This isn't used by CI (see above point), but it can be useful to run all of the load shedder tests.
Copy link
Contributor

@tonidero tonidero left a comment

Choose a reason for hiding this comment

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

Makes sense, thanks for doing this!

@NachoSoto NachoSoto enabled auto-merge (squash) February 12, 2024 17:11
@NachoSoto NachoSoto merged commit d2b1c45 into main Feb 12, 2024
24 checks passed
@NachoSoto NachoSoto deleted the load-shedder-tests-only branch February 12, 2024 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants