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

Charts[Network]: update haproxy to use startup probe and not expose and not expose service port 5555 #852

Closed
1 task
Tracked by #111
jeromy-cannon opened this issue May 20, 2024 · 1 comment · Fixed by #856
Closed
1 task
Tracked by #111
Assignees
Labels
Bug A error that causes the feature to behave differently than what was expected based on design docs P1 High priority issue. Required to be completed in the assigned milestone. released

Comments

@jeromy-cannon
Copy link
Contributor

jeromy-cannon commented May 20, 2024

Implement a startup probe to update the pods PodCondition[Ready] to true/false based on the result of hitting its own Data Plan API: http://localhost:5555/v2/services/haproxy/stats/native?type=backend.
It will need a basic authorization header: "Authorization": "Basic base64(user:pass)"

NOTE: example logic will need to be converted to work in Helm chart: https://github.com/hashgraph/solo/blob/22553743934b50b626423aed6b0d4af6912fe913/src/commands/node.mjs#L1784

Helpful links:

@github-project-automation github-project-automation bot moved this to 🆕 New in FST Suite May 20, 2024
@jeromy-cannon jeromy-cannon self-assigned this May 20, 2024
@jeromy-cannon jeromy-cannon added Bug A error that causes the feature to behave differently than what was expected based on design docs P1 High priority issue. Required to be completed in the assigned milestone. labels May 20, 2024
@jeromy-cannon jeromy-cannon changed the title Charts[Network]: update haproxy to use liveness/readiness probe and not expose and not expose service port 5555 Charts[Network]: update haproxy to use startup probe and not expose and not expose service port 5555 May 22, 2024
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in FST Suite May 23, 2024
@swirlds-automation
Copy link
Contributor

🎉 This issue has been resolved in version 0.26.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A error that causes the feature to behave differently than what was expected based on design docs P1 High priority issue. Required to be completed in the assigned milestone. released
Projects
Status: ✅ Done
2 participants