Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-05-10
12:32:33 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Extract image download method into a mixin https://review.opendev.org/658346
12:35:51 mordred Shrews: https://review.opendev.org/#/c/651534 has 2x+2 - but I left off +A in case you wanna poke at it
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

Earlier   Later