| eandersson | johnsom thanks for the review. I'll merge the current patch, but I'll start working on something more robust. I just rather have this for now, since I don't think I'll have a lot of time over the next few weeks. | 05:23 |
|---|---|---|
| eandersson | (and maybe months who really knows :p) | 05:27 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix support for scoped tokens and default roles https://review.opendev.org/c/openstack/designate/+/800727 | 05:55 |
| opendevreview | Merged openstack/python-designateclient master: Add basic osc unit tests https://review.opendev.org/c/openstack/python-designateclient/+/819761 | 06:22 |
| eandersson | johnsom I really think we should remove the old floating ips implementation, at least deprecate it (if it isn't already). | 17:27 |
| eandersson | I feel like it was more like a early PoC that was later replaced by the current neutron integration | 17:30 |
| eandersson | but I have very little context around it | 17:30 |
| johnsom | eandersson Yeah, I haven't dug into that part of the code deeply myself yet. I am cool with someone proposing deprecation, but I not sure we want to open the microversion pandora's box by removing it. | 17:35 |
| eandersson | Yea - that is the issue. | 17:51 |
| eandersson | Our CI -_- | 20:33 |
| eandersson | I feel like there are days where nothing fails, and other days where everything fails :p | 20:37 |
| eandersson | Actually maybe something broke when I rebased, or maybe it is due to the uwsgi update on devstack | 23:28 |
| eandersson | https://zuul.opendev.org/t/openstack/build/ce7fb2d8cab54350ab3e910c5865e0e4/log/controller/logs/screen-designate-api.txt | 23:28 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!