| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-10-11 | |||
| 13:41:20 | ml | openstackcloud raises Exception from openstack.cloud.exc, it doesn't use anything from openstack.exception. | |
| 13:41:38 | ml | Why is that? | |
| 13:44:12 | mordred | ml: it's just historical - we haven't finished merging all the things yet | |
| 13:45:17 | mordred | ml: geting the openstackcloud code to raise exceptions from openstack.exception instead would be great | |
| 13:46:10 | ml | Can I start using them in my change already? or is it desired to be handled as one change? | |
| 13:48:23 | ml | probably a bad idea to keep both imports at the same time, because one can't import it as "exc" | |
| 13:50:31 | mordred | ml: we could just import it as exceptions ... I think either way you want to go is fine | |
| 13:50:44 | ml | ok, cool | |
| 14:06:59 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rearrange shade image code https://review.openstack.org/609683 | |
| 14:07:00 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Move image methods to sdk image proxy https://review.openstack.org/609684 | |
| 14:25:28 | dtantsur | mordred: is it okay that we ignore the error_message argument in https://github.com/openstack/openstacksdk/blob/master/openstack/_adapter.py#L129 ? | |
| 14:26:06 | dtantsur | it's my third (?) attempt to understand error handling in https://review.openstack.org/#/c/604645/ and I'm still confused... | |
| 14:37:46 | openstackgerrit | Matthias Lisin proposed openstack/openstacksdk master: openstackcloud.py: Implement FWaaS wrapper methods. https://review.openstack.org/604811 | |
| 14:38:36 | mordred | dtantsur: probably not - and you're probably confused because it's probably broken | |
| 14:39:06 | mordred | dtantsur: lemme see if I can wrap my head around it | |
| 14:39:25 | dtantsur | cool. your head has much higher chances of figuring it out than mine :) | |
| 14:39:44 | Shrews | mordred: overall direction on 609684 looks pretty good to me. Maybe make BaseImageProxy abstract for clarity and to enforce _upload_image() is declared in other classes | |
| 14:40:32 | mordred | Shrews: yah - I tried that and hit "TypeError: metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases" | |
| 14:40:42 | Shrews | O.o | |
| 14:40:46 | mordred | Shrews: and at that point I just gav up and decided to farm goats | |
| 14:41:29 | Shrews | oh, maybe because it derives from Proxy | |
| 14:41:31 | Shrews | poo | |
| 14:42:07 | mordred | Shrews: although - I keep thinking that maybe that proxy metaclass is a bad idea | |
| 14:44:45 | Shrews | mordred: you could make BaseImageProxy more of an interface (call it BaseImageInterface), and make v1._proxy.Proxy derive from that *and* proxy.Proxy | |
| 14:47:28 | Shrews | there are other ways around it, but I'm meh about it | |
| 14:51:33 | dtantsur | we need some metaclasses for metaclasses | |
| 15:12:31 | smcginnis | dtantsur: Wouldn't that be metametaclasses? | |
| 15:12:42 | dtantsur | awesome, yeah? | |
| 15:41:02 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rearrange shade image code https://review.openstack.org/609683 | |
| 15:41:02 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Move image methods to sdk image proxy https://review.openstack.org/609684 | |
| 15:41:03 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Revert the Proxy metaclass https://review.openstack.org/609747 | |
| 15:41:25 | mordred | Shrews, dtantsur: ^^ how bout them metaapples? | |
| 15:45:27 | dtantsur | heh | |
| 15:49:42 | mordred | dtantsur: maybe by then I will have figured out the error message scenario | |
| 16:00:41 | elmiko | api sig office hours, HOOOoooooo | |
| 16:00:57 | elmiko | lol, i need to screenshot that | |
| 16:00:58 | edleafe | Office hour for API-SIG starts now | |
| 16:01:13 | edleafe | Try not to flood the channel with all your inquiries | |
| 16:02:08 | elmiko | ++ | |
| 16:02:55 | edleafe | hey, pipe down, elmiko! | |
| 16:06:37 | mordred | edleafe: are we there yet? | |
| 16:07:09 | edleafe | mordred: I swear, if you don't settle down, I'm turning this car around and going back home!! | |
| 16:08:18 | mordred | edleafe: but but but | |
| 16:13:42 | elmiko | edleafe: lol | |
| 16:13:54 | elmiko | edleafe: are we there yet? | |
| 16:14:18 | edleafe | elmiko: I'm going to count to 3... | |
| 16:17:07 | mordred | edleafe: elmiko keeps hitting me! | |
| 16:18:02 | mordred | if people are bored in office hours and want to review some fairly insane python patches - I've got plenty I can point you at ... | |
| 16:18:55 | elmiko | best office hours ever! | |
| 17:02:58 | edleafe | Well, that was underwhelming. :) | |
| 17:29:18 | openstackgerrit | Merged openstack/openstacksdk master: Correct updating baremetal nodes by name or ID https://review.openstack.org/608674 | |
| 18:14:55 | elmiko | edleafe: lol | |
| 18:21:18 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Handle not having cinderclient.v1 available https://review.openstack.org/609473 | |
| 20:02:52 | openstackgerrit | Merged openstack/openstacksdk master: Use discovery instead of config to create proxies https://review.openstack.org/600292 | |
| 20:16:47 | openstackgerrit | Merged openstack/openstacksdk master: Remove api version default values https://review.openstack.org/600330 | |
| 21:02:50 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Handle not having cinderclient.v1 available https://review.openstack.org/609473 | |
| #openstack-sdks - 2018-10-12 | |||
| 03:09:39 | openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | |
| 07:53:00 | openstackgerrit | Merged openstack/openstacksdk master: Add functional tests for clustering https://review.openstack.org/607061 | |
| 09:50:26 | openstackgerrit | Vishakha Agarwal proposed openstack/python-openstackclient master: Modify the help message for 'registered limit set' https://review.openstack.org/609985 | |
| 11:32:24 | openstackgerrit | Matthias Lisin proposed openstack/openstacksdk master: openstackcloud.py: Implement FWaaS wrapper methods. https://review.openstack.org/604811 | |
| 12:51:20 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Switch bare metal NIC actions in OpenStackCloud to baremetal Proxy calls https://review.openstack.org/610024 | |
| 13:01:41 | dtantsur | Shrews: hi, if you have a minute: https://review.openstack.org/#/c/609399/ | |
| 13:16:12 | Shrews | dtantsur|afk: sure, will take a look in a little bit | |
| 13:43:02 | mordred | oh my god the discovery patch landed | |
| 13:46:14 | edleafe | mordred: ?? Which patch? | |
| 13:46:42 | mordred | edleafe: Merged openstack/openstacksdk master: Use discovery instead of config to create proxies https://review.openstack.org/600292 | |
| 13:46:53 | mordred | edleafe: sdk is now fully discovery driven | |
| 13:47:15 | mordred | with config values only used for override | |
| 13:47:47 | edleafe | mordred: ah. For a second there, I thought you were referring to https://review.openstack.org/459405 | |
| 13:48:02 | mordred | edleafe: bah. like we'll ever finish that. it's too hard :) | |
| 13:48:43 | edleafe | nah, we have dtantsur|afk on it. | |
| 13:49:21 | mordred | good point | |
| 13:49:31 | Shrews | mordred: good grief. i +A'd that DAYS ago | |
| 13:49:33 | mordred | mnaser: do you know who a good person is to ping about the broken magnum job for sdk? | |
| 13:49:39 | Shrews | i assumed it merged already | |
| 13:50:00 | mordred | Shrews: there was an oslo.messaging release which broke bifrost which broke the sdk bifrost gate | |
| 13:50:30 | Shrews | see... this is why we should test in a silo | |
| 13:50:33 | mnaser | mordred: I think spyros (PTL) is usually pretty good about that stuff. I can’t recall his IRC nick off the top of my head | |
| 13:51:01 | mnaser | mordred: strigazi | |
| 13:51:05 | mordred | awesome- thanks! | |
| 13:51:37 | mordred | mnaser: openstack-magnum doesn't seem to be their irc channel ... | |
| 13:55:24 | hansmoleman | dtroyer: i was just reminded of this bug i opened for osc https://storyboard.openstack.org/#!/story/2002158 - curious as to how to handle that. you can't set a volume as read-only while it's being created, but the option is on the volume create CLI. | |
| 13:55:42 | hansmoleman | there is a --wait option on the volume create CLI, so if --wait and --read-only, then wait for it to be available and then make the read-only change? | |
| 13:55:50 | hansmoleman | or make --read-only imply --wait? | |
| 13:56:21 | hansmoleman | oh nvm, i guess there is no --wait option for volume create | |
| 13:56:23 | hansmoleman | that's on server create | |
| 13:56:51 | hansmoleman | welp, i could add --wait to volume create and then build on that for --read-only, but that wouldn't work for anyone using older versions of osc | |
| 14:04:45 | mnaser | mordred: #openstack-containers is the one to look at | |
| 14:25:27 | openstackgerrit | Chuck Short proposed openstack/os-client-config master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610051 | |
| 14:42:51 | mordred | hansmoleman: does the volume have to be in ready state to have its readonly attribute set? | |
| 14:43:05 | hansmoleman | yeah it has to be available | |
| 14:43:10 | mordred | how fun | |
| 14:43:17 | hansmoleman | $ openstack volume create --size 1 --read-only read-only-vol1 | |
| 14:43:17 | hansmoleman | Failed to set volume read-only access mode flag: Invalid volume: Volume 70539836-fbb9-478d-9cd1-0e733c0eb0cc status must be available to update readonly flag, but current status is: creating. (HTTP 400) (Request-ID: req-ac54552a-3784-4028-8262-8a600844c186) | |
| 14:43:42 | mordred | well - I'd certainly support --read-only implying --wait | |
| 14:43:50 | mordred | because otherwise it's a pretty useless flag | |
| 14:43:57 | hansmoleman | either that or that option just needs to die because it never worked | |
| 14:44:01 | hansmoleman | and was clearly not tested when it was added | |
| 14:44:04 | mordred | yah | |
| 14:44:14 | openstackgerrit | Chuck Short proposed openstack/os-client-config master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610051 | |