| openstackgerrit | Doron Chen proposed openstack/storlets: storlet engine in docker container - documentation update https://review.openstack.org/370532 | 03:58 |
|---|---|---|
| openstackgerrit | Kota Tsuyuzaki proposed openstack/storlets: Enable 2 or more input for a storlet app https://review.openstack.org/347258 | 05:27 |
| openstackgerrit | Merged openstack/storlets: storlet engine in docker container - documentation update https://review.openstack.org/370532 | 07:17 |
| openstackgerrit | Kota Tsuyuzaki proposed openstack/storlets: Enable 2 or more input for a storlet app https://review.openstack.org/347258 | 09:18 |
| eranrom | kota_: Hi, got a question on your multi-input work | 09:25 |
| eranrom | kota_: Can you explain in what sense your sub-requests to gather the extra sources is a hack? | 09:26 |
| kota_ | eranrom: sorry, I didn't get what you mean | 09:34 |
| kota_ | anyway, hi eranrom | 09:34 |
| eranrom | (kota_): I know this is a crazy hack to set the resp | 09:36 |
| eranrom | # dinamically so that this is a temprorary way to make sure | 09:36 |
| eranrom | # the capability, this aboslutely needs cleanup more genelic | 09:36 |
| eranrom | kota_: this is what you wrote in the patch | 09:36 |
| eranrom | I am afraid I did not understand it | 09:37 |
| eranrom | kota_: BTW: Hi | 09:37 |
| eranrom | :-) | 09:37 |
| kota_ | ok, let's make sure the docs, I might forget which code has the doc.... | 09:38 |
| eranrom | https://review.openstack.org/#/c/347258/20/Engine/swift/storlet_middleware/handlers/proxy.py | 09:39 |
| eranrom | line 203 | 09:40 |
| kota_ | ok | 09:44 |
| kota_ | sorry, got another meeting. will be back in one hour later. | 09:44 |
| kota_ | eranrom: i don't have clear memory for the docs for now, maybe i intended to "I want to make the multi input as more generic" | 10:17 |
| kota_ | eranrom: that is because currently that implemented at the proxy handler so that object handler cannot work for the extra resource | 10:18 |
| kota_ | and that code verifys the header format at that time but imo clean code should be header validation when incomming request arrived at. | 10:19 |
| kota_ | that is the reason I added the comment that says 'this is tentative way' | 11:12 |
| kota_ | eranrom:^^ | 11:12 |
| kota_ | hmm... i have no idea how to fix the functest failure on my 2 input storlet patch | 11:18 |
| kota_ | absolutely that has been affected by repository restructured. probably something wrong at build.xml or functests setup but idk right now which part is broken. | 11:20 |
| kota_ | probably, takashi or eranrom can help me where I have to look at :P | 11:20 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add StorletFile class https://review.openstack.org/367560 | 11:54 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add StorletFile class https://review.openstack.org/367560 | 11:56 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Enable 2 or more input for a storlet app https://review.openstack.org/347258 | 12:33 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add StorletFile class https://review.openstack.org/367560 | 12:41 |
| eranrom | kota_: ok thanks for the explanation. I think Takashi already had a look, checking now... | 13:10 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Use seek to move fd cursol to the range start in StorletRangeFile https://review.openstack.org/370909 | 14:56 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Use seek to move fd cursol to the start of input range https://review.openstack.org/370909 | 14:58 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Use seek to move fd cursol to the start of input range https://review.openstack.org/370909 | 15:03 |
| openstackgerrit | Merged openstack/storlets: Add StorletFile class https://review.openstack.org/367560 | 15:06 |
| *** hrou has joined #openstack-storlets | 18:54 | |
| *** ChanServ sets mode: +o eranrom | 18:58 | |
| openstackgerrit | Merged openstack/storlets: Use seek to move fd cursol to the start of input range https://review.openstack.org/370909 | 19:40 |
| *** eranrom changes topic to "Logs: http://eavesdrop.openstack.org/irclogs/%23openstack-storlets/ | Reviews: https://review.openstack.org/#/q/status:open+project:openstack/storlets,n,z" | 20:10 | |
| *** eranrom changes topic to "Logs: http://eavesdrop.openstack.org/irclogs/%23openstack-storlets/ | Reviews: https://review.openstack.org/#/q/status:open+project:openstack/storlets,n,z | Big-Tent: https://etherpad.openstack.org/p/storlets-big-tent" | 20:10 | |
| eranrom | hrou: Hi, just sent you a thank you note, in the hope it will not get filtered :-) | 20:17 |
| * eranrom thanks hrou | 20:17 | |
| *** mattoliverau has quit IRC | 21:47 | |
| *** matt6434 has joined #openstack-storlets | 21:47 | |
| *** matt6434 is now known as mattoliverau | 21:49 | |
| *** hrou has quit IRC | 22:27 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!