| *** saneax has quit IRC | 00:02 | |
| *** crobertsrh has joined #openstack-sahara | 00:24 | |
| *** elmiko has quit IRC | 00:29 | |
| openstackgerrit | lu huichun proposed openstack/python-saharaclient: Add job execution update https://review.openstack.org/208001 | 00:39 |
|---|---|---|
| *** crobertsrh is now known as _crobertsrh | 00:44 | |
| *** melaks has quit IRC | 01:16 | |
| *** saneax has joined #openstack-sahara | 01:44 | |
| *** saneax has quit IRC | 02:06 | |
| *** logan2 has quit IRC | 02:08 | |
| *** logan2 has joined #openstack-sahara | 02:09 | |
| *** melaks has joined #openstack-sahara | 03:01 | |
| *** logan2 has quit IRC | 03:14 | |
| *** logan2 has joined #openstack-sahara | 03:31 | |
| *** Poornima has joined #openstack-sahara | 03:39 | |
| *** saneax has joined #openstack-sahara | 03:47 | |
| *** coolsvap|away is now known as coolsvap | 03:58 | |
| *** Poornima has quit IRC | 04:47 | |
| *** melaks has quit IRC | 05:27 | |
| *** Poornima has joined #openstack-sahara | 06:01 | |
| *** Nikolay_St has quit IRC | 06:07 | |
| openstackgerrit | Merged openstack/sahara: Changed misleading function name in Heat engine https://review.openstack.org/203843 | 06:08 |
| *** witlessb has joined #openstack-sahara | 07:12 | |
| *** esikachev has joined #openstack-sahara | 07:14 | |
| *** Nikolay_St has joined #openstack-sahara | 07:30 | |
| *** esikachev has quit IRC | 07:38 | |
| openstackgerrit | li,chen proposed openstack/sahara: Job execution cancel timeout https://review.openstack.org/208863 | 07:39 |
| *** esikachev has joined #openstack-sahara | 07:51 | |
| openstackgerrit | li,chen proposed openstack/sahara: Job execution cancel timeout https://review.openstack.org/208863 | 07:53 |
| *** esikachev has quit IRC | 08:47 | |
| *** ylobankov_ is now known as ylobankov | 09:04 | |
| openstackgerrit | Vitaly Gridnev proposed openstack/sahara: Add recommendation support to Cloudera plugin https://review.openstack.org/193098 | 09:09 |
| *** esikachev has joined #openstack-sahara | 09:12 | |
| *** tosky has joined #openstack-sahara | 09:17 | |
| *** esikachev has quit IRC | 09:26 | |
| *** esikachev has joined #openstack-sahara | 09:35 | |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-sahara: Reflect provider change in puppet-openstacklib https://review.openstack.org/209877 | 11:06 |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-sahara: Reflect provider change in puppet-openstacklib https://review.openstack.org/209877 | 11:27 |
| openstackgerrit | Vitaly Gridnev proposed openstack/sahara: Make starting scripts module for vanilla 2 plugin https://review.openstack.org/208474 | 11:31 |
| openstackgerrit | Vitaly Gridnev proposed openstack/sahara: Update vanilla plugin to the latest version https://review.openstack.org/208510 | 11:31 |
| openstackgerrit | Vitaly Gridnev proposed openstack/sahara: Small refactoring for vanilla 2 https://review.openstack.org/208424 | 11:31 |
| *** hdd has joined #openstack-sahara | 11:33 | |
| *** esikachev has quit IRC | 11:38 | |
| *** esikachev has joined #openstack-sahara | 11:41 | |
| tellesnobrega | NikitaKonovalov, _crobertsrh can you take a look at this patch https://review.openstack.org/#/c/204647/ ? | 11:46 |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-sahara: Reflect provider change in puppet-openstacklib https://review.openstack.org/209877 | 11:46 |
| _crobertsrh | tellesnobrega: Yes...sorry I had forgot about it. I will take a look today. | 11:54 |
| *** hdd has quit IRC | 12:00 | |
| *** hdd has joined #openstack-sahara | 12:03 | |
| *** tosky_ has joined #openstack-sahara | 12:07 | |
| *** tosky has quit IRC | 12:11 | |
| tellesnobrega | _crobertsrh: no problem | 12:26 |
| *** hdd has quit IRC | 12:36 | |
| *** raildo has joined #openstack-sahara | 12:39 | |
| *** _crobertsrh is now known as crobertsrh | 12:45 | |
| *** hdd has joined #openstack-sahara | 12:50 | |
| *** elmiko has joined #openstack-sahara | 12:53 | |
| openstackgerrit | Vitaly Gridnev proposed openstack/sahara: Update vanilla plugin to the latest version https://review.openstack.org/208510 | 12:59 |
| *** elmiko has quit IRC | 13:00 | |
| *** elmiko has joined #openstack-sahara | 13:00 | |
| *** tmckay has joined #openstack-sahara | 13:03 | |
| *** esikachev has quit IRC | 13:05 | |
| *** esikachev has joined #openstack-sahara | 13:05 | |
| *** hdd has quit IRC | 13:08 | |
| *** hdd has joined #openstack-sahara | 13:15 | |
| *** hdd has quit IRC | 13:16 | |
| *** saneax has quit IRC | 13:44 | |
| *** egafford has joined #openstack-sahara | 13:55 | |
| *** esikachev has quit IRC | 13:58 | |
| *** esikache1 has joined #openstack-sahara | 13:58 | |
| *** degorenko has quit IRC | 14:10 | |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-sahara: Reflect provider change in puppet-openstacklib https://review.openstack.org/209877 | 14:11 |
| *** degorenko has joined #openstack-sahara | 14:11 | |
| openstackgerrit | Vitaly Gridnev proposed openstack/sahara-image-elements: {WIP} Try fix centos7 building image https://review.openstack.org/209951 | 14:17 |
| *** Poornima has quit IRC | 14:18 | |
| *** esikache1 has quit IRC | 14:39 | |
| openstackgerrit | Andrey Pavlov proposed openstack/sahara: Adding is_public and is_protected fields support https://review.openstack.org/195065 | 14:41 |
| *** esikache1 has joined #openstack-sahara | 14:57 | |
| *** esikache1 has quit IRC | 15:02 | |
| openstackgerrit | Merged openstack/sahara: Updated from global requirements https://review.openstack.org/206854 | 15:07 |
| openstackgerrit | Merged openstack/sahara: Put missing fields to validation schema https://review.openstack.org/207244 | 15:08 |
| *** tosky_ is now known as tosky | 15:17 | |
| *** chen12 has joined #openstack-sahara | 15:19 | |
| *** chen12 has quit IRC | 15:19 | |
| *** esikache1 has joined #openstack-sahara | 15:48 | |
| *** hogepodge has quit IRC | 16:06 | |
| *** hogepodge has joined #openstack-sahara | 16:07 | |
| *** Nikolay_St has quit IRC | 16:11 | |
| *** tmckay has quit IRC | 16:15 | |
| openstackgerrit | Merged openstack/puppet-sahara: Update README.md for Sahara https://review.openstack.org/207804 | 16:16 |
| *** tmckay has joined #openstack-sahara | 16:26 | |
| *** tmckay has quit IRC | 16:27 | |
| *** tmckay has joined #openstack-sahara | 16:27 | |
| *** esikache1 has quit IRC | 16:54 | |
| *** htruta_ has quit IRC | 17:00 | |
| *** htruta has joined #openstack-sahara | 17:01 | |
| *** vgridnev has joined #openstack-sahara | 17:13 | |
| *** chlong has quit IRC | 17:14 | |
| *** tosky has quit IRC | 17:27 | |
| *** Nikolay_St has joined #openstack-sahara | 17:31 | |
| *** tosky has joined #openstack-sahara | 17:54 | |
| *** weiting has joined #openstack-sahara | 17:55 | |
| *** weiting_chen has joined #openstack-sahara | 18:17 | |
| *** weiting has quit IRC | 18:17 | |
| *** Nikolay_St has quit IRC | 18:52 | |
| crobertsrh | wow, david-lyle is like magic. cluster template editing patch finally being merged in horizon. | 18:55 |
| *** weiting_chen has quit IRC | 18:55 | |
| elmiko | ha, nice! | 19:02 |
| elmiko | crobertsrh, also, thanks for the -1 on that api-site review... ಠ_ಠ | 19:03 |
| openstackgerrit | Ethan Gafford proposed openstack/sahara: Check ACLs before adding access for Manila share https://review.openstack.org/209621 | 19:03 |
| tosky | SergeyLukjanov: I forgot to ask during the meeting: if I want to add a ID to scenario tests, should I create a spec or is simple bp (or even a simple review) enough? | 19:05 |
| crobertsrh | elmiko: I figured you'd love that | 19:06 |
| tosky | SergeyLukjanov: also, any estimate for that spec about splitting scenario tests out of sahara repo, if still targeted for liberty? | 19:06 |
| elmiko | crobertsrh, i hope you check again, i did update it! | 19:08 |
| crobertsrh | +1 all the way this time | 19:08 |
| * elmiko fist pumps | 19:10 | |
| elmiko | i really didn't think i'd get pinged on creating a bug for a 1 word change... | 19:11 |
| crobertsrh | HUGE bug though | 19:12 |
| elmiko | lol, totally | 19:12 |
| crobertsrh | plus bonus launchpad points for a bug report | 19:12 |
| elmiko | i think the bug text is actually longer than the CR plus the commit msg | 19:12 |
| elmiko | true, always look on the brightside crobertsrh++ | 19:12 |
| elmiko | ok, i'll be back in a bit | 19:13 |
| *** raildo has quit IRC | 20:11 | |
| *** raildo has joined #openstack-sahara | 20:11 | |
| *** logan2 has quit IRC | 20:12 | |
| *** logan2 has joined #openstack-sahara | 20:12 | |
| *** sgotliv_ has quit IRC | 20:44 | |
| *** logan2 has quit IRC | 20:48 | |
| *** raildo has quit IRC | 20:57 | |
| *** crobertsrh is now known as _crobertsrh | 20:58 | |
| *** tmckay has quit IRC | 21:39 | |
| *** tmckay has joined #openstack-sahara | 21:39 | |
| *** DWfuturetec has joined #openstack-sahara | 21:55 | |
| *** vgridnev has quit IRC | 22:12 | |
| *** vgridnev has joined #openstack-sahara | 22:12 | |
| *** vgridnev has quit IRC | 22:12 | |
| jamielennox | elmiko: i replied to your quesiton on the list, what are you manually handling project_id and user_id for? | 22:17 |
| *** egafford has quit IRC | 22:22 | |
| *** witlessb has quit IRC | 22:34 | |
| elmiko | jamielennox, well, to create the trust | 23:00 |
| *** logan2 has joined #openstack-sahara | 23:06 | |
| jamielennox | elmiko: you shouldn't have to pass those around though right? you create the trust once then you pass the trust_id around | 23:13 |
| elmiko | jamielennox, right, it's more a question of how we pass the info to the module that wants to create the trust | 23:14 |
| elmiko | in the past we've just created keystone Client objects | 23:14 |
| elmiko | now i'm changing it to deal in auth plugin objects instead | 23:15 |
| jamielennox | elmiko: so i would encourage you to ditch the persistent keystoneclient | 23:16 |
| jamielennox | once you have a session / auth then there is no cost to creating a client | 23:16 |
| jamielennox | so make the keystoneclient on demand which will at least make it easier to reason about | 23:17 |
| elmiko | that's kinda what i'm coming to realize | 23:17 |
| jamielennox | as for what you pass around, heat is really the other service i've worked wiht in this sort of way and they pass the context around | 23:17 |
| elmiko | i just need to figure out how to change our methodology | 23:17 |
| elmiko | and that's what we do as well, i've embedded the auth into the context | 23:18 |
| jamielennox | that's what i would have thought | 23:18 |
| elmiko | we've just relied too much on gaining the user_id, auth_token, and project_id from the context | 23:18 |
| jamielennox | so there's no harm to adding methods to the context, like get_keystone_client() and have the context manage the creation there | 23:18 |
| elmiko | it's really just an internal shift for us | 23:19 |
| elmiko | we use a utility module to do this work, but it's functionally the same | 23:19 |
| jamielennox | ok | 23:20 |
| jamielennox | so get_user_id and get_project_id take session just in case the token has expired, they should be considered free to call as almost always it just reads the current auth | 23:20 |
| elmiko | yea, i'm incorporating these into our flow now | 23:21 |
| elmiko | i've been working on the idea of keeping the Session and the auth separate, and then combining them when necessary to create clients and what not | 23:22 |
| elmiko | is this a wise approach? | 23:22 |
| jamielennox | yes, it will let you keep connection pools open | 23:22 |
| elmiko | cool | 23:22 |
| jamielennox | you're still doing gloval session objects right? | 23:23 |
| elmiko | i've got a nice session caching object that allows us to reuse the same Session, and if necessary use a Session bound to specific cert | 23:23 |
| elmiko | yes | 23:23 |
| elmiko | https://github.com/elmiko/sahara/blob/bp/keystone-sessions/sahara/service/sessions.py | 23:24 |
| elmiko | that is my rough draft of the session caching mechanism we are using, it's pretty simple | 23:24 |
| elmiko | mainly it just allows us to keep a generic Session, and then specific Sessions bound to cert's for the clients that need them | 23:24 |
| elmiko | jamielennox, another question, is it safe to ignore the old "insecure" option to the Client? | 23:28 |
| elmiko | (i didn't see an equivalent in the Sessions) | 23:28 |
| jamielennox | elmiko: the equivalent in session is verify=False | 23:29 |
| jamielennox | which is the same syntax as requests uses | 23:29 |
| jamielennox | yes, all those connection and auth objects on the old client are ignored if you're using sessions | 23:30 |
| elmiko | ok, thanks | 23:34 |
| elmiko | this is a big change for us, but i think it's for the best | 23:35 |
| *** tosky has quit IRC | 23:37 | |
| *** DWfuturetec has quit IRC | 23:38 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!