Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-10-30
17:50:30 gyroknight_ Hi, just wondering if anyone's had trouble with list_servers(all_projects=True) in openstack.connection.Connection? It seems to only list instances belonging to the authenticated user's project, even when the user is an admin
17:50:49 gyroknight_ I can't find any related tasks or stories in the StoryBoard
17:53:57 mordred gyroknight_: I feel like I saw a patch related to that recently
17:53:59 mordred looking
17:54:24 gyroknight_ The servers(all_projects=True) generator in the compute v2 proxy also doesn't seem to work. The generator creation seems to work fine, but if I try to iterate through it, Resource throws an error "openstack.exceptions.InvalidResourceQuery: Invalid query params: all_projects"
17:54:34 mordred oh - maybe not
17:55:01 mordred gyroknight_: cool (I mean, not cool, but good to know) - I've gotta run out for a sec, but will investigate in just a little bit
17:55:37 gyroknight_ Thanks
18:22:36 gyroknight_ Figured out the compute proxy, seems the documentation is incorrect, it should be all_tenants instead of all_projects
18:23:47 gyroknight_ I'm using the documentation here: https://docs.openstack.org/openstacksdk/latest/user/proxies/compute.html#openstack.compute.v2._proxy.Proxy
18:56:49 mordred gyroknight: we just recently merged a fix for that ... https://review.openstack.org/#/c/540390/ (like, 2 days ago and it'll be in there when we release later today)
18:57:42 gyroknight Awesome, thanks
18:59:32 gyroknight I figured out the workaround with list_servers(filters={'all_tenants': True}) in the meantime, but it'll be nice to have the alias later on
19:04:17 mordred ++
19:06:30 mordred hrm. list_servers(filters={'all_tenants': True}) works but list_servers(all_projects=True) doesn't work? that's sad making ...
19:06:41 mordred gyroknight: do you have resource caching enabled?
19:06:58 gyroknight Not sure
19:07:19 gyroknight How would I check?
19:08:07 mordred you'd have to do it explicitly by putting a cache: section in a clouds.yaml
19:08:20 mordred so if you weren't intending on it, you are likely not doing it
19:08:38 gyroknight Yeah, don't have one of those
19:08:48 mordred just asking cause we found a bug earlier today with filters not being passed properly when caching was in play
19:09:08 mordred kk. well - I'll see if I can figure out why one form works and the ohter doesn't
19:14:43 gyroknight Interestingly enough, all_projects=True works in get_server(), so maybe it's only isolated to those two occurences?
19:21:25 mordred o_O
19:21:40 mordred that's excessively strange - get_server is built on top of list_server :)
19:41:19 openstackgerrit Merged openstack/openstacksdk master: Filter ports in list_ports when batching is in effect https://review.openstack.org/614213
19:43:50 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Apply list filter fix to servers and floating ips too https://review.openstack.org/614236
21:13:07 openstackgerrit Merged openstack/openstacksdk master: Remove setup.py check from pep8 job https://review.openstack.org/614274
21:57:04 openstackgerrit Tony Breeds proposed openstack/cliff master: Removed unused err variable https://review.openstack.org/614359
23:16:02 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add networks to Limestone vendor https://review.openstack.org/608027
23:17:52 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add networks to Limestone vendor https://review.openstack.org/608027
#openstack-sdks - 2018-10-31
00:01:20 openstackgerrit Merged openstack/os-client-config master: Remove setup.py check from pep8 job https://review.openstack.org/614280
00:39:19 openstackgerrit Merged openstack/openstacksdk master: Added basic CRUD functionality around Host Aggregates https://review.openstack.org/609416
00:39:19 openstackgerrit Merged openstack/openstacksdk master: Make delete_unattached_floating_ips return a count https://review.openstack.org/614075
02:44:19 openstackgerrit Vishakha Agarwal proposed openstack/python-openstackclient master: Updated the take_actions for unified limits https://review.openstack.org/612283
04:40:05 openstackgerrit Merged openstack/openstacksdk master: Add networks to Limestone vendor https://review.openstack.org/608027
08:58:58 openstackgerrit Vishakha Agarwal proposed openstack/python-openstackclient master: Add project param in LimitList parser https://review.openstack.org/611789
09:01:02 openstackgerrit brandon zhao proposed openstack/openstacksdk master: Fix the conflict of urlparse between python2 and python3 https://review.openstack.org/614439
09:08:54 openstackgerrit brandon zhao proposed openstack/os-client-config master: Fix the conflict of urlparse between python2 and python3 https://review.openstack.org/614443
09:34:25 openstackgerrit Vishakha Agarwal proposed openstack/python-openstackclient master: Updated the take_actions for unified limits https://review.openstack.org/612283
13:33:14 openstackgerrit Merged openstack/cliff master: Removed unused err variable https://review.openstack.org/614359
14:15:35 openstackgerrit Monty Taylor proposed openstack/cliff master: Don't try to run issubclass on non-classes https://review.openstack.org/614218
14:46:09 openstackgerrit Chris Dent proposed openstack/service-types-authority master: Update placement project to placement https://review.openstack.org/614527
14:59:49 mordred cdent: ^^ neat! I think you can delete a line
15:00:05 cdent mordred: yeah, I think i just glazed right over that.
15:00:11 cdent A whole lot of glazedness lately
15:01:04 openstackgerrit Chris Dent proposed openstack/service-types-authority master: Update placement project to placement https://review.openstack.org/614527
15:01:06 openstackgerrit Merged openstack/openstacksdk master: Added assign function to identity v3 proxy https://review.openstack.org/612006
15:01:08 cdent there we go
15:05:21 mordred \o/
15:41:50 openstackgerrit Merged openstack/openstacksdk master: Fix the conflict of urlparse between python2 and python3 https://review.openstack.org/614439
15:46:23 openstackgerrit Merged openstack/os-client-config master: Fix the conflict of urlparse between python2 and python3 https://review.openstack.org/614443
16:06:37 openstackgerrit Merged openstack/openstacksdk master: Apply list filter fix to servers and floating ips too https://review.openstack.org/614236
17:44:13 openstackgerrit Corey Wright proposed openstack/openstacksdk master: Fix bugs in debugging with Tox https://review.openstack.org/614596
19:44:30 openstackgerrit Merged openstack/openstacksdk master: Fix latest flake8 issues https://review.openstack.org/614038
20:06:21 openstackgerrit Merged openstack/openstacksdk master: Use sdk for list_servers https://review.openstack.org/530770
23:02:04 openstackgerrit Merged openstack/cliff master: Don't try to run issubclass on non-classes https://review.openstack.org/614218
#openstack-sdks - 2018-11-01
01:08:42 openstackgerrit Corey Wright proposed openstack/openstacksdk master: Explicit set capabilities in VolumeDetail and SnapshotDetail https://review.openstack.org/614659
01:23:03 openstackgerrit Vishakha Agarwal proposed openstack/python-openstackclient master: Updated the take_actions for unified limits https://review.openstack.org/612283
08:17:22 openstackgerrit wangxiyuan proposed openstack/openstacksdk master: Add description property for role https://review.openstack.org/614705
08:23:25 openstackgerrit wangxiyuan proposed openstack/openstacksdk master: Add missing properties for role https://review.openstack.org/614705
08:47:13 openstackgerrit lvxianguo proposed openstack/python-openstackclient master: trivial: modify spelling error of project https://review.openstack.org/614713
09:19:25 vishakha amotoki: Hi. Can you pl look these patches pending from long time. Thank you. https://review.openstack.org/#/q/project:openstack/python-openstackclient+owner:%22Vishakha+Agarwal+%253Cagarwalvishakha18%2540gmail.com%253E%22
12:21:07 openstackgerrit Merged openstack/python-openstackclient master: trivial: modify spelling error of project https://review.openstack.org/614713
14:12:36 mordred kmalloc: https://review.openstack.org/#/c/614705 <-- adds support for 'description' field, but says that was just recently added server side ... how would one detect whether or not it's ok to send description to the server?
14:12:46 mordred cmurphy: ^^ (same question)
14:15:08 mordred edleafe: feel like +A'ing cdent's patch https://review.openstack.org/#/c/614527/ ?
14:15:43 edleafe mordred: sure - will get to it soon
14:16:51 mordred edleafe: woot
14:28:35 mriedem guh, osc moves from launchpad to storyboard and now you can't lookup any old bugs in launchpad for osc...
14:28:36 mriedem fun!
14:31:52 smcginnis mriedem: I think you can still get to them if you have the link, but can't search for them.
14:32:21 mriedem yeah, had to git blame what i was looking for to find the bug
14:33:40 openstack Launchpad bug 1800998 in python-openstackclient "It is impossible to release IP from instance. Error: 'AttributeError: remove_fixed_ip'" [High,New]
14:33:40 mriedem anywho https://bugs.launchpad.net/python-openstackclient/+bug/1800998
14:33:50 mriedem apparently you can still report a bug to another project, and then add osc to it after the fact
14:34:22 mriedem i would try to search for duplicates in storyboard but i've used the SB search before and i won't waste my time
14:36:57 smcginnis That's bad.
14:41:44 smcginnis fungi: Are you aware of that issue? ^ (not the bug, the ability to create LP bugs after a project has migrated to SB)
14:42:46 fungi yes, turning off bug reporting in lp doesn't prevent someone from adding that project to a bug created through other means
14:44:00 fungi it only disables the bug reporting link from that project's page, and gets rid of the project-specific bug list views
14:48:18 openstackgerrit Merged openstack/service-types-authority master: Update placement project to placement https://review.openstack.org/614527
15:08:44 openstackgerrit OpenStack Proposal Bot proposed openstack/os-service-types master: Updated from OpenStack Service Type Authority https://review.openstack.org/614786
15:21:13 openstackgerrit Merged openstack/openstacksdk master: Explicit set capabilities in VolumeDetail and SnapshotDetail https://review.openstack.org/614659
15:28:04 openstackgerrit Merged openstack/openstacksdk master: Fix bugs in debugging with Tox https://review.openstack.org/614596
15:37:47 kmalloc Hmmmmmm
15:38:14 kmalloc mordred: we don't do a good job of communicating that is allowed
15:38:39 kmalloc mordred: you could use our API version (e.g. 3.11)
15:40:04 kmalloc mordred: let me check, roles might be a bit sloppy on this front though and just work.
15:42:45 kmalloc mordred: so, roles are sloppy and store "extra" fields in a JSON blob
15:42:55 kmalloc So if it isn't supported, it still works
15:43:21 kmalloc And stores it as description on the role. The difference is it is a full column now.
15:43:40 kmalloc It should be fine to just support that universally
15:43:59 kmalloc Other resources are not as sloppy, and may not work like that.
15:45:54 kmalloc https://github.com/openstack/keystone/blob/master/keystone/assignment/role_backends/sql.py#L203
15:45:57 kmalloc And
15:46:25 kmalloc https://github.com/openstack/keystone/blob/master/keystone/assignment/schema.py#L26
15:48:09 kmalloc This also highlights that we wrote a broken migration for adding that column :(
15:57:16 mordred kmalloc: oh yay fun

Earlier   Later