Add K8s Gateway API resources#177
Open
franksc wants to merge 12 commits intoAustralian-Imaging-Service:rcfrom
Open
Add K8s Gateway API resources#177franksc wants to merge 12 commits intoAustralian-Imaging-Service:rcfrom
franksc wants to merge 12 commits intoAustralian-Imaging-Service:rcfrom
Conversation
Contributor
|
Contributor
|
we also had to add httproute timeouts to allow large file downloads. could this be allowed with optional values ..? |
specified Create a no-hostname listener and HTTPRoute where no gateway.hosts specified
- .gateway.tls -> .gateway.https - Add .gateway.http, move .gateway.hosts under this hierarchy (aligns with .gateway.https) - .hosts entries can specify listenerName to create in Gateway, reference in HTTPRoute - .gateway.redirectHttpToHttps true/false (default false) - gateway.createNoHostnameListenerAndRoute true/false (default false) to optionally create no-hostname Gateway listener and HTTPRoute
listenersAllowedRoutesFromNamespaces (default 'Same')
Author
|
I have implemented the above suggestions with this batch of commits. Using an existing Gateway resource not managed by this chart should be accommodated with something like: TBD: probably split HTTPRoute stuff into a separate template file, flatten some repeated nested references |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.