| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: de-client-ify baremetal get_machine https://review.openstack.org/499774 | 00:03 |
|---|---|---|
| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: Catch attrbute error for other APIs https://review.openstack.org/499775 | 00:03 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-clientify baremetal create/delete https://review.openstack.org/500119 | 00:03 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-client-ify baremetal machine port list https://review.openstack.org/500120 | 00:03 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-client-ify machine patch operations https://review.openstack.org/500121 | 00:03 |
| *** lbragstad has joined #openstack-sdks | 00:06 | |
| *** gildub has joined #openstack-sdks | 00:07 | |
| *** slaweq has joined #openstack-sdks | 00:13 | |
| *** slaweq has quit IRC | 00:18 | |
| *** bobh has joined #openstack-sdks | 00:32 | |
| *** mgagne has joined #openstack-sdks | 00:34 | |
| *** salv-orl_ has quit IRC | 00:38 | |
| *** salv-orlando has joined #openstack-sdks | 00:38 | |
| *** salv-orlando has quit IRC | 00:43 | |
| *** sdague has quit IRC | 00:44 | |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Fix py35 and pypy tox env https://review.openstack.org/523383 | 01:14 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Cleanup objects that we create on behalf of images https://review.openstack.org/522974 | 01:14 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add method to cleanup autocreated image objects https://review.openstack.org/522975 | 01:14 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Complete move of baremetal machine tests https://review.openstack.org/522976 | 01:14 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Treat clouds.yaml with one cloud like envvars https://review.openstack.org/522977 | 01:14 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Protect against p_opt not having prompt attribute https://review.openstack.org/522978 | 01:14 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Update make_rest_client to work with version discovery https://review.openstack.org/522979 | 01:14 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Added nat_source flag for networks. https://review.openstack.org/522980 | 01:14 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Shift image tests from test_operator https://review.openstack.org/523258 | 01:14 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Switch baremetal nics/ports tests over https://review.openstack.org/523355 | 01:14 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud https://review.openstack.org/523259 | 01:14 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Remove get_legacy_client https://review.openstack.org/523483 | 01:14 |
| * mordred hopes everyone enjoys the current patch-bomb | 01:14 | |
| rods | wow | 01:14 |
| SamYaple | one patch a day mordred. thats my review rate for your patches. youre being throttled | 01:16 |
| *** salv-orlando has joined #openstack-sdks | 01:39 | |
| openstackgerrit | Hengqing Hu proposed openstack/python-openstacksdk master: Add pools attribute to load balancer heath monitor https://review.openstack.org/522962 | 01:40 |
| *** gildub has quit IRC | 01:41 | |
| *** salv-orlando has quit IRC | 01:44 | |
| *** gildub has joined #openstack-sdks | 01:52 | |
| *** ldl has joined #openstack-sdks | 01:54 | |
| *** purplerbot has quit IRC | 01:56 | |
| *** purplerbot has joined #openstack-sdks | 01:56 | |
| *** annp has joined #openstack-sdks | 01:59 | |
| TheJulia | SamYaple: What about for patches written by others? :) | 02:03 |
| SamYaple | TheJulia: dont subvert the spirit of my decleration with your hacker-think! | 02:04 |
| TheJulia | Muahahahahaha! | 02:04 |
| *** shu-mutou-AWAY has quit IRC | 02:09 | |
| *** salv-orlando has joined #openstack-sdks | 02:40 | |
| *** fzdarsky_ has joined #openstack-sdks | 02:40 | |
| *** bobh has quit IRC | 02:42 | |
| *** fzdarsky|afk has quit IRC | 02:42 | |
| *** dave-mccowan has quit IRC | 02:42 | |
| *** salv-orlando has quit IRC | 02:45 | |
| openstackgerrit | Jamie Lennox proposed openstack/keystoneauth master: Mark SAML loader properties as required https://review.openstack.org/523675 | 03:40 |
| *** gildub has quit IRC | 03:46 | |
| *** prg3 has quit IRC | 03:51 | |
| *** prg3 has joined #openstack-sdks | 03:51 | |
| openstackgerrit | Jamie Lennox proposed openstack/keystoneauth master: Mark SAML loader properties as required https://review.openstack.org/523675 | 03:58 |
| *** slaweq has joined #openstack-sdks | 04:15 | |
| *** d0ugal has quit IRC | 04:18 | |
| *** ykarel|away has joined #openstack-sdks | 04:18 | |
| *** slaweq has quit IRC | 04:19 | |
| *** ldl has quit IRC | 04:21 | |
| *** ykarel|away has quit IRC | 04:23 | |
| *** gkadam_ has joined #openstack-sdks | 04:23 | |
| *** bobh has joined #openstack-sdks | 04:25 | |
| *** d0ugal has joined #openstack-sdks | 04:25 | |
| *** bobh has quit IRC | 04:29 | |
| *** shu-mutou has joined #openstack-sdks | 04:29 | |
| *** markvoelker has quit IRC | 04:30 | |
| *** ldl has joined #openstack-sdks | 04:32 | |
| *** prg3 has quit IRC | 04:41 | |
| *** salv-orlando has joined #openstack-sdks | 04:42 | |
| *** gouthamr has quit IRC | 04:43 | |
| *** salv-orlando has quit IRC | 04:46 | |
| *** ldl has quit IRC | 04:47 | |
| *** ldl has joined #openstack-sdks | 04:48 | |
| *** reedip has quit IRC | 05:17 | |
| *** markvoelker has joined #openstack-sdks | 05:30 | |
| *** gouthamr has joined #openstack-sdks | 05:46 | |
| *** gouthamr has quit IRC | 06:04 | |
| *** Kvisle has joined #openstack-sdks | 06:07 | |
| *** alex_xu has quit IRC | 06:10 | |
| *** alex_xu has joined #openstack-sdks | 06:11 | |
| *** slaweq has joined #openstack-sdks | 06:17 | |
| *** salv-orlando has joined #openstack-sdks | 06:25 | |
| *** slaweq has quit IRC | 06:35 | |
| *** slaweq has joined #openstack-sdks | 06:35 | |
| *** slaweq has quit IRC | 06:40 | |
| *** gkadam_ has quit IRC | 06:58 | |
| *** reedip has joined #openstack-sdks | 08:01 | |
| *** jpich has joined #openstack-sdks | 08:17 | |
| *** gkadam_ has joined #openstack-sdks | 08:22 | |
| *** e0ne has joined #openstack-sdks | 08:44 | |
| *** ralonsoh has joined #openstack-sdks | 08:49 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/keystoneauth master: Updated from global requirements https://review.openstack.org/523736 | 08:58 |
| *** fzdarsky_ is now known as fzdarsky | 09:03 | |
| *** ioggstream has joined #openstack-sdks | 09:11 | |
| *** hoangcx has quit IRC | 09:24 | |
| *** dtantsur|afk is now known as dtantsur | 09:26 | |
| *** fzdarsky has quit IRC | 09:43 | |
| *** fzdarsky has joined #openstack-sdks | 09:48 | |
| *** fzdarsky has quit IRC | 09:54 | |
| *** slaweq has joined #openstack-sdks | 09:55 | |
| *** slaweq has quit IRC | 10:00 | |
| *** annp has quit IRC | 10:02 | |
| *** ldl has quit IRC | 10:05 | |
| *** fzdarsky has joined #openstack-sdks | 10:10 | |
| *** ldl has joined #openstack-sdks | 10:11 | |
| openstackgerrit | Édouard Thuleau proposed openstack/python-openstacksdk master: Set empty Tag list if Network Tag API extension not supported https://review.openstack.org/523834 | 10:11 |
| *** shu-mutou is now known as shu-mutou-AWAY | 10:21 | |
| *** ldl has quit IRC | 10:26 | |
| *** ldl has joined #openstack-sdks | 10:28 | |
| *** ldl has quit IRC | 10:43 | |
| *** ldl has joined #openstack-sdks | 10:48 | |
| *** slaweq has joined #openstack-sdks | 11:10 | |
| *** LindaWang1 has joined #openstack-sdks | 11:12 | |
| *** slaweq has quit IRC | 11:19 | |
| *** slaweq has joined #openstack-sdks | 11:19 | |
| *** slaweq has quit IRC | 11:24 | |
| *** sdague has joined #openstack-sdks | 11:27 | |
| *** jkilpatr has quit IRC | 11:46 | |
| *** slaweq has joined #openstack-sdks | 12:10 | |
| *** ScrewTSW has joined #openstack-sdks | 12:10 | |
| *** jkilpatr has joined #openstack-sdks | 12:17 | |
| *** bobh has joined #openstack-sdks | 12:20 | |
| *** slaweq has quit IRC | 12:21 | |
| frickler | mordred: how about doing a shade release fixing https://storyboard.openstack.org/#!/story/2001265 ? | 12:41 |
| *** dtantsur is now known as dtantsur|brb | 12:47 | |
| *** bobh has quit IRC | 12:50 | |
| *** bobh has joined #openstack-sdks | 12:51 | |
| *** thrash|g0ne is now known as thrash | 12:59 | |
| *** bobh has quit IRC | 13:11 | |
| *** chenybd_ has joined #openstack-sdks | 13:15 | |
| *** ldl has quit IRC | 13:17 | |
| *** markvoelker has quit IRC | 13:19 | |
| *** markvoelker has joined #openstack-sdks | 13:19 | |
| *** mmedvede has joined #openstack-sdks | 13:21 | |
| *** LindaWang1 has quit IRC | 13:25 | |
| *** edmondsw has joined #openstack-sdks | 13:32 | |
| *** rcarrillocruz has joined #openstack-sdks | 13:32 | |
| *** dave-mccowan has joined #openstack-sdks | 13:36 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade master: WIP: os_router idempotency check https://review.openstack.org/523873 | 13:38 |
| Shrews | mordred: wow, your changes are not happy moving through to gate | 13:41 |
| Shrews | frickler: i'm looking at that now | 13:41 |
| Shrews | 523873 trying to confirm what's being seen | 13:41 |
| *** openstackstatus has quit IRC | 13:43 | |
| *** openstack has joined #openstack-sdks | 13:46 | |
| *** ChanServ sets mode: +o openstack | 13:46 | |
| frickler | Shrews: the bug is triggered only when Neutron is configured with HA (or maybe DVR) and the router also uses that, so I'm not sure you will see it in gate easily | 13:50 |
| Shrews | frickler: yeah, just noticed that comment. trying to find the api for that | 13:51 |
| Shrews | frickler: not being familiar with that extension, was https://review.openstack.org/482153 totally the wrong thing to do then? | 13:54 |
| Shrews | mordred: i'm wondering what problem 482153 was solving and if it should just be reverted | 13:54 |
| Shrews | the commit message says "L3 HA ports do not correspond to a router interface" but obviously that's not correct | 13:56 |
| Shrews | frickler: do you know if that is something that can be enabled in devstack? | 13:58 |
| frickler | Shrews: iiuc 482153 solved the issue not to list the L3 HA ports that are more or less internal to the Neutron HA implementation | 14:01 |
| frickler | Shrews: and you can sure enable things in devstack, neutron is running various scenarios with DVR, not sure about HA | 14:02 |
| Shrews | oh! seems a fix is already committed | 14:03 |
| Shrews | ok, i thought we needed a fix AND release. but just a release is needed then | 14:03 |
| frickler | Shrews: ah, yes, I did a fix for that, sorry, I assumed that you had seen it. I thought you were trying to verify the fix | 14:03 |
| Shrews | frickler: my fault, i misunderstood. yeah, only mordred can do the release | 14:04 |
| frickler | Shrews: adding your check above and maybe a devstack scenario that runs Neutron in HA mode still seem useful to me | 14:05 |
| Shrews | would be nice if we could find out if devstack supported this though | 14:05 |
| Shrews | frickler: yeah. i don't know the devstack magic for that :( | 14:06 |
| frickler | Shrews: I'll put it on my list, hopefully sometime this week | 14:07 |
| *** cdent has joined #openstack-sdks | 14:10 | |
| cdent | sdague: how did your microversions talk go? was it at apistrat? | 14:15 |
| sdague | it went pretty well, it was at apistrat. It was the end of the last day, so I got a couple of questions on it, but not a lot of space for broader discussion | 14:16 |
| cdent | did anyone have an epiphany? | 14:18 |
| *** LindaWang1 has joined #openstack-sdks | 14:20 | |
| *** dtantsur|brb is now known as dtantsur | 14:24 | |
| mordred | Shrews, frickler: releases are actually run by the release team now, so you can actually submit a patch requesting one without needing me ... but, I can do that this morning - I think we're overdue for a release anyway | 14:38 |
| Shrews | mordred: oh right. is that process described anywhere? | 14:42 |
| *** prg3 has joined #openstack-sdks | 14:46 | |
| mordred | Shrews: it's pretty much making a patch to openstack/releases ... you'll find a file, deliverables/queens/shade.yaml - that has entries you can copy-pasta | 14:46 |
| mordred | Shrews: you need a git sha - so doing a pull on master is usuallya good first step | 14:46 |
| *** openstack has quit IRC | 14:46 | |
| *** openstack has joined #openstack-sdks | 14:52 | |
| *** ChanServ sets mode: +o openstack | 14:52 | |
| celebdor1 | Does anybody know if shade support heat tags? I couldn't find it with simple grepping | 14:52 |
| Shrews | mordred: ah, the README there has the process... but the PTL or release liaison has to submit the request | 14:52 |
| mordred | celebdor1: I don't think it does, no - but adding it would be a great idea | 14:53 |
| celebdor1 | mordred: It is indeed | 14:54 |
| celebdor1 | I need it for queens though :P | 14:54 |
| mordred | celebdor1: :) it's pretty quick to add things to shade and get them released - I don't think getting it in for queens should be a problem | 14:55 |
| *** d0ugal has quit IRC | 14:56 | |
| mordred | Shrews: remote: https://review.openstack.org/523890 Release shade 1.25.0 | 14:57 |
| celebdor1 | mordred: thanks for the info | 14:58 |
| celebdor1 | :-) | 14:58 |
| mordred | Shrews: also, I have learned this morning that there are a few openstack projects, octavia-dashboard, senlin-dashboard, tricircle, bilean, senlin, heat and a couple others - that an sdk release is going to break | 14:59 |
| *** cdent has quit IRC | 14:59 | |
| mordred | Shrews: so I think I need to put 'get patches landed to each of those projects' to the list of things we do before we cut an sdk release with the new changes in it | 15:00 |
| Shrews | mordred: that's fun | 15:00 |
| *** salv-orl_ has joined #openstack-sdks | 15:02 | |
| mordred | Shrews: oh - actually, no - we kept compat for the thing that would break them | 15:02 |
| mordred | so we can release, then change the interface they all use | 15:02 |
| *** salv-orlando has quit IRC | 15:03 | |
| *** d0ugal has joined #openstack-sdks | 15:04 | |
| *** marst has joined #openstack-sdks | 15:06 | |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Sort image update results before comparing https://review.openstack.org/523893 | 15:11 |
| mordred | Shrews: that ^^ should fix the unittest failures that are happening | 15:12 |
| mordred | Shrews: I'm guessing the jsonpatch release on november 25 is to blame, fwiw | 15:13 |
| *** bobh has joined #openstack-sdks | 15:14 | |
| *** thrash is now known as thrash|biab | 15:15 | |
| *** d0ugal_ has joined #openstack-sdks | 15:17 | |
| *** d0ugal has quit IRC | 15:20 | |
| *** gouthamr has joined #openstack-sdks | 15:22 | |
| *** cdent has joined #openstack-sdks | 15:32 | |
| *** d0ugal has joined #openstack-sdks | 15:35 | |
| *** d0ugal_ has quit IRC | 15:37 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Avoid tox_install.sh for constraints support https://review.openstack.org/508061 | 15:48 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Fix basepython setting in tox.ini https://review.openstack.org/523915 | 15:48 |
| *** chenybd_ has quit IRC | 15:49 | |
| *** chenybd_ has joined #openstack-sdks | 15:49 | |
| *** chenybd_ has quit IRC | 15:54 | |
| *** LindaWang1 has quit IRC | 15:56 | |
| *** ScrewTSW has quit IRC | 15:56 | |
| *** e0ne has quit IRC | 15:59 | |
| *** edmondsw_ has joined #openstack-sdks | 16:00 | |
| *** thrash|biab is now known as thrash | 16:02 | |
| *** edmondsw has quit IRC | 16:03 | |
| *** edmondsw_ is now known as edmondsw | 16:03 | |
| mordred | incoming patch bomb ... sorry for the rebase ... it was needed to get in the jsonpatch sorting fix | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add ability to work in other auth contexts https://review.openstack.org/522970 | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add helper property to get the current user id https://review.openstack.org/522971 | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Remove reference to context-managers from release note https://review.openstack.org/522972 | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Document current_user_id in a release note https://review.openstack.org/522973 | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Fix py35 and pypy tox env https://review.openstack.org/523383 | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Cleanup objects that we create on behalf of images https://review.openstack.org/522974 | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add method to cleanup autocreated image objects https://review.openstack.org/522975 | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Complete move of baremetal machine tests https://review.openstack.org/522976 | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Treat clouds.yaml with one cloud like envvars https://review.openstack.org/522977 | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Protect against p_opt not having prompt attribute https://review.openstack.org/522978 | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Update make_rest_client to work with version discovery https://review.openstack.org/522979 | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Added nat_source flag for networks. https://review.openstack.org/522980 | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Shift image tests from test_operator https://review.openstack.org/523258 | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Switch baremetal nics/ports tests over https://review.openstack.org/523355 | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud https://review.openstack.org/523259 | 16:16 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Remove get_legacy_client https://review.openstack.org/523483 | 16:16 |
| mordred | Shrews: those should, you know, pass this time | 16:16 |
| sdague | cdent: I don't know that there were any real epiphanies from folks. I think a new interesting spin on it was with graphql, someone asked if that solved the problem all by itself | 16:17 |
| sdague | which, it solves a related problem in theory, though in practice, if your applications don't consume the API directly, but through a 3rd party SDK, as we've seen, graphql probably just always asks for everything, then you are back to REST resources. | 16:18 |
| cdent | I’ve read elsewhere of graphql being a way out of a versioning problem, if you start there | 16:19 |
| cdent | but if you’re not already there… | 16:19 |
| sdague | yeh | 16:20 |
| sdague | also, I think graphql seems to work best when the front end has intimate knowledge of the costs of asking for things on the backend. So as a layer between the github ui and services, it's awesome | 16:23 |
| mordred | sdague: yah - or, as soon as they finish adding all of the things to the graphql layer, it'll help zuul's github driver make fewer API calls and thus avoid rate-limit issues | 16:27 |
| sdague | mordred: sure | 16:28 |
| mordred | sdague: but I agree, in general, with the assessment about intimate knowledge - in this example, the zuul thing is a cost optimization - it's certainly not an easier/better way to write the first version of the thing | 16:28 |
| sdague | mordred: and, you are hitting the API raw, because you need that level of granularity | 16:29 |
| mordred | yup | 16:29 |
| sdague | but, if you went through someone's random SDK | 16:29 |
| sdague | you are unlikely to get those optimizations | 16:29 |
| mordred | yup. exactly | 16:29 |
| *** calebb has joined #openstack-sdks | 16:30 | |
| mordred | that said - I'd kind of LOVE a graphql api option for openstack service ... for similar reasons | 16:30 |
| *** melwitt has joined #openstack-sdks | 16:30 | |
| mordred | finding the port that's ona subnet on a network is basically doing three full list calls and doing a client-side in-memory join | 16:30 |
| sdague | mordred: honestly, building a graphql porcelin for all of openstack would be a totally legit thing to do | 16:31 |
| sdague | it would be different enough that I don't think it would be confusing between the rest resources at the point sevices and what it returned | 16:31 |
| mordred | sdague: yah. although having a per-service graphql as well that the porcelin could use for push-downs for efficiency would also be nice ... but yes, a graphql openstack porcelain that could do "I want to create a floating IP on a port from the server that is on this subnet of this network"... | 16:32 |
| mordred | would be awesome to do as one call | 16:33 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Fix basepython setting in tox.ini https://review.openstack.org/523915 | 16:41 |
| openstackgerrit | Merged openstack/python-openstackclient master: Set correct designate endpoint in docs https://review.openstack.org/523567 | 16:56 |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | 17:17 |
| *** thrash is now known as thrash|biab | 17:25 | |
| *** gkadam_ has quit IRC | 17:27 | |
| *** jpich has quit IRC | 17:29 | |
| *** ioggstream has quit IRC | 17:34 | |
| *** ralonsoh has quit IRC | 17:36 | |
| *** e0ne has joined #openstack-sdks | 17:44 | |
| *** celebdor1 has quit IRC | 17:46 | |
| *** _alastor_ has quit IRC | 17:53 | |
| *** _alastor_ has joined #openstack-sdks | 17:55 | |
| *** salv-orl_ has quit IRC | 17:59 | |
| *** salv-orlando has joined #openstack-sdks | 17:59 | |
| *** salv-orlando has quit IRC | 17:59 | |
| *** salv-orlando has joined #openstack-sdks | 18:03 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Fix basepython setting in tox.ini https://review.openstack.org/523915 | 18:07 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Avoid tox_install.sh for constraints support https://review.openstack.org/508061 | 18:07 |
| Kvisle | Hi! I can see that https://review.openstack.org/#/c/522004/ looks good with review, but it doesn't seem to be merged yet. Do I need to do anything? | 18:08 |
| Shrews | Kvisle: No. Just waiting for mordred to have a chance to look it over first. | 18:10 |
| Kvisle | Ah =) | 18:10 |
| *** dtantsur is now known as dtantsur|afk | 18:10 | |
| *** fzdarsky is now known as fzdarsky|afk | 18:14 | |
| mordred | Kvisle: +A - thanks! | 18:14 |
| Kvisle | \o/ | 18:15 |
| *** _alastor_ has quit IRC | 18:16 | |
| Shrews | mordred: that's an interesting error you got there on that image sorting fix. rsync seems to think there's a MITM attack | 18:20 |
| mordred | Shrews: yay! | 18:21 |
| Shrews | http://logs.openstack.org/93/523893/1/gate/openstack-tox-py27/8ddfac1/ara/result/38e984ce-db99-4dac-8399-5e4002a0025b/ | 18:21 |
| mordred | Shrews: that sounds like a thing we should bring up over in infra land | 18:25 |
| Shrews | k | 18:25 |
| thingee | mordred: I'm still interested on working on oaktree. I read through the brain dump email and appreciate you writing that up, finally ;) | 18:35 |
| *** gkadam_ has joined #openstack-sdks | 18:35 | |
| *** salv-orlando has quit IRC | 18:36 | |
| *** salv-orlando has joined #openstack-sdks | 18:36 | |
| thingee | mordred: I would like to keep my core seat please. | 18:37 |
| cdent | I’d like to work on oaktree too, but I’m struggling to think I’ll have the time :( | 18:40 |
| *** salv-orlando has quit IRC | 18:41 | |
| openstackgerrit | Merged openstack-infra/shade master: Implement availability_zone_hints for networks and routers. https://review.openstack.org/522004 | 18:51 |
| Shrews | just for the record, if oaktree attempts to handle more than just openstack clouds (which is already insanely complex), it will be doomed to failure. <EOM> | 19:06 |
| cdent | Shrews++ | 19:35 |
| *** salv-orlando has joined #openstack-sdks | 19:37 | |
| *** salv-orlando has quit IRC | 19:42 | |
| SamYaple | Shrews: all we gotta do is abstract a little bit more. easy peasy | 19:43 |
| openstackgerrit | Harald Jensås proposed openstack/python-openstackclient master: Allow setting network-segment on subnet update https://review.openstack.org/523974 | 19:44 |
| thingee | Shrews: +1 | 19:48 |
| *** thrash|biab is now known as thrash | 19:53 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | 20:18 |
| *** salv-orlando has joined #openstack-sdks | 20:27 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | 20:32 |
| openstackgerrit | Harald Jensås proposed openstack/python-openstackclient master: Allow setting network-segment on subnet update https://review.openstack.org/523974 | 20:55 |
| *** salv-orlando has quit IRC | 20:58 | |
| *** salv-orlando has joined #openstack-sdks | 21:04 | |
| mordred | thingee: woot. sorry it took me so long to get that down on paper in any useful way | 21:04 |
| mordred | SamYaple: agree | 21:05 |
| mordred | cdent: I'm going to give you core for now, just because - if it doesn't work out, no biggie | 21:05 |
| cdent | mordred: I think I can handle that responsibly. Thanks. | 21:06 |
| *** d0ugal has quit IRC | 21:07 | |
| openstackgerrit | Merged openstack/python-openstacksdk master: Sort image update results before comparing https://review.openstack.org/523893 | 21:09 |
| openstackgerrit | Merged openstack/python-openstacksdk master: Add ability to work in other auth contexts https://review.openstack.org/522970 | 21:09 |
| openstackgerrit | Merged openstack/python-openstacksdk master: Add helper property to get the current user id https://review.openstack.org/522971 | 21:09 |
| openstackgerrit | Merged openstack/python-openstacksdk master: Remove reference to context-managers from release note https://review.openstack.org/522972 | 21:09 |
| openstackgerrit | Merged openstack/python-openstacksdk master: Document current_user_id in a release note https://review.openstack.org/522973 | 21:09 |
| SamYaple | mordred: ill try to get on the rest of the reviews after i land in denver | 21:10 |
| SamYaple | keep them small and ill keep reviewing :P | 21:11 |
| mordred | SamYaple, Shrews: I think we need a tag or a topic or something to mark "this already landed in shade or os-client-config and is just a cherry-pick to get them up to date" vs. "this is actual code that needs actual review other than 'do the tests pass'" | 21:18 |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | 21:19 |
| dtroyer | mordred: at what point would you want to reverse that? ie, SDK is source of truth and shade/occ are 'backports'? when they become shims? | 21:20 |
| openstackgerrit | Monty Taylor proposed openstack/cliff master: Use in-tree cliffdemo app for docs build https://review.openstack.org/515016 | 21:22 |
| openstackgerrit | Monty Taylor proposed openstack/cliff master: Move doc requirements to doc/requirements.txt https://review.openstack.org/522369 | 21:22 |
| openstackgerrit | Monty Taylor proposed openstack/cliff master: Remove tox_install.sh and just pass -c in tox https://review.openstack.org/523984 | 21:22 |
| openstackgerrit | Monty Taylor proposed openstack/cliff master: DNM testing docs and releasenotes jobs https://review.openstack.org/522370 | 21:22 |
| mordred | dtroyer: yah- when they become shims | 21:25 |
| mordred | dtroyer: and hopefully at that point we don't have to do any cherry-picking/backports *fingers crossed* | 21:25 |
| *** tonythomas has joined #openstack-sdks | 21:27 | |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-client-ify many baremetal calls https://review.openstack.org/523986 | 21:35 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: Fix operator cloud get_nic_by_mac https://review.openstack.org/523987 | 21:35 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-client-ify fixed method get_nic_by_mac https://review.openstack.org/523988 | 21:35 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: Remove python-ironicclient https://review.openstack.org/523989 | 21:35 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Avoid tox_install.sh for constraints support https://review.openstack.org/508061 | 21:37 |
| Shrews | mordred: maybe. i just compared git logs to discover that, but if it would help others, fine with me | 21:37 |
| Shrews | mordred: there is a git option to leave a "cherry picked" comment, iirc | 21:39 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Remove use of tox-siblings role https://review.openstack.org/523993 | 21:39 |
| Shrews | git cherry-pick -x ... or similar | 21:39 |
| Shrews | i thought git-review carried it over, too | 21:39 |
| TheJulia | -X | 21:40 |
| TheJulia | -x doesn't create the comment | 21:40 |
| Shrews | well boo | 21:41 |
| Shrews | oh, but -X does? | 21:41 |
| TheJulia | c'est la vie | 21:41 |
| TheJulia | yes, it does | 21:41 |
| Shrews | whoomp, there it is | 21:42 |
| mordred | Shrews: neat! | 21:44 |
| mordred | oh- also - wow ... TheJulia do I see a Remove python-ironicclient patch up there??? | 21:44 |
| TheJulia | mordred: is it too early to consider it a birthday gift? | 21:44 |
| * mordred hands TheJulia a pastry full of other pastries that are all full of chickens that are stuffed with whiskey | 21:45 | |
| TheJulia | mmm whisky | 21:45 |
| TheJulia | Seriously though, Yes, I hope it is right. Unit tests pass locally, which is a good sign. I fired up a dnm test on bifrost to pull it in and see if it breaks. I also moved my wip tag accordingly. | 21:46 |
| Shrews | we should, ya know, merge that and stuff | 21:48 |
| *** kmalloc has joined #openstack-sdks | 21:49 | |
| TheJulia | Shrews: yeah, for some reason my removal of my wip tag earlier today from the patch chain didn't take... :\ | 21:50 |
| mordred | TheJulia, Shrews: +2 with comment on https://review.openstack.org/#/c/499774/3 - comment applies to the previous patch too (which is already +A'd) - I think it's totally fine for us to deal with in a followup | 21:50 |
| *** edmondsw has quit IRC | 21:51 | |
| mordred | TheJulia: also, I think there is a rebase oops in https://review.openstack.org/#/c/500119/3 | 21:52 |
| openstackgerrit | Merged openstack-infra/shade master: Fix basepython setting in tox.ini https://review.openstack.org/523915 | 21:52 |
| TheJulia | mordred: replied, and agreed, I'll rip that out in the morning. | 21:53 |
| Shrews | what's funny is, that it still passed tests | 21:53 |
| *** edmondsw has joined #openstack-sdks | 21:53 | |
| Shrews | s/funny/sad/ ?? | 21:53 |
| TheJulia | wow... fun rebase failure :( | 21:54 |
| * TheJulia pulls the rev down to fix and then cry | 21:54 | |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-clientify baremetal create/delete https://review.openstack.org/500119 | 21:57 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-client-ify baremetal machine port list https://review.openstack.org/500120 | 21:57 |
| mordred | TheJulia: AHA - normalize_machines only removes novaclient noise - I was wondering how the old method worked with normalize_machines(MachienPortList()) :) | 21:57 |
| * TheJulia sighs | 21:57 | |
| TheJulia | mordred: my guess, cp -a python-novaclient python-ironicclient | 21:58 |
| *** edmondsw has quit IRC | 21:58 | |
| mordred | TheJulia: also, left a note for later in https://review.openstack.org/#/c/500119 - I don't think we should do anything about it today or tomorrow | 22:01 |
| mordred | TheJulia: thank you for leaving the comment there though - it was very helpful :) | 22:01 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade master: De-client-ify machine patch operations https://review.openstack.org/500121 | 22:01 |
| mordred | TheJulia, Shrews: stack up to the WIP is all +2 from me. | 22:02 |
| adriant | Out of curiosity, for the openstackSDK, if "param profile: DEPRECATED" what am I meant to use in code programatically to define what region I need to talk to? The docs for openstackSDK do not mention the profile deprecation, and as much as I dislike the profile mechanism, it made sense (although the ability/need to define a different region for mul | 22:02 |
| adriant | tiple services in the same connection object was odd). | 22:02 |
| *** cdent has quit IRC | 22:03 | |
| mordred | TheJulia: THANK YOU! I think as soon as that last patch lands we should cut another release, since removing that depend will make life **WAY** easier on packagers | 22:03 |
| mordred | adriant: that'sa great question - and we should clearly make a document about that | 22:03 |
| mordred | adriant: one second - lemme link you to something | 22:03 |
| TheJulia | mordred: awesome, I'm going to disappear shortly for at least the next few hours | 22:04 |
| TheJulia | mordred: I just feel bad I'm just now getting to it.... | 22:04 |
| * TheJulia swears it is not stress hacking on code... but it really is | 22:04 | |
| adriant | mordred: I'm trying to switch a bunch of tools to openstackSDK, but none of them rely on clouds.yaml (it doesn't make sense to) and I need to pass in config stuff. From keystoneauth I can get my session, find, but region I've always done with profile. | 22:05 |
| mordred | adriant: well, this isn't nearly as useful a link as I'd hoped, but ... | 22:05 |
| mordred | http://git.openstack.org/cgit/openstack/python-openstackclient/tree/openstackclient/network/client.py#n39 | 22:05 |
| adriant | mordred: that still uses profile :P | 22:05 |
| mordred | adriant: yah- the answer for you is gonna be to make a CloudConfig object basically just like you've been making a profile | 22:05 |
| *** jkilpatr has quit IRC | 22:05 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Release note cleanup https://review.openstack.org/524004 | 22:06 |
| mordred | adriant: lemme make a quick patch to something else that shows the ACTUAL answer | 22:06 |
| mordred | adriant: and also a doc patch :) | 22:06 |
| adriant | mordred: k, I had a feeling i'd end up having to make a config object :P | 22:06 |
| mordred | adriant: out of curiosity - do you want your use to explicitly *not* support clouds.yaml - or more you don't have any clouds.yaml and don't care | 22:07 |
| mordred | (example patch is different depending on which of those is true - I should probably make both examples) | 22:07 |
| adriant | mordred: the thing I'm working on in question is a django site/service that has it's own config file. So clouds.yaml doesn't make sense. | 22:08 |
| mordred | adriant: nod. cool - example coming | 22:08 |
| adriant | same again with a different API service I've got | 22:08 |
| adriant | then the rest of my tools are built on top of the openstackclient, so they natively just use clouds.yaml | 22:08 |
| adriant | I find that if I'm writing any scripts for openstack, I just make them an openstackclient plugin and forget all the tiresome stuff and focus on functionality | 22:09 |
| adriant | mordred: and thanks :) | 22:09 |
| *** jkilpatr has joined #openstack-sdks | 22:29 | |
| mordred | adriant: https://review.openstack.org/524011 Update openstacksdk construction to be forward compatible | 22:31 |
| mordred | adriant: I'm going to extract that into a proper doc - but that should have all the pieces you need (and more) | 22:32 |
| *** edmondsw has joined #openstack-sdks | 22:32 | |
| openstackgerrit | Merged openstack-infra/shade master: De-client-ify baremetal node_set_provision_state https://review.openstack.org/499773 | 22:34 |
| *** edmondsw has quit IRC | 22:37 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Release note cleanup https://review.openstack.org/524004 | 22:38 |
| adriant | mordred: ty, and love the comment | 22:42 |
| mordred | adriant: :) | 22:46 |
| adriant | mordred: I have to admin, that's much nicer than the profile stuff. The profile stuff was my main reason for not liking the SDK since it just seemed overcomplicated | 22:48 |
| adriant | admit* | 22:48 |
| *** edmondsw has joined #openstack-sdks | 22:48 | |
| adriant | like sure, you could have swift in one region, nova in another, and cinder in a third, all in the same connection object. the question though is why!? | 22:49 |
| *** e0ne has quit IRC | 22:49 | |
| *** e0ne has joined #openstack-sdks | 22:50 | |
| *** e0ne has quit IRC | 22:51 | |
| *** e0ne has joined #openstack-sdks | 22:51 | |
| *** e0ne has quit IRC | 22:51 | |
| *** e0ne has joined #openstack-sdks | 22:52 | |
| *** e0ne has quit IRC | 22:52 | |
| *** e0ne has joined #openstack-sdks | 22:53 | |
| *** e0ne has quit IRC | 22:53 | |
| *** e0ne has joined #openstack-sdks | 22:54 | |
| *** e0ne has quit IRC | 22:55 | |
| *** e0ne has joined #openstack-sdks | 22:55 | |
| *** e0ne has quit IRC | 22:55 | |
| *** prg3 has quit IRC | 23:03 | |
| *** d0ugal has joined #openstack-sdks | 23:04 | |
| adriant | mordred: and the thing you were talking about at the summit was "get_session_client" right? So in the context of openstackSDK you can just do raw API calls if the code doesn't support the feature you. | 23:05 |
| adriant | mordred: but that appears to be on the config class, rather than the connection/service class. Which means I need to keep the config class around if I need to call APIs directly, or is there a way I can get at that on the connection/service class? | 23:06 |
| adriant | I was imagining it as "resp = conn.object_store._client.get("/thing") | 23:07 |
| adriant | I'm sure the adapter is in there somewhere, and i can probably find it by trawling the code some more, just was curious if there was a nice way to do access it. | 23:09 |
| *** salv-orlando has quit IRC | 23:22 | |
| *** salv-orlando has joined #openstack-sdks | 23:22 | |
| adriant | mordred: also, a passthrough or shortcut for "OpenStackConfig(load_yaml_config=False).get_one_cloud" might not be a bad idea. Since we shouldn't assume clouds.yaml is the default. It's only the default from a cli/tools perspective, and we really should make use the SDK in services and such much easier. Or at least initialising it. | 23:25 |
| *** bobh has quit IRC | 23:25 | |
| *** bobh has joined #openstack-sdks | 23:26 | |
| *** salv-orlando has quit IRC | 23:27 | |
| *** bobh has quit IRC | 23:30 | |
| openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add some docs about not using Profile https://review.openstack.org/524021 | 23:31 |
| mordred | adriant: sorry - was writing that ^^ | 23:32 |
| mordred | adriant: SO - in the master branch currently you don't need do do a get_session_client at all - once you have a connectoin, each service's Proxy object is actually also an Adapter | 23:32 |
| mordred | adriant: so you can do conn.compute.get('/foo') | 23:32 |
| mordred | adriant: also, fwiw, the service proxy objects all all by service_type as per the service type registry, and there's also one for any known alias ... although still todo is to make sure old aliases like volumev2 actually construct appropriately versioned adapters | 23:35 |
| mordred | adriant: AND - yes, definitely shortcuts such as what you say - there are several littered about, but we need to rationalize it all since the codebase merges | 23:35 |
| mordred | adriant: so, like, in master, you can do "import openstack ; openstack.connect(**parameters_you_would_have_passed_to_get_one_cloud)" | 23:36 |
| SamYaple | TheJulia: w00t! congrats on getting whiskey chicken. seems like a fair trade for that work (now you just have to hold monty to it) | 23:37 |
| mordred | adriant: maybe we should make 2 factory functions - one for no-clouds.yaml and one for yes-clouds.yaml ... cause for utility scripts I gotta tell you I would never ever use not clouds.yaml and the extra typing to turn on loading the files would drive me batty | 23:39 |
| mordred | adriant: *waves hands* ... fundamentally, totally agree with the things you're saying, we can work out what the shortcut things do pretty easily so we're all happy :) | 23:39 |
| *** marst has quit IRC | 23:49 | |
| *** bobh has joined #openstack-sdks | 23:53 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!