Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-02-21
15:22:04 dtantsur I also have a conflicting meeting btw
15:22:24 elmiko haha
15:23:03 elmiko ok, i'll keep an eye on the chat
15:56:01 elmiko API SIG office hour starts in 5 minutes, prepare your inquiries!
16:17:04 edleafe ...and I'm back
16:28:59 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/openstacksdk master: Add "name" filter in "list" call when retrieving a single register https://review.openstack.org/637238
16:39:24 cdent elmiko, edleafe : If I have a resource which returns a json object which contains nested json objects, if I have a GETtable subresoruce for one of those nested json objects am I obliged to have subresources for all the nested objects?
16:39:57 elmiko cdent: good question
16:40:03 edleafe "Obliged"?
16:40:28 edleafe It would seem likely that you would need that, but I don't think that you are under any sort of obligation
16:40:32 mordred cdent: when you say nested json objects ...
16:40:39 elmiko yeah "obliged" sounds kinda strong to me, especially if there is no value in exposing the subresources by themselves
16:40:54 mordred (yeah, I agree with edleafe and elmiko on the obligation part)
16:42:18 cdent As I hope is fairly obvious, I don't have a real use case here, I just want office hours to have _something_
16:42:51 dtantsur :)
16:43:04 edleafe Ah, I thought someone had posed such a question to you and you were passing it along
16:43:10 dtantsur I don't think we should create subresources for the sake of having them (we're quite guilty of that in ironic)
16:43:13 elmiko haha, cdent ++
16:43:29 cdent beer?
16:43:41 elmiko i say no to "obliged", yes to "if there is value in exposing the subresource"
16:45:21 mordred cdent: maybe?
16:45:59 cdent I also say no to "obliged" but I know there are some who would feel that way for sake of grammatical parallelism
16:46:28 elmiko i mean, if you have the cycles to burn....
16:52:38 dtantsur everyone here has a lot of spare time, why even ask?
16:52:45 dtantsur :)
16:56:54 elmiko now if you are using automagical tooling to create your server... XD
16:57:42 dtantsur sounds ironic!
16:57:47 elmiko haha
17:12:14 elmiko gonna wander off for some lunch, good chatting with y'all as usual =)
17:13:06 dtantsur enjoy
19:34:51 openstackgerrit David Rabel proposed openstack/python-openstackclient master: Only allow project ID in 'image add project' https://review.openstack.org/638495
19:35:13 openstackgerrit David Rabel proposed openstack/python-openstackclient master: Fix help message of image add project https://review.openstack.org/638113
20:50:21 openstackgerrit Merged openstack/openstacksdk master: Add image attributes from v2.7 https://review.openstack.org/637538
20:53:38 elmiko anyone around who understand pbr reasonably well?
21:05:04 mordred elmiko: yup.
21:13:18 elmiko mordred: so, i'm using it for a new project and trying to work my through the devel cycle kinks. and i've run into this issue where i create a tag, push the dists to pypi, then i'll do work and when i do a `pip install -U .` of my devel stuff pbr complains about semvers and git stuff. have you run into this?
21:15:12 mordred elmiko: I have not run in to that although that makes me have sads :(
21:15:52 elmiko mordred: definitely giving me a case of the sads
21:16:02 elmiko i can work around it, just curious why it happens
21:17:14 mordred elmiko: I'm trying to think of a good explanation of why that would happen - if you have the git repo locally, that should Just Work (although fwiw I tend to just do a pip install -e . in my source dir and then months later forget that I'm running off of my local branch :) )
21:25:32 elmiko mordred: ack, i am working locally and usually this works. i think i must have futzed with the versions or something.
21:26:14 elmiko what does the `-e` flag do?
21:27:46 mordred elmiko: it does an 'editable' install - it's like python setup.py develop
21:28:00 elmiko mordred: interesting, thanks!
21:28:03 mordred elmiko: so, basically, it installs links to your source dir - and as you change python files it's immediately reflected
21:28:13 elmiko that is super useful
21:28:29 mordred if you add new entrypoints in setup.cfg you'll need to re-run pip install -e .
21:28:37 mordred just fwiw
21:33:38 elmiko ack, thanks again =)
21:33:43 elmiko mordred ++
21:44:29 njohnston list of objects returned?
21:44:29 njohnston Hi! I have a quick question about adding functionality into the openstack sdk. I want to create functionality in the sdk that will use the neutron bulk port creation method, so that you can supply an array of port information in a single request and all of those ports will be created at once and in streamlined manner. Will I need to use something other than the generic _create, or can it handle a list of inputs in and a
21:49:18 mordred njohnston: that is a a great question! I don't think the generic _create will do what you want - so you might need to just do your own magic method
21:49:47 mordred njohnston: that said - maybe once you've got that we can abstract out a _create_list or something like that
21:50:17 njohnston mordred: Great, that is what I thought! I'll start working on it. Thanks!
21:51:40 mordred \o/
22:11:45 njohnston mordred: If I am understanding this code right, it sounds like I might need a different fundamental class than Resource, something like BulkResource that understands that instead of getting **attrs and returning Resource will be getting [attrs, attrs, attrs] and returning [Resource, Resource, Resource]. Does that sound like a sane proposition?
22:18:33 mordred njohnston: maybe? but also - maybe it's just a method on the proxy? (like, making a BulkResource might be a significant engineering effort - and the main bits of Resource are largely about expressing the remote objects
22:19:05 mordred njohnston: that said - maybe you do want a BulkResource object that takes another resource class as a parameter and does like you said
22:20:16 mordred njohnston: so that instead of conn.network.create_ports([args, args, args]) returning [Port, Port, Port] it returns BulkResource which wraps a list of Ports?
22:20:43 mordred njohnston: I could go either way - might be one of those "get in to it and see which sucks less" sorts of things
22:24:48 njohnston yeah, sounds like it. :-)
22:37:08 johnsom Thank you for the rechecks. Was just going to check on the gate status.
22:41:29 mordred johnsom: I was so excited about the gate status
23:03:48 openstackgerrit Merged openstack/openstacksdk master: Add Octavia (load_balancer) load balancer failover https://review.openstack.org/634010
23:52:50 openstackgerrit Merged openstack/openstacksdk master: Add Octavia (load_balancer) provider API support https://review.openstack.org/634488
23:52:52 openstackgerrit Merged openstack/openstacksdk master: Add Octavia (load_balancer) flavor profile API https://review.openstack.org/634518
23:52:54 openstackgerrit Merged openstack/openstacksdk master: Add Octavia (load_balancer) flavor API https://review.openstack.org/634532
#openstack-sdks - 2019-02-22
01:14:24 openstackgerrit Paul Belanger proposed openstack/openstacksdk master: Fix syntax error with exception handling https://review.openstack.org/638564
02:15:08 openstackgerrit Merged openstack/openstacksdk master: Add Octavia (load_balancer) amphora API https://review.openstack.org/634538
02:31:12 openstackgerrit Merged openstack/openstacksdk master: Add support for bodyless commits https://review.openstack.org/634558
04:56:49 openstackgerrit Merged openstack/openstacksdk master: Fix syntax error with exception handling https://review.openstack.org/638564
11:09:30 openstackgerrit David Rabel proposed openstack/python-openstackclient master: Only allow project ID in 'image add project' https://review.openstack.org/638495
14:22:56 openstackgerrit David Rabel proposed openstack/python-openstackclient master: Add --attach-status parameter to image set https://review.openstack.org/638671
14:24:11 mordred dtantsur, Shrews: https://review.openstack.org/#/c/637563/ is happy now - and has tests!
14:25:28 Shrews mordred: https://review.openstack.org/#/c/637563/3/openstack/service_description.py line 197... that return will never happen
14:25:44 Shrews will it? can you both raise and return?
14:25:54 Shrews is this some secret python black magic?
14:26:19 mordred Shrews: you are right - that return will never happen
14:26:48 Shrews aww shoot. i was hoping for the voodoo stuff
14:27:28 mordred sorry bout that - let me rev that real quick
14:28:14 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Bail earlier on a version mismatch for a supported service https://review.openstack.org/637563
14:28:19 mordred Shrews: thereyago
14:28:57 Shrews +2
14:29:51 frickler openstacksdk-ansible-devel-functional-devstack seems to be failing for some time and the error looks consistently like this http://logs.openstack.org/38/637238/3/check/openstacksdk-ansible-devel-functional-devstack/ca428a7/job-output.txt.gz#_2019-02-21_20_47_46_673039
14:32:37 mordred frickler: I think gtema had identified an ansible-side issue the other day
14:32:52 gtema yupp
14:33:11 gtema port_security_enabled was set to false by default
14:33:20 gtema in the os_network module
14:33:32 mordred and it probably wants to be None right?
14:33:34 gtema but I didn't manage to create PR in ansible for that
14:33:39 gtema yes, should be none
14:33:42 mordred I can do that real quick
14:36:13 frickler yeah, port_security_enabled=False is a very bad default
14:40:06 mordred frickler, gtema, Shrews, mnaser: https://github.com/ansible/ansible/pull/52821
14:41:09 frickler mordred: lgtm, thx. odyssey4me ^^
14:41:45 mordred frickler, odyssey4me: we should be able to see that -devel job go green on that PR
14:46:23 frickler mordred: is it expected that the details for that job on github only point to https://zuul.openstack.org/ and not to specific results?
14:47:33 frickler makes fixing issues probably a bit difficult. see e.g. https://github.com/ansible/ansible/pull/52699
14:47:59 frickler the shippable check points to results instead

Earlier   Later