| opendevreview | James Denton proposed openstack/openstack-ansible master: feat: Add skyline deployment capability https://review.opendev.org/c/openstack/openstack-ansible/+/859446 | 11:43 |
|---|---|---|
| admin1 | finally .. skyline :) | 11:55 |
| hamburgler | admin1: also :) | 22:23 |
| hamburgler | damiandabrowski: re: skyline deployment capability - Don't see James Denton online but: https://review.opendev.org/c/openstack/openstack-ansible/+/859446/3/inventory/group_vars/skyline_all/haproxy_service.yml#24 haproxy_backend_port doesn't look to have default(openstack_service_backend_ssl) - I realize this may be too early also for it to be even considered as an option, but maybe note worthy to help keep in | 22:25 |
| hamburgler | line with everything else | 22:25 |
| hamburgler | jrosser: tagging you for the above too since you are a co-author :) | 22:28 |
| damiandabrowski | hamburgler: hi! you're right, technically we can use the same backend port for TLS and non-TLS environments(as we do for other openstack services), but variables like `haproxy_backend_ssl` should be there | 22:29 |
| damiandabrowski | ex. https://opendev.org/openstack/openstack-ansible/src/commit/ea0bbd2c582b313fc7174cc714a52e17e27e6e14/inventory/group_vars/keystone_all/haproxy_service.yml#L25-L27 | 22:29 |
| damiandabrowski | feel free to leave a comment in gerrit ;) | 22:29 |
| hamburgler | damiandabrowski: will do! thanks again :D | 22:31 |
| hamburgler | done :) | 22:38 |
| opendevreview | James Denton proposed openstack/openstack-ansible master: feat: Add skyline deployment capability https://review.opendev.org/c/openstack/openstack-ansible/+/859446 | 23:04 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!