Skip to content
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

Traefik generator support #159

Merged
merged 10 commits into from
Sep 15, 2021
Merged

Traefik generator support #159

merged 10 commits into from
Sep 15, 2021

Conversation

Tarick
Copy link
Contributor

@Tarick Tarick commented Sep 9, 2021 •
edited by kylehodgetts
Loading

This PR adds an initial support for Traefik IngressRoute generation

Changes

  • generation of Traefik IngressRoute and Middleware using CRD objects and marshalling to Yaml
  • added related tests
  • wizard detection of intsalled Traefik based on available CRD API in the cluster

Fixes

resolves #134

Checklist

  • tested locally
  • added new dependencies
  • updated the docs
  • added a test

Kyle Hodgetts and others added 6 commits September 2, 2021 14:26
Go fmt
747a3a9
Add boilerplate code for Traefik generator
cd7c974
Add cluster detection for traefik v2. Assumes it lives in traefik v2 …
cedb22e 
…namespace
basic path prefix and trim implementation
b54c9c2
@Tarick
Merge branch 'kylehodgetts/feature/134-traefik-support' into tyatsura…
a9e0fe2 
…k/feature/134-traefic-support
@Tarick
Add initial Traefik v2 support
a8eb473 
 * generation of Traefik IngressRoute and Middleware using CRD objects
   and marshalling to Yaml
 * added related tests
 * wizard detection of intsalled Traefik based on available CRD API in
   the cluster
@Tarick Tarick marked this pull request as draft September 9, 2021 07:25
@Tarick Tarick added this to In progress in Generators via automation Sep 9, 2021
@Tarick Tarick linked an issue Sep 9, 2021 that may be closed by this pull request
Traefik Proxy support #134
Closed
4 tasks
@Tarick Tarick self-assigned this Sep 9, 2021
@Tarick Tarick marked this pull request as ready for review September 14, 2021 15:06
kylehodgetts
kylehodgetts suggested changes Sep 15, 2021
View reviewed changes
Copy link
Contributor

@kylehodgetts kylehodgetts left a comment

Choose a reason for hiding this comment

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

Good initial PR!

cluster/cluster_test.go Show resolved Hide resolved
examples/petstore/README.md Show resolved Hide resolved
wizard/wizard.go Outdated Show resolved Hide resolved
@Tarick Tarick merged commit ffc9022 into main Sep 15, 2021
Generators automation moved this from In progress to Done Sep 15, 2021
@Tarick Tarick deleted the tyatsurak/feature/134-traefic-support branch September 15, 2021 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kylehodgetts kylehodgetts kylehodgetts approved these changes

@dobegor dobegor dobegor approved these changes

Assignees

@Tarick Tarick

Labels
area/generators kind/feature
Projects
Generators
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Traefik Proxy support
3 participants
@Tarick @kylehodgetts @dobegor