| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add python sdaemon support to storlet gateway https://review.openstack.org/362326 | 04:04 |
|---|---|---|
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add functional tests for python storlet execution https://review.openstack.org/359983 | 04:04 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add Python sdaemon support in daemon_factory https://review.openstack.org/356224 | 04:04 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Implement Python SDaemon https://review.openstack.org/358252 | 04:04 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Make python storlet daemon load applications https://review.openstack.org/363303 | 04:04 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Make python storlet daemon load applications https://review.openstack.org/363303 | 06:14 |
| eranrom | Hi All, you might be interested in this one: http://itsonlyme.name/blog | 06:23 |
| eranrom | Will be very interested to get your feedback on this. | 06:24 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Make python storlet daemon load applications https://review.openstack.org/363303 | 06:39 |
| *** takashi_ has joined #openstack-storlets | 06:41 | |
| takashi_ | eranrom: interesting! I'll have a look later | 06:42 |
| kota_ | eranrom: awesome. book it to my favarit in my browser, I will take a look later. | 06:46 |
| *** takashi_ has quit IRC | 07:11 | |
| eranrom | takashi: kota_: Thanks! | 07:33 |
| *** takashi_ has joined #openstack-storlets | 07:48 | |
| openstackgerrit | Eran Rom proposed openstack/storlets: Creanup redundant README.md and README.rst https://review.openstack.org/347278 | 07:59 |
| eranrom | flaper87: I have just read the meeting log again. (1) Thanks very much for your help on this. | 08:34 |
| eranrom | flaper87: (2) Once Takashi took care of switching to OpenJDK, is seems to me that the next bit todo item is to refactor the structure of the project so that it is python centric | 08:34 |
| eranrom | flaper87: while we be looking at this, is there anything else you think we should do at this point? | 08:35 |
| flaper87 | eranrom: hey there! | 11:13 |
| flaper87 | eranrom: yeah, I noticed a couple of other things I wanted to share | 11:13 |
| * flaper87 takes his notes | 11:13 | |
| flaper87 | 1. Change the copyright header to something like: https://github.com/openstack/zaqar/blob/master/zaqar/i18n.py#L3-L14 | 11:15 |
| eranrom | flaper87: waiting patiently | 11:15 |
| flaper87 | 2. change the package path to not mention ibm: https://github.com/openstack/storlets/tree/master/Engine/SBus/SBusJavaFacade/src/com/ibm/storlet/sbus | 11:16 |
| flaper87 | 3. Wonder if this should test master: https://github.com/openstack/storlets/blob/master/tox.ini#L13 | 11:17 |
| flaper87 | although swift has independent releases | 11:17 |
| flaper87 | eranrom: I don't recall if storlets will have independent releases too | 11:17 |
| flaper87 | will it? | 11:17 |
| eranrom | flaper87: In the patch I have 'stated' independent releases | 11:17 |
| flaper87 | eranrom: perfect | 11:17 |
| flaper87 | I guess it's fine for now. It's weird to download swift code from the tox file | 11:18 |
| flaper87 | we'll get to that later | 11:18 |
| flaper87 | let's first take care of the restructuring work | 11:18 |
| eranrom | flaper87: ok | 11:18 |
| flaper87 | eranrom: btw, where are you based? | 11:18 |
| eranrom | flaper87: Israel, you? | 11:18 |
| flaper87 | eranrom: EU | 11:18 |
| flaper87 | perfect | 11:18 |
| flaper87 | almost the same TZ | 11:18 |
| eranrom | perfect indeed | 11:18 |
| flaper87 | 4. It'd be great to add py35 to the list of envs https://github.com/openstack/storlets/blob/master/tox.ini#L3 | 11:19 |
| flaper87 | that means adding a py35 job | 11:19 |
| flaper87 | btw, if there are things you've no clue how to do, let me know. I'm here to help | 11:20 |
| flaper87 | like the ci jobs stuff | 11:20 |
| flaper87 | Question: Why do you need this file? https://github.com/openstack/storlets/blob/master/.unittests#L1 | 11:21 |
| flaper87 | I understand what it does | 11:21 |
| flaper87 | but I'm curious to know what problems it solves | 11:21 |
| flaper87 | it'd be great to have storlet use testr | 11:21 |
| eranrom | flaper87: well, I mostly followed what's happening in Swift. | 11:22 |
| eranrom | flaper87: I agree its 'redundant' | 11:22 |
| eranrom | testr instead of unittest? | 11:23 |
| flaper87 | eranrom: gotcha | 11:23 |
| flaper87 | yeah, instead of nostests | 11:23 |
| flaper87 | we can remove the entire file, tbh | 11:23 |
| eranrom | you mean .unittest? | 11:23 |
| flaper87 | Ok, here's a plan of actions, let me know if you agree | 11:23 |
| flaper87 | eranrom: yup | 11:23 |
| eranrom | ok, with you... | 11:23 |
| flaper87 | eranrom: let's take care of the easy stuff first like restructuring the code base, fixing the imports order, etc | 11:24 |
| flaper87 | once that's done, we'll drill down into the testing side of things | 11:24 |
| flaper87 | that way I'll have some time to run the unittests and see if I can help there too | 11:24 |
| flaper87 | does that sound good? | 11:24 |
| eranrom | flaper87: sounds good to me. At some point I will also need to get kota's / takashi's takwe | 11:25 |
| eranrom | s/takwe/take | 11:25 |
| flaper87 | yeah! Perfect | 11:25 |
| * flaper87 runs tox on storlets | 11:26 | |
| eranrom | to be honest, I am running into prb / setuptools issues around the '>' markers. | 11:27 |
| eranrom | Jenkins,however, seems to be happy. I still haven't cracked it yet. | 11:27 |
| flaper87 | mhh, fun. It's usually the other way around. Jenkins is cranky and I'm happy | 11:28 |
| eranrom | :-) | 11:28 |
| kota_ | eranrom: can i do something? | 11:29 |
| kota_ | sorry, was away | 11:29 |
| kota_ | looking at scroll back... | 11:30 |
| eranrom | kota_: great that you have joined. | 11:30 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add python sdaemon support to storlet gateway https://review.openstack.org/362326 | 11:33 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add functional tests for python storlet execution https://review.openstack.org/359983 | 11:33 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add Python sdaemon support in daemon_factory https://review.openstack.org/356224 | 11:33 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Implement Python SDaemon https://review.openstack.org/358252 | 11:33 |
| kota_ | it looks like takashi is also still working | 11:34 |
| openstackgerrit | Kota Tsuyuzaki proposed openstack/storlets: WIP: use Swift master to run unit test https://review.openstack.org/364246 | 11:34 |
| kota_ | addressed item 3 from flaper87 | 11:35 |
| kota_ | i think we can track master right now because Akihito did something to fix on copy middleware. | 11:36 |
| kota_ | hopefully it works well on the gate. | 11:36 |
| eranrom | kota_: I lost the count :-) what is item 3? .unittest? | 11:37 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Close metadata read md https://review.openstack.org/364247 | 11:38 |
| kota_ | py35... hmm, that makes my headake. I think it's a good comment but I don't know when Swift can finish the py35 migration :/ | 11:38 |
| kota_ | ah, changing tox Swift envrion from 2.7.0 to master | 11:38 |
| kota_ | .unittest seems extra after item 4 | 11:38 |
| flaper87 | eranrom: I can write them down on an etherpad | 11:42 |
| eranrom | flaper87: That would be great, thanks! | 11:43 |
| flaper87 | actually, going to do that | 11:43 |
| * flaper87 dances | 11:43 | |
| eranrom | :-) | 11:43 |
| takashi_ | kota_, flaper87: IMO, we need 2 steps to add py35 env. First, we can create an entry point in tox for py35, and execute very 'limited' tests as current swift does | 11:43 |
| takashi_ | as the second step, we need to fix the things to add complete support for py35 | 11:43 |
| flaper87 | eranrom: kota_ takashi_ https://etherpad.openstack.org/p/storlets-big-tent | 11:44 |
| flaper87 | takashi_: sounds good | 11:44 |
| flaper87 | I'll keep digging and adding stuff there | 11:44 |
| flaper87 | feel free to scratch the ones that are done | 11:44 |
| flaper87 | it'd be cool if you could add the review links that address those points to the ehterpad too | 11:45 |
| flaper87 | and thanks a bunch for working on this | 11:45 |
| flaper87 | appreciated | 11:45 |
| eranrom | flaper87: thanks. kota: let's add your comment on py35 there. | 11:45 |
| eranrom | flaper87: kota_: I need to go for a while | 11:45 |
| eranrom | talk to you later. | 11:45 |
| eranrom | flaper87: thanks again! | 11:45 |
| flaper87 | eranrom: rock on! ttyl | 11:46 |
| takashi_ | flaper87: thanks for creating etherpad. I wrote my ideas about py3 in that. | 11:47 |
| flaper87 | takashi_: ++ | 11:47 |
| kota_ | flaper87: thank you for maitaining us :-) | 11:47 |
| flaper87 | kota_: my pleasure :) | 11:56 |
| kota_ | yup, https://review.openstack.org/364254 | 11:57 |
| kota_ | eranrom: patcch 364254 is the prep to avoid .unittests for running tox | 11:58 |
| kota_ | i'm not sure if we want to remove it because Swift looks like still keeping .unitest file even though they don't run the unittest via .unittest file. | 11:59 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Make python storlet daemon load applications https://review.openstack.org/363303 | 12:05 |
| eranrom | kota_: regarding .unittest I am not sure how significant the point is. We can add t link to the patch on the etherpad, and decide later. | 12:05 |
| kota_ | ok, at this time, it's late for today, i'll take a look. | 12:06 |
| *** takashi_ has quit IRC | 13:10 | |
| *** openstackgerrit has quit IRC | 13:49 | |
| *** openstackgerrit has joined #openstack-storlets | 13:49 | |
| openstackgerrit | Doron Chen proposed openstack/storlets: skip to relevant section of YouTube video https://review.openstack.org/364350 | 14:23 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add python sdaemon support to storlet gateway https://review.openstack.org/362326 | 15:51 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add functional tests for python storlet execution https://review.openstack.org/359983 | 15:51 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add Python sdaemon support in daemon_factory https://review.openstack.org/356224 | 15:51 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Make python storlet daemon load applications https://review.openstack.org/363303 | 15:51 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add python sdaemon support to storlet gateway https://review.openstack.org/362326 | 16:35 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add functional tests for python storlet execution https://review.openstack.org/359983 | 16:35 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Add Python sdaemon support in daemon_factory https://review.openstack.org/356224 | 16:35 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Make python storlet daemon load applications https://review.openstack.org/363303 | 16:35 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Change the package name to not mention ibm https://review.openstack.org/364473 | 17:17 |
| openstackgerrit | Takashi Kajinami proposed openstack/storlets: Make python storlet daemon load applications https://review.openstack.org/363303 | 17:23 |
| *** openstackgerrit has quit IRC | 18:18 | |
| *** openstackgerrit has joined #openstack-storlets | 18:19 | |
| *** takashi has quit IRC | 18:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!