kuadrant-operator icon indicating copy to clipboard operation
kuadrant-operator copied to clipboard

[wip] refactor: parrallel tests

Open KevFan opened this issue 1 year ago • 3 comments

KevFan avatar May 22 '24 09:05 KevFan

What is the reasoning for this refactor? This PR conflicts with a WIP test refactor I am doing for making room to onboard EnvoyGateway. Basically I am creating "istio tests", so we can later add "envoy gateway tests".

eguzki avatar May 22 '24 12:05 eguzki

What is the reasoning for this refactor? This PR conflicts with a WIP test refactor I am doing for making room to onboard EnvoyGateway. Basically I am creating "istio tests", so we can later add "envoy gateway tests".

@eguzki Just seeing how easy / difficult it is to do this / what issues are there and to see if it's worth it generally (can be seen as part of https://github.com/Kuadrant/kuadrant-operator/issues/564). Not particularly looking to merge this soon if it's a concern

KevFan avatar May 22 '24 13:05 KevFan

I thought we had already did this parallel test thing, but it was on the limitador-operator https://github.com/Kuadrant/limitador-operator/pull/134.

There is a big conflict with https://github.com/Kuadrant/kuadrant-operator/pull/645 but we will workout as needed. :+1:

eguzki avatar May 23 '24 18:05 eguzki

Closing this and opening up another PR since it's a big rebase

KevFan avatar Jun 05 '24 08:06 KevFan