Add tests for other cable drivers as well

[Jaanki]

Currently, subctl tests for default cable driver only. Add tests to run against all PRs for other cable drivers as well.

[tpantelis]

We decided to run a minimal set of tests in subctl. Also submariner-operator doesn't even run cable driver tests. The submariner repo tests all those.

[mkolesnik]

What's the value of having a full e2e matrix?

There's nothing in the e2e tests that actually verifies that any given flag that you set is actually getting deployed (which is a bigger problem but not directly related to this).

You could have simple unit tests to achieve the same goal - see that a flag that you set gets passed down as is to the CR that gets created.
Or if you want you could say that we want to add it to system tests, but since the whole point is to test that a flag gets passed down to a CR I think it's overkill.

[tpantelis]

You could have simple unit tests to achieve the same goal - see that a flag that you set gets passed down as is to the CR that gets created.

👍

[stale]

This issue has been automatically marked as stale because it has not had activity for 60 days. It will be closed if no further activity occurs. Please make a comment if this issue/pr is still valid. Thank you for your contributions.

[dfarrell07]

Discussing on the PR scrub, it seems to make sense to have system tests in the operator to verify cable drivers we request are deployed and/or to have unit tests in subctl to verify the submariner object's cable driver flag is set properly.

[dfarrell07]

Unit test part: #408

[dfarrell07]

System tests: submariner-io/submariner-operator#2373