kubernetes-sigs / gateway-api

Repository for the next iteration of composite service (e.g. Ingress) and load balancing APIs.
https://gateway-api.sigs.k8s.io
Apache License 2.0
1.86k stars 482 forks source link

conformance: spammy logs from TLS test failures #1903

Closed howardjohn closed 1 year ago

howardjohn commented 1 year ago

What happened:

Response expectation failed for request: {URL:{Scheme:https Opaque: User: Host:172.18.0.106:443 Path:/https-listener-on-443-and-redirect-port-not-set RawPath: OmitHost:false ForceQuery:false RawQuery: Fragment: RawFragment:} Host:example Protocol:HTTPS Method:GET Headers:map[X-Echo-Set-Header:[]] UnfollowRedirect:true CertPem:[45 45 45 45 45 66 69 71 73 78 32 67 69 82 84 73 70 73 67 65 84 69 45 45 45 45 45 10 77 73 73 68 70 106 67 67 65 102 54 103 65 119 73 66 65 103 73 82 65 75 49 55 85 65 101 104 50 65 110 65 50 69 113 73 71 122 56 78 67 86 115 119 68 81 89 74 75 111 90 73 104 118 99 78 65 81 69 76 66 81 65 119 10 74 68 69 81 77 65 52 71 65 49 85 69 67 104 77 72 81 87 78 116 90 83 66 68 98 122 69 81 77 65 52 71 65 49 85 69 65 120 77 72 90 71 86 109 89 88 86 115 100 68 65...

(log truncated)

What you expected to happen:

Log is not 100s of lines long for a failure. We should hide the cert in some way probably

How to reproduce it (as minimally and precisely as possible): Fail on a TLS test Anything else we need to know?:

shaneutt commented 1 year ago

/help

k8s-ci-robot commented 1 year ago

@shaneutt: This request has been marked as needing help from a contributor.

Guidelines

Please ensure that the issue body includes answers to the following questions:

For more details on the requirements of such an issue, please see here and ensure that they are met.

If this request no longer meets these requirements, the label can be removed by commenting with the /remove-help command.

In response to [this](https://github.com/kubernetes-sigs/gateway-api/issues/1903): >/help Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
gauravkghildiyal commented 1 year ago

/assign