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

Reduce smoke test suite size #2182

Merged
merged 2 commits into from
Jun 21, 2022

Conversation

carolynvs
Copy link
Member

The smoke tests are taking too long, 5m on linux and 10m+ on windows on our CI servers. The goal for the smoke tests is <3m so that it's easy to run and vet that you didn't completely break porter.

I've moved some of the smoke tests into the integration test suite so that we are only focusing on the most important checks in the smoke test suite.

The smoke tests are taking too long, 5m on linux and 10m+ on windows on
our CI servers. The goal for the smoke tests is <3m so that it's easy to
run and vet that you didn't completely break porter.

I've moved some of the smoke tests into the integration test suite so
that we are only focusing on the most important checks in the smoke test
suite.

Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>
@carolynvs carolynvs marked this pull request as ready for review June 20, 2022 20:23
@carolynvs carolynvs requested a review from VinozzZ June 20, 2022 20:23
@carolynvs
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 4 pipeline(s).

@carolynvs
Copy link
Member Author

This brought the windows smoke test run down from 10m to ~4m. The integration tests are taking 20m and will need to be looked at for redundant test cases and optimizations so that it runs faster (usually by stacking more assertions onto a single bundle run).

@carolynvs carolynvs merged commit 778eb49 into getporter:release/v1 Jun 21, 2022
@carolynvs carolynvs deleted the move-smoke-to-int branch June 21, 2022 20:40
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