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: increase storage in GH runners and run integration tests in different machines #187

Merged
merged 3 commits into from
Aug 1, 2023

Conversation

DnPlas
Copy link
Contributor

@DnPlas DnPlas commented Aug 1, 2023

This PR includes the following changes:

  • ci: increase storage of GH runners for integration testing

Increasing the storage of the GH runners will avoid potential issues when running integration
tests that deploy SeldonDeployments that require a significant amount of them.

Part of #185

  • ci: run integration tests in different machines to avoid deploying seldon twice

Building and deploying seldon twice in a row may lead to errors while executing test cases from two different
test files. This change ensures each individual test file runs on its own environment.

Part of #185

Expected CI failures

…ldon twice

Building and deploying seldon twice in a row may lead to errors while executing test cases from two different
test files. This change ensures each individual test file runs on its own environment.

Part of #185
Increasing the storage of the GH runners will avoid potential issues when running integration
tests that deploy SeldonDeployments that require a significant amount of them.

Part of #185
Copy link
Contributor

@i-chvets i-chvets left a comment

Choose a reason for hiding this comment

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

Approved. Pending CI.

@DnPlas DnPlas merged commit 7fb43b2 into dnplas-fix-ci Aug 1, 2023
8 checks passed
@DnPlas DnPlas deleted the KF-4054-fix-185 branch August 1, 2023 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants