Skip to content
This repository has been archived by the owner on Feb 27, 2023. It is now read-only.

v1.13.1 cherrypicks #229

Merged
merged 1 commit into from
Mar 1, 2021

Conversation

skriss
Copy link
Member

@skriss skriss commented Mar 1, 2021

HTTPSRoute does not exist as per gateway API
docs
.
For HTTPS protocol, HTTPRoute + spec.tls needs to be used.

Signed-off-by: Kenjiro Nakayama nakayamakenjiro@gmail.com

HTTPSRoute does not exist as per [gateway API
docs](https://gateway-api.sigs.k8s.io/spec/#networking.x-k8s.io/v1alpha1.HTTPRoute).
For HTTPS protocol, `HTTPRoute` + `spec.tls` needs to be used.

Signed-off-by: Kenjiro Nakayama <nakayamakenjiro@gmail.com>
@skriss skriss requested a review from a team as a code owner March 1, 2021 19:07
@skriss
Copy link
Member Author

skriss commented Mar 1, 2021

#230

Copy link
Member

@stevesloka stevesloka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't really a "breaking" change since that field isn't parsed by Contour yet so I don't know that it's super important to cherrypick.

Copy link
Member

@stevesloka stevesloka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem to merge in though to keep things proper. =)

@skriss
Copy link
Member Author

skriss commented Mar 1, 2021

Eh, guess I'll merge it since I have to do a release anyway.

@skriss skriss merged commit 8caffa7 into projectcontour:release-1.13 Mar 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants