init httproute controller #345
Merged
Conversation
This file contains 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
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @Abdiramen and the rest of your teammates on |
bcb13c3 to
1b270a3
Compare
f3a953e to
d3d394d
Compare
1b270a3 to
a066e2c
Compare
d3d394d to
810522a
Compare
97baeb3 to
f840f27
Compare
c8ac507 to
dfbc8c4
Compare
Abdiramen
added a commit
that referenced
this pull request
Mar 14, 2024
* add experimetnal gateway flag * update store * sperate experimental gateway deploy from normal deploy process * split http edge for gateway * update helm snapshot tests * lint * fix build issue * remove comments * use flag in driver seed/sync * move util functions * clean-up controller * add DeleteNamedGateway function * set domainNaim var * block httproutes from seeding * get domainmaps * init httproute controller (#345) * init httproute controller * only start HTTPRouteReconciler if gateway flag is set * setup reconciler struct * update reconciler markers * run httproute edge creation * Gateway handles traffic * remove debug log * map HTTProute to HTTPSEdge * implement HTTPRouteRules (#347) * init httproute controller * only start HTTPRouteReconciler if gateway flag is set * setup reconciler struct * update reconciler markers * run httproute edge creation * Gateway handles traffic * remove debug log * map HTTProute to HTTPSEdge * add HTTPRouteMatch * convert filters to traffic policy actions * fix RequestRedirect filter * fix request redirect filter * fix domain issue * switch request redirect to redirect action * remove log * emmit error if regex path match is created * remove unsupported match types * add AllowedRoutes check * rename utils/ to controllers/ * update helm snapshots * remove spaces * check if listener has hostname field * move functions with their structs * don't finalize ingored gateway * boolean logic fix * remove unsed function param * rename module set func to endpoint policy * clean up some comments * fix spelling/copy-paste errors * fix more boolean errors * Revert "remove unsed function param" This reverts commit 02b6795. * remove hostnames not in gateway domain map * turn gateway domains to a map * skip non HTTPS listeners * pull out boolean inside of getDriver * fix copy-paste error * don't build edge if traffic unsupported filters are set * handle error case for all filters * merge edgemaps * don't overwrite gateway domain map * cleanup comments * use loop instead of func * throw more errors * fix lint * use guard statements * rename to hasReference * inline error check * encapsulate double pointer to struct
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.
What
Add the http route controller which corresponds to an ngrok https edge
How
Breaking Changes
nope