| *** dave-mccowan has quit IRC | 00:08 | |
| *** tosky has quit IRC | 00:27 | |
| *** slaweq has joined #openstack-sdks | 00:45 | |
| *** slaweq has quit IRC | 00:49 | |
| *** dave-mccowan has joined #openstack-sdks | 01:50 | |
| *** mriedem_afk has quit IRC | 02:19 | |
| *** dave-mccowan has quit IRC | 02:46 | |
| *** dave-mccowan has joined #openstack-sdks | 02:55 | |
| *** dave-mccowan has quit IRC | 03:33 | |
| *** dave-mccowan has joined #openstack-sdks | 03:34 | |
| *** dave-mccowan has quit IRC | 04:30 | |
| *** markvoelker has quit IRC | 04:32 | |
| *** markvoelker has joined #openstack-sdks | 05:02 | |
| *** dasp has quit IRC | 05:03 | |
| *** dasp_ has joined #openstack-sdks | 05:03 | |
| *** e0ne has joined #openstack-sdks | 06:32 | |
| openstackgerrit | Rabi Mishra proposed openstack/osc-lib master: Fix formatter handling for python 3.7 https://review.openstack.org/618137 | 06:52 |
|---|---|---|
| openstackgerrit | Filippo Inzaghi proposed openstack/python-openstackclient master: Add Python 3.6 classifier to setup.cfg https://review.openstack.org/621096 | 07:13 |
| *** e0ne has quit IRC | 07:31 | |
| *** slaweq has joined #openstack-sdks | 07:34 | |
| openstackgerrit | Radoslaw Smigielski proposed openstack/python-openstackclient master: Fix --limit option in image list sub-command https://review.openstack.org/617139 | 07:36 |
| *** Luzi has joined #openstack-sdks | 07:38 | |
| *** gtema has joined #openstack-sdks | 07:56 | |
| *** jpena|off is now known as jpena | 08:01 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Refactor tag support https://review.openstack.org/618993 | 08:26 |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Refactor tag support https://review.openstack.org/618993 | 08:27 |
| *** tosky has joined #openstack-sdks | 08:42 | |
| *** jpich has joined #openstack-sdks | 08:53 | |
| gtema | dtantsur|msg, are you here? | 09:23 |
| dtantsur|mtg | gtema: I am | 09:24 |
| gtema | dtantsur|msg: as a continue to removal of xxxDetails I have found similar issue in orchestrate | 09:25 |
| gtema | but the difference is that it is not list, but create/update | 09:25 |
| dtantsur|mtg | I think all Details are broken more or less | 09:25 |
| gtema | basically StackPreview | 09:25 |
| gtema | I feel it would be good to apply the same approach as for Details here, but this would mean extension for also other request types | 09:26 |
| gtema | and extending basic proxy/resource/test_proxy | 09:26 |
| gtema | this is however becoming relatively big | 09:26 |
| gtema | I mean everywhere adding optional base_path | 09:26 |
| gtema | wanted to ask, what is your opinion here? | 09:27 |
| *** e0ne has joined #openstack-sdks | 09:27 | |
| dtantsur|mtg | I need to think about it, I'm quite sleepy right now :) and I'm not familiar with orchestration enough | 09:28 |
| gtema | it's not about orchestration itself. I am asking more about adding base_path optional parameter not only for "list", what you did for Details, but also for other request types | 09:28 |
| gtema | use case: upon some parameter to proxy (i.e. preview) the endpoint for "create" is different | 09:30 |
| *** ttsiouts has joined #openstack-sdks | 09:41 | |
| *** ttsiouts has quit IRC | 09:44 | |
| dtantsur|mtg | gtema: this can be done in _prepare_request (used by everything except for list) | 09:44 |
| *** ttsiouts has joined #openstack-sdks | 09:44 | |
| gtema | sure, this is what I have done. But you would first need to also extend Proxy._create (and _update) and pass this optional parameter into Resource | 09:45 |
| gtema | and this mean that test_proxy_base should be also extended with additional check of base_path for such calls | 09:45 |
| gtema | that's what I ment "is getting big" | 09:46 |
| gtema | and wanted to check before I land a patch not to face resistence | 09:47 |
| gtema | s/resistence/resistance/ | 09:47 |
| *** ttsiouts has quit IRC | 09:49 | |
| dtantsur|mtg | the approach seems good to me at first glance, but I really need to see the code | 09:58 |
| gtema | ok, than I will work further | 09:58 |
| gtema | thanks | 09:58 |
| *** ShilpaSD has joined #openstack-sdks | 10:06 | |
| *** olivierbourdon38 has joined #openstack-sdks | 10:09 | |
| *** ttsiouts has joined #openstack-sdks | 10:27 | |
| *** slaweq has quit IRC | 10:32 | |
| *** ttsiouts has quit IRC | 10:55 | |
| *** ttsiouts has joined #openstack-sdks | 10:55 | |
| *** cdent has joined #openstack-sdks | 10:59 | |
| *** ShilpaSD has quit IRC | 11:00 | |
| *** ttsiouts has quit IRC | 11:00 | |
| *** dtantsur|mtg is now known as dtantsur|afk | 11:00 | |
| *** cdent has quit IRC | 11:20 | |
| *** gtema has quit IRC | 11:38 | |
| *** cdent has joined #openstack-sdks | 11:46 | |
| *** gtema has joined #openstack-sdks | 11:49 | |
| *** slaweq has joined #openstack-sdks | 11:51 | |
| *** Luzi has quit IRC | 12:00 | |
| *** slaweq has quit IRC | 12:24 | |
| *** jpena is now known as jpena|lunch | 12:41 | |
| *** e0ne has quit IRC | 12:54 | |
| *** ttsiouts has joined #openstack-sdks | 13:00 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153 | 13:02 |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154 | 13:04 |
| *** dave-mccowan has joined #openstack-sdks | 13:20 | |
| *** annp has quit IRC | 13:29 | |
| *** cdent has quit IRC | 13:37 | |
| *** e0ne has joined #openstack-sdks | 13:37 | |
| *** jpena|lunch is now known as jpena | 13:40 | |
| *** ttsiouts has quit IRC | 13:54 | |
| *** ttsiouts has joined #openstack-sdks | 13:55 | |
| *** slaweq has joined #openstack-sdks | 13:56 | |
| *** ttsiouts has quit IRC | 14:00 | |
| *** nicholas has joined #openstack-sdks | 14:02 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153 | 14:11 |
| *** ttsiouts has joined #openstack-sdks | 14:15 | |
| *** mriedem has joined #openstack-sdks | 14:19 | |
| *** olivierbourdon38 has quit IRC | 14:20 | |
| *** olivierbourdon38 has joined #openstack-sdks | 14:20 | |
| *** cdent has joined #openstack-sdks | 14:32 | |
| *** lbragstad is now known as elbragstad | 14:36 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154 | 14:46 |
| *** ttsiouts has quit IRC | 14:54 | |
| *** ttsiouts has joined #openstack-sdks | 14:55 | |
| *** ttsiouts has quit IRC | 14:56 | |
| *** ttsiouts has joined #openstack-sdks | 14:57 | |
| *** ttsiouts has quit IRC | 14:58 | |
| *** dayou_ has joined #openstack-sdks | 14:59 | |
| *** dayou has quit IRC | 15:00 | |
| *** dayou_ has quit IRC | 15:10 | |
| *** dayou_ has joined #openstack-sdks | 15:11 | |
| *** mriedem has quit IRC | 15:50 | |
| *** dayou_ has quit IRC | 16:19 | |
| *** mriedem has joined #openstack-sdks | 16:21 | |
| *** ttsiouts has joined #openstack-sdks | 16:38 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154 | 16:39 |
| *** ttsiouts has quit IRC | 16:39 | |
| *** ttsiouts has joined #openstack-sdks | 16:40 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153 | 16:41 |
| *** dayou_ has joined #openstack-sdks | 16:42 | |
| *** ttsiouts has quit IRC | 16:44 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154 | 16:48 |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Rework orchestration to add update preview https://review.openstack.org/621154 | 16:57 |
| *** jpena is now known as jpena|off | 16:59 | |
| *** jpich has quit IRC | 17:00 | |
| *** gtema has quit IRC | 17:04 | |
| *** mriedem is now known as mriedem_lunch | 17:11 | |
| *** fungi has joined #openstack-sdks | 17:15 | |
| *** e0ne has quit IRC | 17:45 | |
| *** openstackgerrit has quit IRC | 17:51 | |
| *** openstackgerrit has joined #openstack-sdks | 18:40 | |
| openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: WIP Fix version discovery for rackspace public cloud https://review.openstack.org/621257 | 18:40 |
| kmalloc | mordred: ^ that makes me sad. | 18:49 |
| Shrews | kmalloc: yah, he's working on a better thilng | 18:49 |
| kmalloc | i figured | 18:49 |
| kmalloc | but man. | 18:49 |
| kmalloc | gross. | 18:50 |
| kmalloc | cmurphy: ^ cc | 18:50 |
| openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Fix version discovery for clouds with int project_ids https://review.openstack.org/621257 | 18:58 |
| *** olivierbourdon38 has quit IRC | 19:05 | |
| openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Fix version discovery for clouds with int project_ids https://review.openstack.org/621257 | 19:09 |
| *** mriedem_lunch is now known as mriedem | 19:13 | |
| openstackgerrit | Nathan Kinder proposed openstack/osc-lib master: Default to using public interface for all clients https://review.openstack.org/621276 | 19:35 |
| mordred | kmalloc: ^^ care to weigh in on that one? | 20:00 |
| kmalloc | the most recent one? | 20:01 |
| kmalloc | or the nkinder change? | 20:01 |
| kmalloc | cause YES PUBLIC INTERFACES | 20:02 |
| kmalloc | mordred: +1 (since i don't have +2 on that repo) | 20:03 |
| kmalloc | but i'd +2 that if i could | 20:03 |
| mordred | kmalloc: sweet. perhaps dhellmann or dtroyer will do the +3 | 20:18 |
| dhellmann | lgtm | 20:21 |
| * dhellmann takes his cold back to bed now | 20:21 | |
| *** slaweq has quit IRC | 20:34 | |
| *** openstackgerrit has quit IRC | 20:50 | |
| larsks | mordred: https://review.openstack.org/#/c/618519/ has two +2s and +verified, but it hasn't merged yet. Is something stuck? | 21:14 |
| mordred | larsks: it's marked cannot merge | 21:15 |
| *** openstackgerrit has joined #openstack-sdks | 21:15 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: syntax errors and undefined exceptions in service_description.py https://review.openstack.org/618519 | 21:15 |
| mordred | larsks: I rebased it - let's see if that fixes the issue | 21:15 |
| larsks | mordred: I must not have been looking at the right spot. Thanks! | 21:16 |
| mordred | larsks: fingers crossed gerrit likes us this time | 21:16 |
| openstackgerrit | Monty Taylor proposed openstack/promstat master: Add initial code https://review.openstack.org/621295 | 21:40 |
| *** slaweq has joined #openstack-sdks | 22:36 | |
| *** slaweq has quit IRC | 22:43 | |
| *** slaweq has joined #openstack-sdks | 22:50 | |
| *** slaweq has quit IRC | 22:52 | |
| *** mriedem has quit IRC | 22:54 | |
| *** slaweq has joined #openstack-sdks | 22:54 | |
| *** Miouge- has quit IRC | 23:05 | |
| *** cdent has quit IRC | 23:07 | |
| *** Miouge has joined #openstack-sdks | 23:08 | |
| *** slaweq has quit IRC | 23:22 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Transform server with munch before normalizing https://review.openstack.org/621316 | 23:29 |
| mordred | Shrews: ^^ | 23:30 |
| *** tosky has quit IRC | 23:42 | |
| *** pbourke has quit IRC | 23:47 | |
| *** pbourke has joined #openstack-sdks | 23:49 | |
| *** slaweq has joined #openstack-sdks | 23:54 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!