Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-10-30
15:45:59 Shrews mordred: i totes want a be_awesome() method
15:52:52 mordred Shrews: we should make one
15:54:51 mordred gtema: looking at your story from earlier about caching and wait_for methods ... we really need to do some work in the caching layer
15:55:05 gtema mordred: agree
15:55:20 gtema just spend some time trying to figure out why it suddenly does not work properly
15:55:43 gtema mordred: can I help there?
15:58:49 mordred gtema: almost certainly! I think once we get this task manager updates finished it'll be a good time to work on it
15:59:07 gtema ok
16:00:21 gtema mordred: what about backup support in block-storage (https://review.openstack.org/#/c/591445/)
16:00:45 openstackgerrit Sean McGinnis proposed openstack/os-client-config master: Remove setup.py check from pep8 job https://review.openstack.org/614280
16:01:46 mordred gtema: "Note that backup restore is not documented in the official API, but is implemented in the cinderclient"
16:01:49 mordred that's so awesome
16:02:11 gtema yes, I was clarifying it - it was a bug promised to be fixed in docs
16:02:53 gtema oh, it is even now fixed - will update commit message
16:04:07 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: implement block-storage backup resource https://review.openstack.org/591445
16:04:10 smcginnis Which "official API" is Cinder backup not documented in?
16:04:32 gtema backup restore was not documented
16:04:36 smcginnis Where?
16:04:46 gtema https://developer.openstack.org/api-ref/block-storage/v2/index.html?expanded=restore-backup-detail#backups-backups
16:05:04 gtema but now it is documented
16:05:11 mordred \o/
16:05:22 smcginnis Must have been a really old reference.
16:06:14 gtema from what I remember I was told there was a content error somewhere in sources, so that this method was not shown properly
16:06:30 smcginnis Ah, that could be then.
16:07:17 smcginnis OK, yeah, I fixed that just in August. https://review.openstack.org/#/c/592026/
16:08:15 gtema yes, I was asking in maillist sometime back in august - my change was for this time forgotten and I rebased it yesterday
16:08:29 gtema mordred: thanks
16:08:35 smcginnis gtema: Cool, thanks!
16:09:01 gtema welcome
16:11:51 dtroyer mordred: +2… there is an isclass() call in OSC that has gone back and forth a time or two, I'm wondering if a similar solution would be appropriate there (without looking before typing of course)
17:13:38 openstackgerrit Merged openstack/openstacksdk master: Add close method to shutdown threadpool https://review.openstack.org/613976
17:13:39 openstackgerrit Merged openstack/openstacksdk master: Make Connection a context manager https://review.openstack.org/613978
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/ ?

Earlier   Later