Skip to content

Enable LB health-monitor to fulfill ADR from Docs/#227 #312

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
Closed
garloff opened this issue Dec 5, 2022 · 3 comments
Closed

Enable LB health-monitor to fulfill ADR from Docs/#227 #312

garloff opened this issue Dec 5, 2022 · 3 comments
Assignees
@garloff @batistein
Labels
bug Something isn't working Container Issues or pull requests relevant for Team 2: Container Infra and Tooling SCS is standardized SCS is standardized standards Issues / ADR / pull requests relevant for standardization & certification
Milestone
R4 (v5.0.0)

Comments

@garloff
Copy link
Member

garloff commented Dec 5, 2022 •
edited
Loading

As KaaS Operator using the SCS Ref.Impl. I want to be sure it fulfills the LB requirements tested for in issues#212 and documented in the ADR tracked in SovereignCloudStack/issues#227.
This has been discussed already here: #174, in particular with commit 002cecd
@garloff garloff self-assigned this Dec 5, 2022
@garloff garloff added bug Something isn't working Container Issues or pull requests relevant for Team 2: Container Infra and Tooling standards Issues / ADR / pull requests relevant for standardization & certification SCS is standardized SCS is standardized labels Dec 5, 2022
@garloff garloff moved this to Backlog in Sovereign Cloud Stack Dec 5, 2022
@garloff garloff moved this from Backlog to Doing in Sovereign Cloud Stack Dec 5, 2022
@garloff
Copy link
Member Author

garloff commented Dec 7, 2022

See also SovereignCloudStack/issues#227 (comment)

@garloff garloff moved this from Doing to Blocked / On hold in Sovereign Cloud Stack Jan 16, 2023
@garloff
Copy link
Member Author

garloff commented Mar 5, 2023

PR #351 aligns our implementation with the decisions:

  • Use proxy proto for nginx, fixing it for internal access (with hostname setting)
  • Do not enable health-monitor by default (as it only addresses half of the issues with traficPolicyExternal: local)
  • Document this in the doc directory

@garloff
Copy link
Member Author

garloff commented Mar 12, 2023

Done with the merging of #351.

@garloff garloff closed this as completed Mar 12, 2023
@github-project-automation github-project-automation bot moved this from Blocked / On hold to Done in Sovereign Cloud Stack Mar 12, 2023
@garloff garloff added this to the R4 (v5.0.0) milestone Mar 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@garloff garloff

@batistein batistein

Labels
bug Something isn't working Container Issues or pull requests relevant for Team 2: Container Infra and Tooling SCS is standardized SCS is standardized standards Issues / ADR / pull requests relevant for standardization & certification
Projects
Sovereign Cloud Stack
Archived in project
Milestone
R4 (v5.0.0)
Development

No branches or pull requests
2 participants
@garloff @batistein