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

Handle maintenance mode for yggdrasil #7

Merged
merged 38 commits into from
Jul 29, 2024
Merged

Conversation

SoulKyu
Copy link

@SoulKyu SoulKyu commented Jun 11, 2024

Add a new maintenance mode :

This mode should be configured on the cluster configuration.
When a cluster is in maintenance mode :

  • Add a weight of 0 on all endpoint that are on more than 1 cluster (or just don't add it)
  • Don't touch any endpoint that are only on the cluster in maintenance mode
  • Yggdrasil wont start if all cluster are in a maintenance mode
  • Yggdrasil should send WARNING when their 1 or more cluster in maintenance
  • Add a metrics that permit to know cluster in maintenance mode
  • Add a metrics that provide informations on which ingress belong to which endpoint

Aluxima and others added 30 commits June 20, 2022 18:21
feat(weight-annotation): Annotation to add load-balancing weight
The new annotations got the priority, the simple 'timeout' annotation still sets the 3 timeouts but if there is, at least one of the new ones, it's overwriting the values set by the simple annotation.
Applying @Aluxima suggestion

Co-authored-by: Laurent Marchaud <[email protected]>
feat(split-time): split timeout annotations into 3 different ones, make defaults configurable
Signed-off-by: Laurent Marchaud <[email protected]>
Signed-off-by: Laurent Marchaud <[email protected]>
…esses

Adds multiple envoy listener ipv4 adresses
Add healthcheck-host annotation, deduplicate redundant cluster ingresses
@SoulKyu SoulKyu self-assigned this Jun 11, 2024
@SoulKyu
Copy link
Author

SoulKyu commented Jun 11, 2024

@Solvik @Aluxima

cmd/root.go Outdated Show resolved Hide resolved
…to cross informations between ingress and envoy
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
pkg/envoy/ingress_translator.go Show resolved Hide resolved
pkg/envoy/ingress_translator.go Outdated Show resolved Hide resolved
pkg/envoy/metrics.go Outdated Show resolved Hide resolved
SoulKyu and others added 5 commits June 11, 2024 16:38
Co-authored-by: Laurent Marchaud <[email protected]>
Co-authored-by: Laurent Marchaud <[email protected]>
Co-authored-by: Laurent Marchaud <[email protected]>
Co-authored-by: Laurent Marchaud <[email protected]>
Copy link
Author

@SoulKyu SoulKyu left a comment

Choose a reason for hiding this comment

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

Ty

@SoulKyu SoulKyu force-pushed the handle-maintenance-mode branch 7 times, most recently from 3620354 to c912ebc Compare July 29, 2024 09:16
@SoulKyu SoulKyu merged commit 65aa824 into staging Jul 29, 2024
15 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.

4 participants