Earlier  
Posted Nick Remark
#openstack-sdks - 2017-12-04
20:58:35 slaweq__ mordred: hi
20:58:42 slaweq__ mordred: I have quick question to You
20:58:53 mordred slaweq__: heya!
20:59:07 slaweq__ mordred: now, when shade is merged with OpenstackSDK, what is status of patches for shade?
20:59:31 slaweq__ should we still review them? or should them be abandoned and proposed to sdk repo?
21:00:17 mordred I think for now we still review them - at least until we make shade into a compatability layer on top of sdk
21:00:32 slaweq__ ok
21:00:48 mordred slaweq__: I think that might take a little while to accomplish - in the meantime, I've just been cherry-picking patches over to sdk after they land in shade
21:01:07 openstackgerrit Merged openstack-infra/shade master: Merge OpenstackCloud and OperatorCloud files https://review.openstack.org/524877
21:01:08 openstackgerrit Merged openstack-infra/shade master: Remove shade-functional-devstack-legacy https://review.openstack.org/525258
21:01:12 slaweq__ ok, that's now clear for me
21:01:14 slaweq__ thx
21:01:30 mordred excellent
21:07:14 openstackgerrit Merged openstack/osc-lib master: Updated from global requirements https://review.openstack.org/520481
21:26:59 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Make functional-tips job voting https://review.openstack.org/525283
21:35:42 slaweq__ dtroyer: hello
21:36:09 slaweq__ dtroyer: can You maybe take a look at https://review.openstack.org/#/c/515401/ and tell me if it's something what You wanted more or less?
21:36:34 slaweq__ it's not finished yet but I wan't to hear Your opinion about it
21:36:37 slaweq__ thx in advance
21:36:51 dtroyer slaweq__: sure, I'll try to get to it today yet
21:37:01 slaweq__ dtroyer: thx a lot
22:08:51 openstackgerrit Merged openstack/python-openstackclient master: Remove a bunch of things we promised to remove in 2H 2017 https://review.openstack.org/524399
#openstack-sdks - 2017-12-05
00:10:25 SamYaple oh yea mordred. the keystone shade salt module/states merged
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: Implement availability_zone_hints for networks and routers. https://review.openstack.org/525279
15:02:18 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Fix doc enforcer module for python3 https://review.openstack.org/525642
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

Earlier   Later