| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-04-06 | |||
| 16:50:27 | mordred | (or find other things for you to try so we can figure out what's wrong) | |
| 16:51:07 | _alastor_ | Ubuntu 16.04, devstack sha: 4dde3b8b7cf0a0ea96a9b8fe8cb1de8cfa97250e | |
| 17:01:34 | mordred | _alastor_: not that it helps you - but fwiw, http://paste.openstack.org/show/718614/ | |
| 17:02:31 | mordred | _alastor_: you might also try print(openstack.__file__) and print(openstack.version.__version__) just to make sure there isn't something really weird going on with paths | |
| 17:03:32 | _alastor_ | Definitely something weird. Python thinks that openstack is a built-in | |
| 17:04:14 | _alastor_ | http://paste.openstack.org/show/718615/ | |
| 17:10:54 | _alastor_ | The python2 version is pointing at the correct dist-packages module: '/usr/local/lib/python2.7/dist-packages/openstack' | |
| 17:11:11 | _alastor_ | And all the files appear to be present, but python insists on non fully loading it | |
| 17:18:22 | _alastor_ | Reinstalling openstacksdk didn't fix it | |
| 17:20:09 | _alastor_ | Installing openstacksdk in a python2 virtualenv fixes the problem, but it still doesn't explain why the system openstacksdk is messed up | |
| 17:20:59 | _alastor_ | Nothing was done on this system except installing Devstack, copying a few Cinder driver files and running Tempest | |
| 17:44:35 | johnsom | mordred I want to get on that version discovery issue, but the docs are still TODO: https://specs.openstack.org/openstack/api-wg/guidelines/discoverability.html Do you have a link or pointer to what I need to fix for Octavia? | |
| 18:05:20 | dtruong | morded, I added a senlin devstack job in https://review.openstack.org/#/c/555762/ to run senlin functional test. | |
| 18:05:22 | dtruong | However the senlin-api service fails to connect to keystone with SSL error: http://logs.openstack.org/62/555762/7/check/openstacksdk-functional-devstack-senlin/2de7149/controller/logs/screen-sl-api.txt.gz#_Apr_06_01_27_32_527646 | |
| 18:05:40 | dtruong | Is there anything that I missed in the senlin job definition? | |
| 18:10:49 | mordred | johnsom: best thing I've got is the example dicts here: http://specs.openstack.org/openstack/api-wg/guidelines/microversion_specification.html#version-discovery | |
| 18:11:15 | johnsom | Ah, ok, thanks for the pointer! | |
| 18:11:55 | mordred | johnsom: tl;dr - the links entry is what octavia is missing right now | |
| 18:12:08 | johnsom | Ok | |
| 18:12:21 | mordred | johnsom: it just needs a single entry with your /v2 endpoint in it | |
| 18:12:32 | mordred | and then you should be all set! | |
| 18:13:11 | mordred | johnsom: if you get a patch, we should do a depends-on with an openstacksdk patch that removes the octavia workaround | |
| 18:13:59 | mordred | _alastor_: I'm having a similar but not *quite* the same issue myself, with the same symptoms - works in a virtualenv, has issues non-venv | |
| 18:14:11 | johnsom | Ok, yeah, ping me with it. I can use that as a test case too. However it will likely be later today before I have something. I pivoted to another project. | |
| 18:14:12 | mordred | _alastor_: I don't have answers yet, but clearly this is a big problem | |
| 18:14:42 | mordred | johnsom: let's just circle back to it early next week maybe - I got up super early today and am almost done for the day | |
| 18:14:59 | johnsom | Sounds good. Have a good weekend | |
| 18:15:49 | mordred | you too! | |
| 18:16:06 | _alastor_ | mordred: Ok, I'm both glad and sad it's not a unique issue :) | |
| 18:16:45 | mordred | _alastor_: me too - I really would have preferred it if your machine was just weird :) | |
| 18:21:54 | gtmanfred | mordred: when you have time, i would like to talk about this https://storyboard.openstack.org/#!/story/2001794 We have just released our new cloud driver for salt-cloud that uses shade, but are running into this race condition with one of our customers | |
| 18:50:33 | ssurana | Hey Guys! need some more reviews on https://review.openstack.org/#/c/558461/ thanks! | |
| 18:53:09 | mordred | gtmanfred: yes - I'd love to talk about that - I'm not 100% here at the moment, so maybe I can catch you on Monday | |
| 18:53:23 | gtmanfred | That would work perfectly! | |
| 18:53:37 | mordred | gtmanfred: fwiw - there are two mechanisms shade uses for doing auto floating ips - one of them is safe for multi-process use, the other is not | |
| 18:53:59 | gtmanfred | cool, i might need to just switch to the safe one, because right now I am only using create_server | |
| 18:54:03 | gtmanfred | with wait=True | |
| 18:54:46 | gtmanfred | yeah, we have updated the customer that we are going to work with upstream to get this fixed, so we are clear for a bit on my end. | |
| 18:54:46 | mordred | gtmanfred: gotcha. so - lemme dig up the part that does the safe mechanism | |
| 18:54:51 | gtmanfred | :+1: | |
| 18:55:02 | mordred | sweet | |
| 18:56:20 | gtmanfred | Thanks! | |
| 18:56:34 | mordred | gtmanfred: ok. the code isn't friendly to link to - the tl;dr is - rather than doing create floating ip + attach floating ip - if you find the neutron port that is assocated with the target fixed ip on the server, you can pass that port to the create floating ip call - and if you do, neutron will create-and-attach the floating ip -so it'll never be in a state where another process can find it and try | |
| 18:56:36 | mordred | to attach it | |
| 18:57:48 | mordred | gtmanfred: I'll see if I can't write up a little something about it that would be more readable than attempting to trade the shade logic itself (which is a bit crazier than one would like) | |
| 18:59:52 | gtmanfred | that would be awesome! thanks! take your time, I can wait till monday if you need | |
| 19:00:29 | gtmanfred | also, here is one other question | |
| 19:01:02 | gtmanfred | the only other problem is if you use reuse=True, you could just run into a race condition where multiple servers grab the same available ip, and try to use it | |
| #openstack-sdks - 2018-04-08 | |||
| 01:37:37 | openstackgerrit | Doug Hellmann proposed openstack/openstackclient master: add lower-constraints job https://review.openstack.org/556037 | |
| 03:50:34 | openstackgerrit | chenyb4 proposed openstack/openstacksdk master: Fix wait for futures append 'result' error https://review.openstack.org/559523 | |
| 11:37:59 | openstackgerrit | Dongcan Ye proposed openstack/python-openstackclient master: Remove deprecated ip floating commands https://review.openstack.org/546924 | |
| 12:05:14 | openstackgerrit | Dongcan Ye proposed openstack/python-openstackclient master: Network: Add tag support for floating ip https://review.openstack.org/547316 | |
| 15:14:59 | openstackgerrit | Dongcan Ye proposed openstack/python-openstackclient master: Network: Add tag support for floating ip https://review.openstack.org/547316 | |
| #openstack-sdks - 2018-04-09 | |||
| 09:16:30 | openstackgerrit | Merged openstack/os-client-config master: Add osc tips jobs back https://review.openstack.org/526706 | |
| 13:13:44 | openstackgerrit | Jens Harbott (frickler) proposed openstack/python-openstackclient master: Add help for nova interface-list to decoder https://review.openstack.org/559734 | |
| 13:52:45 | openstackgerrit | Jens Harbott (frickler) proposed openstack/python-openstackclient master: Add bgp commands to neutron decoder https://review.openstack.org/559742 | |
| 14:21:12 | openstackgerrit | Doug Hellmann proposed openstack/keystoneauth master: add lower-constraints job https://review.openstack.org/555625 | |
| 14:24:46 | openstackgerrit | Doug Hellmann proposed openstack/cliff master: add lower-constraints job https://review.openstack.org/555405 | |
| 15:01:48 | openstackgerrit | Doug Hellmann proposed openstack/os-client-config master: add lower-constraints job https://review.openstack.org/556042 | |
| 16:40:20 | openstackgerrit | Duc Truong proposed openstack/openstacksdk master: Add function examples test for clustering https://review.openstack.org/555762 | |
| 17:26:45 | openstackgerrit | Jens Harbott (frickler) proposed openstack/python-openstackclient master: Add bgp commands to neutron decoder https://review.openstack.org/559742 | |
| 17:31:27 | openstackgerrit | Doug Hellmann proposed openstack/keystoneauth master: add lower-constraints job https://review.openstack.org/555625 | |
| 20:11:47 | openstackgerrit | Merged openstack/python-openstackclient master: Add help for nova interface-list to decoder https://review.openstack.org/559734 | |
| #openstack-sdks - 2018-04-10 | |||
| 14:48:43 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Remove invalid 'unlock-volume' migration arg https://review.openstack.org/558910 | |
| 18:29:40 | openstackgerrit | Doug Hellmann proposed openstack/cliff master: exclude cmd2 0.8.3 and update to 0.8.4 https://review.openstack.org/560121 | |
| 18:52:34 | eandersson | mordred, would this patch work for pike? https://review.openstack.org/#/c/558710/ | |
| 19:35:57 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Remove invalid 'unlock-volume' migration arg https://review.openstack.org/558910 | |
| 19:35:58 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Clean up W503 and E402 pep8 errors https://review.openstack.org/560132 | |
| 21:04:06 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Remove invalid 'unlock-volume' migration arg https://review.openstack.org/558910 | |
| 21:07:55 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Clean up W503 and E402 pep8 errors https://review.openstack.org/560132 | |
| 21:30:02 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Remove invalid 'unlock-volume' migration arg https://review.openstack.org/558910 | |
| 22:27:32 | openstackgerrit | Merged openstack/openstacksdk master: Fix resource not exist the resource.status error https://review.openstack.org/558757 | |
| 22:38:46 | openstackgerrit | Merged openstack/os-client-config master: add lower-constraints job https://review.openstack.org/556042 | |
| #openstack-sdks - 2018-04-11 | |||
| 04:03:56 | openstackgerrit | Merged openstack/openstacksdk master: Strip the version prefix from the next link for pagination https://review.openstack.org/558461 | |
| 08:53:44 | frickler | amotoki: mordred: dtroyer_zz: these three would seem to close the last gap in DNS support between neutron and OSC https://review.openstack.org/516701 https://review.openstack.org/558820 https://review.openstack.org/558824 | |
| 09:24:34 | jpich | dtroyer_zz: Hi! I submitted a couple of release patches for osc stable/ocata as well, if you would have time to give them a look over at some point I would be grateful :) Thank you | |
| 11:38:45 | openstackgerrit | Jacky Hu proposed openstack/openstacksdk master: Allow cascade deletion of load balancer https://review.openstack.org/553404 | |
| 12:25:45 | dtroyer_zz | jpich: I'll have a look this afternoon, thanks! | |
| 13:01:46 | jpich | dtroyer_zz: Thank you! | |
| 13:05:43 | pooja_jadhav | mordred: Hi | |
| 13:17:39 | openstackgerrit | Jens Harbott (frickler) proposed openstack/openstacksdk master: WIP: Add functional tests for Neutron DNS extension https://review.openstack.org/559068 | |
| 13:40:42 | pooja_jadhav | hi all, any body knows about split logging? | |
| 13:41:10 | pooja_jadhav | I need some help | |
| 15:22:36 | frickler | mugsie: do you happen to know why there is a broken functional recordset test in the sdk? might that be due to the ovo changes? http://logs.openstack.org/68/559068/4/check/openstacksdk-functional-devstack-tips/796096e/testr_results.html.gz | |
| 15:22:52 | mugsie | I freaking hope not | |
| 15:22:56 | mugsie | let me look | |
| 15:27:17 | frickler | mugsie: might also be imported from shade and never tested before. in that patch I enable designate for the functional test for the first time in an attempt to work around an issue with neutron integration | |
| 15:27:46 | mugsie | shade was pretty well tested against designate afaik | |
| 15:27:58 | frickler | mugsie: just a bit disappointed that I got the original issue resolved and now things are still red because of this failure | |
| 15:28:04 | mugsie | for sure | |
| 15:28:30 | frickler | anyway, I'm off to enjoy the remainder of the beautiful weather here, will continue tomorrow | |
| 15:28:51 | mugsie | frickler: enjoy :) | |
| 15:32:29 | mugsie | frickler: I think I found it - let me see if I can get a fix together | |
| 15:45:32 | openstackgerrit | Graham Hayes proposed openstack/openstacksdk master: Fix DNS Recordset CRUD to use IDs https://review.openstack.org/560477 | |
| 15:45:43 | mugsie | frickler: ^ should fix it | |
| 17:00:49 | frickler | mugsie: good job, I had only looked at the broken test so far and not yet noticed that the implemented function indeed doesn't live up to what it promises. added comments | |
| 17:01:44 | mugsie | frickler: https://review.openstack.org/#/c/560545/1 was added to test the fix as well :) | |
| 17:03:12 | frickler | mugsie: ah, I didn't see that, marvelous | |
| 17:16:13 | mugsie | gah - that is failing unit tests anyway | |
| #openstack-sdks - 2018-04-12 | |||
| 06:44:57 | frickler | mugsie: seems the unit test will need added mocking for the new calls you inserted, let me know if you want help with that | |