| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-04-06 | |||
| 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 | mordred | gtmanfred: gotcha. so - lemme dig up the part that does the safe mechanism | |
| 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: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 | |
| 07:02:00 | mhen | Hello! | |
| 07:03:17 | mhen | This channel is also related to the OpenStack client, correct? | |
| 07:08:28 | openstackgerrit | Jens Harbott (frickler) proposed openstack/openstacksdk master: WIP: Add functional tests for Neutron DNS extension https://review.openstack.org/559068 | |
| 07:11:51 | frickler | mhen: yes, feel free to ask your real question. we should also consider making the channel topic more inviting | |
| 07:12:22 | mhen | okay | |
| 07:20:34 | mhen | Is there no IRC meeting for the client anymore? I found a schedule and agendas online but the most recent seems to be mid 2017. | |
| 07:24:04 | mhen | background is: I'm trying to implement image signing functionality into the OpenStack client (which currently has to be done manually using 'openssl' commands). I'd like to discuss if this would be worth a contribution and if so, where it would be best to put related crypto code (openstack-sdk or cursive library?). | |
| 07:29:58 | frickler | mhen: I haven't seen an IRC meeting and the schedule on eavesdrop doesn't seem to list one. probably outdated information in the wiki? | |
| 07:30:59 | frickler | mhen: regarding the other question I have no idea, maybe dtroyer_zz or mordred have some idea. but that topic may also better be suited for a discussion on the dev-ml | |
| 07:37:04 | mhen | For reference, this is the meeting schedule I'm referring to: http://eavesdrop.openstack.org/#OpenStackClient_Team_Meeting | |
| 07:39:51 | mhen | frickler, you referring to this I guess? http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev | |
| 07:41:45 | openstackgerrit | Jens Harbott (frickler) proposed openstack/openstacksdk master: Fix DNS Recordset CRUD to use IDs https://review.openstack.org/560477 | |
| 07:42:19 | frickler | mugsie: fixed a small glitch in your patch first ^^ | |
| 07:44:41 | frickler | mhen: you are correct about th mailing list | |
| 07:45:46 | frickler | mhen: and no idea why I missed the meeting in the schedule. probably should get dropped there if it doesn't happen anyway | |
| 07:46:12 | mhen | frickler, okay. Thanks for your guidance! I will try there :) | |
| 08:57:50 | openstackgerrit | Jens Harbott (frickler) proposed openstack/openstacksdk master: Add designate to the services deployed for functional tests https://review.openstack.org/560831 | |
| 09:01:18 | frickler | mugsie: FYI ^^ waiting to see what others think about that now | |
| 09:17:02 | openstackgerrit | Graham Hayes proposed openstack/openstacksdk master: Fix DNS Recordset CRUD to use IDs https://review.openstack.org/560477 | |
| 09:19:02 | openstackgerrit | Graham Hayes proposed openstack/openstacksdk master: DNM: Fix DNS Recordset CRUD to use IDs https://review.openstack.org/560545 | |
| 09:19:43 | mugsie | frickler: cool. ^^ should fix the unit tests, and I reup'd the testing patch on top as well | |
| 13:11:57 | openstackgerrit | Nick Jones proposed openstack/os-client-config master: Remove DataCentred from list of vendors https://review.openstack.org/560894 | |
| 13:47:22 | mordred | mugsie, frickler: I'd be fine just adding designate to the normal functional test config for openstacksdk as long as doing so doesn't increase the memory load on the devstack such that stuff starts failing | |
| 13:49:02 | mordred | mugsie, frickler: however, if adding it to the default doesn't work, the senlin folks have a patch up adding a senlin-enabled job, we could also add designate to that (and name it somethin gdifferent) | |
| 14:01:10 | gtmanfred | mordred: have you had a chance to look into the floating ip thing? This is basically all we are doing right now, is create_server with wait=True, and auto_ip=True https://github.com/saltstack/salt/blob/2018.3/salt/cloud/clouds/openstack.py#L644 | |
| 14:04:09 | mugsie | mordred: cool - I don't think we will add a huge amount of extra pressure, and chewing up an entire extra node for 3 tests feels like a bad thing :/ - lets hope it fits :) | |
| 16:00:46 | elmiko | cdent around? | |
| 16:00:53 | cdent | dag nabbit | |