Skip to content

Consolidate wa backend servers and merge waHttp into wa#532

Merged
bsuh merged 1 commit into
mainfrom
pr532
Jul 10, 2026
Merged

Consolidate wa backend servers and merge waHttp into wa#532
bsuh merged 1 commit into
mainfrom
pr532

Conversation

@bsuh

@bsuh bsuh commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Summary:
Merge the separate wa and wa_http HAProxy backends into a single wa
backend that lists multiple servers (5222, 80, and a new 443). The
haproxyV4Http frontend now points its defaultBackend at wa, and the
backend template iterates over a servers list instead of hardcoding a
single server per backend.

Test Plan: Rendered the chart and verified the generated HAProxy config lists all three wa servers under the wa backend and that the wa_http backend is removed.

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 10, 2026
@bsuh bsuh requested a review from jaltenmueller July 10, 2026 22:09
Summary:
Merge the separate wa and wa_http HAProxy backends into a single wa
backend that lists multiple servers (5222, 80, and a new 443). The
haproxyV4Http frontend now points its defaultBackend at wa, and the
backend template iterates over a servers list instead of hardcoding a
single server per backend.

Test Plan: Rendered the chart and verified the generated HAProxy config lists all three wa servers under the wa backend and that the wa_http backend is removed.
@bsuh bsuh merged commit 913c81f into main Jul 10, 2026
5 checks passed
@bsuh bsuh deleted the pr532 branch July 10, 2026 22:53
@jackelinejohanaorantes-cell

Copy link
Copy Markdown

Ya volvi

@orantesjackeline2003-crypto

Copy link
Copy Markdown

@orantesjackeline2003-crypto

Copy link
Copy Markdown

🫂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants