Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

tests: fix flaky HTTPRoute creation #13

Merged
merged 3 commits into from
Mar 15, 2024
Merged

tests: fix flaky HTTPRoute creation #13

merged 3 commits into from
Mar 15, 2024

Conversation

pmalek
Copy link
Member

@pmalek pmalek commented Mar 15, 2024

Similar to https://github.com/Kong/gateway-operator-archive/pull/1620 but for Ingress.

Occurred in https://github.com/Kong/gateway-operator/actions/runs/8295838796/job/22703706632#step:4:1140

    test_httproute.go:161: 
        	Error Trace:	/home/runner/work/gateway-operator/gateway-operator/test/integration/test_httproute.go:161
        	Error:      	Received unexpected error:
        	            	admission webhook "httproutes.validation.ingress-controller.konghq.com" denied the request: Unable to validate HTTPRoute schema: making HTTP request: Post "https://10-244-106-29.dataplane-admin-c2f97339-f437-4b4e-b535-ae248e8aee21-p2rctgkbsd.58d2076e-cb13-4c3a-9115-9e07a705c564.svc:8444/schemas/routes/validate": dial tcp 10.244.106.29:8444: connect: connection refused

@pmalek pmalek self-assigned this Mar 15, 2024
@pmalek pmalek marked this pull request as ready for review March 15, 2024 13:15
@pmalek pmalek requested a review from a team March 15, 2024 13:15
@pmalek pmalek enabled auto-merge (squash) March 15, 2024 13:23
@pmalek pmalek merged commit 2ebefd2 into main Mar 15, 2024
13 checks passed
@pmalek pmalek deleted the fix-flaky-httproute-creation branch March 15, 2024 13:31
This was referenced Mar 15, 2024
Merged
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@programmer04 programmer04 programmer04 approved these changes

Assignees

@pmalek pmalek

Labels
area/tests test-flake
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pmalek @programmer04