Skip to content

Commit 6e8ccdf

Browse files
authored
Merge branch 'main' into feat/gunzip-virtualserver
2 parents 94cb841 + 85a353a commit 6e8ccdf

File tree

3 files changed

+79
-50
lines changed

3 files changed

+79
-50
lines changed

tests/suite/test_ingress_mtls.py

Lines changed: 74 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -145,13 +145,21 @@ def test_ingress_mtls_policy(
145145
virtual_server_setup.namespace,
146146
)
147147
wait_before_test()
148-
resp = session.get(
149-
virtual_server_setup.backend_1_url_ssl,
150-
cert=(crt, key),
151-
headers={"host": virtual_server_setup.vs_host},
152-
allow_redirects=False,
153-
verify=False,
154-
)
148+
resp = mock.Mock()
149+
resp.status_code == 502
150+
counter = 0
151+
152+
while resp.status_code != expected_code and counter < 10:
153+
resp = session.get(
154+
virtual_server_setup.backend_1_url_ssl,
155+
cert=(crt, key),
156+
headers={"host": virtual_server_setup.vs_host},
157+
allow_redirects=False,
158+
verify=False,
159+
)
160+
wait_before_test()
161+
counter += 1
162+
155163
vs_res = read_vs(kube_apis.custom_objects, test_namespace, virtual_server_setup.vs_name)
156164
teardown_policy(kube_apis, test_namespace, tls_secret, pol_name, mtls_secret)
157165

@@ -208,21 +216,28 @@ def test_ingress_mtls_policy_cert(
208216
)
209217
wait_before_test()
210218
ssl_exception = ""
211-
resp = ""
212-
try:
213-
resp = session.get(
214-
virtual_server_setup.backend_1_url_ssl,
215-
cert=certificate,
216-
headers={"host": virtual_server_setup.vs_host},
217-
allow_redirects=False,
218-
verify=False,
219-
)
220-
except requests.exceptions.SSLError as e:
221-
print(f"SSL certificate exception: {e}")
222-
ssl_exception = str(e)
223-
resp = mock.Mock()
224-
resp.status_code = "None"
225-
resp.text = "None"
219+
resp = mock.Mock()
220+
resp.status_code == 502
221+
counter = 0
222+
223+
while resp.status_code != expected_code and counter < 10:
224+
try:
225+
resp = session.get(
226+
virtual_server_setup.backend_1_url_ssl,
227+
cert=certificate,
228+
headers={"host": virtual_server_setup.vs_host},
229+
allow_redirects=False,
230+
verify=False,
231+
)
232+
wait_before_test()
233+
counter += 1
234+
235+
except requests.exceptions.SSLError as e:
236+
print(f"SSL certificate exception: {e}")
237+
ssl_exception = str(e)
238+
resp = mock.Mock()
239+
resp.status_code = "None"
240+
resp.text = "None"
226241

227242
teardown_policy(kube_apis, test_namespace, tls_secret, pol_name, mtls_secret)
228243

@@ -267,7 +282,7 @@ def test_ingress_mtls_policy_cert(
267282
),
268283
],
269284
)
270-
def test_ingress_mtls_polciy_crl(
285+
def test_ingress_mtls_policy_crl(
271286
self,
272287
kube_apis,
273288
crd_ingress_controller,
@@ -297,13 +312,21 @@ def test_ingress_mtls_polciy_crl(
297312
virtual_server_setup.namespace,
298313
)
299314
wait_before_test()
300-
resp = session.get(
301-
virtual_server_setup.backend_1_url_ssl,
302-
cert=(crt_not_revoked, key_not_revoked),
303-
headers={"host": virtual_server_setup.vs_host},
304-
allow_redirects=False,
305-
verify=False,
306-
)
315+
resp = mock.Mock()
316+
resp.status_code == 502
317+
counter = 0
318+
319+
while resp.status_code != expected_code and counter < 10:
320+
resp = session.get(
321+
virtual_server_setup.backend_1_url_ssl,
322+
cert=(crt_not_revoked, key_not_revoked),
323+
headers={"host": virtual_server_setup.vs_host},
324+
allow_redirects=False,
325+
verify=False,
326+
)
327+
wait_before_test()
328+
counter += 1
329+
307330
vs_res = read_vs(kube_apis.custom_objects, test_namespace, virtual_server_setup.vs_name)
308331
teardown_policy(kube_apis, test_namespace, tls_secret, pol_name, mtls_secret)
309332

@@ -360,21 +383,27 @@ def test_ingress_mtls_policy_cert_crl(
360383
)
361384
wait_before_test()
362385
ssl_exception = ""
363-
resp = ""
364-
try:
365-
resp = session.get(
366-
virtual_server_setup.backend_1_url_ssl,
367-
cert=certificate,
368-
headers={"host": virtual_server_setup.vs_host},
369-
allow_redirects=False,
370-
verify=False,
371-
)
372-
except requests.exceptions.SSLError as e:
373-
print(f"SSL certificate exception: {e}")
374-
ssl_exception = str(e)
375-
resp = mock.Mock()
376-
resp.status_code = "None"
377-
resp.text = "None"
386+
resp = mock.Mock()
387+
resp.status_code == 502
388+
counter = 0
389+
390+
while resp.status_code != expected_code and counter < 10:
391+
try:
392+
resp = session.get(
393+
virtual_server_setup.backend_1_url_ssl,
394+
cert=certificate,
395+
headers={"host": virtual_server_setup.vs_host},
396+
allow_redirects=False,
397+
verify=False,
398+
)
399+
wait_before_test()
400+
counter += 1
401+
except requests.exceptions.SSLError as e:
402+
print(f"SSL certificate exception: {e}")
403+
ssl_exception = str(e)
404+
resp = mock.Mock()
405+
resp.status_code = "None"
406+
resp.text = "None"
378407

379408
teardown_policy(kube_apis, test_namespace, tls_secret, pol_name, mtls_secret)
380409

tests/suite/test_jwt_policies_jwksuri.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ def test_jwt_policy_jwksuri(
111111
resp_no_token.status_code == 502
112112
counter = 0
113113

114-
while resp_no_token.status_code != 401 and counter < 10:
114+
while resp_no_token.status_code != 401 and counter < 20:
115115
resp_no_token = requests.get(
116116
virtual_server_setup.backend_1_url,
117117
headers={"host": virtual_server_setup.vs_host},
@@ -230,7 +230,7 @@ def test_jwt_policy_subroute_jwksuri(
230230
resp_no_token.status_code == 502
231231
counter = 0
232232

233-
while resp_no_token.status_code != 401 and counter < 10:
233+
while resp_no_token.status_code != 401 and counter < 20:
234234
resp_no_token = requests.get(
235235
virtual_server_setup.backend_1_url + "/subpath1",
236236
headers={"host": virtual_server_setup.vs_host},
@@ -304,7 +304,7 @@ def test_jwt_policy_subroute_jwksuri_multiple_vs(
304304
resp_2_no_token.status_code == 502
305305
counter = 0
306306

307-
while resp_1_no_token.status_code != 401 and counter < 10:
307+
while resp_1_no_token.status_code != 401 and counter < 20:
308308
resp_1_no_token = requests.get(
309309
virtual_server_setup.backend_1_url + "/subpath1",
310310
headers={"host": virtual_server_setup.vs_host},
@@ -314,7 +314,7 @@ def test_jwt_policy_subroute_jwksuri_multiple_vs(
314314

315315
counter = 0
316316

317-
while resp_2_no_token.status_code != 401 and counter < 10:
317+
while resp_2_no_token.status_code != 401 and counter < 20:
318318
resp_2_no_token = requests.get(
319319
virtual_server_setup.backend_1_url + "/subpath1",
320320
headers={"host": "virtual-server-2.example.com"},

tests/suite/test_jwt_policies_jwksuri_vsr.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ def test_jwt_policy_jwksuri(
101101
resp_no_token.status_code == 502
102102
counter = 0
103103

104-
while resp_no_token.status_code != 401 and counter < 10:
104+
while resp_no_token.status_code != 401 and counter < 20:
105105
resp_no_token = requests.get(
106106
f"{req_url}{v_s_route_setup.route_m.paths[0]}",
107107
headers={"host": v_s_route_setup.vs_host},

0 commit comments

Comments
 (0)