Add integration test #16
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
This was referenced Dec 13, 2024
Merged
tjamet
force-pushed
the
maiao.Ic213a480fb2bf026cbda0d8879375d459eb90724
branch
from
4a0c20b to
eebfa86
Compare
tjamet
force-pushed
the
maiao.I46f513341ce01ec6e9d67897c6d0cb58aeec3021
branch
from
4b17c88 to
e23b773
Compare
zonzamas
approved these changes
Dec 13, 2024
tjamet
force-pushed
the
maiao.Ic213a480fb2bf026cbda0d8879375d459eb90724
branch
from
eebfa86 to
a05a181
Compare
tjamet
force-pushed
the
maiao.I46f513341ce01ec6e9d67897c6d0cb58aeec3021
branch
2 times, most recently
from
023e6f8 to
6f3c338
Compare
tjamet
force-pushed
the
maiao.Ic213a480fb2bf026cbda0d8879375d459eb90724
branch
from
a05a181 to
3606613
Compare
Base automatically changed from
maiao.I46f513341ce01ec6e9d67897c6d0cb58aeec3021
to
main
December 16, 2024 09:39
added 2 commits
December 16, 2024 10:41
Goal --- Ensure that the chart provides enough privileges for the controller to work Change-Id: Ic213a480fb2bf026cbda0d8879375d459eb90724
tjamet
force-pushed
the
maiao.Ic213a480fb2bf026cbda0d8879375d459eb90724
branch
from
e7eb4bf to
9ed742d
Compare
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.
Goal
Ensure that the chart provides enough privileges for the controller to work
Committer details
Local-Branch: HEADRelated changes
Future changes
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