rancher / turtles

Rancher CAPI extension
https://turtles.docs.rancher.com
Apache License 2.0
41 stars 15 forks source link

🐛 Separate auto-migration e2e from the import tests #586

Closed Danil-Grigorev closed 3 weeks ago

Danil-Grigorev commented 3 weeks ago

What this PR does / why we need it:

Due to migration suite running together with import tests, the chart state in the same bootstrap cluster was modified from one test to another. This was causing e2e flakes, and this change should fix it.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Fixes #

Special notes for your reviewer:

Checklist:

furkatgofurov7 commented 3 weeks ago

@Danil-Grigorev https://github.com/rancher/turtles/pull/582 is not passing with these changes on top?

furkatgofurov7 commented 3 weeks ago

I am fine merging this (since it fixes the current e2e test failure) and building on top of it the other changes