| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-05-10 | |||
| 12:36:58 | Shrews | mordred: i was going to get to that this morning, but if it already has two 2's, go ahead | |
| 13:03:01 | mordred | Shrews: I dunno - one of them is from me - and who knows what I've been drinking | |
| 13:12:07 | openstackgerrit | Merged openstack/python-openstackclient master: document the --timing option https://review.opendev.org/657640 | |
| 13:38:42 | openstackgerrit | Jeremy Houser proposed openstack/python-openstackclient master: Refactor test_snapshot.py and test_transfer_request.py https://review.opendev.org/655554 | |
| 15:28:14 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.opendev.org/651534 | |
| 15:29:00 | gtema | shrews, mordred, dtantsur: with the image refactor there was small conflict with yesterdays merged https://review.opendev.org/#/c/637238/ | |
| 15:29:40 | gtema | just pushed a tiny update with changes only in cloud.test_image (few lines adding name query param) | |
| 15:30:06 | gtema | please re-review again not to loose time. Was already going to party that it gets merged | |
| 15:41:37 | mordred | gtema: done! | |
| 15:42:33 | gtema | thanks | |
| 15:47:10 | openstackgerrit | Brian Haley proposed openstack/python-openstackclient master: Stop leaving temp files after unit test runs https://review.opendev.org/658190 | |
| 16:34:44 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.opendev.org/651534 | |
| 16:36:07 | gtema | mordred, Shrews: sorry, was bad for pep8 gods | |
| 16:36:28 | mordred | gtema: DO NOT ARGUE WITH PEP8 GODS | |
| 16:36:56 | gtema | I want to run agains wall with this patch | |
| 16:37:18 | mordred | gtema: just think how good it will feel when it's in :) | |
| 16:38:16 | gtema | trying to, dreaming of it | |
| 16:39:25 | gtema | if nobody objects and zuul gives +1 and today noone else will give +2 or +3 I can do this myself, since it was already approved few times | |
| 16:39:42 | gtema | again: if noone objects | |
| 17:01:24 | mordred | gtema: yeah - I think it's ok to push it on through | |
| 17:05:40 | gtema | mordred: thanks. zuul looks green so far | |
| 20:04:03 | openstackgerrit | Merged openstack/openstacksdk master: Continue refactoring of the image https://review.opendev.org/651534 | |
| #openstack-sdks - 2019-05-11 | |||
| 05:08:58 | openstackgerrit | Akihiro Motoki proposed openstack/cliff master: Blacklist bandit 1.6.0 due to directory exclusion bug https://review.opendev.org/658522 | |
| 07:40:50 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Extract image download method into a mixin https://review.opendev.org/658346 | |
| 07:45:21 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Replace use of log.warn with log.warning https://review.opendev.org/658529 | |
| 20:41:30 | openstackgerrit | melissaml proposed openstack/python-openstackclient master: Rename review.openstack.org to review.opendev.org https://review.opendev.org/658562 | |
| #openstack-sdks - 2019-05-12 | |||
| 02:53:40 | openstackgerrit | Brin Zhang proposed openstack/python-openstackclient master: Add changes-before attribute to server list https://review.opendev.org/657373 | |
| #openstack-sdks - 2019-05-13 | |||
| 02:10:49 | openstackgerrit | ZhouHeng proposed openstack/python-openstackclient master: openstack port create support --extra-dhcp-opt https://review.opendev.org/658712 | |
| 02:19:41 | openstackgerrit | ZhouHeng proposed openstack/python-openstackclient master: openstack port create support --extra-dhcp-opt https://review.opendev.org/657519 | |
| 02:52:50 | openstackgerrit | qian wang proposed openstack/js-openstack-lib master: Modify README.rst https://review.opendev.org/658720 | |
| 09:26:27 | brtknr | Hi all, just wondering about the delay in release of 0.28.0 | |
| 12:32:58 | mordred | brtknr: remote: https://review.opendev.org/658800 Release 0.28.0 of openstacksdk | |
| 12:38:49 | brtknr | mordred: excellent! thanks :) | |
| 12:39:26 | mordred | brtknr: sorry for the delay - releases and summits and whatnot | |
| 12:40:17 | brtknr | mordred: understood! yes, I forgot about the summit! :) | |
| 12:49:12 | openstackgerrit | Natal Ngétal proposed openstack/python-openstackclient master: Fix bandit errors. https://review.opendev.org/658804 | |
| 12:51:37 | openstackgerrit | Natal Ngétal proposed openstack/python-openstackclient master: Fix bandit errors. https://review.opendev.org/658804 | |
| 12:53:31 | openstackgerrit | Natal Ngétal proposed openstack/python-openstackclient master: Fix bandit errors. https://review.opendev.org/658804 | |
| 13:03:55 | efried | mordred, dustinc: At some point can we have a quick brainstorm on how to bootstrap clouds.yaml setup from nova? | |
| 13:04:38 | mordred | efried: yes! that sounds like a great thing to brainstorm. I'm going to be a little flaky until thursday - could we do it then? | |
| 13:05:06 | efried | mordred: Sure thing, I'll spread the word that you'll stop being flaky on Thursday. | |
| 13:06:17 | Shrews | ha | |
| 13:06:31 | Shrews | flaky is a lifestyle choice | |
| 13:06:54 | efried | My goal is to become independently wealthy so I can be flaky 24/7 | |
| 13:07:27 | mordred | efried: please to let me know when you figure that out because I would like to subscribe to your newsletter | |
| 13:07:31 | Shrews | efried: i think mordred achieves that with pennies | |
| 13:08:41 | Shrews | my bonus! | |
| 13:21:23 | gtema | efried: I would be also totally interested in that. Was loading my head lots of times how this can be achieved. Keep me in the loop, please | |
| 13:22:27 | efried | gtema: Sure thing. How much background do you have on Nova's situation here? | |
| 13:23:07 | gtema | not necessarily much, but very interested in the result and possible impl. options | |
| 13:24:01 | efried | Nova talks to a bunch of other APIs (glance, cinder, neutron, ...) to do its thing. Around queens we did some work so that operators could (mostly) set up those communications using common ksa oslo.config options. But they're still talking through python-*client. | |
| 13:24:37 | efried | which means that, for the most part, we use the ksa config opts to create an adapter to get an endpoint, then throw away the rest of the adapter and pass the endpoint to the python-*client, which uses it to create... an adapter. | |
| 13:24:42 | efried | all of which kind of sucks. | |
| 13:26:22 | gtema | sure, and I am exactly on the front of replacing python-*client to SDK, so should be fitting | |
| 13:27:16 | efried | gtema: so step 1 is getting openstacksdk to understand ksa oslo.config opts, which is wip here: https://review.opendev.org/#/c/643601/ | |
| 13:27:39 | efried | and proven in this series https://review.opendev.org/#/c/643664/ | |
| 13:28:13 | efried | and dustinc has been working in that series to replace python-ironicclient with sdk calls, one by one. | |
| 13:28:54 | efried | That first one just needs tests so we can merge it. Then get it in a release of openstacksdk so nova can require it. | |
| 13:29:21 | mordred | yeah - I've got adding tests to that patch on my todo list for this week | |
| 13:29:21 | efried | so we can continue the work of swapping out client pieces with sdk calls. | |
| 13:29:39 | efried | mordred: maybe gtema is volunteering to take over :P | |
| 13:29:43 | gtema | efried: great. Will be looking at. Need to run away unfortunately. Count me in for SDK parts as told during PTG | |
| 13:29:51 | efried | thanks gtema :) | |
| 15:48:45 | AJaeger | could somebody review https://review.opendev.org/656704 for os-service-types to add testing jobs | |
| 15:49:23 | AJaeger | (update not add) | |
| 15:54:42 | AJaeger | thanks, cmurphy ! | |
| 15:59:21 | cmurphy | yw AJaeger | |
| 16:03:50 | openstackgerrit | Merged openstack/os-service-types master: Update testing jobs https://review.opendev.org/656704 | |
| 16:22:37 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Use cliff formattable columns in volume v1 commands https://review.opendev.org/658856 | |
| 16:53:12 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Use cliff formattable columns in volume v2 commands https://review.opendev.org/658863 | |
| 17:20:12 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Use cliff formattable columns in object storage commands https://review.opendev.org/658871 | |
| 18:15:28 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add support for vendor hooks https://review.opendev.org/657848 | |
| 21:46:23 | dustinc | QQ fo ryou guys..I see baremetal is implementing patch_node in next release according to docs (https://docs.openstack.org/openstacksdk/latest/user/proxies/baremetal.html#openstack.baremetal.v1._proxy.Proxy.patch_node) but not sure where to find spec for it, is there a spec somewhere? | |
| 22:45:27 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: [Compute]Make column content readable for both human and machine https://review.opendev.org/658933 | |
| #openstack-sdks - 2019-05-14 | |||
| 01:22:27 | openstackgerrit | caoyuan proposed openstack/keystoneauth master: Replace git.openstack.org URLs with opendev.org URLs https://review.opendev.org/655016 | |
| 04:02:48 | openstackgerrit | qian wang proposed openstack/microversion-parse master: Add .idea/* in .gitignore https://review.opendev.org/658956 | |
| 06:49:06 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add image.stage methods https://review.opendev.org/652981 | |
| 10:10:55 | openstackgerrit | Merged openstack/openstacksdk master: Extract image download method into a mixin https://review.opendev.org/658346 | |
| 12:52:53 | efried | dtantsur: see question from dustinc about 15h ago | |
| 13:13:28 | mordred | efried, dustinc: what do you mean by "spec" in this context? | |
| 13:14:08 | openstackgerrit | Merged openstack/openstacksdk master: Replace use of log.warn with log.warning https://review.opendev.org/658529 | |
| 13:18:44 | efried | mordred: FYI dustinc is on Pacific time, so it may be a couple hours before he gets back to you. (And I don't know what he meant by 'spec') | |
| 13:20:48 | mordred | efried: neat! I'm also in and out today - so we may or may not manage to connect with each other :) | |
| 13:21:29 | efried | mordred: In general, what's the process when someone wants to enhance one of these proxies? Is a spec (as in rst document describing the change) required? | |
| 13:22:30 | mordred | nope - one just dives in. if it's a big change, checking in with other folks first is a good idea - but we don't have nearly enough active devs to warrant the overhead of specs atm | |
| 13:23:21 | efried | ack | |
| 13:23:22 | gtema | mordred, why do you say so, a 3 of us is not enough? ;-) | |
| 13:23:42 | efried | dustinc: FYI the change appears to have been delivered via https://review.opendev.org/#/c/647730/ | |
| 13:26:43 | efried | dustinc: If you're looking for more information on what the json payload should look like, I would start with the corresponding baremetal API: https://developer.openstack.org/api-ref/baremetal/?expanded=update-node-detail#update-node | |
| 13:27:16 | mordred | gtema: 3 of us is perfect :) | |
| 13:27:28 | gtema | we are nearly almighty | |
| 13:27:34 | mordred | nearly | |
| 13:27:35 | gtema | :D | |
| 13:27:38 | efried | (which, mordred/dtantsur/gtema, would be a good thing to link from any of these proxy methods) | |
| 13:27:49 | efried | (especially the ones that have **kwargs) | |
| 13:27:51 | gtema | for baremetal its dtantsur | |
| 13:27:53 | mordred | efried: I agree - that's not a ba idea at all | |
| 13:29:27 | efried | prolly not | |
| 13:29:38 | efried | since it's in another repo | |
| 13:29:48 | efried | I was thinking of the :oslo.config:`group.option` role | |