| *** markvoelker has joined #openstack-sdks | 00:19 | |
| *** tosky has quit IRC | 00:19 | |
| mordred | slaweq: let me know if I can be helpful investigating that devstack/neutron thing - it's hitting all of our patches now | 00:48 |
|---|---|---|
| mordred | slaweq: I'm pretty open tomorrow and can debug or whatnot if it's helpful | 00:48 |
| *** slaweq has quit IRC | 00:52 | |
| *** whoami-rajat has joined #openstack-sdks | 01:12 | |
| *** nicholas has quit IRC | 02:03 | |
| *** keekz has joined #openstack-sdks | 02:05 | |
| *** purplerbot has quit IRC | 02:43 | |
| *** purplerbot has joined #openstack-sdks | 02:44 | |
| *** markvoelker has quit IRC | 03:20 | |
| *** markvoelker has joined #openstack-sdks | 04:21 | |
| *** markvoelker has quit IRC | 04:55 | |
| *** markvoelker has joined #openstack-sdks | 05:52 | |
| *** markvoelker has quit IRC | 06:25 | |
| *** Luzi has joined #openstack-sdks | 06:40 | |
| *** slaweq has joined #openstack-sdks | 07:08 | |
| *** markvoelker has joined #openstack-sdks | 07:22 | |
| *** markvoelker has quit IRC | 07:55 | |
| slaweq | mordred: frickler: I found the culprit of Your issue :) | 08:03 |
| slaweq | mordred: frickler: it is this patch https://review.openstack.org/#/q/e8b7e768a2545621ee98511b8dd271c5117f76bd | 08:03 |
| slaweq | I proposed revert (just in case): https://review.openstack.org/#/c/638359/1 | 08:03 |
| slaweq | but I will try to fix it in better way than only reverting | 08:03 |
| slaweq | maybe it will not be very hard :) | 08:04 |
| *** jpich has joined #openstack-sdks | 08:23 | |
| *** tosky has joined #openstack-sdks | 08:26 | |
| *** jpena|off is now known as jpena | 08:29 | |
| *** ttsiouts has joined #openstack-sdks | 08:36 | |
| *** ralonsoh has joined #openstack-sdks | 08:51 | |
| *** markvoelker has joined #openstack-sdks | 08:52 | |
| *** dtantsur|afk is now known as dtantsur\ | 08:53 | |
| *** dtantsur\ is now known as dtantsur | 08:53 | |
| frickler | slaweq: oh, great, so this is only triggered by port-forwarding being enabled? and that scenario is missing in neutron gate? | 09:13 |
| *** e0ne has joined #openstack-sdks | 09:14 | |
| slaweq | frickler: to be strict it is problem with multiple loading L3 service plugin | 09:15 |
| slaweq | because now it is added as "requirement" for port forwarding service plugin and is loaded from config too in Your scenario | 09:15 |
| slaweq | and that cause this issue | 09:15 |
| slaweq | I'm know what is wrong and now I'm trying to change the logic in such way that it will not init service plugin class twice | 09:16 |
| frickler | slaweq: o.k., that sounds plausible, I'm still worried that the patch did pass neutron gate | 09:19 |
| slaweq | frickler: yes, that is something else what I will have to check | 09:19 |
| slaweq | frickler: but first I would like to fix the issue :) | 09:20 |
| *** markvoelker has quit IRC | 09:25 | |
| *** gtema has joined #openstack-sdks | 09:26 | |
| frickler | slaweq: sure. fyi I confirmed that your revert fixes the issue in my local devstack, but I'm assuming you are much further in your analysis anyway by now ;) | 09:53 |
| slaweq | frickler: yes, I have locally patch which should fix this issue without reverting | 09:54 |
| slaweq | frickler: I hope I will finish it in 1-2 hours and will send to review | 09:54 |
| *** markvoelker has joined #openstack-sdks | 10:17 | |
| openstackgerrit | Jens Harbott (frickler) proposed openstack/openstacksdk master: DNM: Dummy patch to verify neutron fix https://review.openstack.org/638385 | 10:37 |
| frickler | slaweq: ^^ testing your patch here | 10:37 |
| slaweq | frickler: ok, thx | 10:38 |
| slaweq | frickler: I think I know why it didn't failed in neutron gate | 10:39 |
| slaweq | frickler: we simply don't have any API/scenario tests for port forwarding and we are not configuring this service plugin in our jobs | 10:40 |
| slaweq | frickler: and that is the only plugin which for now could trigger this issue :) | 10:40 |
| slaweq | I will open another bug for that but I don't know exactly when I will have some time to get to it | 10:40 |
| frickler | slaweq: maybe just running one of the sdk jobs in neutron as non-voting could be good enough. or at least until you get something proper into neutron-tempest-plugin scenarios | 10:43 |
| Shrews | mordred: coolio | 10:44 |
| Shrews | slaweq: thx for looking into that neutron issue | 10:44 |
| slaweq | frickler: in last denver PTG we agreed that we can add some stadium projects' jobs to neutron queue to avoid such breaks | 10:44 |
| slaweq | frickler: I think that adding SDK job there would be also good idea | 10:44 |
| slaweq | as it is know it may have some value :) | 10:45 |
| slaweq | frickler: should I do it or You will propose patch? | 10:45 |
| frickler | slaweq: I can do it, then you can +2 more easily ;) | 10:45 |
| slaweq | Shrews: no problem :) I was one of those who approved broken patch ;) | 10:45 |
| slaweq | frickler: sure, good idea | 10:45 |
| *** ttsiouts has quit IRC | 11:13 | |
| *** ttsiouts has joined #openstack-sdks | 11:14 | |
| *** cdent has joined #openstack-sdks | 11:15 | |
| *** ttsiouts has quit IRC | 11:18 | |
| *** cdent has quit IRC | 11:23 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: handle "paginated" argument in test_list properly https://review.openstack.org/638401 | 11:56 |
| *** jpena is now known as jpena|lunch | 11:56 | |
| *** ttsiouts has joined #openstack-sdks | 11:56 | |
| slaweq | frickler: looks like my patch works and sdk job is happy http://logs.openstack.org/85/638385/1/check/openstacksdk-functional-devstack-networking/a2a55f6/testr_results.html.gz :) | 12:01 |
| frickler | slaweq: cool, good work :-) | 12:08 |
| *** dayou has quit IRC | 12:19 | |
| *** dayou has joined #openstack-sdks | 12:23 | |
| *** e0ne has quit IRC | 12:24 | |
| slaweq | frickler: thx :) | 12:24 |
| *** edleafe has joined #openstack-sdks | 12:30 | |
| *** e0ne has joined #openstack-sdks | 12:31 | |
| *** gtema has quit IRC | 12:59 | |
| *** jpena|lunch is now known as jpena | 13:15 | |
| *** cdent has joined #openstack-sdks | 13:16 | |
| *** keekz is now known as nicholas | 13:17 | |
| *** markvoelker has quit IRC | 13:17 | |
| *** markvoelker has joined #openstack-sdks | 13:18 | |
| *** markvoelker has quit IRC | 13:22 | |
| *** gtema has joined #openstack-sdks | 13:35 | |
| *** gtema has quit IRC | 14:18 | |
| *** markvoelker has joined #openstack-sdks | 14:18 | |
| *** elmiko has joined #openstack-sdks | 14:39 | |
| *** dave-mccowan has joined #openstack-sdks | 14:47 | |
| *** ttsiouts has quit IRC | 14:51 | |
| *** ttsiouts has joined #openstack-sdks | 14:52 | |
| *** markvoelker has quit IRC | 14:53 | |
| *** ttsiouts has quit IRC | 14:54 | |
| *** ttsiouts has joined #openstack-sdks | 14:54 | |
| mordred | slaweq, frickler: yay! | 14:59 |
| *** Luzi has quit IRC | 15:12 | |
| elmiko | edleafe: just a heads up, i have a conflicting meeting today so i will be around but on a high latency delay | 15:18 |
| elmiko | dtantsur too ^^ | 15:18 |
| edleafe | elmiko: I'm around, but I have a PT sessions scheduled that will keep me from my computer | 15:18 |
| elmiko | k, i'll try to keep an eye out | 15:18 |
| *** dave-mccowan has quit IRC | 15:21 | |
| dtantsur | I also have a conflicting meeting btw | 15:22 |
| elmiko | haha | 15:22 |
| elmiko | ok, i'll keep an eye on the chat | 15:23 |
| *** seeni has joined #openstack-sdks | 15:35 | |
| *** dave-mccowan has joined #openstack-sdks | 15:41 | |
| *** dave-mccowan has quit IRC | 15:45 | |
| *** markvoelker has joined #openstack-sdks | 15:50 | |
| *** tosky has quit IRC | 15:53 | |
| *** tosky has joined #openstack-sdks | 15:54 | |
| elmiko | API SIG office hour starts in 5 minutes, prepare your inquiries! | 15:56 |
| *** ttsiouts has quit IRC | 16:01 | |
| *** ttsiouts has joined #openstack-sdks | 16:02 | |
| *** ttsiouts has quit IRC | 16:06 | |
| edleafe | ...and I'm back | 16:17 |
| *** e0ne has quit IRC | 16:23 | |
| *** markvoelker has quit IRC | 16:23 | |
| 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:28 |
| 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 |
| elmiko | cdent: good question | 16:39 |
| edleafe | "Obliged"? | 16:40 |
| 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 |
| mordred | cdent: when you say nested json objects ... | 16:40 |
| elmiko | yeah "obliged" sounds kinda strong to me, especially if there is no value in exposing the subresources by themselves | 16:40 |
| mordred | (yeah, I agree with edleafe and elmiko on the obligation part) | 16:40 |
| *** seeni has quit IRC | 16:41 | |
| 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 |
| dtantsur | :) | 16:42 |
| edleafe | Ah, I thought someone had posed such a question to you and you were passing it along | 16:43 |
| 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 |
| elmiko | haha, cdent ++ | 16:43 |
| * mordred hands cdent a bucket of sloshing something | 16:43 | |
| cdent | beer? | 16:43 |
| elmiko | i say no to "obliged", yes to "if there is value in exposing the subresource" | 16:43 |
| mordred | cdent: maybe? | 16:45 |
| cdent | I also say no to "obliged" but I know there are some who would feel that way for sake of grammatical parallelism | 16:45 |
| elmiko | i mean, if you have the cycles to burn.... | 16:46 |
| *** dave-mccowan has joined #openstack-sdks | 16:47 | |
| dtantsur | everyone here has a lot of spare time, why even ask? | 16:52 |
| dtantsur | :) | 16:52 |
| elmiko | now if you are using automagical tooling to create your server... XD | 16:56 |
| dtantsur | sounds ironic! | 16:57 |
| elmiko | haha | 16:57 |
| *** jpich has quit IRC | 17:06 | |
| elmiko | gonna wander off for some lunch, good chatting with y'all as usual =) | 17:12 |
| dtantsur | enjoy | 17:13 |
| *** markvoelker has joined #openstack-sdks | 17:20 | |
| *** dtantsur is now known as dtantsur|afk | 17:29 | |
| *** markvoelker has quit IRC | 17:53 | |
| *** jpena is now known as jpena|off | 18:29 | |
| *** markvoelker has joined #openstack-sdks | 18:50 | |
| *** ralonsoh has quit IRC | 18:56 | |
| *** markvoelker has quit IRC | 19:23 | |
| openstackgerrit | David Rabel proposed openstack/python-openstackclient master: Only allow project ID in 'image add project' https://review.openstack.org/638495 | 19:34 |
| openstackgerrit | David Rabel proposed openstack/python-openstackclient master: Fix help message of image add project https://review.openstack.org/638113 | 19:35 |
| *** whoami-rajat has quit IRC | 20:02 | |
| *** e0ne has joined #openstack-sdks | 20:14 | |
| *** markvoelker has joined #openstack-sdks | 20:21 | |
| *** e0ne has quit IRC | 20:37 | |
| *** cdent has quit IRC | 20:38 | |
| openstackgerrit | Merged openstack/openstacksdk master: Add image attributes from v2.7 https://review.openstack.org/637538 | 20:50 |
| elmiko | anyone around who understand pbr reasonably well? | 20:53 |
| *** markvoelker has quit IRC | 20:55 | |
| mordred | elmiko: yup. | 21:05 |
| 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:13 |
| mordred | elmiko: I have not run in to that although that makes me have sads :( | 21:15 |
| elmiko | mordred: definitely giving me a case of the sads | 21:15 |
| elmiko | i can work around it, just curious why it happens | 21:16 |
| 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:17 |
| elmiko | mordred: ack, i am working locally and usually this works. i think i must have futzed with the versions or something. | 21:25 |
| elmiko | what does the `-e` flag do? | 21:26 |
| mordred | elmiko: it does an 'editable' install - it's like python setup.py develop | 21:27 |
| elmiko | mordred: interesting, thanks! | 21:28 |
| mordred | elmiko: so, basically, it installs links to your source dir - and as you change python files it's immediately reflected | 21:28 |
| elmiko | that is super useful | 21:28 |
| mordred | if you add new entrypoints in setup.cfg you'll need to re-run pip install -e . | 21:28 |
| mordred | just fwiw | 21:28 |
| elmiko | ack, thanks again =) | 21:33 |
| elmiko | mordred ++ | 21:33 |
| *** slaweq has quit IRC | 21:35 | |
| *** njohnston has joined #openstack-sdks | 21:41 | |
| 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:44 |
| njohnston | list of objects returned? | 21:44 |
| *** slaweq has joined #openstack-sdks | 21:48 | |
| 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 |
| mordred | njohnston: that said - maybe once you've got that we can abstract out a _create_list or something like that | 21:49 |
| njohnston | mordred: Great, that is what I thought! I'll start working on it. Thanks! | 21:50 |
| mordred | \o/ | 21:51 |
| *** markvoelker has joined #openstack-sdks | 21:53 | |
| 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:11 |
| 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:18 |
| 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:19 |
| 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 |
| 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:20 |
| njohnston | yeah, sounds like it. :-) | 22:24 |
| *** markvoelker has quit IRC | 22:25 | |
| johnsom | Thank you for the rechecks. Was just going to check on the gate status. | 22:37 |
| *** dave-mccowan has quit IRC | 22:40 | |
| mordred | johnsom: I was so excited about the gate status | 22:41 |
| *** slaweq has quit IRC | 22:43 | |
| *** slaweq has joined #openstack-sdks | 22:55 | |
| *** slaweq has quit IRC | 23:00 | |
| openstackgerrit | Merged openstack/openstacksdk master: Add Octavia (load_balancer) load balancer failover https://review.openstack.org/634010 | 23:03 |
| *** slaweq has joined #openstack-sdks | 23:11 | |
| *** tosky has quit IRC | 23:11 | |
| *** slaweq has quit IRC | 23:16 | |
| *** markvoelker has joined #openstack-sdks | 23:22 | |
| openstackgerrit | Merged openstack/openstacksdk master: Add Octavia (load_balancer) provider API support https://review.openstack.org/634488 | 23:52 |
| openstackgerrit | Merged openstack/openstacksdk master: Add Octavia (load_balancer) flavor profile API https://review.openstack.org/634518 | 23:52 |
| openstackgerrit | Merged openstack/openstacksdk master: Add Octavia (load_balancer) flavor API https://review.openstack.org/634532 | 23:52 |
| *** markvoelker has quit IRC | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!