*** kbroughton has joined #savanna | 00:20 | |
*** nosnos has joined #savanna | 00:50 | |
*** IlyaE has joined #savanna | 02:40 | |
*** SergeyLukjanov has joined #savanna | 05:08 | |
*** SergeyLukjanov has quit IRC | 05:14 | |
*** SergeyLukjanov has joined #savanna | 05:15 | |
*** SergeyLukjanov has quit IRC | 06:15 | |
*** SergeyLukjanov has joined #savanna | 07:18 | |
*** DinaBelova has joined #savanna | 07:19 | |
*** alazarev has joined #savanna | 07:28 | |
*** alazarev has quit IRC | 07:35 | |
*** kbroughton has quit IRC | 07:44 | |
*** alazarev has joined #savanna | 07:52 | |
*** DinaBelova has quit IRC | 08:00 | |
*** DinaBelova has joined #savanna | 08:08 | |
openstackgerrit | Sergey Lukjanov proposed a change to stackforge/savanna: Fix python-savannaclient version in reqs https://review.openstack.org/48961 | 09:20 |
---|---|---|
openstackgerrit | Sergey Lukjanov proposed a change to stackforge/savanna-dashboard: Migrate to using python-savannaclient 0.3a2 https://review.openstack.org/48607 | 09:21 |
openstackgerrit | Sergey Lukjanov proposed a change to stackforge/savanna-dashboard: Remove usage of pyvirtualdisplay https://review.openstack.org/49118 | 09:22 |
openstackgerrit | A change was merged to stackforge/python-savannaclient: Bump oslo.config to >=1.2.0 https://review.openstack.org/49061 | 09:40 |
openstackgerrit | A change was merged to stackforge/savanna: Removing line breaks from default configs https://review.openstack.org/49065 | 09:42 |
*** DinaBelova has quit IRC | 09:59 | |
*** SergeyLukjanov has quit IRC | 10:25 | |
*** SergeyLukjanov has joined #savanna | 10:27 | |
*** DinaBelova has joined #savanna | 10:30 | |
*** alazarev has quit IRC | 10:55 | |
*** kbroughton has joined #savanna | 11:00 | |
*** alazarev has joined #savanna | 11:09 | |
*** alazarev has quit IRC | 11:11 | |
*** nosnos has quit IRC | 11:33 | |
openstackgerrit | Yaroslav Lobankov proposed a change to stackforge/savanna: Migration to new integration tests https://review.openstack.org/48652 | 11:36 |
*** matsuhashi has quit IRC | 11:37 | |
*** alazarev has joined #savanna | 11:58 | |
openstackgerrit | Sergey Lukjanov proposed a change to stackforge/savanna: Migration to new integration tests https://review.openstack.org/48652 | 12:04 |
openstackgerrit | Nikita Konovalov proposed a change to stackforge/python-savannaclient: Fix config handling for Node Group Templates https://review.openstack.org/49145 | 12:17 |
*** alazarev has quit IRC | 12:28 | |
*** _crobertsrh is now known as crobertsrh | 12:29 | |
openstackgerrit | A change was merged to stackforge/python-savannaclient: Fix config handling for Node Group Templates https://review.openstack.org/49145 | 12:30 |
*** alazarev has joined #savanna | 12:31 | |
openstackgerrit | Nikita Konovalov proposed a change to stackforge/savanna-dashboard: Migrate to using python-savannaclient 0.3a3 https://review.openstack.org/48607 | 12:39 |
*** NikitaKonovalov has joined #savanna | 12:40 | |
openstackgerrit | Nikita Konovalov proposed a change to stackforge/savanna: Migration to new integration tests https://review.openstack.org/48652 | 12:46 |
*** _mattf is now known as mattf | 12:57 | |
openstackgerrit | Sergey Lukjanov proposed a change to stackforge/savanna: Add admin context for non-request ops https://review.openstack.org/49149 | 13:03 |
openstackgerrit | Sergey Lukjanov proposed a change to stackforge/savanna: Impl multitenancy support https://review.openstack.org/49150 | 13:03 |
*** kbroughton has quit IRC | 13:18 | |
*** kbroughton has joined #savanna | 13:34 | |
openstackgerrit | A change was merged to stackforge/savanna-dashboard: Migrate to using python-savannaclient 0.3a3 https://review.openstack.org/48607 | 13:38 |
openstackgerrit | A change was merged to stackforge/savanna: Added missed default configs for Oozie-4.0.0 https://review.openstack.org/49084 | 13:48 |
*** tmckay has joined #savanna | 13:58 | |
tmckay | freenode webchat client, cool | 13:58 |
tmckay | I'm behind a web service at a car service place and pidgin can't seem to make it through (I think it's the "Accept" screen) | 13:59 |
tmckay | git review doesn't seem to work either :) | 14:00 |
openstackgerrit | Dmitry Mescheryakov proposed a change to stackforge/savanna: Implement threaded SSH for provisioning and Vanilla plugin https://review.openstack.org/48646 | 14:03 |
*** tmckay has quit IRC | 14:09 | |
*** IlyaE has quit IRC | 14:21 | |
*** tmckay has joined #savanna | 14:21 | |
*** IlyaE has joined #savanna | 14:22 | |
*** tmckay has quit IRC | 14:34 | |
*** tmckay has joined #savanna | 14:34 | |
*** tmckay has quit IRC | 14:58 | |
*** DinaBelova has quit IRC | 15:04 | |
*** tmckay has joined #savanna | 15:04 | |
openstackgerrit | Nikita Konovalov proposed a change to stackforge/savanna-dashboard: Fix EDP launch Job Execution https://review.openstack.org/49170 | 15:30 |
openstackgerrit | A change was merged to stackforge/savanna-dashboard: Fix EDP launch Job Execution https://review.openstack.org/49170 | 15:39 |
*** NikitaKonovalov has left #savanna | 15:40 | |
*** SergeyLukjanov has quit IRC | 15:54 | |
*** mattf is now known as _mattf | 16:11 | |
*** SergeyLukjanov has joined #savanna | 16:20 | |
*** DinaBelova has joined #savanna | 16:20 | |
*** DinaBelova has quit IRC | 16:20 | |
*** DinaBelova has joined #savanna | 16:38 | |
openstackgerrit | A change was merged to stackforge/savanna: Migration to new integration tests https://review.openstack.org/48652 | 16:59 |
tmckay | SergeyLukjanov, ping re 49150 and tenant_id | 17:07 |
*** sacharya has joined #savanna | 17:27 | |
tmckay | aignatov, ping | 17:30 |
openstackgerrit | A change was merged to stackforge/savanna: Add admin context for non-request ops https://review.openstack.org/49149 | 17:37 |
SergeyLukjanov | tmckay, hi | 17:37 |
SergeyLukjanov | tmckay, do you have questions about 49150? | 17:38 |
tmckay | SergeyLukjanov, hi, you already answered them in comments :) Thanks | 17:39 |
SergeyLukjanov | tmckay, ok :) | 17:39 |
tmckay | I pinged you here before I wrote comments | 17:39 |
SergeyLukjanov | oh, sorry for delay | 17:39 |
tmckay | oh, no problem, I assumed you were otherwise occupied so I used gerrit instead | 17:39 |
tmckay | SergeyLukjanvo, since you are here... on swift data retrieval, there is a facility that we can use (chunk_size) so that the object returned internal_swift.py are generators, not entire files as strings | 17:41 |
tmckay | SergeyLukjanov, I mean ^^ | 17:41 |
*** _mattf is now known as mattf | 17:42 | |
tmckay | I am considering whether or not to put in a "throttle" value, so that if the data returned for an entire file would be > some maximum it returns the generator from swiftclient instead. | 17:42 |
tmckay | This would protect against someone uploading a 5GB file to swift and then referencing it from a JobBinary and retrieving it (either through the API or a job run). That might cause memory issues on the host. | 17:43 |
tmckay | Seems safer. A caller would have to test whether the result was a generator or a string. This above is my question for aignatov, too -- do you think it is worth the effort to code this? | 17:45 |
tmckay | The JobManager would be an example of a consumer that would loop through the generate piece by piece instead of loading the whole thing in memory at once. | 17:46 |
* tmckay knows QE guys who try to break things like this all the time | 17:47 | |
aignatov | tmckay, actually I didn't understand actual your question on the review. Yes, defenitely you should expose some limit to protect uploading binaries to swift or internal db | 17:53 |
aignatov | sorry for delay | 17:54 |
tmckay | aignatov, well, we can't limit swift uploads from savanna. We can limit what we read... | 17:54 |
tmckay | aignatov, my thought was this. Instead of saying "that file is too big, you cannot retrieve it from swift...." | 17:54 |
SergeyLukjanov | tmckay, mm, I think that it's not very important atm | 17:55 |
tmckay | if we use the generator facility in the swiftclient we can handle a large file in pieces and avoid saying "no" while limiting memory usage | 17:55 |
SergeyLukjanov | tmckay, I hope that we'll switch from our current rest api solution to OpenStack-approved solution like Pecan and WSME | 17:55 |
*** DinaBelova has quit IRC | 17:55 | |
SergeyLukjanov | and then we can impl theottling | 17:55 |
tmckay | SergeyLukjanov, aignatov, I am thinking of the JobManager, too, in addition to REST. The JobManager retrieves binaries as strings and then transfers them to hdfs. | 17:56 |
aignatov | so, Sergey you mean that we can add big TODO to the code: "Here we should think and Implement throttling"? | 17:56 |
tmckay | So, that string exists in memory in Python | 17:57 |
SergeyLukjanov | yep, I think that we should add TODO | 17:57 |
tmckay | Maybe for now we should just have a configurable limit. No retrieval of binary bigger than ??MB | 17:58 |
SergeyLukjanov | such throttling is needed not for the all providers | 17:58 |
SergeyLukjanov | as tmckay said it could be handled ok with swiftclient | 17:58 |
tmckay | I'll add a size check with a configurable value to the draft CR, how about that? And the community can comment :) | 18:00 |
aignatov | I'm ok with that. | 18:00 |
tmckay | okay, thanks. I had bad dreams of the JobManager crashing ;-) | 18:01 |
*** DinaBelova has joined #savanna | 18:04 | |
openstackgerrit | Sergey Lukjanov proposed a change to stackforge/savanna: Impl multitenancy support https://review.openstack.org/49150 | 18:06 |
SergeyLukjanov | tmckay, it's ok for me too, just thinking about code/feature freeze | 18:10 |
*** alazarev has quit IRC | 18:13 | |
tmckay | SergeyLukjanov, ah, yes. When is that? | 18:14 |
SergeyLukjanov | http://lists.openstack.org/pipermail/openstack-dev/2013-September/015800.html | 18:15 |
tmckay | thanks :) I should read openstack-dev more | 18:15 |
*** rnirmal has joined #savanna | 18:31 | |
SergeyLukjanov | tmckay, btw you can subscribe only to [savanna] marked emails | 18:32 |
tmckay | SergeyLukjanov, that is a great idea | 18:33 |
SergeyLukjanov | the rest emails could be important too, so, it's hard to choose... | 18:33 |
SergeyLukjanov | personally, I'm reading the hole ML | 18:33 |
tmckay | ack, I would like to pay attention to all of it. Maybe 2 folders... I should be able to implement the size check today, I see how to do it in the swiftclient, and publish the CR this evening. Plenty of time for the freez. | 18:34 |
*** DinaBelova has quit IRC | 18:44 | |
*** DinaBelova has joined #savanna | 19:11 | |
*** DinaBelova has quit IRC | 19:24 | |
*** aignatov has quit IRC | 19:27 | |
*** DinaBelova has joined #savanna | 19:28 | |
openstackgerrit | Chad Roberts proposed a change to stackforge/savanna-dashboard: Adding help text to EDP forms https://review.openstack.org/49243 | 19:35 |
*** alazarev has joined #savanna | 19:37 | |
openstackgerrit | Chad Roberts proposed a change to stackforge/savanna-dashboard: Adding help text to EDP forms https://review.openstack.org/49243 | 19:54 |
*** DinaBelova has quit IRC | 20:01 | |
*** alazarev has quit IRC | 20:04 | |
*** rnirmal has quit IRC | 20:08 | |
*** aignatov has joined #savanna | 20:08 | |
openstackgerrit | Trevor McKay proposed a change to stackforge/savanna: Allow job binaries to be retrieved from internal swift https://review.openstack.org/48751 | 20:54 |
*** IlyaE has quit IRC | 21:03 | |
SergeyLukjanov | for your info folks, we'll move savanna to openstack org this weekend | 21:06 |
SergeyLukjanov | oct 5 1600 utc | 21:07 |
crobertsrh | :) | 21:08 |
*** crobertsrh is now known as _crobertsrh | 21:08 | |
tmckay | yay | 21:08 |
*** IlyaE has joined #savanna | 21:18 | |
*** IlyaE has quit IRC | 21:25 | |
*** mattf is now known as _mattf | 21:29 | |
*** sacharya has quit IRC | 21:49 | |
*** IlyaE has joined #savanna | 21:51 | |
*** SergeyLukjanov has quit IRC | 21:54 | |
*** SergeyLukjanov has joined #savanna | 21:58 | |
*** SergeyLukjanov has quit IRC | 22:01 | |
*** tmckay has left #savanna | 22:35 | |
*** tmckay has joined #savanna | 22:36 | |
*** kbroughton has quit IRC | 22:50 | |
*** sacharya has joined #savanna | 23:07 | |
*** IlyaE has quit IRC | 23:29 | |
*** IlyaE has joined #savanna | 23:40 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!