| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-09-12 | |||
| 13:48:38 | gtema | ah, ok. Sad, that I lost some time I promised to work on that topic | |
| 13:48:43 | dtroyer | the push for the major rev was to remove deprecated and back-compat bits preparing for the swithc to totally using sdk for auth | |
| 13:49:11 | dtroyer | gtema: no worries here, I was working on this the first time we went to Denver, so talk about lost time doing other things… :) | |
| 13:49:35 | gtema | yeah. I will reset my work on that tomorrow - promise | |
| 13:52:17 | dtroyer | amotoki: if you are still around, want to +W https://review.opendev.org/#/c/681375/ please? that will be the release commit for the metapackage | |
| 13:52:29 | amotoki | dtroyer: sure | |
| 14:03:58 | openstackgerrit | Merged openstack/openstackclient master: Prepare for OSC 4.0 release https://review.opendev.org/681375 | |
| 16:19:14 | elmiko28 | hey, lost power here, but if anyone has api-sig questions i'm around | |
| 16:19:16 | elmiko28 | sorta | |
| #openstack-sdks - 2019-09-13 | |||
| 16:11:51 | openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: WIP: Switch image to use SDK https://review.opendev.org/650374 | |
| 16:12:44 | gtema | dtroyer - here it is. It is still WIP, but I would really appreciate first review. It's big and relatively complex, so better now than later | |
| #openstack-sdks - 2019-09-15 | |||
| 10:38:56 | openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: WIP: Switch image to use SDK https://review.opendev.org/650374 | |
| #openstack-sdks - 2019-09-16 | |||
| 00:09:48 | openstackgerrit | Brin Zhang proposed openstack/python-openstackclient master: WIP: Add function test for server migrate list https://review.opendev.org/681473 | |
| 01:48:26 | openstackgerrit | Brin Zhang proposed openstack/python-openstackclient master: WIP: Add function test for server migrate list https://review.opendev.org/681473 | |
| 04:50:42 | openstackgerrit | Vishakha Agarwal proposed openstack/keystoneauth master: [WIP] Generate pdf documentation https://review.opendev.org/682272 | |
| 06:41:06 | openstackgerrit | Vishakha Agarwal proposed openstack/keystoneauth master: [WIP] Generate pdf documentation https://review.opendev.org/682272 | |
| 08:57:58 | openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: WIP: Switch image to use SDK https://review.opendev.org/650374 | |
| 09:34:10 | openstackgerrit | Vishakha Agarwal proposed openstack/keystoneauth master: [WIP] Generate pdf documentation https://review.opendev.org/682272 | |
| 11:29:03 | gtema | dtroyer, are you here? | |
| 11:34:26 | mordred | gtema: morning! I'm back from PTO - anything I should look at? | |
| 11:34:46 | gtema | morning mordred. Long time haven't seen you | |
| 11:35:07 | gtema | nothing urgent. I am working on the mswitching OSC to SDK for image | |
| 11:35:15 | mordred | yah - I saw that - very cool! | |
| 11:35:33 | gtema | perhaps you can say your opinion - currently OSC supports creating image from STDIN | |
| 11:35:41 | gtema | this is something we can't do in SDK as of now | |
| 11:35:54 | gtema | but this also leads to inability to do checksum check | |
| 11:36:22 | mordred | yeah. glance cli I think also supported image from stdin ... is this a thing that people really do? | |
| 11:36:31 | gtema | so I'm curios, what should we better do - deprecate stdin/flush to temp file/teach SDK to accept this | |
| 11:36:56 | gtema | no idea, but exactly this is used in the devstack, so even devstack setup already fails | |
| 11:37:38 | mordred | does devstack use this because it needs to stream something - or it just happens to do it? | |
| 11:38:01 | gtema | seems just do it this way. First download image and then pass it as stdin | |
| 11:38:02 | gtema | https://refstack.openstack.org/#/community_results | |
| 11:38:08 | gtema | sorry, wrong link | |
| 11:38:14 | gtema | https://github.com/openstack/devstack/blob/master/functions#L380 | |
| 11:38:35 | gtema | sidenote: using github instead of opendev because of response performance | |
| 11:38:50 | mordred | I mean - we could teach sdk to understand it - but it would make some parts, like multi-part chunked upload in swift, much slower, since you can't seek on stdin | |
| 11:39:05 | gtema | exactly | |
| 11:39:18 | gtema | but deprecating this is already quite a "loud" change | |
| 11:39:31 | gtema | or we can flush stdin silently in OSC | |
| 11:39:36 | mordred | yeah. oh wow - it's actually zcatting stuff | |
| 11:39:38 | gtema | hehe, funny | |
| 11:39:58 | gtema | yeah, in archive mode - agree | |
| 11:40:02 | gtema | it make sense | |
| 11:40:07 | mordred | I mean - I guess that's an example of a use case where stdin makes sense | |
| 11:40:09 | mordred | yeah | |
| 11:41:12 | gtema | I think then really flushing stdin to temp file might be a relatively good idea, since you can use swift for upload | |
| 11:41:33 | mordred | it might be - although it would cause more disk to be used | |
| 11:41:46 | gtema | agree | |
| 11:41:55 | mordred | but yeah - multi-threaded upload to swift is, you know, important :) | |
| 11:42:01 | gtema | sure | |
| 11:42:09 | gtema | and also md5 check is | |
| 11:42:17 | mordred | yeah | |
| 11:42:59 | gtema | ok, then will implement this in OSC | |
| 11:43:22 | gtema | I don't think it make sense to be done in SDK | |
| 11:43:24 | mordred | maybe that's the best bet then - and then we update docs (and devstack) to use a direct filename argument when we can - because it has better performance | |
| 11:43:27 | mordred | yah | |
| 11:43:37 | gtema | ok, cool | |
| 11:43:39 | gtema | thks | |
| 11:44:01 | gtema | btw, project cleanup. We need to start working on it | |
| 11:44:24 | gtema | and solve caching | |
| 11:44:55 | mordred | yah | |
| 11:45:44 | gtema | I'm currently having own need for project cleanup | |
| 12:11:57 | mordred | gtema: got a sec for an easy one? https://review.opendev.org/#/c/601659 <-- I didn't actually write it, ignore gerrit. :) | |
| 12:12:14 | gtema | sure | |
| 12:12:50 | gtema | sone | |
| 12:13:14 | gtema | done I mean | |
| 12:13:38 | mordred | \o/ | |
| 13:29:36 | openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: WIP: Switch image to use SDK https://review.opendev.org/650374 | |
| 13:52:14 | openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: WIP: Switch image to use SDK https://review.opendev.org/650374 | |
| 16:15:27 | openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: WIP: Switch image to use SDK https://review.opendev.org/650374 | |
| 16:57:03 | openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: WIP: Switch image to use SDK https://review.opendev.org/650374 | |
| 17:14:26 | openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: WIP: Switch image to use SDK https://review.opendev.org/650374 | |
| 18:41:25 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Fix image create with tags https://review.opendev.org/682454 | |
| 18:42:19 | gtema_ | mordred: ^^^. This came up in some of the devstack functional tests. We cast tags to string currently for the image | |
| 18:45:31 | mordred | gtema_: you left in a print - otherwise LGTM | |
| 18:46:29 | gtema_ | right | |
| 18:46:32 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Fix image create with tags https://review.opendev.org/682454 | |
| 18:46:47 | gtema_ | fixed | |
| 18:47:29 | gtema_ | I hoped to escape issues with followup SDK changes, since I need to raise SDK in lower-constraints as well (and would require release) | |
| 18:49:28 | openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: WIP: Switch image to use SDK https://review.opendev.org/650374 | |
| 19:03:23 | gtema | otherwise remaining tests in the OSC are green, so the change is on the good track | |
| #openstack-sdks - 2019-09-17 | |||
| 00:09:17 | openstackgerrit | Merged openstack/keystoneauth master: Fix misspell word https://review.opendev.org/680600 | |
| 07:04:56 | gtema | morning, was anyone able to find the reason for devstack-networking failures? It blames octavia (argument --bind_ip: Invalid IPAddress value:), but I can't find any recent suspicious change | |
| 07:23:28 | gtema | solved, the fix already landed | |
| 07:45:44 | mordred | gtema: oh yeah? good - I was just looking at that | |
| 07:45:54 | mordred | gtema: was just a devstack issue? | |
| 07:46:05 | gtema | https://review.opendev.org/#/c/682327/ | |
| 07:48:30 | mordred | oh. hah | |
| 07:48:34 | mordred | yeah | |
| 07:49:50 | gtema | let's see whether it helps :) | |
| 07:50:48 | gtema | I didn't know the image is so deep in OSC. The change is now similar huge to the what it was in SDK :D | |
| 07:50:59 | gtema | it would be fun to wait for review | |
| 08:02:19 | mordred | gtema: image is such a fun topic | |
| 08:02:37 | gtema | yeah | |
| 11:20:55 | openstackgerrit | Vishakha Agarwal proposed openstack/keystoneauth master: Generate pdf documentation https://review.opendev.org/682272 | |
| 11:34:20 | gtema | what the heck is going on, that still every change fails. Argh.... | |
| 11:40:21 | mordred | gtema: there was a mirror issue for a second - although I think that's fixed now | |
| 11:40:48 | gtema | really? | |
| 11:41:20 | gtema | ah, yeah. lets see whether 3rd or 5th round of rechecks helps | |
| 13:11:13 | openstackgerrit | Merged openstack/openstacksdk master: Properly convert baremetal fields to server-side values https://review.opendev.org/680649 | |