| *** pm90_ has quit IRC | 00:11 | |
| *** thrash is now known as thrash|g0ne | 00:33 | |
| *** Qiming has joined #openstack-sdks | 00:47 | |
| *** jose4183 has quit IRC | 00:48 | |
| *** gildub has joined #openstack-sdks | 00:53 | |
| *** gildub has quit IRC | 00:55 | |
| openstackgerrit | Monty Taylor proposed openstack/os-client-config: Do not treat project_name and project_id the same https://review.openstack.org/214382 | 01:13 |
|---|---|---|
| *** Yanyanhu has joined #openstack-sdks | 01:15 | |
| *** Yanyan has joined #openstack-sdks | 01:22 | |
| *** gildub has joined #openstack-sdks | 01:23 | |
| *** pm90_ has joined #openstack-sdks | 01:24 | |
| *** Yanyanhu has quit IRC | 01:25 | |
| *** liangbo has joined #openstack-sdks | 01:26 | |
| *** liangbo has quit IRC | 01:31 | |
| mordred | terrylhowe, GheRivero, Shrews: ^^ that cropped up as a problem for asselin - we were relying on a ?bug? from before that you could pass a name to the id parameter - but that apparently does not work in all cases | 01:37 |
| *** Qiming has quit IRC | 01:39 | |
| *** britthouser has joined #openstack-sdks | 01:48 | |
| *** britthou_ has joined #openstack-sdks | 01:50 | |
| *** britthouser has quit IRC | 01:53 | |
| *** Qiming has joined #openstack-sdks | 01:53 | |
| *** liangbo has joined #openstack-sdks | 01:58 | |
| *** Qiming_ has joined #openstack-sdks | 02:03 | |
| *** Qiming has quit IRC | 02:05 | |
| *** Qiming__ has joined #openstack-sdks | 02:05 | |
| *** Qiming_ has quit IRC | 02:06 | |
| *** Qiming__ has quit IRC | 02:15 | |
| *** gildub has quit IRC | 02:27 | |
| *** Qiming has joined #openstack-sdks | 02:41 | |
| *** gildub has joined #openstack-sdks | 02:47 | |
| *** Qiming has quit IRC | 02:55 | |
| *** Qiming has joined #openstack-sdks | 03:03 | |
| *** liangbo has quit IRC | 03:09 | |
| *** Qiming has quit IRC | 03:55 | |
| *** Qiming has joined #openstack-sdks | 03:56 | |
| *** Qiming has quit IRC | 04:06 | |
| *** aufi has joined #openstack-sdks | 05:01 | |
| *** liangbo has joined #openstack-sdks | 05:12 | |
| *** Qiming has joined #openstack-sdks | 05:14 | |
| *** boris-42 has quit IRC | 05:20 | |
| *** Qiming has quit IRC | 05:26 | |
| *** liangbo has quit IRC | 05:27 | |
| *** Qiming has joined #openstack-sdks | 05:29 | |
| *** Qiming has quit IRC | 05:29 | |
| *** Qiming has joined #openstack-sdks | 05:30 | |
| *** Qiming has quit IRC | 05:43 | |
| *** Qiming has joined #openstack-sdks | 05:43 | |
| *** Qiming_ has joined #openstack-sdks | 05:46 | |
| *** Qiming_ has quit IRC | 05:47 | |
| *** Qiming has quit IRC | 05:50 | |
| *** Qiming has joined #openstack-sdks | 05:55 | |
| *** britthou_ has quit IRC | 06:21 | |
| *** britthouser has joined #openstack-sdks | 06:22 | |
| *** markus_z has joined #openstack-sdks | 06:30 | |
| *** woodster_ has quit IRC | 06:49 | |
| *** jlvillal has quit IRC | 06:58 | |
| *** jlvillal has joined #openstack-sdks | 06:58 | |
| *** boris-42 has joined #openstack-sdks | 07:58 | |
| *** gildub has quit IRC | 08:19 | |
| *** liangbo has joined #openstack-sdks | 08:25 | |
| *** liangbo has quit IRC | 09:19 | |
| *** Qiming has quit IRC | 09:30 | |
| *** Qiming has joined #openstack-sdks | 09:38 | |
| *** sigmavirus24_awa has quit IRC | 10:06 | |
| *** Yanyan has quit IRC | 10:07 | |
| *** sigmavirus24_awa has joined #openstack-sdks | 10:11 | |
| terrylhowe | modred That might of come from OSC where it always does a find on name or id so you could use either. My only concern with that patch is OSC is going to have to look at project_id so that just needs to be coordinated. | 10:57 |
| *** britthouser has quit IRC | 11:10 | |
| *** boris-42 has quit IRC | 11:20 | |
| *** Kiall_ is now known as Kiall | 11:34 | |
| *** Qiming has quit IRC | 11:35 | |
| *** thrash|g0ne is now known as thrash | 11:39 | |
| Shrews | terrylhowe: agreed. i want OSC to be tested with it before it gets approved, so I'll let you have final approval on it. | 11:44 |
| terrylhowe | I’ll get a patch together for osc today Shrews | 11:45 |
| Kiall | terrylhowe: Q - we're adding a CLI for Designate's V2 API - It's coded up as a osc plugin, but we're thinking it should land in the osc repo rather than ours.. Thoughts? | 11:51 |
| Kiall | Reviews either way would also be cool https://review.openstack.org/#/c/133676/ :) | 11:51 |
| terrylhowe | Kiall: I’ll take a look. I think the OSC PTL Dean Troyer (who is afk for another week or two) wants to help projects with integrations as a plugin, but not necessarily include them in the osc repo. The advantage being projects can turn around changes a lot faster and as a plugin you someone else is dealing with a lot of the basic CLI options, auth, and transport/session setup. | 11:56 |
| Kiall | Ah - I've been wondering why he's not been on IRC recently :) | 11:57 |
| *** jose4183 has joined #openstack-sdks | 12:00 | |
| *** jose4183 has quit IRC | 12:05 | |
| *** jose4183 has joined #openstack-sdks | 12:21 | |
| *** chlong has joined #openstack-sdks | 12:26 | |
| *** britthouser has joined #openstack-sdks | 12:28 | |
| *** gildub has joined #openstack-sdks | 12:43 | |
| openstackgerrit | Asha proposed openstack/python-openstackclient: Added note to install openstackclient https://review.openstack.org/214304 | 12:45 |
| *** jose4183 has quit IRC | 13:07 | |
| *** petertr7_away is now known as petertr7 | 13:11 | |
| *** jose4183 has joined #openstack-sdks | 13:14 | |
| *** pm90__ has joined #openstack-sdks | 13:23 | |
| *** pm90_ has quit IRC | 13:25 | |
| openstackgerrit | Merged openstack/os-client-config: Do not treat project_name and project_id the same https://review.openstack.org/214382 | 13:33 |
| *** zz_steveortiz is now known as steveortiz | 13:40 | |
| *** gildub has quit IRC | 14:02 | |
| *** petertr7 is now known as petertr7_away | 14:10 | |
| *** boris-42 has joined #openstack-sdks | 14:15 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:22 | |
| *** edleafe has quit IRC | 14:27 | |
| *** edleafe has joined #openstack-sdks | 14:28 | |
| etoews | terrylhowe: i'm looking at https://review.openstack.org/#/c/186140/ and wondering if ksa should be considered for the sdk 1.0 release. | 14:40 |
| *** petertr7_away is now known as petertr7 | 14:40 | |
| terrylhowe | I don’t know the exact schedule for ksa etoews I’ve been meaning to check with jamielennox | 14:41 |
| terrylhowe | that patch definitely needs work | 14:41 |
| *** Qiming has joined #openstack-sdks | 14:42 | |
| terrylhowe | I’ve been trying to pull changes out of it for general use. Some of them have been merged. | 14:42 |
| terrylhowe | If ksa is going to be ready in time, I think we should definitely use it because that will make us compatible with a lot of clients right away | 14:43 |
| etoews | one question for that patch. i see it deletes test_transport.py but not transport.py. is that just a consequence of needing more work? | 14:43 |
| terrylhowe | I think in rebasing some files came back and/or I forgot to merge some. I delete the whole auth dir except service_filter, but some files are there | 14:44 |
| terrylhowe | s/merge some/delete some | 14:45 |
| etoews | so ksa would remove the transport module? | 14:46 |
| terrylhowe | well, I think that would be best although I like our transport/session design a bit better in some ways. It will make us more compatible and there are some improvements in the ksa session | 14:47 |
| terrylhowe | is briancurtin around? | 14:47 |
| etoews | terrylhowe: so utilizing ksa would make it easier for the clients to adopt the sdk? | 14:48 |
| terrylhowe | definitely, they are using that session now from ksc | 14:48 |
| etoews | lemme check | 14:48 |
| etoews | he should be around in 10-15 | 14:49 |
| etoews | we need to follow up with the keystone people sooner than later on ksa. yes, it's an internal architectural change that could possibly happen after 1.0 but it's a pretty major change and (a) we need to figure out if it's appropriate (maybe that discussion already happened while i was away???) and (b) if so, figure out if it's pre or post 1.0. | 14:52 |
| *** jose4183 has quit IRC | 14:59 | |
| *** jose4183 has joined #openstack-sdks | 14:59 | |
| terrylhowe | maybe morgan_2549 knows the likely release schedule for ksa | 15:00 |
| morgan_2549 | Soon™ | 15:02 |
| morgan_2549 | We really are close | 15:03 |
| etoews | morgan_2549: is Soon™ measured in days, weeks, or months? | 15:07 |
| morgan_2549 | etoews: we have a couple more cleanup things to land and then need to do a serious review of everything | 15:07 |
| morgan_2549 | I hope weeks at most | 15:08 |
| etoews | morgan_2549: what's the best way to track progress on it? | 15:08 |
| morgan_2549 | Bug me and jamielennox and mordred | 15:08 |
| morgan_2549 | We find a gap and we fix it | 15:09 |
| morgan_2549 | But basically we need to respin a patch or two and get them approved | 15:09 |
| etoews | thx! | 15:09 |
| morgan_2549 | The major review is to make sure we didn't miss anything before we commit to the contract | 15:10 |
| *** jose4183 has quit IRC | 15:19 | |
| briancurtin | etoews: if ksa is measured in weeks, that'll fit within our 1.0, so should we maybe shift what you're trying to do with the logging stuff into that project? i had thought it was much further out, which was why a few days ago i had said to continue working out the anon logging with our current setup | 15:19 |
| *** jose4183 has joined #openstack-sdks | 15:20 | |
| etoews | from my understanding ksa scrubs headers and doesn’t log auth payloads so the anon stuff may be unnecessary. terrylhowe knows more. | 15:21 |
| etoews | as such, i think i'll let that patch cool for now until something definitive happens with ksa. | 15:22 |
| terrylhowe | ksa definitely handles redacting passwords and tokens | 15:22 |
| mordred | what did I do? | 15:22 |
| terrylhowe | we’d just need to watch if we log configuration | 15:22 |
| etoews | i'm thinking i'll just switch to chipping away at those user guides. | 15:22 |
| *** petertr7 is now known as petertr7_away | 15:26 | |
| *** petertr7_away is now known as petertr7 | 15:27 | |
| openstackgerrit | Asha proposed openstack/python-openstackclient: Added note to install openstackclient https://review.openstack.org/214304 | 15:29 |
| *** jose4183 has quit IRC | 15:34 | |
| *** britthouser has quit IRC | 15:35 | |
| briancurtin | etoews: sounds good | 15:36 |
| *** aufi has quit IRC | 15:47 | |
| *** petertr7 is now known as petertr7_away | 15:49 | |
| *** jose4183 has joined #openstack-sdks | 16:06 | |
| *** steveortiz is now known as zz_steveortiz | 16:07 | |
| *** Qiming has quit IRC | 16:07 | |
| *** zz_steveortiz is now known as steveortiz | 16:07 | |
| *** steveortiz is now known as zz_steveortiz | 16:10 | |
| *** zz_steveortiz is now known as steveortiz | 16:10 | |
| *** jose4183 has quit IRC | 16:11 | |
| *** pm90__ has quit IRC | 16:14 | |
| openstackgerrit | Merged stackforge/python-openstacksdk: Remove spaces around data in transport debug print https://review.openstack.org/214375 | 16:19 |
| *** petertr7_away is now known as petertr7 | 16:24 | |
| *** petertr7 is now known as petertr7_away | 16:34 | |
| *** steveortiz is now known as zz_steveortiz | 16:40 | |
| *** zz_steveortiz is now known as steveortiz | 16:44 | |
| *** pm90_ has joined #openstack-sdks | 17:00 | |
| *** britthouser has joined #openstack-sdks | 17:00 | |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: Update the plugin docs for designate https://review.openstack.org/214712 | 17:01 |
| *** mpmsimo has joined #openstack-sdks | 17:07 | |
| *** petertr7_away is now known as petertr7 | 17:07 | |
| *** mpmsimo has quit IRC | 17:10 | |
| *** markus_z has quit IRC | 17:14 | |
| *** steveortiz is now known as zz_steveortiz | 17:21 | |
| openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Properly pass on Container in path_args https://review.openstack.org/213278 | 17:24 |
| *** mpmsimo has joined #openstack-sdks | 17:25 | |
| *** pm90__ has joined #openstack-sdks | 17:26 | |
| *** britthouser has quit IRC | 17:27 | |
| *** britthouser has joined #openstack-sdks | 17:28 | |
| *** pm90_ has quit IRC | 17:29 | |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: SDK integration into OSC for network CRUD https://review.openstack.org/138745 | 17:32 |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: SDK integration into OSC for network CRUD https://review.openstack.org/138745 | 17:38 |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: SDK integration into OSC for network CRUD https://review.openstack.org/138745 | 17:41 |
| *** ameybhide has joined #openstack-sdks | 17:51 | |
| *** mpmsimo has quit IRC | 17:58 | |
| *** jose4183 has joined #openstack-sdks | 18:09 | |
| *** jose4183 has quit IRC | 18:12 | |
| *** jose4183 has joined #openstack-sdks | 18:12 | |
| *** stevemar has joined #openstack-sdks | 18:28 | |
| *** rm_work is now known as rm_work|away | 18:36 | |
| *** mpmsimo has joined #openstack-sdks | 18:41 | |
| briancurtin | terrylhowe, etoews: beware...test_debug_post seems to be somewhat flaky. i ran once and py27 failed, then i ran again and only pep8 passed, and now i just ran it and everything passed (i'm not doing anything remotely close to this code) | 18:46 |
| terrylhowe | strange | 18:48 |
| openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Fix find_extension for Network and Compute proxies https://review.openstack.org/214753 | 18:50 |
| *** rm_work|away is now known as rm_work | 18:57 | |
| *** stevemar has quit IRC | 19:07 | |
| *** zz_steveortiz is now known as steveortiz | 19:10 | |
| *** nikhil_k_ has quit IRC | 19:13 | |
| briancurtin | terrylhowe: i'm going to do some work on find -- the way it's written to use id= and name= query params will only work for network APIs. none of the other services seem to allow that level of filtering - they only support it on other attributes (like minRam, for example). i noticed it during the find_extensions change, and it is currently not possible to | 19:17 |
| briancurtin | find_flavors by name | 19:17 |
| *** mpmsimo has quit IRC | 19:18 | |
| openstackgerrit | Terry Howe proposed stackforge/python-openstacksdk: Make sure there is data for the meter test https://review.openstack.org/214763 | 19:18 |
| terrylhowe | well find by name does seem to work for things like servers I thought | 19:19 |
| terrylhowe | are you going to revive the client side filter briancurtin ? | 19:20 |
| briancurtin | terrylhowe: i have no idea yet, i just know something has to happen because looking at this right now there's zero chance of find_flavors("m1.tiny") working. it will probably have to just pull everything and look at the names | 19:21 |
| briancurtin | so, maybe, at least a little bit, but it'll fit in that existing sub-method we have inside of find. I'm not going to make it the all-knowing super power method i once did, we just need to look at name attrs that come back and see if one matches | 19:21 |
| briancurtin | currently we're making a request that thinks it will only receive the match, but its sending query params the server doesn't care about, so we get a full list of things and quit once we hit the second one, thinking it's a duplicate | 19:22 |
| briancurtin | which actually explains the bug i just entered saying that sometimes we get DuplicateResource when we should be getting NoResourceFound, like if you search a bogus name | 19:23 |
| terrylhowe | some functional tests would be awesome | 19:23 |
| briancurtin | terrylhowe: yeah, i'm going to work on the base Resource.find first and then try to get some functional tests of a few resources to prove it out | 19:24 |
| terrylhowe | based on the examples, find was working for me for servers and keypairs briancurtin and I think there are a few functional tests | 19:26 |
| terrylhowe | resource is pretty big, so I guess if it helps, pull that get_one_match thing out make some sort of helper class maybe | 19:26 |
| briancurtin | can probably just remove the query param stuff and it will work almost immediately | 19:27 |
| briancurtin | servers does accept the query param though for name, so that's one that currently works. if it can make use of a query param and it wants to do that, it should provide a specialization for it to use it | 19:28 |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: SDK integration into OSC for network CRUD https://review.openstack.org/138745 | 19:28 |
| terrylhowe | we aren’t gettin openstacksdk gerrit notifications here anymore I guess | 19:29 |
| briancurtin | terrylhowe: there's one from 40 minutes ago | 19:29 |
| briancurtin | actually and even more recent than that, your meter test one | 19:30 |
| terrylhowe | yeh, odd. oh well | 19:30 |
| terrylhowe | oh, I guess some packets are being dropped because I don’t see that one | 19:30 |
| *** petertr7 is now known as petertr7_away | 19:41 | |
| *** petertr7_away is now known as petertr7 | 19:41 | |
| *** e0ne has joined #openstack-sdks | 19:45 | |
| *** e0ne has quit IRC | 19:47 | |
| openstackgerrit | Terry Howe proposed stackforge/python-openstacksdk: Fix find available floating IP test https://review.openstack.org/214777 | 19:48 |
| openstackgerrit | Terry Howe proposed stackforge/python-openstacksdk: Add a functional test for find_extension https://review.openstack.org/214780 | 19:57 |
| *** e0ne has joined #openstack-sdks | 20:02 | |
| *** woodster_ has joined #openstack-sdks | 20:13 | |
| *** stevemar has joined #openstack-sdks | 20:14 | |
| *** kelvin has joined #openstack-sdks | 20:16 | |
| briancurtin | terrylhowe: with relatively minimal changes, find can now find anything by id or name, and it now properly raises DuplicateResource only when there's a dupe, and either returns None or raises ResourceNotFound depending on what you called with when nothing is found. that should solve any of the outstanding find-doesn't-work bugs. i'm not going to add | 20:17 |
| briancurtin | functional tests for every resource, but at least for right now some finds for server will prove it out | 20:17 |
| briancurtin | terrylhowe: another thing we need to get to is that BaseProxy should have a _find -- that's the one oddity while looking at all of the Proxy classes, that they still call through resources to do their find | 20:17 |
| terrylhowe | well, find for server worked before | 20:18 |
| *** e0ne has quit IRC | 20:18 | |
| briancurtin | only because it was written specifically for it | 20:18 |
| briancurtin | basically all others were broken in some way | 20:18 |
| briancurtin | especially when finding by name | 20:18 |
| terrylhowe | well, might not be much to generalize in the base class for find | 20:20 |
| briancurtin | i just did it? | 20:20 |
| briancurtin | the fix is just don't use query params | 20:20 |
| terrylhowe | oh, you added a _find to the base proxy? | 20:21 |
| terrylhowe | anyway, I ahven’t seen the patch yet | 20:21 |
| briancurtin | no. well i did, but not as part of fixing Resource.find | 20:21 |
| terrylhowe | ah, cool | 20:21 |
| openstackgerrit | Merged openstack/python-openstackclient: Update the plugin docs for designate https://review.openstack.org/214712 | 20:24 |
| *** e0ne has joined #openstack-sdks | 20:29 | |
| *** mpmsimo has joined #openstack-sdks | 20:33 | |
| openstackgerrit | Terry Howe proposed stackforge/python-openstacksdk: Add support for resource foreign keys in base_path https://review.openstack.org/207972 | 20:33 |
| *** petertr7 is now known as petertr7_away | 20:34 | |
| *** e0ne has quit IRC | 20:35 | |
| *** e0ne has joined #openstack-sdks | 20:36 | |
| *** petertr7_away is now known as petertr7 | 20:37 | |
| openstackgerrit | Terry Howe proposed stackforge/python-openstacksdk: Add support for resource foreign keys in base_path https://review.openstack.org/207972 | 20:42 |
| openstackgerrit | Kelvin Lui proposed openstack/python-openstackclient: Adds documentation on weekly meeting https://review.openstack.org/214785 | 20:46 |
| *** mpmsimo has quit IRC | 20:59 | |
| *** stevemar has quit IRC | 21:05 | |
| *** mpmsimo has joined #openstack-sdks | 21:05 | |
| *** petertr7 is now known as petertr7_away | 21:07 | |
| *** e0ne has quit IRC | 21:15 | |
| *** e0ne has joined #openstack-sdks | 21:19 | |
| *** mpmsimo has quit IRC | 21:20 | |
| *** mpmsimo has joined #openstack-sdks | 21:22 | |
| *** e0ne has quit IRC | 21:28 | |
| *** kelvin_ has joined #openstack-sdks | 21:29 | |
| *** kelvin has quit IRC | 21:31 | |
| *** mpmsimo has quit IRC | 21:33 | |
| *** kelvin__ has joined #openstack-sdks | 21:37 | |
| *** kelvin_ has quit IRC | 21:37 | |
| *** thrash is now known as thrash|g0ne | 21:47 | |
| *** kelvin__ has quit IRC | 21:57 | |
| *** steveortiz is now known as zz_steveortiz | 21:57 | |
| *** pm90__ has quit IRC | 22:16 | |
| *** gildub has joined #openstack-sdks | 22:23 | |
| *** jose4183 has quit IRC | 22:34 | |
| *** mpmsimo has joined #openstack-sdks | 22:39 | |
| *** mpmsimo has quit IRC | 22:40 | |
| openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Make Resource.find more generically applicable https://review.openstack.org/214822 | 22:43 |
| openstackgerrit | Merged openstack/python-openstackclient: Added note to install openstackclient https://review.openstack.org/214304 | 22:45 |
| *** bnemec has quit IRC | 22:56 | |
| *** britthouser has quit IRC | 22:56 | |
| *** pm90_ has joined #openstack-sdks | 23:01 | |
| *** mpmsimo1 has joined #openstack-sdks | 23:10 | |
| *** bnemec has joined #openstack-sdks | 23:20 | |
| *** chlong has quit IRC | 23:28 | |
| *** mpmsimo1 has quit IRC | 23:40 | |
| *** bnemec has quit IRC | 23:43 | |
| *** bnemec has joined #openstack-sdks | 23:45 | |
| *** ameybhide has quit IRC | 23:46 | |
| *** bnemec has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!