Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-03-04
19:45:58 mordred yah
19:46:00 umbSublime testing now
19:46:09 mordred and we aim to support older clouds too - so it's fair for us to deal with that
19:46:35 umbSublime I put the endpoint I get for `endpoint list` as a value right ?
19:46:43 mordred yeah
19:47:55 umbSublime \o/ print(conn.placement) -> <openstack.proxy.Proxy object at 0x7ffbdaf5a5c0>
19:47:58 mordred WOOT
19:48:17 mordred cool. I can work up a test case that has a broken old placement discovery dict and then figure out how to fix it :)
19:48:22 mordred thanks for helping track that down
19:48:27 umbSublime Man I'm going to get rid of so much hacky code today ^^
19:48:32 mordred \o/
19:48:40 mordred my main goal in life is facilitating the deletion of code
19:48:49 umbSublime thanks a bunch !!!
19:48:55 umbSublime hahaha it's working XD
20:05:45 openstackgerrit Merged openstack/openstacksdk master: Return uuid alias for coe_cluster in non strict mode https://review.opendev.org/710823
20:23:52 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Handle old status-less placement service https://review.opendev.org/711328
20:25:03 mordred umbSublime: ^^ that produces the same error - so there's that at leat
20:25:05 mordred least
20:28:40 umbSublime :)
20:29:29 umbSublime Do you want me to file a bug for this. I'm not sure if the fix has to be implemented in keystoneauth1 or the SDK directly
20:32:40 mordred umbSublime: nah - I'm working on it
20:33:03 mordred umbSublime: although I feel lke I'm going to need a shower when I'm done :)
20:33:04 umbSublime <3
21:04:27 umbSublime Say I wanted (one day) to try and tackle some low hanging fruit bugs on the SDK where are issues tracked. I don't see much stuff related to sdk in launchpad
21:06:28 mordred we're in storyboard ... https://storyboard.openstack.org/#!/project/openstack/openstacksdk
21:06:40 mordred I'm pretty bad about using it myself though (hides)
21:09:50 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Handle old status-less placement service https://review.opendev.org/711328
21:10:02 mordred umbSublime: and there's the workaround :)
21:10:40 mordred Shrews: I feel like it's been too long since you've had to review patches from me touching version discovery in the guts of SDK ^^
21:10:58 mordred efried: ^^ same story
21:13:01 efried mordred: can you link to the commit that corrected the missing status field in placement?
21:19:18 mordred efried: uh ... I was sort of hoping you just magically new ... let me go look
21:19:40 efried mordred: I could have gone and looked myself. <lazy>
21:19:55 efried I probably wrote that patch. Or at least reviewed it.
21:20:20 efried But I'm way behind memorizing the gerrit numbers and commit hashes of all the patches I've been involved in.
21:21:28 mordred efried: https://review.opendev.org/#/c/575117/
21:21:45 mordred efried: you want me to put a referene to that in the commit message?
21:21:54 efried mordred: yes please, but also one other comment...
21:22:57 efried mordred: ...posted
21:24:24 efried JSON doesn't allow comments???
21:24:56 efried I'll be back to +2 that in ~.5h
21:24:58 Shrews mordred: oh, i can't wait. really. no, seriously.
21:24:58 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Handle old status-less placement service https://review.opendev.org/711328
21:25:20 mordred efried: nope. that's one of teh nice things about yaml
21:25:32 umbSublime Thanks mordred !
21:25:40 mordred Shrews: well - no need to wait - there you are, all nice and ready for you
21:25:42 efried mordred: +2
21:25:44 mordred \o/
21:26:09 mordred umbSublime: thak you - that was very helpful debugging info - and I imagine you won't be the only one to hit that
21:31:48 umbSublime That was the easy part ;)
21:36:25 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Add bindep file https://review.opendev.org/711055
21:36:25 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Build utility image for using osc https://review.opendev.org/711246
22:04:25 openstackgerrit Sam Morrison proposed openstack/python-openstackclient master: Always display direction for security group rules https://review.opendev.org/709017
22:05:19 openstackgerrit Sam Morrison proposed openstack/python-openstackclient master: Always display direction for security group rules https://review.opendev.org/709017
22:22:03 openstackgerrit Merged openstack/ansible-collections-openstack master: Fix H236 and remove exclusion https://review.opendev.org/698066
22:27:00 openstackgerrit Merged openstack/ansible-collections-openstack master: Remove from __future__ lines https://review.opendev.org/698067
22:27:01 openstackgerrit Merged openstack/ansible-collections-openstack master: Test with and assert support for python3 https://review.opendev.org/710641
22:57:07 openstackgerrit Merged openstack/openstacksdk master: Fix: Set image name correctly if filename is not passed https://review.opendev.org/710368
23:19:52 openstackgerrit Merged openstack/openstacksdk stable/train: Handle HTTP errors in add/remove router interface calls https://review.opendev.org/710994
23:19:56 openstackgerrit Merged openstack/openstacksdk stable/train: Fix error handling on add/remove router iface calls https://review.opendev.org/710995
#openstack-sdks - 2020-03-05
00:20:16 openstackgerrit Merged openstack/openstacksdk master: Handle old status-less placement service https://review.opendev.org/711328
10:26:19 dtantsur mordred: yep, as far as I remember, futurist extends stdlib
11:43:29 openstackgerrit Merged openstack/openstacksdk master: Add port property: ip_allocation https://review.opendev.org/711237
12:06:01 mgoddard I think ansible 2.9.6 has broken openstack modules
12:06:12 mgoddard we're seeing this everywhere in kolla:
12:06:15 mgoddard https://b317bafa0db642d61e9b-babe6b68bec526aecbe80deed799da2b.ssl.cf2.rackcdn.com/711295/3/check/kolla-ansible-centos-source/b02cb5e/primary/logs/ansible/deploy
12:06:18 mgoddard py2 and py3
13:30:38 openstackgerrit Merged openstack/openstacksdk master: Implement If-Match support for Neutron resources https://review.opendev.org/710030
13:45:16 frickler mordred: you just got me slightly confused by adding sdk core to osc core, doesn't that preempt the governance patch a bit?
14:00:45 openstackgerrit Merged openstack/openstacksdk master: Include "fields" to "SecurityGroup" query parameters https://review.opendev.org/710820
14:22:52 mgoddard odyssey4me: hi
14:50:22 mordred mgoddard: AWESOME. I'm poking and trying to reproduce the traceback at the moment but havne't been able to yet - might need to hold a test node
14:50:39 mgoddard mordred: I have more info
14:50:48 mordred frickler: yeah - I was in there doing core team update from the mailing list and figured I just go ahead and take care f that too ... might be a little ahead of myself
14:50:51 mgoddard mordred: see my comment on https://github.com/ansible/ansible/pull/67577/files
14:50:56 mgoddard it's an easy fix
14:51:11 mgoddard just patching up kolla then I'll raise a bug in ansible and push a fix
14:51:21 mgoddard seems like everything is breaking this week
14:51:27 mordred aha! cool. and yes - it does seem that way
14:51:57 mordred mgoddard: fwiw - once we've switched the modules to the collection, we can co-gate them on kolla
14:52:03 mordred and prevent stuff like that
14:52:42 mgoddard mordred: I think the existing jobs would have caught it. It was a bit of custom code for 2.8 in a backport
14:52:50 mordred AH
14:52:51 mgoddard looks like no CI for 2.8?
14:53:08 mordred yeah - I don't think we run openstack jobs on stable-2.8 - which is probably a mistake :)
14:53:14 mordred let me go see if I can rectify that
14:54:06 mgoddard that would be nice
14:57:26 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add ansible stable-2.9 job and run 2.8 and 2.9 https://review.opendev.org/711471
14:58:48 mordred mgoddard: ^^ that - and remote: https://review.opendev.org/711474 Run openstacksdk functional jobs on ansible 2.8 and 2.9
14:59:21 mordred and yeah - if the tests had been hooked up that should have caught that
14:59:49 mordred mgoddard: ping me when you push up the ansible fix and I can lgtm it
15:06:54 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Switch to futurist for concurrency https://review.opendev.org/711301
15:07:10 mordred dtantsur: that one ^^ should be green now
15:08:33 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Deal with collection build modifying tree https://review.opendev.org/711033
15:08:39 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Fix license metadata https://review.opendev.org/711035
15:08:45 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Clean up minor build quibbles https://review.opendev.org/711036
15:12:51 dtantsur nice!
15:12:52 mordred brtknr: the magnum function tests for sdk seem to always be failing - and it seems to be related to detecting magnum: https://4689b6461ef076a1f80e-32e19aab5c30ebc59039fa10c249436d.ssl.cf5.rackcdn.com/711301/1/check/openstacksdk-functional-devstack-magnum/9a10a15/testr_results.html
15:13:30 mordred brtknr: this makes me think our devstack job is not configured properly ... do you know anything about magnum devstack job config?
15:14:24 mordred dtantsur: and yeah - I think that's a nice improvement and should let us make things nicer without making sdk unuable in the services

Earlier   Later