| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2020-03-04 | |||
| 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 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Handle old status-less placement service https://review.opendev.org/711328 | |
| 21:24:58 | Shrews | mordred: oh, i can't wait. really. no, seriously. | |
| 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: Build utility image for using osc https://review.opendev.org/711246 | |
| 21:36:25 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Add bindep file https://review.opendev.org/711055 | |
| 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 | |
| 15:17:50 | Shrews | oh good. i was also trying to reproduce the strictversion thing without success. glad to page irc back in and see the issue was found | |
| 15:18:10 | mgoddard | mordred: https://github.com/ansible/ansible/pull/68043 | |
| 15:20:25 | rm_work | what is the plan for the next SDK release? | |
| 15:22:29 | mgoddard | mordred, Shrews: I'll send something to the ML about ansible 2.8.9. Any [tags] I should use? | |
| 15:23:47 | rm_work | (I ask about the next release, because I need to use the feature I just added in a client change) | |
| 15:26:30 | mordred | brtknr: https://zuul.opendev.org/t/openstack/build/9a10a15e2a2e4af18021948078429d1c/log/job-output.txt#50351 this might have something to do with it - from what I can tell magnum IS running in that cloud | |
| 15:26:52 | mordred | rm_work: I think we're pretty good for a next release - all of the recent things people need have landed - let me double check that | |