test: check migration to v3 controller

rancher / turtles

Rancher CAPI extension

https://turtles.docs.rancher.com

Apache License 2.0

41 stars 15 forks source link

test: check migration to v3 controller #585

Open salasberryfin opened 3 weeks ago

salasberryfin commented 3 weeks ago

What this PR does / why we need it:

We decided to split into a separate PR the E2E tests added to validate v1 to v3 controller migration. The changes to the controller can be found in https://github.com/rancher/turtles/pull/575. This PR only affects E2E scenarios.

Which issue(s) this PR fixes: Fixes #

Special notes for your reviewer:

Checklist:

[x] squashed commits into logical changes
[ ] includes documentation
[ ] adds unit tests
[x] adds or updates e2e tests

furkatgofurov7 commented 3 weeks ago

@salasberryfin can we rebase this PR?

Danil-Grigorev commented 1 week ago

@salasberryfin Can you please rebase this PR?

furkatgofurov7 commented 1 week ago

Does this need a rebase again?

salasberryfin commented 1 week ago

This was rebased today but it is still failing when trying to connect to the downstream cluster after migration using kubeconfig. Still figuring out the reason behind this.

Danil-Grigorev commented 1 week ago

@salasberryfin The problem you are facing might be fixed by https://github.com/rancher/turtles/commit/7653a8182ee99e9fbdf7c36c417b4331eeac3bd7#diff-7be87c7e06c4b33e0c43cce272e500934fca8631072faf36237af336d4c1c562R185-R190