Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-01-31
01:58:13 mordred the complex problems are the best :)
02:12:54 mordred adriant: BWAHAHAHAHA. found it and it's silly
02:13:17 mordred adriant: http://paste.openstack.org/show/658082
02:13:59 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Add a descriptor object for each service proxy https://review.openstack.org/539232
02:14:00 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Add OpenStackCloud object to Connection https://review.openstack.org/539234
02:14:00 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Ensure Connection can be made from keyword arguments https://review.openstack.org/539233
02:14:01 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Rename BaseProxy to Proxy https://review.openstack.org/539271
02:14:01 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Cleanup the BaseProxy docs a bit https://review.openstack.org/539270
02:14:02 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove openstack_cloud factory function https://review.openstack.org/539395
02:14:02 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Merge Connection and OpenStackCloud https://review.openstack.org/539394
02:14:03 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Shift config exceptions to openstack.exceptions https://review.openstack.org/539396
02:14:14 adriant mordred: hah!
02:14:30 mordred and there's tonight's patch bomb ... I shall now remove myself from the compute and eat some fish
02:14:48 adriant good plan, will play with those some more and see how it looks
02:14:53 adriant mostly looks great
06:38:45 openstackgerrit Hunt Xu proposed openstack/python-openstacksdk master: resource: don't early terminate list https://review.openstack.org/539424
08:45:03 openstackgerrit Hunt Xu proposed openstack/python-openstacksdk master: Implement list projects for user https://review.openstack.org/539460
08:46:40 openstackgerrit Hunt Xu proposed openstack/python-openstacksdk master: resource: don't early terminate list https://review.openstack.org/539424
10:48:09 openstackgerrit Chen Hanxiao proposed openstack/python-openstackclient master: Add --image-property parameter in 'server create' https://review.openstack.org/535664
11:08:14 openstackgerrit Hunt Xu proposed openstack/python-openstacksdk master: Fix TypeError for overrided get methods https://review.openstack.org/539494
14:20:44 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Merge Connection and OpenStackCloud https://review.openstack.org/539394
14:20:45 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Shift config exceptions to openstack.exceptions https://review.openstack.org/539396
14:20:45 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove openstack_cloud factory function https://review.openstack.org/539395
14:20:46 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Rename BaseProxy to Proxy https://review.openstack.org/539271
14:20:46 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Cleanup the BaseProxy docs a bit https://review.openstack.org/539270
14:57:18 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Shift voting flag for ansible-devel job https://review.openstack.org/539563
14:57:37 tosky hi! I noticed a regression when running some tempest tests; while requirements and upper-constraints pins to openstacksdk 0.9.19, tempest install the last version (0.11.0)
14:57:49 tosky and when running a simple `openstack network list --external`, I get
14:57:53 tosky Invalid query params: router:external
14:59:33 tosky I don't seem to see an existing bug, so I'm going to file one, but I'm not sure if openstackclient 3.14.0 is supposed to work with openstacksdk 0.11.0
15:06:12 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Shift voting flag and test_matrix_branch for ansible-devel job https://review.openstack.org/539563
15:10:38 mordred tosky: yes - we do expect openstackclient 3.14.0 to work with 0.11 - so yes please to the bug
15:11:05 tosky mordred: so again openstacksdk only, not against openstackclient?
15:11:25 mordred tosky: yah - I'd consider it a bug in the 0.11 release that we should figure out and get a point-release cut for
15:11:40 tosky mordred: thanks, I was halfway through it, incoming
15:12:29 tosky here you are: https://bugs.launchpad.net/python-openstacksdk/+bug/1746535
15:12:30 openstack Launchpad bug 1746535 in OpenStack SDK "`openstack network list --external` breaks with openstacksdk 0.11.0 + openstackclient 3.14.0" [Undecided,New]
15:33:39 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Split request logging into four different loggers https://review.openstack.org/505764
15:38:10 mnaser mordred: forgive me if i'm misusing shade, but get_volume() seems to want to list volumes and then find it (assumingly trying to filter by name first)
15:38:20 mnaser so trying to do get_volume() in the admin tenant results in no volume found
15:38:43 mnaser is there some switch i'm not turning on?
15:38:59 mordred mnaser: yes - that is the default behavior. there is a constructor flag "use_direct_get" which will cause it to use get instead of list for gets
15:39:15 mnaser mordred: wonderful, that's exactly what i needed, thank you
15:40:01 mnaser hmm
15:40:36 mordred mnaser: I believe we'll flip the value of that default in the openstacksdk version of shade - we filter list calls to support nodepool, which I promise you do not want doing 1000s of get calls ... but for humans who are not nodepool it can be unexpected behavior :)
15:41:06 mnaser mordred: i think that flag might have been lost somehow
15:41:17 openstackgerrit Merged openstack-infra/shade master: Add devel branches and override-checkout for ansible-devel job https://review.openstack.org/536159
15:41:22 mnaser it seems like it is not a constructor value but a thing to set as an attribute to cloud
15:41:28 mnaser (aka cloud.use_direct_get = True)
15:41:38 mordred mnaser: I see it on the constructor?
15:41:54 mnaser note to self don't trust github search
15:42:16 mordred mnaser: http://git.openstack.org/cgit/openstack-infra/shade/tree/shade/openstackcloud.py#n137
15:42:21 mordred mnaser: never trust github
15:42:37 mnaser it doesn't seem to be in the release shipped with rdo pike, maybe i'll ask for a bump sometime soon
15:42:51 mordred oh - yah - it's a new flag
15:43:00 mordred got implemented this cycle
15:43:15 mnaser shade==1.22.2 and this merged in 1.24.0
15:43:21 mnaser ok cool venv it is
15:43:27 mordred \o/
15:43:43 mordred mnaser: luckily 1.24.0 is post-client-ectomy, so the venv should be quite small
15:43:53 mnaser oh awesome
15:45:44 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Shift voting flag and test_matrix_branch for ansible-devel job https://review.openstack.org/539563
15:46:19 tosky client-ectomy++
15:47:40 mordred tosky: ++ seriously. so much better
15:48:39 tosky and it helps packagers
15:51:02 mnaser ok slip up
15:51:11 mnaser shade.openstack_cloud != shade.OpenStackCloud
15:51:37 mnaser that got me confused for a while why it was still false
16:13:14 mordred mnaser: oh - well piddle. we should add that parameter to the shade.openstack_cloud constructor
16:14:24 mnaser mordred: and looks like get_server doesn't respect use_direct_get because it uses the find function all the time, but get_server_by_id also has 2 extra calls that get fips and ports
16:14:36 mnaser (i dont have time to look into it right now just fyi)
16:16:30 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Plumb use-direct-get through factory functions https://review.openstack.org/539586
16:17:53 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Plumb use-direct-get through factory functions https://review.openstack.org/539586
16:17:53 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Switch to providing created_at field for servers https://review.openstack.org/536003
16:18:02 mordred mnaser: ^^ there you go - I'll look at servers real quick
16:44:54 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Updated from global requirements https://review.openstack.org/533991
16:44:55 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Split docs requirements and update tox.ini https://review.openstack.org/539594
17:28:40 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove openstack_cloud factory function https://review.openstack.org/539395
17:28:40 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Merge Connection and OpenStackCloud https://review.openstack.org/539394
17:37:05 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Split docs requirements and update tox.ini https://review.openstack.org/539594
17:37:06 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Updated from global requirements https://review.openstack.org/533991
17:52:04 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Split request logging into four different loggers https://review.openstack.org/505764
18:03:35 openstackgerrit Merged openstack-infra/shade master: Shift voting flag and test_matrix_branch for ansible-devel job https://review.openstack.org/539563
20:03:35 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Shift config exceptions to openstack.exceptions https://review.openstack.org/539396
20:03:40 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Cleanup the BaseProxy docs a bit https://review.openstack.org/539270
20:03:45 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Rename BaseProxy to Proxy https://review.openstack.org/539271
20:08:20 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Fix get_server to work with use_direct_get https://review.openstack.org/539663
20:08:20 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Plumb use-direct-get through factory functions https://review.openstack.org/539586
20:08:27 mordred mnaser: ^^ ok. those should work for you now
20:52:07 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Add get_client_config method to CloudRegion https://review.openstack.org/539678
20:59:32 openstackgerrit Merged openstack-infra/shade master: Switch to providing created_at field for servers https://review.openstack.org/536003
20:59:45 mordred dtroyer: feel like merging https://review.openstack.org/#/c/525283/ ?
21:01:50 dtroyer mordred: ugh, I didn't do that yet???
21:02:02 dtroyer done
21:02:19 adriant mordred: I'm look at https://bugs.launchpad.net/python-openstacksdk/+bug/1746535
21:02:20 openstack Launchpad bug 1746535 in OpenStack SDK "`openstack network list --external` breaks with openstacksdk 0.11.0 + openstackclient 3.14.0" [Undecided,New]
21:02:52 adriant it was introduced by my patch, but... what worries me is this likely means that that never actually worked.
21:03:10 adriant my patch just exposes the error rather than drops the param on the floor
21:03:28 adriant although, it is also possible my patch's logic is busted, so that is an option
21:03:28 openstackgerrit Merged openstack/os-client-config master: Add betacloud to the vendors https://review.openstack.org/527248

Earlier   Later