| *** sapd1 has joined #senlin | 01:31 | |
| *** __ministry has quit IRC | 01:57 | |
| *** __ministry has joined #senlin | 01:58 | |
| *** sapd1 has quit IRC | 11:04 | |
| *** jmlowe has joined #senlin | 13:28 | |
| rm_work | dtruong: err, so I think what you told me before might not be totally right... availability_zone is a param for the LB-create, which looks a bit less simple to change | 14:04 |
|---|---|---|
| rm_work | need to change the signature of https://github.com/openstack/senlin/blob/master/senlin/drivers/os/octavia_v2.py#L38-L39 | 14:06 |
| rm_work | also it seems like https://github.com/openstack/senlin/blob/master/senlin/drivers/os/lbaas.py#L29-L38 | 14:09 |
| rm_work | is a thin wrapper around octavia, even though it claims to be for Neutron-LBaaS? | 14:09 |
| rm_work | ah maybe I get what you were saying | 14:15 |
| openstackgerrit | Adam Harwell proposed openstack/senlin master: Add option to choose LB availability_zone https://review.opendev.org/710464 | 15:06 |
| rm_work | dtruong: can you review ^^ if you get a chance and tell me if this is basically correct? if so, I'll test it in my cloud | 15:07 |
| rm_work | ah it does probably need a depends-on for https://review.opendev.org/710452 | 18:12 |
| dtruong | rm_work LGTM. The only thing missing is an entry in https://github.com/openstack/senlin/blob/master/setup.cfg#L64 for the new policy version | 19:37 |
| *** jmlowe has quit IRC | 23:00 | |
| *** jmlowe has joined #senlin | 23:03 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!