| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-09-11 | |||
| 11:22:06 | frickler | amotoki: dtroyer: this breaks the plugin check because it doesn't chdir into the correct project directory https://opendev.org/openstack/openstackclient/commit/32ef09e8ada7b4a7c38d55e91c1bcb7aa6d69f73 , see results at https://zuul.opendev.org/t/openstack/builds?job_name=openstackclient-check-plugins | |
| 11:36:54 | amotoki | frickler: thanks for letting me know it. Hmm.... it worked when I fixed it a couple of months ago but something might be changed. will look into it. | |
| 11:48:02 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Properly convert baremetal fields to server-side values https://review.opendev.org/680649 | |
| 12:31:17 | openstackgerrit | Bence Romsics proposed openstack/python-openstackclient master: Add command: router add/remove route --route https://review.opendev.org/674325 | |
| 13:20:43 | dtroyer | amotoki: the change of the checkplugins job to tox has another new assumption in it about the local dev directory layout, install siblings tried to install my entire source tree on my laptop, including non-openstack stuffs. I am going to revert that bit for now, we can work out a better way to run the check locally. | |
| 13:21:24 | amotoki | dtroyer: I think I forgot to specify zuul_work_dir to the opensatckclient repo. | |
| 13:21:27 | openstackgerrit | Dean Troyer proposed openstack/openstackclient master: Revert "check-plugins job: convert the job to tox" https://review.opendev.org/681470 | |
| 13:22:07 | openstackgerrit | Dean Troyer proposed openstack/openstackclient master: Revert "check-plugins job: switch to python3" https://review.opendev.org/681471 | |
| 13:22:16 | amotoki | dtroyer: I am thinking I need to do the thing like https://github.com/openstack/python-openstackclient/blob/master/.zuul.yaml#L19 | |
| 13:23:47 | dtroyer | the breakage is when running the job in other projects, no tox target… yes, that. | |
| 13:24:11 | dtroyer | but there are other issues, I need to get 4.0 done today, this can follow... | |
| 13:27:40 | amotoki | dtroyer: I see. Let's revert the related patches now and then send follow-up fix(es) | |
| 13:29:37 | openstackgerrit | Brin Zhang proposed openstack/python-openstackclient master: WIP: Add function test for server migrate list https://review.opendev.org/681473 | |
| 13:41:55 | openstackgerrit | Dean Troyer proposed openstack/openstackclient master: Revert "check-plugins job: convert the job to tox" https://review.opendev.org/681470 | |
| 13:49:48 | openstackgerrit | Merged openstack/openstackclient master: Revert "check-plugins job: switch to python3" https://review.opendev.org/681471 | |
| 16:30:11 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Update release table for Train and 4.0.0 https://review.opendev.org/681067 | |
| 16:37:32 | openstackgerrit | Merged openstack/openstackclient master: Revert "check-plugins job: convert the job to tox" https://review.opendev.org/681470 | |
| 19:35:53 | openstackgerrit | Merged openstack/python-openstackclient master: Update release table for Train and 4.0.0 https://review.opendev.org/681067 | |
| 22:23:34 | openstackgerrit | Dean Troyer proposed openstack/openstackclient master: Prepare for OSC 4.0 release https://review.opendev.org/681375 | |
| #openstack-sdks - 2019-09-12 | |||
| 02:56:01 | openstackgerrit | Brin Zhang proposed openstack/python-openstackclient master: Microversion 2.79: Add delete_on_termination to volume-attach API https://review.opendev.org/673725 | |
| 03:42:21 | openstackgerrit | Brin Zhang proposed openstack/python-openstackclient master: Microversion 2.79: Add delete_on_termination to volume-attach API https://review.opendev.org/673725 | |
| 06:17:21 | amotoki | dtroyer: regarding the checkplugins job reverted yesterday, it did not introduce the new assumption you mentioned. playbooks/openstackclient-check-plugins/run.yaml already assumes that ../../x/pbrx (non-opensatck stuffs). | |
| 06:20:05 | amotoki | dtroyer: Apart from that the job broke other openstackclient related jobs due to the lack of chdir in .zuul.yaml, I believe nothing was changed. | |
| 06:20:53 | amotoki | dtroyer: I don't plan to propose it again until Ussuri is open. I just would like to clarify what needs to be improved. | |
| 09:25:24 | openstackgerrit | Brin Zhang proposed openstack/python-openstackclient master: WIP: Add function test for server migrate list https://review.opendev.org/681473 | |
| 13:12:07 | dtroyer | amotoki: you are right there, I had not noticed that before and assumed it was ner | |
| 13:12:15 | dtroyer | new* | |
| 13:21:42 | amotoki | dtroyer: no worries. thanks for the clarification. | |
| 13:37:36 | openstackgerrit | Dean Troyer proposed openstack/openstackclient master: Prepare for OSC 4.0 release https://review.opendev.org/681375 | |
| 13:38:09 | dtroyer | amotoki: also, osc4 is real now! I am working on a release for the 'openstackclient' package also 4.0.0 so the major release numbers match…and the check_plugins is failing now on rsd-lib… <sigh> | |
| 13:39:15 | amotoki | dtroyer: yay for osc4! | |
| 13:44:29 | dtroyer | it only took <mumble-mumble> years :) | |
| 13:45:27 | amotoki | :) | |
| 13:46:49 | gtema | dtroyer, do you still want glance patch in osc4, or not. The patch is not yet ready, but I can try to speed up | |
| 13:46:55 | gtema | ? | |
| 13:48:04 | dtroyer | gtema: osc4 is released. I still want the patch but I had to release for the freeze. all that really means is it will not make the stable/train branch | |
| 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 | |