Allow specifying default site / vhost
Bug #1863170 reported by
Haw Loeung
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Content Cache Charm |
Fix Released
|
Wishlist
|
Haw Loeung | ||
Bug Description
Hi,
In some cases, we have multiple sites configured, with HAProxy generated config as:
| listen combined-443
| bind 0.0.0.0:443 ssl crt /var/lib/haproxy
| bind :::443 ssl crt /var/lib/haproxy
| use_backend backend-
| use_backend backend-
We should have an option to specify a default. Or perhaps make the first or last in the defined YAML the default. Or both.
Related branches
~hloeung/content-cache-charm:master
- Stuart Bishop (community): Approve
- Content Cache Charmers: Pending requested
-
Diff: 286 lines (+84/-7)13 files modifiedlib/haproxy.py (+29/-5)
reactive/content_cache.py (+4/-0)
tests/unit/files/config_test_config.txt (+5/-0)
tests/unit/files/content_cache_rendered_haproxy_test_output.txt (+1/-0)
tests/unit/files/content_cache_rendered_haproxy_test_output3.txt (+1/-1)
tests/unit/files/content_cache_rendered_haproxy_test_output_auto_maxconns.txt (+1/-0)
tests/unit/files/content_cache_rendered_haproxy_test_output_processes_and_threads.txt (+1/-0)
tests/unit/files/haproxy_config_rendered_listen_stanzas_test_output.txt (+1/-0)
tests/unit/files/haproxy_config_rendered_listen_stanzas_test_output2.txt (+1/-1)
tests/unit/files/haproxy_config_rendered_listen_stanzas_test_output3.txt (+14/-0)
tests/unit/files/haproxy_config_rendered_test_output.txt (+1/-0)
tests/unit/files/haproxy_config_rendered_test_output2.txt (+1/-0)
tests/unit/test_haproxy.py (+24/-0)
| Changed in content-cache-charm: | |
| importance: | Undecided → Wishlist |
| Changed in content-cache-charm: | |
| status: | New → Triaged |
Revision history for this message
| Haw Loeung (hloeung) wrote | #1 |
| Changed in content-cache-charm: | |
| assignee: | nobody → Haw Loeung (hloeung) |
| status: | Triaged → In Progress |
| Changed in content-cache-charm: | |
| status: | In Progress → Fix Committed |
| Changed in content-cache-charm: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
We also want to have a default for sites not yet set up or people hitting the servers by IP. For this, we should have a flag 'default' or similar instead.