| elmiko | alazarev: ping | 00:02 |
|---|---|---|
| alazarev | elmiko, here | 00:03 |
| elmiko | is Vitaly Gridnev in Saratov? | 00:03 |
| alazarev | elmiko, yes | 00:04 |
| elmiko | alazarev: i hate to -1 https://review.openstack.org/#/c/134504/ again, but i feel it's getting kind complicated | 00:05 |
| *** chlong has joined #openstack-sahara | 00:05 | |
| elmiko | alazarev: also, i was reading some of your specs. i like the multi-threaded stuff you are proposing, what's your opinion about migrating to pecan? | 00:06 |
| alazarev | elmiko, actually no options about migrating to pecan... it looks like decision was made on high level, but noone want to spend resources on that | 00:07 |
| *** tmckay has joined #openstack-sahara | 00:07 | |
| alazarev | elmiko, the only project supporting it is ceilometer | 00:08 |
| elmiko | alazarev: ceilometer is the only projet supporting pecan? | 00:08 |
| alazarev | elmiko, yeap | 00:08 |
| elmiko | alazarev: barbican uses it as well | 00:08 |
| elmiko | i thought a few others did too | 00:09 |
| *** tellesnobrega has quit IRC | 00:09 | |
| alazarev | elmiko, oh, didn't take a look to barbican, so may be some other projects | 00:09 |
| elmiko | alazarev: is the issue that pecan is better performance than flask? | 00:09 |
| alazarev | elmiko, but definitely not a majority | 00:09 |
| alazarev | elmiko, the main reason - py3 support | 00:10 |
| alazarev | elmiko, I don't see any other advantages | 00:10 |
| *** Longgeek has quit IRC | 00:10 | |
| elmiko | alazarev: hmm, that doesn't seem like a convincing enough reason. oh well... i've been digging into pecan recently, that's why i'm curious. | 00:11 |
| *** tellesnobrega has joined #openstack-sahara | 00:11 | |
| alazarev | elmiko, there is no way to go to py3 with flask, openstack board doesn't like dead ends | 00:12 |
| elmiko | alazarev: good point | 00:12 |
| elmiko | take care, i'm out | 00:13 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/sahara: Updated from global requirements https://review.openstack.org/146710 | 00:15 |
| *** tmckay has quit IRC | 00:26 | |
| *** zhiyan has joined #openstack-sahara | 00:48 | |
| *** shakamunyi has quit IRC | 01:13 | |
| *** amcrn has quit IRC | 01:24 | |
| *** egafford has quit IRC | 01:40 | |
| *** Longgeek has joined #openstack-sahara | 01:55 | |
| *** zhiyan has quit IRC | 02:05 | |
| *** zhiyan has joined #openstack-sahara | 02:07 | |
| *** tellesnobrega has quit IRC | 02:09 | |
| *** Longgeek has quit IRC | 02:11 | |
| *** tellesnobrega has joined #openstack-sahara | 02:20 | |
| *** hdd has quit IRC | 02:26 | |
| *** shakamunyi has joined #openstack-sahara | 02:41 | |
| *** shakamunyi has quit IRC | 02:47 | |
| *** shakamunyi has joined #openstack-sahara | 02:48 | |
| *** zhiyan has quit IRC | 02:55 | |
| *** witlessb has joined #openstack-sahara | 02:56 | |
| *** zhiyan has joined #openstack-sahara | 02:57 | |
| *** macjack has joined #openstack-sahara | 03:00 | |
| *** shakamunyi has quit IRC | 03:05 | |
| *** macjack has quit IRC | 03:08 | |
| *** Longgeek has joined #openstack-sahara | 03:51 | |
| *** Longgeek has quit IRC | 04:42 | |
| *** Longgeek has joined #openstack-sahara | 04:44 | |
| *** zigo has quit IRC | 05:20 | |
| *** zigo has joined #openstack-sahara | 05:21 | |
| *** hdd has joined #openstack-sahara | 05:39 | |
| *** Longgeek has quit IRC | 06:19 | |
| *** Longgeek has joined #openstack-sahara | 06:34 | |
| *** Poornima has joined #openstack-sahara | 06:44 | |
| *** sgotliv has quit IRC | 06:51 | |
| *** Longgeek has quit IRC | 06:53 | |
| *** Longgeek has joined #openstack-sahara | 06:59 | |
| *** zhiyan has quit IRC | 07:08 | |
| *** zhiyan has joined #openstack-sahara | 07:12 | |
| *** hdd has quit IRC | 07:13 | |
| *** chlong has quit IRC | 07:47 | |
| *** sgotliv has joined #openstack-sahara | 08:17 | |
| *** witlessb has joined #openstack-sahara | 08:34 | |
| *** sgotliv has quit IRC | 08:34 | |
| openstackgerrit | Sergey Reshetnyak proposed openstack/sahara-image-elements: Migrate to openjdk - part 2 https://review.openstack.org/146434 | 09:46 |
| *** coolsvap|afk is now known as coolsvap | 09:59 | |
| openstackgerrit | Sergey Reshetnyak proposed openstack/sahara-image-elements: Migrate to openjdk - part 2 https://review.openstack.org/146434 | 10:00 |
| openstackgerrit | Sergey Reshetnyak proposed openstack/sahara-image-elements: Migrate to openjdk - part 2 https://review.openstack.org/146434 | 10:09 |
| openstackgerrit | Sergey Kolekonov proposed stackforge/sahara-ci-config: devstack automation improvements https://review.openstack.org/147100 | 10:19 |
| openstackgerrit | Sergey Reshetnyak proposed openstack/sahara-image-elements: Migrate to openjdk - part 2 https://review.openstack.org/146434 | 10:22 |
| *** sgotliv has joined #openstack-sahara | 10:28 | |
| *** coolsvap is now known as coolsvap|afk | 10:39 | |
| *** tosky has joined #openstack-sahara | 10:45 | |
| *** Poornima has quit IRC | 10:46 | |
| *** coolsvap|afk is now known as coolsvap | 10:51 | |
| openstackgerrit | Pedro Navarro Pérez proposed openstack/sahara: Removing warnings in the MAPR doc plugin https://review.openstack.org/147110 | 11:00 |
| *** coolsvap is now known as coolsvap|afk | 11:02 | |
| *** coolsvap|afk is now known as coolsvap | 11:09 | |
| openstackgerrit | Vitaly Gridnev proposed openstack/sahara: Added endpoint and utils to work with events https://review.openstack.org/134504 | 11:16 |
| openstackgerrit | Sergey Reshetnyak proposed openstack/sahara: Make vanilla 2.4.1 plugin deprecated https://review.openstack.org/147125 | 11:51 |
| openstackgerrit | Andrey Pavlov proposed openstack/sahara: Adding cluster, instance, job_execution ids to logs https://review.openstack.org/143964 | 12:04 |
| *** chlong has joined #openstack-sahara | 12:05 | |
| openstackgerrit | Sergey Kolekonov proposed stackforge/sahara-ci-config: devstack automation improvements https://review.openstack.org/147100 | 12:06 |
| *** sgotliv_ has joined #openstack-sahara | 12:47 | |
| *** sgotliv has quit IRC | 12:47 | |
| openstackgerrit | Sergey Kolekonov proposed stackforge/sahara-ci-config: Reduce Jenkins slaves number to 8 https://review.openstack.org/147131 | 12:51 |
| openstackgerrit | Merged stackforge/sahara-ci-config: Reduce Jenkins slaves number to 8 https://review.openstack.org/147131 | 12:57 |
| *** chlong has quit IRC | 13:02 | |
| openstackgerrit | Merged stackforge/sahara-ci-config: Mark Vanilla 2.4 as non-voting https://review.openstack.org/146483 | 13:02 |
| *** Networkn3rd has quit IRC | 13:33 | |
| *** hdd has joined #openstack-sahara | 13:35 | |
| openstackgerrit | Denis Egorenko proposed stackforge/sahara-ci-config: Add saharaclient tempest tests for Sahara https://review.openstack.org/143686 | 13:40 |
| *** _crobertsrh is now known as crobertsrh | 14:05 | |
| openstackgerrit | Denis Egorenko proposed stackforge/sahara-ci-config: Move Saharaclient tempest tests to Sahara https://review.openstack.org/143686 | 14:12 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/sahara: Updated from global requirements https://review.openstack.org/147184 | 14:42 |
| *** tmckay has joined #openstack-sahara | 14:47 | |
| *** hdd has quit IRC | 14:52 | |
| *** egafford has joined #openstack-sahara | 14:53 | |
| *** egafford has quit IRC | 14:59 | |
| *** egafford has joined #openstack-sahara | 15:12 | |
| *** hdd has joined #openstack-sahara | 15:28 | |
| *** Networkn3rd has joined #openstack-sahara | 15:54 | |
| openstackgerrit | Sergey Reshetnyak proposed openstack/sahara-specs: New integration tests for Sahara https://review.openstack.org/147219 | 16:13 |
| openstackgerrit | Denis Egorenko proposed stackforge/sahara-ci-config: Back ubuntu_cdh_latest image for CDH integration tests https://review.openstack.org/147222 | 16:22 |
| *** uvirtbot has quit IRC | 16:25 | |
| *** Longgeek has quit IRC | 16:29 | |
| *** hdd has quit IRC | 16:45 | |
| *** IvanBerezovskiy has left #openstack-sahara | 16:49 | |
| *** Networkn3rd has quit IRC | 16:54 | |
| *** Networkn3rd has joined #openstack-sahara | 16:55 | |
| *** tmckay has quit IRC | 16:57 | |
| *** hdd has joined #openstack-sahara | 16:58 | |
| *** skolekonov has quit IRC | 17:03 | |
| *** coolsvap is now known as coolsvap|afk | 17:03 | |
| *** Longgeek has joined #openstack-sahara | 17:10 | |
| *** sgotliv_ has quit IRC | 17:23 | |
| *** Longgeek has quit IRC | 17:24 | |
| *** saurabhs has joined #openstack-sahara | 17:48 | |
| *** Longgeek has joined #openstack-sahara | 17:49 | |
| *** tmckay has joined #openstack-sahara | 17:56 | |
| *** Networkn3rd has quit IRC | 18:06 | |
| *** hdd has quit IRC | 18:08 | |
| *** chlong has joined #openstack-sahara | 18:10 | |
| openstackgerrit | Trevor McKay proposed openstack/sahara: Add options supporting DataSource identifiers in job_configs https://review.openstack.org/142248 | 18:18 |
| openstackgerrit | Trevor McKay proposed openstack/sahara: Add options supporting DataSource identifiers in job_configs https://review.openstack.org/142248 | 18:20 |
| openstackgerrit | Merged openstack/sahara: Removing warnings in the MAPR doc plugin https://review.openstack.org/147110 | 18:41 |
| *** Networkn3rd has joined #openstack-sahara | 18:54 | |
| *** hdd has joined #openstack-sahara | 18:56 | |
| *** macjack has joined #openstack-sahara | 19:00 | |
| *** tosky has quit IRC | 19:08 | |
| *** macjack has quit IRC | 19:22 | |
| *** rharwood has left #openstack-sahara | 19:30 | |
| openstackgerrit | Merged openstack/sahara: fix the edp and hive test issue for CDH5.3 https://review.openstack.org/145429 | 19:48 |
| *** sgotliv has joined #openstack-sahara | 20:38 | |
| egafford | tmckay: Looking at https://review.openstack.org/#/c/142248/11/sahara/db/sqlalchemy/api.py. I'm not ridiculously familiar with sqlalchemy: at the point at which you're assessing the search_opts kvps, have "%"s already been escaped? That is, if a search string legitimately contains an escaped percentage sign that is not a SQL control character, will this method successfully recognize that that string shouldn't be a LIKE expression? | 21:22 |
| *** amcrn has joined #openstack-sahara | 21:27 | |
| tmckay | egafford, no, it's not handling escapes | 21:28 |
| egafford | tmckay: Okay, so any % is assumed to be a LIKE? Particularly given that we have been known to store Python format strings in our DB, that seems like it could be unfortunate for perf. | 21:28 |
| egafford | I can see the argument that as this is just parsing the search string, there's no real need to make it more complex, though. It would take a fairly large cloud to make this a possible problem. | 21:33 |
| tmckay | egafford, I think it's unlikely that a literal '%' will end up in the URL column values for a data source, which is the only place this is being used at present. But it could be improved. | 21:35 |
| tmckay | hmm | 21:36 |
| egafford | Ah, I'm sorry; I didn't note that this was only being used in data_source_count, and was not used otherwise. | 21:36 |
| tmckay | well, chars outside ASCII could be encoded that way | 21:37 |
| tmckay | in urls | 21:37 |
| egafford | True; they frequently are. | 21:37 |
| egafford | Beyond that, the method set (in_filter, like_filter) has a lot of broad applicability, and I wouldn't be surprised if others built on this in the future for other cases. | 21:38 |
| *** Networkn3rd has quit IRC | 21:40 | |
| tmckay | so the problem comes when we use a like filter on a search where the search term includes a literal '%'. Currently only for counting data sources, and only for one internal call. So I don't think it's worth holding up the CR, but it may be worth a low priority bug | 21:42 |
| egafford | I completely agree. | 21:43 |
| tmckay | egafford, feel free to file a bug against it | 21:43 |
| egafford | tmckay: Cool; I'll file away a note to do so after CR. | 21:44 |
| tmckay | great, thanks | 21:44 |
| egafford | Clearly not a huge deal in any event. Thanks tmckay. | 21:44 |
| *** chlong has quit IRC | 21:47 | |
| *** sgotliv has quit IRC | 21:51 | |
| *** crobertsrh is now known as _crobertsrh | 21:54 | |
| *** sgotliv has joined #openstack-sahara | 22:09 | |
| openstackgerrit | Trevor McKay proposed openstack/sahara: Initial work on spark-swift integration https://review.openstack.org/146659 | 22:33 |
| *** tmckay has quit IRC | 22:35 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/sahara: Updated from global requirements https://review.openstack.org/147184 | 23:25 |
| *** sgotliv has quit IRC | 23:31 | |
| *** witlessb has quit IRC | 23:36 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!