| kota_ | eronrom: Did the e-mail include us as c.c.? | 00:48 |
|---|---|---|
| kota_ | eranrom: sounds good to me though. | 00:49 |
| *** takashi has joined #openstack-storlets | 01:00 | |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Use context manager for os.pipe https://review.openstack.org/294352 | 01:20 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Use contextmanager about StorletLogger https://review.openstack.org/294355 | 01:26 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Use contextmanager for storlet invocation descriptors https://review.openstack.org/294358 | 01:34 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Use contextmanager when opening file https://review.openstack.org/294359 | 01:37 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Use contextmanager about input data writer to storlet container https://review.openstack.org/294362 | 01:47 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Use contextmanager about input data writer to storlet container https://review.openstack.org/294362 | 02:27 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Use contextmanager for storlet invocation descriptors https://review.openstack.org/294358 | 02:29 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Use contextmanager about input data writer to storlet container https://review.openstack.org/294362 | 03:30 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Use contextmanager when opening file https://review.openstack.org/294359 | 03:37 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Fix an error in FakeApp https://review.openstack.org/294380 | 03:50 |
| takashi | I devided my patch 287738 introducing contextmanager about file descriptors into some smaller patches. | 04:04 |
| patchbot | takashi: https://review.openstack.org/#/c/287738/ - storlets - Re-implement restart_docker_container by python (ABANDONED) | 04:04 |
| takashi | oh.. I pointed a wrong patch... patch 282211 is the right one | 04:05 |
| patchbot | takashi: https://review.openstack.org/#/c/282211/ - storlets - Use contextmanager to manager file descriptors | 04:05 |
| takashi | I think the smaller ones are better. We can make the effect of each patch clear. | 04:06 |
| takashi | The original patch is still WIP state (not abandoned), because I didn't comitted unittests included in it. | 04:08 |
| takashi | I'll abandon after doing it, so please see the smaller patch set when reviewing it. | 04:08 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: DO NOT MERGE: just testing https://review.openstack.org/294406 | 06:07 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: DO NOT MERGE: just testing https://review.openstack.org/294406 | 06:22 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Merge StorletInvocationXXXProtocol https://review.openstack.org/294406 | 07:01 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Merge StorletInvocationXXXProtocol https://review.openstack.org/294406 | 07:48 |
| *** takashi has quit IRC | 08:45 | |
| *** openstackgerrit has quit IRC | 12:48 | |
| *** openstackgerrit has joined #openstack-storlets | 12:49 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!