| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2020-03-02 | |||
| 17:53:38 | rm_work | my current quest is fixing octavia's AZ handling and compatibility with nova/neutron routed-network stuff, which requires also ... fixing nova/neutron routed-network stuff :D | |
| 17:53:43 | mordred | this is what we get for gtema not being able to come to shanghai - and has nothing to do with me forgetting things at all | |
| 17:53:50 | rm_work | well, it's also not nearly a top priority | |
| 17:53:58 | rm_work | which is the other reason it's been sidelined thus far for me | |
| 17:54:01 | mordred | yeah | |
| 17:54:20 | mordred | and oh goodie - it sounds like you have a wonderful rabbit hole to go down with routed networks :) | |
| 17:54:25 | rm_work | ugh yes | |
| 17:54:35 | rm_work | i was hoping i could take the easy route but i was denied :D | |
| 17:55:24 | mordred | where would the fun in that be? | |
| 17:56:15 | dtroyer | damn, you guys have been busy | |
| 17:56:16 | mordred | it's a dtroyer! | |
| 17:56:27 | dtroyer | where? | |
| 17:56:34 | mordred | ssssh. it's a sekrit | |
| 17:56:48 | dtroyer | oh good, don't let him out of the asylum | |
| 17:56:58 | mordred | I hear he's bringing vegetables | |
| 17:57:31 | dtroyer | he hates vegetables, which is why being vegetarian for 8 years was, um, interesting | |
| 17:57:41 | mordred | mmm. eggs | |
| 17:58:00 | mordred | could I have some eggs with those eggs? maybe with a side of eggs? | |
| 17:59:06 | dtroyer | So I think it is time to acknowledge the clear truth that I'm not going to continue to be working on OSC, not OpenStack for that matter. I owe an apology and major thanks to everyone who has kept the lights on. | |
| 17:59:42 | mordred | dtroyer: that is sad panda - but thanks for your many years of keeping the lights on yourself! | |
| 17:59:46 | dtroyer | I don't have a new job yet, but nothing I am looking at includes room here, unfortunately | |
| 18:06:15 | rm_work | hmm | |
| 18:06:34 | rm_work | is that because you're explicitly not looking at stuff that's related, because you need to GTFO for your sanity? :D | |
| 18:06:37 | rm_work | or just coincidental | |
| 18:07:43 | dtroyer | more coincidental. dis-investment in OpenStack is a real thing and more than just Intel | |
| 18:08:22 | rm_work | I feel like it's just ... shifting | |
| 18:08:23 | dtroyer | but it may be time, too. I almost made it to Nova's 10th birthday, but that's a long time in one place | |
| 18:08:24 | rm_work | but yes | |
| 18:12:13 | mordred | dtroyer: tell me about it - I have certainly never worked 10 years on anything else | |
| 18:21:17 | rm_work | me either! | |
| 18:22:34 | rm_work | I think I'm going on 7 years on openstack now tho :) | |
| 18:24:29 | dtantsur | tosky: dumped some microversion microthoughts to the patch :) | |
| 18:25:24 | mordred | Shrews: https://review.opendev.org/#/c/710823 is the sdk-side of that uuid thing from earlier | |
| 18:26:17 | dtantsur | mordred: will it be too much to ask for a unit test on ^^^? | |
| 18:26:46 | dtantsur | I guess it's a question for brtknr really | |
| 18:26:47 | rm_work | oh hey, good place for this since i'm already here: https://review.opendev.org/#/c/710452/ | |
| 18:27:12 | rm_work | need that for senlin ASAP, and will need that to do a conversion for OSC to SDK | |
| 18:27:30 | dtantsur | rm_work: does this resource even handle microversions? otherwise you'll always see None there | |
| 18:28:04 | rm_work | which, the one i just linked? | |
| 18:28:35 | rm_work | or the glance thing | |
| 18:29:50 | dtantsur | rm_work: the one you just linked | |
| 18:30:06 | rm_work | no microversioned involved | |
| 18:30:24 | rm_work | there's just a note in the commit message about which octavia version it appears in | |
| 18:30:31 | dtantsur | mmm, so 2.14 is not a microversion | |
| 18:30:33 | dtantsur | ? | |
| 18:30:40 | rm_work | there's no code around verifying that (though the octavia API will let you see what version it's on, yes) | |
| 18:30:46 | rm_work | but it's not strictly microversions | |
| 18:30:48 | mordred | so - it'll be there if octavia supports it - and none otherwise yeah? | |
| 18:31:07 | dtantsur | ah, similar to what ironic-inspector does, information-only microversions | |
| 18:31:59 | rm_work | https://github.com/openstack/octavia/blob/master/octavia/api/root_controller.py#L93-L95 | |
| 18:32:39 | rm_work | err, in a get? it'd be missing in a GET if the octavia version is older | |
| 18:32:49 | mordred | kinda like what glance does | |
| 18:32:57 | rm_work | non-present on the octavia side, illustrated by None on the SDK object side | |
| 18:33:02 | mordred | yeah | |
| 18:33:07 | Shrews | mordred: should there be a non-strict-mode test to validate that? We have an entire TestNormalize/TestStrictNormalize suite of tests for such things currently | |
| 18:33:13 | rm_work | and I guess a 400 if passed in fromt he SDK on a create | |
| 18:33:18 | johnsom | dtroyer Thanks for all of your excellent work on OSC. Good luck on your next adventure. | |
| 18:33:24 | Shrews | but apparently for not all things :/ | |
| 18:33:47 | mordred | Shrews: yeah, probably. I think you and dtantsur had the same thought | |
| 18:33:55 | mordred | it's almost like you value tests or something | |
| 18:34:01 | dtantsur | rm_work: +A then | |
| 18:34:10 | rm_work | whaaaaaaat, tests are important? who knew :D | |
| 18:34:12 | Shrews | mordred: we're such silly people folk | |
| 18:34:26 | dtantsur | can confirm ^^ | |
| 18:34:54 | rm_work | Oh my, yes.</farnsworthy> | |
| 18:36:18 | openstackgerrit | Merged openstack/ansible-collections-openstack master: os_coe_cluster: Retrive id/uuid correctly https://review.opendev.org/709734 | |
| 18:36:40 | KeithMnemonic | mordred: so is there anyway to get this merged or is it too late now? https://review.opendev.org/703683 | |
| 19:26:38 | mordred | KeithMnemonic: there you go | |
| 19:28:58 | KeithMnemonic | thank you!! | |
| 19:40:39 | KeithMnemonic | mordred: while you are in the giving mood, care to also help on this one for train? https://review.opendev.org/#/c/701299/ | |
| 19:41:44 | mordred | KeithMnemonic: done | |
| 20:01:59 | openstackgerrit | Merged openstack/openstacksdk master: Add availability_zone param to load balancer https://review.opendev.org/710452 | |
| 21:19:43 | openstackgerrit | Merged openstack/python-openstackclient stable/train: Fix openstack server list --deleted --marker option https://review.opendev.org/701299 | |
| 21:22:30 | KeithMnemonic | looking for review on this backport now that the train branch merged. thank you https://review.opendev.org/#/c/702260/4 | |
| #openstack-sdks - 2020-03-03 | |||
| 00:37:24 | openstackgerrit | Merged openstack/python-openstackclient stable/rocky: Fix functional tests for py3 https://review.opendev.org/703683 | |
| 11:03:46 | openstackgerrit | Riccardo Pittau proposed openstack/openstacksdk master: Import generate_temp_url from swiftclient https://review.opendev.org/710973 | |
| 12:41:32 | openstackgerrit | Bernard Cafarelli proposed openstack/openstacksdk stable/train: Handle HTTP errors in add/remove router interface calls https://review.opendev.org/710994 | |
| 12:41:35 | openstackgerrit | Bernard Cafarelli proposed openstack/openstacksdk stable/train: Fix error handling on add/remove router iface calls https://review.opendev.org/710995 | |
| 13:10:43 | openstackgerrit | Michał Dulko proposed openstack/openstacksdk stable/train: Fix error handling in network trunks operations https://review.opendev.org/711001 | |
| 13:10:43 | openstackgerrit | Michał Dulko proposed openstack/openstacksdk stable/train: Fix error handling in network trunks operations https://review.opendev.org/711001 | |
| 13:15:52 | KeithMnemonic | Could I also please get some reviews on this change https://review.opendev.org/#/c/701066/ | |
| 14:46:31 | openstackgerrit | Riccardo Pittau proposed openstack/openstacksdk master: Add retired and retired_reason fields to baremetal node https://review.opendev.org/711032 | |
| 14:46:45 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add "fields" parameter to ListSecurityGroup query https://review.opendev.org/710825 | |
| 14:50:52 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Deal with collection build modifying tree https://review.opendev.org/711033 | |
| 14:54:35 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/openstacksdk master: Include "fields" to "SecurityGroup" query parameters https://review.opendev.org/710820 | |
| 14:55:37 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Fix license metadata https://review.opendev.org/711035 | |
| 14:57:11 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Clean up minor build quibbles https://review.opendev.org/711036 | |
| 14:59:57 | openstackgerrit | Riccardo Pittau proposed openstack/openstacksdk master: Add retired and retired_reason fields to baremetal node https://review.opendev.org/711032 | |
| 15:27:16 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Add Openstack guidelines spec from Ansible https://review.opendev.org/704558 | |
| 15:29:11 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Test with and assert support for python3 https://review.opendev.org/710641 | |
| 15:30:00 | mordred | dtantsur: https://review.opendev.org/#/c/711032 has your name all over it | |
| 15:31:37 | dtantsur | done! | |
| 15:33:32 | mordred | dtantsur: also - looking at futurist (it does seem like a good idea to use it) | |
| 15:34:01 | mordred | dtantsur: I don't suppose we could drop the prettytable dependency ... | |
| 15:39:19 | dtantsur | mordred: I don't recall why we have it, honestly | |
| 15:40:13 | dtantsur | mordred: I would +2 a change making it optional, seems really used in one corner case | |
| 15:41:34 | mordred | dtantsur: https://review.opendev.org/711041 Make PrettyTable optional | |
| 15:42:24 | mordred | dtantsur: I think with that it's a great idea to use futurist in sdk - and if we do that global executor idea it should make it really nice for people instantiating a Connection in an eventlet service or similar | |
| 15:43:27 | dtantsur | mordred: one small suggestion inline, I remembered that this code path is often used by default | |
| 15:48:38 | mordred | dtantsur: updated - and good call | |