Earlier  
Posted Nick Remark
#openstack-sdks - 2017-12-05
00:10:32 SamYaple glance and neutron will merge soon
00:19:29 mordred \o/
00:22:03 SamYaple i think im goign to not touch the nova onewith a 10 foot pole lol
00:22:08 SamYaple ill leave that to someone else
00:22:14 SamYaple ill do cinder though
01:17:31 openstackgerrit Julia Kreger proposed openstack-infra/shade master: Add retry logic mechanism https://review.openstack.org/525037
01:21:20 TheJulia mordred: ^^^ should take care of the most likely places where a retry will be needed due to a occasional lock. I didn't write a super low level unit test for the wrapper since the rest calls largely cover it. Let me know what you guys would prefer to see, and I'll toss it up as a follow-up, or maybe do it during lunch tomorrow if I have brain
01:56:19 SamYaple TheJulia: really really good work on python-irnoicclient. You have made me happier than you know
01:56:37 SamYaple when im feeling down i just think about that and smile
01:56:59 SamYaple (smile because i didnt have to do it heh)
02:56:47 openstackgerrit OpenStack Proposal Bot proposed openstack-infra/shade master: Updated from global requirements https://review.openstack.org/525368
03:26:23 openstackgerrit OpenStack Proposal Bot proposed openstack/os-client-config master: Updated from global requirements https://review.openstack.org/520472
03:26:42 openstackgerrit OpenStack Proposal Bot proposed openstack/os-service-types master: Updated from global requirements https://review.openstack.org/525388
03:27:14 openstackgerrit OpenStack Proposal Bot proposed openstack/osc-lib master: Updated from global requirements https://review.openstack.org/525389
11:02:17 openstackgerrit Slawek Kaplonski proposed openstack/python-openstackclient master: [WIP] Add support for get details of Quota https://review.openstack.org/515401
11:03:41 openstackgerrit Slawek Kaplonski proposed openstack/python-openstackclient master: Add support for get details of Quota https://review.openstack.org/515401
12:19:21 openstackgerrit Slawek Kaplonski proposed openstack/python-openstackclient master: Add support for get details of Quota https://review.openstack.org/515401
12:29:39 openstackgerrit Hengqing Hu proposed openstack/python-openstacksdk master: Remove build_sphinx section from setup.cfg https://review.openstack.org/525579
12:32:57 openstackgerrit Hengqing Hu proposed openstack/os-client-config master: Remove build_sphinx section from setup.cfg https://review.openstack.org/525583
14:47:00 openstackgerrit Slawek Kaplonski proposed openstack/python-openstackclient master: Add support for get details of Quota https://review.openstack.org/515401
14:56:22 Shrews TheJulia: mordred: could we maybe figure out a way to change the sleep time for the tests in 525037? I worry that if we start using that every place that our already slow tests could slow down considerably
15:02:18 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Fix doc enforcer module for python3 https://review.openstack.org/525642
15:02:18 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Implement availability_zone_hints for networks and routers. https://review.openstack.org/525279
15:03:55 mordred Shrews: I believe we already mock out sleep in our unittests
15:04:52 mordred Shrews: http://git.openstack.org/cgit/openstack/python-openstacksdk/tree/openstack/tests/unit/base.py#n91
15:05:14 mordred http://git.openstack.org/cgit/openstack-infra/shade/tree/shade/tests/unit/base.py#n90
15:18:33 Shrews mordred: oh, i totally forgot we did that
15:40:36 openstackgerrit Andreas Jaeger proposed openstack/cliff master: DNM testing docs and releasenotes jobs https://review.openstack.org/522370
15:45:26 openstackgerrit Merged openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/525402
16:04:10 openstackgerrit OpenStack Proposal Bot proposed openstack-infra/shade master: Updated from global requirements https://review.openstack.org/525368
16:53:35 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/525689
17:16:04 openstackgerrit LIU Yulong proposed openstack/python-openstacksdk master: Add FloatingIP qos_policy_id attribute https://review.openstack.org/525702
20:25:50 openstackgerrit John Dennis proposed openstack/os-client-config master: Do not apply format expansions to passwords https://review.openstack.org/525744
20:44:31 mordred hah. that's thebest patch ever^^ :)
20:45:06 mordred dtroyer, Shrews: ^^ that seems ... important :)
20:48:49 Shrews that's a super wordy commit message (in a good way)
20:49:55 Shrews dtroyer: left the +A for you
20:54:04 dtroyer strange pep8 job failure in that review…
20:55:49 dtroyer ah, what I'm seeing isn't the job failing, but there is an odd message in the logs just before the pep8 output
20:57:11 dtroyer (and to complete the trifecta) ok, nevermind, that seems to be in other pep8 job outputs too…
21:00:41 openstackgerrit Dean Troyer proposed openstack/os-client-config master: Do not apply format expansions to passwords https://review.openstack.org/525744
21:08:18 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Do not apply format expansions to passwords https://review.openstack.org/525756
21:08:28 mordred dtroyer, Shrews: ^^ there it is cherry-picked to sdk
22:24:26 slaweq dtroyer: hi, my patch for quota details list is ready for Your review: https://review.openstack.org/#/c/515401/
22:24:38 slaweq dtroyer: please look at it if You will have few minutes :)
22:39:20 openstackgerrit Merged openstack/python-openstacksdk master: Add FloatingIP qos_policy_id attribute https://review.openstack.org/525702
22:44:57 SamYaple mordred: glance and neutron merged just in time to make it into the Oxygen release of saltstack
23:17:34 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Fix SDK Connection creation alternative to Profile https://review.openstack.org/523938
#openstack-sdks - 2017-12-06
00:15:35 samueldmq heya, I heard Shade has moved house?
00:15:36 samueldmq :)
00:19:59 samueldmq We look a bigger team now, with awesome people in the channel :)
00:20:01 samueldmq Glad to be back
08:25:51 frickler is there a way to deprecate an option to a command? this seems to only reference deprecation a complete command https://docs.openstack.org/python-openstackclient/pike/contributor/command-wrappers.html
08:26:23 frickler see https://bugs.launchpad.net/python-designateclient/+bug/1736161 for context
08:26:24 openstack Launchpad bug 1736161 in python-designateclient "The records option gobbles up all trailing parameters" [Undecided,In progress] - Assigned to Dr. Jens Harbott (j-harbott)
12:13:59 openstackgerrit Jake Yip proposed openstack/osc-lib master: Fix sorting in Python 3 https://review.openstack.org/526046
12:55:46 openstackgerrit Jake Yip proposed openstack/osc-lib master: Fix sorting in Python 3 https://review.openstack.org/526046
12:57:56 openstackgerrit Jake Yip proposed openstack/osc-lib master: Fix sorting in Python 3 https://review.openstack.org/526046
13:54:18 dtroyer frickler: we've done it "by hand" in the past, ie set help to SUPPRESS and log a warning in take_action() if it is used
14:01:35 frickler dtroyer: thx, I did the second part, do you have a sample for the help change? https://review.openstack.org/526008
14:05:33 dtroyer it's just using help=argparse.SUPPRESS in the option declaration
14:06:05 dtroyer so the old option doesn't appear any more…
14:11:37 frickler ah, that's easy, thanks again
15:59:28 openstackgerrit Merged openstack/os-service-types master: Updated from global requirements https://review.openstack.org/525388
16:05:32 openstackgerrit Mark Goddard proposed openstack-infra/shade master: Baremetal NIC list should return a list https://review.openstack.org/526105
16:21:16 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Use devstack functional test base job https://review.openstack.org/526116
16:22:39 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Use devstack functional base job https://review.openstack.org/526117
16:24:15 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Fix SDK Connection creation alternative to Profile https://review.openstack.org/523938
17:13:08 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Fix batching for floating ips and ports https://review.openstack.org/526127
17:36:20 dmsimard mordred: ohai
17:41:57 dmsimard supplemental is being called from here https://github.com/openstack-infra/shade/blob/master/shade/meta.py#L414 and add_server_interfaces is used in a couple places
17:43:23 dmsimard like in these two blocks https://github.com/openstack-infra/shade/blob/master/shade/openstackcloud.py#L3144-L3155
17:44:39 dmsimard so 'addresses' is not populated until _get_supplemental_addresses runs at least once ?
17:45:25 dmsimard so we're falling in that block and then recursing https://github.com/openstack-infra/shade/blob/6b613f1a387f0c49ce834b0396ac70efb848d565/shade/meta.py#L372
17:45:35 dmsimard which ends up doing all those floating IP calls ? /me adds debug
17:47:01 dmsimard for fip in cloud.search_floating_ips( filters=dict(port_id=port['id'])):
17:47:21 dmsimard sounds very much like GET call to network for https://phx2.cloud.rdoproject.org:13696/v2.0/floatingips.json?port_id=2fde8e86-ba92-448b-bbee-29c16f635cc3 used request id req-2d6942ca-2eab-41d1-a411-188aa6c5305e
17:48:40 dmsimard but we have the mapping of port id <-> floating ip in list_floating_ips(), let me try that
17:50:41 mordred dmsimard: ah - hrm.
17:52:03 dmsimard hang on, I might have a patch
17:54:04 openstackgerrit Dean Troyer proposed openstack/osc-lib master: Use converged SDK insead of os-client-config https://review.openstack.org/518240
18:24:44 peter___ hello all, does anybody know why after making connection: conn = connection.Connection(**auth_args) and I want display project projects = conn.identity.list_projects() in ossdk doesnt working?
18:25:31 peter___ looks like object conn doesnt have identity.list_projects() method.
18:47:47 dmsimard mordred: yeah... so I'm seeing the problem but I'm not sure what's the best way to fix it. Here's a summary of what's happening for a server with 5 ports: http://paste.openstack.org/raw/628306/
18:48:11 dmsimard mordred: We're doing 5 calls to list_floating_ips to check if any of those ports are a floating ip.
18:48:31 dmsimard mordred: It sounds like we should be filtering client side -- do one list_ports, one list_floating_ips, make them friends
18:49:06 dmsimard That's the two for loops from here: https://github.com/openstack-infra/shade/blob/6b613f1a387f0c49ce834b0396ac70efb848d565/shade/meta.py#L377-L380
18:50:30 dmsimard If you're thinking "why does this server have like 5 ports", it's a tenant in which tripleo runs heat stacks for nested tripleo testing with isolated networks
18:57:33 dmsimard So if you have 100 servers with 5 ports each, we're doing 1*list_port, 5*list_floating_ips per server, that's 600 API calls just for _get_supplemental_addresses
19:18:10 peter___ import openstack.config ModuleNotFoundError: No module named 'openstack.config'
20:59:28 openstackgerrit Julia Kreger proposed openstack-infra/shade master: Baremetal NIC list should return a list https://review.openstack.org/526105
21:19:57 openstackgerrit Dean Troyer proposed openstack/os-client-config master: Make the get_service_type() overrides tolernat of no defaults https://review.openstack.org/526201
21:20:54 openstackgerrit Dean Troyer proposed openstack/python-openstacksdk master: Make the get_service_type() overrides tolernat of no defaults https://review.openstack.org/526202
21:26:11 openstackgerrit Dean Troyer proposed openstack/osc-lib master: Use converged SDK insead of os-client-config https://review.openstack.org/518240
21:26:46 openstackgerrit Dean Troyer proposed openstack/osc-lib master: Use converged SDK insead of os-client-config https://review.openstack.org/518240
21:51:45 SamYaple mordred: i just found out there was a netsplit because i couldnt see you in any of the channels. so thanks forthat
21:52:24 SamYaple im wondering if you have any ideas on how to do an introspective valid_kwargs type thing with the saltstack shade modules
22:47:29 openstackgerrit Dean Troyer proposed openstack/python-openstacksdk master: Make the get_service_type() overrides tolernat of no defaults https://review.openstack.org/526202
23:44:29 openstackgerrit Jake Yip proposed openstack/osc-lib master: Fix sorting in Python 3 https://review.openstack.org/526046
23:46:23 openstackgerrit Jake Yip proposed openstack/osc-lib master: Fix sorting in Python 3 https://review.openstack.org/526046

Earlier   Later