| *** pm90_ has quit IRC | 00:01 | |
| *** shakamunyi has joined #openstack-sdks | 00:12 | |
| *** barra204 has joined #openstack-sdks | 00:12 | |
| *** Caleb-- has quit IRC | 00:38 | |
| *** Caleb-- has joined #openstack-sdks | 00:43 | |
| openstackgerrit | Jens Rosenboom proposed openstack/python-openstackclient: Add a reference to the IRC channels https://review.openstack.org/187831 | 00:54 |
|---|---|---|
| *** Qiming has joined #openstack-sdks | 00:56 | |
| *** superfly_ has joined #openstack-sdks | 01:12 | |
| *** superfly_ is now known as superflyy | 01:14 | |
| *** thrash is now known as thrash|g0ne | 01:14 | |
| *** barra204 has quit IRC | 01:24 | |
| *** shakamunyi has quit IRC | 01:24 | |
| *** superflyy has quit IRC | 01:24 | |
| *** Yanyanhu has joined #openstack-sdks | 01:27 | |
| *** tobe has joined #openstack-sdks | 01:33 | |
| *** hitalia has quit IRC | 01:34 | |
| openstackgerrit | Yanyan Hu proposed stackforge/python-openstacksdk: Complete property definition in some lb resources https://review.openstack.org/187452 | 01:41 |
| openstackgerrit | Merged stackforge/python-openstacksdk: Complete property definition in some lb resources https://review.openstack.org/187452 | 02:19 |
| *** jose4183 has quit IRC | 02:40 | |
| briancurtin | etoews: nothing special. do you have a specific example? resource.props are sort of fancy wrappers, in a way | 02:46 |
| etoews | briancurtin: so i'm totally redesigning the message service stuff | 02:47 |
| etoews | the interfaces were too ugly | 02:47 |
| etoews | i'm moving the information into the objects. | 02:48 |
| etoews | (at the expense of just a bit of extra validation) | 02:48 |
| etoews | so now there will be | 02:49 |
| etoews | client = None | 02:49 |
| etoews | on a Queue | 02:49 |
| etoews | it's not a real property on a queue so i wasn't going to do | 02:49 |
| etoews | client = resource.prop("client") | 02:50 |
| etoews | because i thought that would be misleading | 02:50 |
| etoews | it's almost ready for review. | 02:50 |
| etoews | trying to get it out there before my brain melts. | 02:51 |
| openstackgerrit | Qiming Teng proposed stackforge/python-openstacksdk: Add Heat resource support https://review.openstack.org/181063 | 02:56 |
| Qiming | etoews, about https://review.openstack.org/#/c/181063/, could you please do me a favor pasting your functional test results? | 02:59 |
| Qiming | I'm not feeling happy with the template used for functional tests. It doesn't work in my local devstack environment, because I have both public and private neutron networks, so I have to add a network id/name as a hack | 03:01 |
| Qiming | the template also demands for a glance image with specific name, which is not necessarily needed for this functional test | 03:02 |
| Qiming | maybe we should use a slightly lightweight template for functional testing | 03:03 |
| openstackgerrit | Merged openstack/python-openstackclient: Add a reference to the IRC channels https://review.openstack.org/187831 | 03:05 |
| openstackgerrit | Everett Toews proposed stackforge/python-openstacksdk: Claim messages and delete messages https://review.openstack.org/187852 | 03:37 |
| openstackgerrit | Qiming Teng proposed stackforge/python-openstacksdk: Add Heat resource support https://review.openstack.org/181063 | 03:38 |
| openstackgerrit | Steve Martinelli proposed openstack/python-openstackclient: Add domain support for ec2creds in v3 identity https://review.openstack.org/187854 | 03:39 |
| etoews | Qiming: i added the a link to the functional test output as a comment on the review | 03:48 |
| etoews | that heat template was chosen because it's the "default" one for heat and devstack http://docs.openstack.org/developer/heat/getting_started/on_devstack.html | 03:49 |
| etoews | i don't know that we'd want anything less complicated than that. | 03:49 |
| Qiming | etoews, I understand the intent to use a full template, however, I'm inclined to use a stack with just a OS::Heat::RandomString as resource | 03:50 |
| Qiming | that will make the whole functional test much faster, much predictable, and less depencies on your environment | 03:50 |
| Qiming | as for your functional test output, I'll check it later, thanks for the help!! | 03:51 |
| etoews | np | 03:53 |
| etoews | remember you can always run devstack in a public cloud to get a default instance that doesn't need tweaks for a local env. | 03:55 |
| etoews | but i understand if you have other constraints requiring you to use a local env. | 03:56 |
| etoews | i certainly understand your desire to use a simple template. | 03:56 |
| etoews | but even if we had that the code would still need be tested against the more complicated templates. | 03:57 |
| etoews | overall functional testing of heat won't be easy because it's probably the most complicated service. | 03:58 |
| etoews | i wouldn't rule out having a mix of templates in the future. | 03:58 |
| etoews | i need to get some sleep now. have a good day! | 03:59 |
| *** notmars has joined #openstack-sdks | 03:59 | |
| *** stevemar has quit IRC | 04:58 | |
| *** notmars has quit IRC | 05:07 | |
| openstackgerrit | Amey Bhide proposed openstack/python-openstackclient: Add support for volume v2 commands https://review.openstack.org/187480 | 05:08 |
| *** notmars has joined #openstack-sdks | 05:32 | |
| *** Shrews has quit IRC | 05:51 | |
| *** terrylhowe has quit IRC | 05:53 | |
| *** dguerri`away has quit IRC | 05:55 | |
| openstackgerrit | Qiming Teng proposed stackforge/python-openstacksdk: Add Heat resource support https://review.openstack.org/181063 | 05:55 |
| *** Shrews has joined #openstack-sdks | 05:56 | |
| *** dguerri`away has joined #openstack-sdks | 05:57 | |
| *** dguerri`away is now known as dguerri | 05:57 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-openstackclient: Imported Translations from Transifex https://review.openstack.org/186273 | 06:05 |
| *** shakamunyi has joined #openstack-sdks | 06:16 | |
| *** barra204 has joined #openstack-sdks | 06:17 | |
| *** aufi has joined #openstack-sdks | 06:37 | |
| *** chlong has quit IRC | 07:37 | |
| *** karimb has joined #openstack-sdks | 07:43 | |
| *** tobe has quit IRC | 07:52 | |
| *** karimb has quit IRC | 07:57 | |
| *** tobe has joined #openstack-sdks | 08:14 | |
| *** heyongli has quit IRC | 08:34 | |
| Yanyanhu | hi, briancurtin, are you around? | 08:44 |
| *** heyongli has joined #openstack-sdks | 08:49 | |
| *** heyongli has quit IRC | 08:51 | |
| *** heyongli has joined #openstack-sdks | 09:06 | |
| *** heyongli has quit IRC | 09:07 | |
| openstackgerrit | Yanyan Hu proposed stackforge/python-openstacksdk: Add path_args when invoking Resource.list() from proxy layer https://review.openstack.org/187919 | 09:08 |
| *** Qiming_ has joined #openstack-sdks | 10:16 | |
| *** Qiming_ has quit IRC | 10:16 | |
| *** Qiming has quit IRC | 10:16 | |
| openstackgerrit | Yanyan Hu proposed stackforge/python-openstacksdk: Remove path_args attrs from creation request body https://review.openstack.org/187934 | 10:20 |
| *** Yanyanhu has quit IRC | 10:25 | |
| *** jlibosva has joined #openstack-sdks | 10:42 | |
| *** trown|outttypeww is now known as trown | 11:40 | |
| *** tobe has quit IRC | 11:56 | |
| *** tobe has joined #openstack-sdks | 11:57 | |
| *** tobe has quit IRC | 12:02 | |
| *** thrash|g0ne is now known as thrash | 12:05 | |
| *** aufi has quit IRC | 12:26 | |
| *** aufi has joined #openstack-sdks | 12:26 | |
| *** zz_steveortiz is now known as steveortiz | 12:53 | |
| *** notmars has joined #openstack-sdks | 12:54 | |
| *** bknudson has joined #openstack-sdks | 12:56 | |
| *** mattfarina has joined #openstack-sdks | 13:04 | |
| *** karimb has joined #openstack-sdks | 13:10 | |
| *** terrylhowe has joined #openstack-sdks | 13:15 | |
| *** stevemar has joined #openstack-sdks | 13:17 | |
| stevemar | dtroyer, cant believe i forgot the docs | 13:25 |
| *** petertr7_away is now known as petertr7 | 13:25 | |
| dtroyer | I know, right? | 13:32 |
| dtroyer | you of all people! ;) | 13:32 |
| openstackgerrit | Merged openstack/python-openstackclient: Imported Translations from Transifex https://review.openstack.org/186273 | 13:44 |
| *** sigmavirus24_awa is now known as sigmavirus24 | 13:54 | |
| *** steveortiz is now known as zz_steveortiz | 14:04 | |
| *** pm90_ has joined #openstack-sdks | 14:05 | |
| *** jose4183 has joined #openstack-sdks | 14:12 | |
| *** jose4183 has quit IRC | 14:13 | |
| *** petertr7 is now known as petertr7_away | 14:15 | |
| *** petertr7_away is now known as petertr7 | 14:16 | |
| *** elarson_ has joined #openstack-sdks | 14:22 | |
| *** david-lyle has quit IRC | 14:25 | |
| *** elarson has quit IRC | 14:27 | |
| *** elarson_ is now known as elarson | 14:27 | |
| *** jose4183 has joined #openstack-sdks | 14:27 | |
| *** petertr7 is now known as petertr7_away | 14:30 | |
| *** petertr7_away is now known as petertr7 | 14:31 | |
| openstackgerrit | Merged openstack/python-openstackclient: Get rid of oslo_i18n deprecation notice https://review.openstack.org/187631 | 14:35 |
| *** barra204 has quit IRC | 14:36 | |
| *** shakamunyi has quit IRC | 14:36 | |
| openstackgerrit | Merged openstack/python-openstackclient: Fix security group list command https://review.openstack.org/186385 | 14:37 |
| *** Qiming has joined #openstack-sdks | 14:44 | |
| Qiming | terrylhowe, hi | 14:44 |
| terrylhowe | Hey Qiming I’m looking at that fix to list now | 14:44 |
| Qiming | terrylhowe, the functional test failure is not caused by the list() function | 14:45 |
| Qiming | it is caused by find() | 14:45 |
| Qiming | find_stack() | 14:45 |
| terrylhowe | Hmm, may be a similar problem there | 14:45 |
| Qiming | stack.Stack.find(...) not working as expected | 14:45 |
| terrylhowe | I don’t think that find should take path_args, it should use path_args=self | 14:46 |
| terrylhowe | I think the problem is in resource | 14:46 |
| Qiming | in the find() case, query params are not passed in, so each time I'm running the functional tests, I'm getting two stacks, thanks to the 'limit=2' argument | 14:46 |
| Qiming | it is not about path_args, it is about params | 14:46 |
| Qiming | ref: http://git.openstack.org/cgit/stackforge/python-openstacksdk/tree/openstack/resource.py#n936 | 14:48 |
| Qiming | args are hard-coded, this is no good | 14:48 |
| *** pm90_ has quit IRC | 14:48 | |
| terrylhowe | yes, it doesn’t have path_args there either which it should | 14:49 |
| Qiming | for find() operation, the request will be http://....../stacks?name=expected_name | 14:49 |
| Qiming | path_args, according to my understanding is something you will substitute in the base url, not the query string | 14:50 |
| terrylhowe | yes, what are you thinking is missing? Stack doesn’t have any foreign keys in the URL, does it? | 14:51 |
| *** pm90_ has joined #openstack-sdks | 14:51 | |
| Qiming | for example, I may want to find a stack whose status is CREATE_COMPLETED | 14:51 |
| Qiming | that is a query string to be appended, the find() function today is not doing that, so functional tests always lead me to line 951 | 14:52 |
| terrylhowe | find wasn’t built to be a generic query. It was originally built just to do a name->id mapping. | 14:53 |
| terrylhowe | definitely an area of contention though since it’s use is limited | 14:54 |
| Qiming | okay, then we can treat it as a different problem, see line 897, the filters dictionary is not accepting new query key-value pairs | 14:54 |
| *** petertr7 is now known as petertr7_away | 14:55 | |
| terrylhowe | Ah, I see what you mean know Qiming filters should probably be based of params | 14:56 |
| Qiming | yip, maybe merge params into filters before doing session.get() | 14:57 |
| terrylhowe | yeh, good catch | 14:57 |
| Qiming | we can do that later | 14:57 |
| Qiming | for the moment, the heat resource patch, still two questions | 14:58 |
| terrylhowe | Open a bug if you like, but there are some bugs already for find that may cover that problem | 14:58 |
| Qiming | one is about leaving functional test into separate patch, I think you are okay with that | 14:58 |
| terrylhowe | wouldn’t hurt to have another one to track that particular issue though | 14:58 |
| terrylhowe | I am so we can get your other changes through while we work the list and path_args problem | 14:59 |
| Qiming | okay, will search bug list to find out | 14:59 |
| Qiming | the other question is about the stack template used | 14:59 |
| Qiming | if you check this: https://review.openstack.org/#/c/181063/8/openstack/tests/functional/orchestration/v1/test_stack.py | 15:00 |
| terrylhowe | I would like to have those values defaulted in the tests with possible overrides out of clouds.yaml | 15:00 |
| Qiming | you will find that I am proposing using a simpler (much simpler) stack for functional tests | 15:01 |
| terrylhowe | but I have not set up the configuration yet | 15:01 |
| terrylhowe | a stack that works with basic devstack | 15:01 |
| Qiming | aha, clouds.yaml is a magic | 15:01 |
| terrylhowe | etoews: had some opinions on the template. He put together the test. | 15:02 |
| Qiming | no one told me how to use it, I happen to make a copy at the sdk root dir than run tox -e functional | 15:02 |
| Qiming | yes, talked to him yesterday, seems he is at US timezone | 15:02 |
| terrylhowe | etoews: set that up, it is very nice. I think he is in TX so he should be around soon. | 15:03 |
| Qiming | the goal here, after all, is not testing whether heat behaves properly, that is heat functional tests' job | 15:03 |
| Qiming | what we really need in this context is to make sure SDk interacts with heat in the expected way | 15:03 |
| terrylhowe | agreed | 15:03 |
| Qiming | okay, in the next patch, I will leave the functional tests out, then I'll try make find() works as expected, finally, check in functional tests for this | 15:05 |
| *** zz_steveortiz is now known as steveortiz | 15:05 | |
| terrylhowe | https://bugs.launchpad.net/python-openstacksdk/+bug/1461582 | 15:07 |
| openstack | Launchpad bug 1461582 in OpenStack SDK "Find uses page but page doesn't pass params" [Undecided,New] | 15:07 |
| Qiming | okay, assigned, ;) | 15:08 |
| terrylhowe | thanks Qiming ! | 15:09 |
| Qiming | thanks for you time | 15:09 |
| *** Qiming has quit IRC | 15:17 | |
| stevemar | dtroyer, HA, we wrote the same thing at the same time | 15:24 |
| stevemar | twinsies! | 15:24 |
| *** petertr7_away is now known as petertr7 | 15:26 | |
| openstackgerrit | Steve Martinelli proposed openstack/python-openstackclient: Add domain support for ec2creds in v3 identity https://review.openstack.org/187854 | 15:28 |
| stevemar | dtroyer, head up on https://review.openstack.org/#/c/187480/2 | 15:30 |
| dtroyer | stevemar: there were not bugs for those ec2 change, right? | 15:30 |
| openstackgerrit | Terry Howe proposed stackforge/python-openstacksdk: Fix proxy docs https://review.openstack.org/187757 | 15:31 |
| stevemar | dtroyer, nope, i'll retroactively create one | 15:31 |
| dtroyer | I saw that, was planning to wait on it until we get 1.3.1 out | 15:31 |
| stevemar | i think jamielennox had a keystoneclient one | 15:31 |
| stevemar | https://bugs.launchpad.net/horizon/+bug/1236326 | 15:32 |
| openstack | Launchpad bug 1236326 in python-keystoneclient "AttributeError: 'Client' object has no attribute 'ec2'" [High,Fix committed] - Assigned to Jamie Lennox (jamielennox) | 15:32 |
| *** notmars has quit IRC | 15:32 | |
| dtroyer | that'll work, can just to the updates manually | 15:33 |
| openstackgerrit | Steve Martinelli proposed openstack/python-openstackclient: Add EC2 support for identity v3 API https://review.openstack.org/187127 | 15:33 |
| openstackgerrit | Steve Martinelli proposed openstack/python-openstackclient: Add domain support for ec2creds in v3 identity https://review.openstack.org/187854 | 15:33 |
| openstackgerrit | Steve Martinelli proposed openstack/python-openstackclient: Add domain support for ec2creds in v3 identity https://review.openstack.org/187854 | 15:33 |
| stevemar | dtroyer, so how often do you have to google "periodic table" | 15:35 |
| stevemar | or do you have it bookmarked at this point? | 15:35 |
| dtroyer | it pops up in my browser history at the top of 'per' I think | 15:36 |
| *** notmars has joined #openstack-sdks | 15:38 | |
| * stevemar shakes head | 15:41 | |
| stevemar | part of hopes that m16 (sulfur) breaks things, so it's 'poisonous' | 15:42 |
| dtroyer | this can be arranged | 15:42 |
| stevemar | though if we make all poisonous elements bad releases we're going to have a lot of bad releases | 15:43 |
| *** notmars has quit IRC | 15:43 | |
| stevemar | theres a heck of a lot of deadly on that table | 15:43 |
| *** thrash has quit IRC | 15:50 | |
| *** jlibosva has quit IRC | 15:52 | |
| dtroyer | I just realized doing this, it's not a point release by adding the v3 ec2 creds commands… 1.4.0 here we come | 15:54 |
| *** thrash has joined #openstack-sdks | 15:54 | |
| *** notmars has joined #openstack-sdks | 15:55 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: Clean up ec2 credentials help text https://review.openstack.org/188057 | 15:57 |
| openstackgerrit | Terry Howe proposed stackforge/python-openstacksdk: Rename keystore key-manager https://review.openstack.org/187716 | 16:04 |
| *** pm90_ has quit IRC | 16:10 | |
| *** jamielennox is now known as jamielennox|away | 16:21 | |
| *** barra204 has joined #openstack-sdks | 16:21 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: Create 1.4.0 release notes https://review.openstack.org/188068 | 16:24 |
| stevemar | dtroyer, btw, we need to bump up ksc req | 16:27 |
| *** barra204 has quit IRC | 16:28 | |
| dtroyer | can you do that and we'll just make sure it merges before release notes? | 16:28 |
| stevemar | dtroyer, y, i'll take a quick look to make sure thats all in order | 16:34 |
| stevemar | dtroyer, fyi, we added some v2 volume support (delete and show) https://review.openstack.org/#/c/186612/ | 16:35 |
| stevemar | not sure if you wanted to add that to the release notes | 16:35 |
| dtroyer | um, right, I missed that getting merged. what if we wait to brag about all of that at once? | 16:36 |
| dtroyer | this point release keeps growing… ;) | 16:36 |
| stevemar | dtroyer, fair enough, we can wait to brag | 16:36 |
| stevemar | dtroyer, things go much faster when it's not just us :) | 16:37 |
| dtroyer | that's a nice problem to have | 16:37 |
| *** karimb has quit IRC | 16:38 | |
| *** petertr7 is now known as petertr7_away | 16:38 | |
| *** trown is now known as trown|lunch | 16:39 | |
| *** pm90_ has joined #openstack-sdks | 16:41 | |
| stevemar | indeed | 16:42 |
| *** pm90_ has quit IRC | 17:12 | |
| *** dguerri is now known as dguerri`away | 17:15 | |
| *** LinuxHorn is now known as K3rk | 17:18 | |
| *** K3rk is now known as Linuxhorn | 17:18 | |
| *** Linuxhorn is now known as LinuxHorn | 17:19 | |
| *** notmars has quit IRC | 17:20 | |
| *** petertr7_away is now known as petertr7 | 17:29 | |
| *** pm90_ has joined #openstack-sdks | 17:36 | |
| *** trown|lunch is now known as trown | 17:38 | |
| *** hitalia has joined #openstack-sdks | 17:39 | |
| *** hitalia has quit IRC | 17:50 | |
| *** notmars has joined #openstack-sdks | 17:50 | |
| mordred | dtroyer: could I buy a review of the outstanding occ stack when you get a sec? | 17:54 |
| *** jose4183 has quit IRC | 17:54 | |
| mordred | dtroyer: I'l trade you having said nice things about you to your boss yesterday ... | 17:55 |
| *** dguerri`away is now known as dguerri | 17:59 | |
| *** hitalia has joined #openstack-sdks | 17:59 | |
| dtroyer | mordred: I'll take that trade… | 18:01 |
| *** aufi has quit IRC | 18:03 | |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: Add functional tests for volume set and unset https://review.openstack.org/188129 | 18:04 |
| openstackgerrit | Merged stackforge/python-openstacksdk: Fix proxy docs https://review.openstack.org/187757 | 18:14 |
| openstackgerrit | Merged openstack/os-client-config: Add list of image params needed to disable agents https://review.openstack.org/186722 | 18:27 |
| openstackgerrit | Merged openstack/os-client-config: Add auro to list of known vendors https://review.openstack.org/187005 | 18:27 |
| *** notmars has quit IRC | 18:29 | |
| dtroyer | mordred: all good, except a comment and possible discussion on 187772 based on an OSC bug I don't particularly want to call a bug… | 18:34 |
| mordred | dtroyer: oh jeez | 18:35 |
| *** notmars has joined #openstack-sdks | 18:36 | |
| *** keekz has joined #openstack-sdks | 18:38 | |
| *** notmars has quit IRC | 18:39 | |
| *** notmars has joined #openstack-sdks | 18:40 | |
| openstackgerrit | Merged openstack/os-client-config: Change naming in vendor doc to match vendors.py https://review.openstack.org/187006 | 18:43 |
| openstackgerrit | Merged openstack/os-client-config: Add more defaults to our defaults file https://review.openstack.org/187771 | 18:43 |
| *** hitalia has quit IRC | 18:48 | |
| *** jose4183 has joined #openstack-sdks | 18:48 | |
| *** mordred has quit IRC | 18:55 | |
| *** mordred has joined #openstack-sdks | 18:55 | |
| *** barra204 has joined #openstack-sdks | 18:57 | |
| *** hitalia has joined #openstack-sdks | 18:59 | |
| mordred | dtroyer: so - hrm | 19:00 |
| mordred | dtroyer: I think I also agree that --insecure should have that effect | 19:00 |
| mordred | dtroyer: if we're adding this method - perhaps we should have it look at --insecure too? | 19:01 |
| dtroyer | insecure is ! verify unless verify == None | 19:01 |
| dtroyer | but with your defaults that isn't going to be the case for o-c-c | 19:02 |
| dtroyer | so then it's just switching that if construct to check verify first | 19:02 |
| mordred | well, shade actually doesn't have an insecure argument at all | 19:02 |
| mordred | so I may have just mised this | 19:02 |
| dtroyer | it isn't needed, OSC has it only because compat | 19:02 |
| mordred | ah | 19:02 |
| dtroyer | but people like it for some reason | 19:02 |
| *** barra204 has quit IRC | 19:02 | |
| *** pm90__ has joined #openstack-sdks | 19:03 | |
| dtroyer | so assuming verify has the correct value of —verify/—insecure, it's only a check for verify == True and cacert != None to set verify to cacert | 19:04 |
| mordred | dtroyer: maybe for compat I should get occ to check for insecure and if it's there, set verify and the others to none? | 19:04 |
| dtroyer | I wouldn't…osc already converts —insecure to !—verify to | 19:04 |
| dtroyer | if shade doesn't use to don't add it | 19:04 |
| mordred | cool | 19:05 |
| mordred | dtroyer: I don't see any insecure processing in oss | 19:05 |
| mordred | oh | 19:05 |
| mordred | gah | 19:05 |
| mordred | osc | 19:05 |
| mordred | I can't read | 19:05 |
| mordred | coo | 19:05 |
| dtroyer | it's somewhere in shell.py initialize_app() | 19:06 |
| mordred | dtroyer: so that means potentially that that method is fine if osc is doing the right thing currently | 19:06 |
| * mordred is a little slow today on the thinking | 19:06 | |
| *** pm90_ has quit IRC | 19:06 | |
| dtroyer | no, as is, get_requests_verify_args does what osc does today | 19:06 |
| dtroyer | if we fix bug 1447784 then get_requests_verify_args should flip too | 19:07 |
| openstack | bug 1447784 in python-openstackclient "--insecure is ignored if OS_CACERT env var is set" [High,Confirmed] https://launchpad.net/bugs/1447784 | 19:07 |
| dtroyer | either way, I'll use get_requests_verify_args eventually | 19:08 |
| dtroyer | want I should do a flipped version of 187772? | 19:08 |
| openstackgerrit | Merged stackforge/python-openstacksdk: Remove fingerprint as keypair name https://review.openstack.org/187728 | 19:10 |
| *** pm90__ has quit IRC | 19:14 | |
| *** pm90_ has joined #openstack-sdks | 19:15 | |
| openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: WIP: path_args needed in compute lists https://review.openstack.org/188155 | 19:18 |
| mordred | dtroyer: sure! | 19:21 |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: Add functional tests for server CRUD https://review.openstack.org/188160 | 19:40 |
| *** hitalia has quit IRC | 19:49 | |
| *** petertr7 is now known as petertr7_away | 19:49 | |
| *** petertr7_away is now known as petertr7 | 19:50 | |
| openstackgerrit | Dean Troyer proposed openstack/os-client-config: Provide a helper method to get requests ssl values https://review.openstack.org/187772 | 19:55 |
| dtroyer | mordred: ^^^ + a quick-n-dirty test or two | 19:55 |
| mordred | woot | 19:56 |
| mordred | dtroyer: yeah - that looks good to me | 19:57 |
| *** barra204 has joined #openstack-sdks | 19:58 | |
| *** barra204 has quit IRC | 20:03 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: Clean up ec2 credentials help text https://review.openstack.org/188057 | 20:04 |
| mordred | Shrews: ^^ dtroyer updated my ssl arg patch | 20:08 |
| Shrews | *nod* | 20:09 |
| *** hitalia has joined #openstack-sdks | 20:14 | |
| openstackgerrit | Merged openstack/python-openstackclient: Add EC2 support for identity v3 API https://review.openstack.org/187127 | 20:20 |
| *** hitalia has quit IRC | 20:27 | |
| *** steveortiz is now known as zz_steveortiz | 20:29 | |
| *** zz_steveortiz is now known as steveortiz | 20:34 | |
| *** pm90_ has quit IRC | 20:41 | |
| *** pm90_ has joined #openstack-sdks | 20:41 | |
| etoews | briancurtin: you around? | 20:42 |
| briancurtin | etoews: what's up | 20:42 |
| etoews | your comment "Does delete truly not have a version..." | 20:42 |
| etoews | so the href returned by zaqar has a version on it e.g. /v1/queues/test_queue | 20:43 |
| etoews | but the sdk appends a v1 to the endpoint | 20:43 |
| etoews | ultimately you wind up with http://api.com/v1/v1/queues/test_queue | 20:44 |
| etoews | which is why i needed to strip the /v1 | 20:44 |
| briancurtin | ah | 20:45 |
| etoews | is there a convenient way to get the sdk to not append version when building the url? | 20:45 |
| briancurtin | i thought we sorted that out with some other service that was doing that. perhaps image | 20:45 |
| briancurtin | terrylhowe: does that ring a bell? | 20:45 |
| etoews | it's a real stick in the eye because i can't just use the href directly | 20:45 |
| *** stevemar has quit IRC | 20:51 | |
| openstackgerrit | Merged openstack/os-client-config: Provide a helper method to get requests ssl values https://review.openstack.org/187772 | 20:53 |
| *** hitalia has joined #openstack-sdks | 20:54 | |
| openstackgerrit | Merged openstack/cliff: Print help on help command https://review.openstack.org/176478 | 20:58 |
| *** mattfarina has quit IRC | 20:59 | |
| etoews | briancurtin: i poked through the image models and didn't see anything like that... | 21:00 |
| *** pm90_ has quit IRC | 21:01 | |
| *** trown is now known as trown|outttypeww | 21:03 | |
| *** tzero has left #openstack-sdks | 21:07 | |
| openstackgerrit | Merged openstack/python-openstackclient: Add domain support for ec2creds in v3 identity https://review.openstack.org/187854 | 21:08 |
| *** petertr7 is now known as petertr7_away | 21:09 | |
| *** hitalia has quit IRC | 21:30 | |
| *** karimb has joined #openstack-sdks | 21:38 | |
| *** abhide has joined #openstack-sdks | 21:56 | |
| etoews | briancurtin: is it just me or is the behaviour of enumerate starting with a negative index undocumented? https://docs.python.org/3/library/functions.html#enumerate | 22:04 |
| *** pm90_ has joined #openstack-sdks | 22:08 | |
| *** pm90_ has quit IRC | 22:09 | |
| *** pm90_ has joined #openstack-sdks | 22:09 | |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: Add functional tests for server CRUD https://review.openstack.org/188160 | 22:11 |
| openstackgerrit | Everett Toews proposed stackforge/python-openstacksdk: Claim messages and delete messages https://review.openstack.org/187852 | 22:11 |
| terrylhowe | etoews: there is code that is supposed tos trip the endpoint out of what is in the service catalog. Maybe that is broken for your case | 22:16 |
| etoews | terrylhowe: one sec. lemme double check. | 22:16 |
| terrylhowe | https://github.com/stackforge/python-openstacksdk/blob/master/openstack/auth/service_catalog.py#L42 | 22:17 |
| etoews | terrylhowe: it's not in the service catalog. it gets appended by the sdk. | 22:17 |
| mordred | dtroyer, Shrews: occ patch queue is clear - I'm goign to cut a 1.3.0 | 22:20 |
| *** jamielennox|away is now known as jamielennox | 22:20 | |
| terrylhowe | etoews: i don’t really understand what you are doing, but this code pastes the version the user selected back on https://github.com/stackforge/python-openstacksdk/blob/master/openstack/auth/service_catalog.py#L99 | 22:24 |
| mordred | have I mentioned recently that I can't wait for the service catalog overhaul? | 22:26 |
| terrylhowe | if the service filter said the user requested ‘’ for a version, you’d get no version, but that is kind of kludgy | 22:26 |
| terrylhowe | I hear ya mordred | 22:26 |
| etoews | terrylhowe: for most ops we need the version on there. but for delete i'm using the href returned from zaqar, which already includes the version. | 22:28 |
| etoews | so for a delete you wind up with http://api.com/v1/v1/queues/test_queue | 22:29 |
| etoews | terrylhowe: how do i force a recheck? just 'recheck' | 22:29 |
| etoews | ? | 22:29 |
| *** dguerri is now known as dguerri`away | 22:29 | |
| *** sigmavirus24 is now known as sigmavirus24_awa | 22:30 | |
| *** bknudson has quit IRC | 22:31 | |
| etoews | cool. 'recheck' worked. | 22:44 |
| *** steveortiz is now known as zz_steveortiz | 22:44 | |
| terrylhowe | etoews: maybe we could add a get_unversioned_url or something to service catalog | 22:45 |
| *** barra204 has joined #openstack-sdks | 22:48 | |
| *** karimb has quit IRC | 22:51 | |
| *** barra204 has quit IRC | 22:53 | |
| etoews | terrylhowe: that would be super handy! but let's put that off for a subsequent patch set please. | 22:54 |
| openstackgerrit | Merged openstack/python-openstackclient: Clean up ec2 credentials help text https://review.openstack.org/188057 | 23:04 |
| *** pm90_ has quit IRC | 23:18 | |
| openstackgerrit | Terry Howe proposed openstack/python-openstackclient: Add functional tests for image set https://review.openstack.org/188206 | 23:19 |
| *** chlong has joined #openstack-sdks | 23:19 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!