Add Event Loop #56

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

Merged

pleshakov merged 7 commits into main from add-controller

Feb 10, 2022

Contributor

pleshakov commented Feb 4, 2022

Add Main Controller -- the main (apart from the Manager) control loop
of the Gateway
Old controller.go was renamed appropriately to manager.go
Add HTTPRoute Controller and Implementation
Add example HTTPRoute YAMLs

Implements https://github.com/nginxinc/nginx-gateway-kubernetes/issues/52


          Add Main Controller

c77383d

- Add Main Controller -- the main (apart from the Manager) control loop
of the Gateway
- Old controller.go was renamed appropriately to manager.go
- Add HTTPRoute Controller and Implementation
- Add example HTTPRoute YAMLs

pleshakov requested a review from f5yacobucci

February 4, 2022 22:13

pleshakov commented

View reviewed changes

internal/controller/controller.go Outdated Show resolved Hide resolved


          Fix linting errors

5037c04

f5yacobucci reviewed

View reviewed changes

internal/controller/controller.go Outdated Show resolved Hide resolved

f5yacobucci reviewed

View reviewed changes

internal/controller/controller.go Outdated Show resolved Hide resolved

f5yacobucci reviewed

View reviewed changes

internal/controller/controller.go Outdated Show resolved Hide resolved

f5yacobucci reviewed

View reviewed changes

internal/controller/controller.go Outdated Show resolved Hide resolved

f5yacobucci reviewed

View reviewed changes

internal/implementations/httproute/httproute.go Show resolved Hide resolved

f5yacobucci reviewed

View reviewed changes

internal/implementations/httproute/httproute.go Show resolved Hide resolved

f5yacobucci reviewed

View reviewed changes

internal/state/configuration.go Show resolved Hide resolved

f5yacobucci reviewed

View reviewed changes

internal/state/fake_configuration.go Outdated Show resolved Hide resolved

f5yacobucci reviewed

View reviewed changes

pkg/sdk/interfaces.go Show resolved Hide resolved

pleshakov added 5 commits

February 10, 2022 10:46


          Use auto-generated fake for Configuration


          Rename Controller

627753e


          Add logging

fef69af


          Add TO-DOs

3f27362


          Add one more TO-DO

529bb1e

f5yacobucci approved these changes

View reviewed changes

pleshakov merged commit 889cfad into main

pleshakov deleted the add-controller branch

February 10, 2022 23:14

pleshakov changed the title ~~Add Main Controller~~ Add Event Loop

pammecrandall added the enhancement label

miledxz added a commit to miledxz/nginx-gateway-fabric that referenced this pull request


          Add Event Loop (nginx#56)

c81cd8e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels