chore: Split e2e by type

[JorTurFer]

Signed-off-by: Jorge Turrado jorge_turrado@hotmail.es

Checklist

• Commits are signed with Developer Certificate of Origin (DCO - learn more)
• Tests have been added
• Changelog has been updated and is aligned with our changelog requirements

Fixes #3270

[JorTurFer]

/run-e2e
Update: You can check the progress here

[v-shenoy]

Just some suggestions for folder names here. Maybe auth instead of secret-providers. Also assets sounds somewhat off, how about common?

Otherwise, looks good!

[JorTurFer]

I'm not sure about auth, because it could be confusing with scaler auths, my idea is to split external secret providers like vault, key vault, or pod identities here...
I like common over assets, tomorrow I'll change it :)

[v-shenoy]

I'm not sure about auth, because it could be confusing with scaler auths, my idea is to split external secret providers like vault, key vault, or pod identities here... I like common over assets, tomorrow I'll change it :)

Makes sense. To get a bit more clarity, where do tests for things like pod identity / workload identity go?

[JorTurFer]

I'm not sure about auth, because it could be confusing with scaler auths, my idea is to split external secret providers like vault, key vault, or pod identities here... I like common over assets, tomorrow I'll change it :)

Makes sense. To get a bit more clarify, where do tests for things like pod identity / workload identity go?

My idea is put them also in this secret-provider folders because they provide the tokens under the hood

[JorTurFer]

/run-e2e
Update: You can check the progress here

[v-shenoy]

keda/tests/setup_test.go

Line 101 in b4299cc

out, err := ExecuteCommandWithDir("make deploy", "..")

This is causing the problem. The path is incorrect, @JorTurFer

[JorTurFer]

/run-e2e
Update: You can check the progress here

[v-shenoy]

Just one thing, I added some tests for workload identity in scalers_go that need to be moved.

[JorTurFer]

Just one thing, I added some tests for workload identity in scalers_go that need to be moved.

Rebased, PTAL @v-shenoy and I'll merge after is ready to avoid more rebases xD

[JorTurFer]

/run-e2e
Update: You can check the progress here

[v-shenoy]

Just one thing, I added some tests for workload identity in scalers_go that need to be moved.

Rebased, PTAL @v-shenoy and I'll merge after is ready to avoid more rebases xD

Yeah, I am not adding any changes rn. But there are some test PRs in review it seems, maybe this should be merged before them :P

[JorTurFer]

Yeah, I am not adding any changes rn. But there are some test PRs in review it seems, maybe this should be merged before them :P

I think so, to not rebase them again xD