| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Use contextmanager to manager file descriptors https://review.openstack.org/282211 | 01:03 |
|---|---|---|
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Prohibit storlet execution with range header https://review.openstack.org/282931 | 01:10 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add X-Storlet-Range header to specify input range https://review.openstack.org/283423 | 01:20 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Improve handling about Timeout https://review.openstack.org/276545 | 01:24 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Make stub_gateway pass through headers https://review.openstack.org/285156 | 01:32 |
| takashi | I'm afraid gate job is now broken... :-( | 01:37 |
| takashi | it fails to install required packages after apt-get upgrade, and fails to install swift/storlets on the testing node. | 01:38 |
| takashi | It seems that "apt-get upgrade" breaks sudo | 01:38 |
| takashi | no, no. sudo failures is not caused by upgrade, but caused by jenkins... | 01:48 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add AUTHORS file https://review.openstack.org/276553 | 01:53 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Re-implement restart_docker_container by python https://review.openstack.org/287738 | 02:17 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: [DO NOT MERGE] This is for gate testing https://review.openstack.org/289668 | 02:47 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: [DO NOT MERGE] This is for gate testing https://review.openstack.org/289668 | 02:47 |
| takashi | I asked infra guys about gate job failure, and the fix will be merged soon. :-) | 03:39 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Improve handling about Timeout https://review.openstack.org/276545 | 04:01 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: [DO NOT MERGE] This is for gate testing https://review.openstack.org/289668 | 04:38 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: [DO NOT MERGE] This is for gate testing https://review.openstack.org/289668 | 05:42 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Fix gate job failure https://review.openstack.org/289668 | 05:55 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Make sure to install apt-transport-https https://review.openstack.org/289668 | 06:15 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Make sure to install apt-transport-https https://review.openstack.org/289668 | 06:22 |
| takashi | eranrom: Can you review patch 289668 ? It is needed to solves gate job failure. | 07:04 |
| patchbot | takashi: https://review.openstack.org/#/c/289668/ - storlets - Make sure to install apt-transport-https | 07:04 |
| eranrom | takashi,: done | 07:38 |
| eranrom | takashi,: thanks for the quick fix | 07:38 |
| takashi | eranrom: thx | 07:41 |
| takashi | project-config is now fixed, but we should still be careful because sometimes jenkins uses old version and fails to run tests. :-( | 07:42 |
| eranrom | takashi,: Do you know what got broken in infra? | 07:53 |
| takashi | eranrom: see the fix patch, patch 289684 | 07:55 |
| patchbot | takashi: https://review.openstack.org/#/c/289684/ - openstack-infra/project-config - Remove revoke-sudo from storlets, trove functional (MERGED) | 07:55 |
| openstackgerrit | Merged openstack/storlets: Make sure to install apt-transport-https https://review.openstack.org/289668 | 07:55 |
| takashi | eranrom: infra team convert all jobs from bare-trusty to ubuntu-trusty. | 07:56 |
| takashi | s/convert/converted | 07:56 |
| takashi | There were two problems. (1) infra team added an wrong cleaning up about sudoers (2) we now have to install add-transport-https. (1) is fixed in patch 289684 and (2) is fixed in patch 289668 | 07:59 |
| patchbot | takashi: https://review.openstack.org/#/c/289684/ - openstack-infra/project-config - Remove revoke-sudo from storlets, trove functional (MERGED) | 07:59 |
| patchbot | takashi: https://review.openstack.org/#/c/289668/ - storlets - Make sure to install apt-transport-https (MERGED) | 07:59 |
| takashi | I'm not sure why we now have to install apt-transport-https in our install scripts, but surely it fixes some failures in add-apt-repository. | 08:00 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add AUTHORS file https://review.openstack.org/276553 | 08:00 |
| takashi | I'm going to rebase all patches which got -1 from jenkins. | 08:02 |
| eranrom | takashi: ok. Any particular order to review the patches? range patches firsr? | 08:12 |
| takashi | eranrom: Can you review patch 282135 first? | 08:14 |
| patchbot | takashi: https://review.openstack.org/#/c/282135/ - storlets - Refactor unit tests about storlet docker gateway | 08:14 |
| eranrom | sure | 08:14 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Prohibit storlet execution with range header https://review.openstack.org/282931 | 08:19 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add X-Storlet-Range header to specify input range https://review.openstack.org/283423 | 08:20 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add some more comments about storlet_handler https://review.openstack.org/282174 | 08:24 |
| takashi | eranrom: I just rebased range patches, and I think they're going to be ready for review. | 08:31 |
| takashi | eranrom: I found some merge conflicts in refactoring patches with range pathces, so I'll fix refactoring patches after range patches get merged. | 08:32 |
| takashi | I think the range patches are more important, because they fix possible bugs. | 08:33 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add X-Storlet-Range header to specify input range https://review.openstack.org/283423 | 08:39 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add some more comments about storlet_handler https://review.openstack.org/282174 | 08:40 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Make stub_gateway pass through headers https://review.openstack.org/285156 | 08:42 |
| eranrom | takashi: ok so I will do 282135 followed by range patches. Will do so later today | 09:15 |
| takashi | eranrom: thx! :-) | 09:19 |
| *** takashi has quit IRC | 10:07 | |
| *** openstackgerrit has quit IRC | 10:33 | |
| *** openstackgerrit has joined #openstack-storlets | 10:34 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!