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.

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

Bump dependencies #15

Merged
merged 1 commit into from
Dec 13, 2024
Merged

Conversation

tjamet
Copy link
Contributor

@tjamet tjamet commented Dec 13, 2024

Committer details Local-Branch: HEAD
Related changes
Future changes
refactor ingress backend pod detection (#11) Move the detection at the ingress level rather than per rule

Goals

  • Prepare the case to support multiple ingress controllers through CRDs
  • Increase code readability and testeability
Prevent changing dnsEndpoint name and namespace attributes in createOrUpdate (#12) This is not permitted by controllerRuntime and should not be performed
Refactor DNS name extraction (#13) Goal ---

Ease the implementation of CRD based multi-ingress controller traffic routing

Add integration test (#16) Goal ---

Ensure that the chart provides enough privileges for the controller to work

Add fine-grain ingress DNS control through CRD (#14) Context ---

Handling usual cluster operations, we often come to operate higher risk changes like bumping ingress controller
versions, changing the underneath ingress service type (for example moving from an AWS ELB to an AWS NLB).

Doing so, the safest way would be to be able to provision a new ingress controller and progressively migrating traffic
to the new instance.

Problems

Currently, traffic controller reads the host and load balancer reference using
the ingress status.

This prevents from being able to handle and control weighted records across the different ingress controllers.

Goal

Enable fine-grained routing between various ingress controllers in the same cluster.

Unblocked use-cases

  • Progressively change and test the ingress infrastructure (load balancer, ...) and versions
  • Allow sharding ingress controllers at the DNS level

Change-Id: I9146086e1ed524e21ae073616ccbc73335769e68
@tjamet tjamet merged commit 559c285 into main Dec 13, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants