Kuadrant / testsuite

4 stars 14 forks source link

Add geo load-balancing tests #516

Closed averevki closed 1 month ago

averevki commented 2 months ago

Changes

Tests

Closes #496, closes #515

trepel commented 2 months ago

@averevki Could you please resolve the conflicts? It would make reviewing easier and you need to that anyway

trepel commented 2 months ago

I don't understand the reasoning behind doing this refactoring to Change the list of additional clusters to only 2. Why was it required?

trepel commented 2 months ago

I am seeing The DNS operation timed out. error while running resolver.resolve(hostname.hostname). Did you come across that?

averevki commented 2 months ago

I don't understand the reasoning behind doing this refactoring to Change the list of additional clusters to only 2. Why was it required?

We don't need this functionality for now. Right now, it's committing the same object on each cluster, and I need more flexibility in CR configuration on each cluster

averevki commented 2 months ago

I am seeing The DNS operation timed out. error while running resolver.resolve(hostname.hostname). Did you come across that?

You may be using wrong DNS servers. Or you are running under the internal network, which blocks non-internal dns

trepel commented 2 months ago

Only thing that confuses me a little is that if I tried to resolve the hostname in https://dnschecker.org/ it took a few minutes till at least one AU server started resolving to "AU" Gateway external IP.