| *** jwcroppe has quit IRC | 00:27 | |
| *** jwcroppe has joined #cloudkitty | 00:40 | |
| *** jwcroppe has quit IRC | 00:45 | |
| *** gongysh has quit IRC | 00:47 | |
| *** dinghh has left #cloudkitty | 01:16 | |
| *** openstackgerrit has joined #cloudkitty | 01:26 | |
| openstackgerrit | Aaron Ding proposed openstack/cloudkitty: Modify policy of get total https://review.openstack.org/404680 | 01:26 |
|---|---|---|
| *** liujiong has joined #cloudkitty | 01:28 | |
| *** jwcroppe has joined #cloudkitty | 01:47 | |
| *** gongysh has joined #cloudkitty | 02:00 | |
| *** jwcroppe has quit IRC | 02:14 | |
| openstackgerrit | Aaron Ding proposed openstack/cloudkitty: Modify policy of get total https://review.openstack.org/404680 | 02:36 |
| *** aolwas has quit IRC | 03:05 | |
| *** cgayan has quit IRC | 03:05 | |
| *** cgayan_ has joined #cloudkitty | 03:05 | |
| *** aolwas has joined #cloudkitty | 03:05 | |
| *** jwcroppe has joined #cloudkitty | 03:54 | |
| *** jwcroppe has quit IRC | 04:05 | |
| *** jwcroppe has joined #cloudkitty | 05:43 | |
| *** jwcroppe has quit IRC | 05:54 | |
| *** jwcroppe has joined #cloudkitty | 07:33 | |
| *** jwcroppe has quit IRC | 07:44 | |
| *** openstackgerrit has quit IRC | 07:50 | |
| *** zhurong has joined #cloudkitty | 08:22 | |
| *** jwcroppe has joined #cloudkitty | 08:28 | |
| *** jwcroppe has quit IRC | 08:37 | |
| *** jwcroppe has joined #cloudkitty | 09:22 | |
| *** jwcroppe has quit IRC | 09:33 | |
| *** zhurong has quit IRC | 09:40 | |
| *** openstack has joined #cloudkitty | 10:15 | |
| *** jwcroppe has joined #cloudkitty | 10:17 | |
| *** jwcroppe has quit IRC | 10:27 | |
| *** gongysh has quit IRC | 10:31 | |
| *** liujiong has quit IRC | 10:47 | |
| *** gongysh has joined #cloudkitty | 11:45 | |
| *** gpocentek has quit IRC | 11:55 | |
| *** sileht has quit IRC | 11:55 | |
| *** zeus has quit IRC | 11:55 | |
| *** zeus- has joined #cloudkitty | 11:55 | |
| *** gpocentek has joined #cloudkitty | 11:56 | |
| *** zeus- is now known as zeus | 11:56 | |
| *** gpocentek has quit IRC | 11:56 | |
| *** gpocentek has joined #cloudkitty | 11:56 | |
| *** zeus is now known as Guest74290 | 11:56 | |
| *** sileht has joined #cloudkitty | 11:57 | |
| *** jwcroppe has joined #cloudkitty | 12:07 | |
| *** gongysh has quit IRC | 12:14 | |
| *** jwcroppe has quit IRC | 12:15 | |
| *** zhangguoqing has joined #cloudkitty | 12:58 | |
| huats | Hello everyone ! | 13:00 |
| aolwas | hi | 13:00 |
| huats | We are supposed to have a meeting now :) | 13:01 |
| huats | we can give a few minutes for the late arrivals ! | 13:02 |
| zhangguoqing | yes. :) | 13:02 |
| huats | zhangguoqing: o/ | 13:02 |
| huats | do you know if someone will arrive too ? | 13:07 |
| zhangguoqing | maybe not | 13:08 |
| huats | ok then let's start | 13:08 |
| huats | and if someone arrive they will catch up | 13:08 |
| huats | so we are going closer and closer from the full freeze (26th of January) | 13:09 |
| huats | so it means we just have 3 weeks before the end of all development for this release | 13:09 |
| huats | I think it is important to have that meeting | 13:10 |
| huats | to see what is going on | 13:10 |
| huats | we don't have many features that have landed yet and some are clearly under development | 13:10 |
| zhangguoqing | yes, how to plan the meeting ? | 13:11 |
| huats | I think it would be too much time consumming to take every review one by one | 13:11 |
| *** liujiong has joined #cloudkitty | 13:12 | |
| huats | may be we can just tackle all the reviews that we are working on/expecting ? | 13:12 |
| zhangguoqing | liujiong, welcome! | 13:12 |
| liujiong | thx, guoqing. | 13:13 |
| huats | hey liujiong we are just starting | 13:13 |
| huats | I was saying that we need to be able to integrate stuffs /review things | 13:13 |
| huats | because the end of the cycle is near (3 weeks...) | 13:13 |
| liujiong | Yep. | 13:13 |
| liujiong | Community almost stopped for two weeks. | 13:14 |
| huats | liujiong: yep which is quite understable | 13:14 |
| huats | but now we need to start again :) | 13:14 |
| huats | so the question is how can we do that ? | 13:14 |
| zhangguoqing | I think, we should review the easy or small BPs quickly. | 13:15 |
| zhangguoqing | Then all of us will do the best to the rest of big or hard BPs. | 13:17 |
| liujiong | We have several good patches that deserve to be merged this cycle. zhangguoqing, Max, Aaron Ding's patches. | 13:18 |
| aolwas | we have some electrical power issue here in toulouse ... huats will reconnect shrtly | 13:19 |
| liujiong | all right! | 13:19 |
| aolwas | luijiong and your's but I was waiting for Aaron's patches to be merged before working on | 13:21 |
| aolwas | it | 13:21 |
| zhangguoqing | aolwas: which one? | 13:22 |
| liujiong | Seems like Aaron is working on other stuffs these days. | 13:22 |
| aolwas | get total by tenant and user | 13:22 |
| zhangguoqing | https://review.openstack.org/#/c/393021/ | 13:22 |
| huats | guys I am sorry for the power shortage... | 13:23 |
| huats | it was the whole block | 13:23 |
| liujiong | It's all right, huats. | 13:23 |
| zhangguoqing | It does not matter. | 13:24 |
| huats | so that was my idea : that we share together the first feedback on small patches while we all try to work on integrating bigger patches | 13:24 |
| huats | if you agree I can point each of you with the small review to do | 13:24 |
| huats | (privately) | 13:25 |
| zhangguoqing | yes, agree with you. | 13:25 |
| aolwas | ok for me | 13:25 |
| huats | so that we won't all spend time on the same | 13:25 |
| huats | (I mean for the first review...) | 13:25 |
| huats | and when someone is OK here with a patch, he can announce it here so that other would knonw that it is a good time to have a look | 13:25 |
| huats | zhangguoqing/liujiong/gpocentek/aolwas what do you think ? | 13:26 |
| liujiong | Fine with me. I will try my best to review patches. | 13:26 |
| zhangguoqing | no problem. | 13:26 |
| huats | I guess the first think would be to continue to review the patches that are already reviewed and +2 by a core | 13:27 |
| aolwas | it's fine for me ... and it will also be useful for core reviewer that patches are ready to be merged | 13:27 |
| huats | patches that have been +2 by a core | 13:28 |
| huats | are : | 13:28 |
| huats | https://review.openstack.org/#/c/402117/ | 13:28 |
| huats | https://review.openstack.org/#/c/404089/ | 13:28 |
| huats | https://review.openstack.org/#/c/365575/ | 13:28 |
| liujiong | BTW, I do not have the rights to give +2 or +w. :( | 13:28 |
| huats | https://review.openstack.org/#/c/393021/ | 13:28 |
| huats | https://review.openstack.org/#/c/397192/ | 13:28 |
| huats | https://review.openstack.org/#/c/397198/ | 13:28 |
| huats | I know liujiong, but it is something that I'd like to change soon... | 13:29 |
| huats | but you can put act like anyone else (I am not giving +2 myself...) | 13:29 |
| huats | and your inputs are as important as any others | 13:29 |
| liujiong | thanks, huats. | 13:29 |
| huats | I have already asked pabardina to have a look at all the dashboard related reviews | 13:30 |
| huats | he will do them really really soon | 13:31 |
| huats | so liujiong you can focus on the other reviews (the one that haven't received a +2 yet) | 13:31 |
| huats | I'll give to each of you a like of the small one today | 13:31 |
| liujiong | OK... | 13:32 |
| huats | my main focus is that someone merge https://review.openstack.org/#/c/393021/ | 13:33 |
| huats | zhangguoqing: can you do that ? | 13:33 |
| zhangguoqing | I have not tested it, but the static review is ok for me. | 13:34 |
| huats | I know that aolwas did extensive testing on it | 13:34 |
| huats | and once that one is merged many other patches needs to be rebase | 13:34 |
| huats | so it is a good starting point | 13:34 |
| zhangguoqing | OK, I will fetch and test this one quickly. | 13:35 |
| huats | The https://review.openstack.org/#/c/404680/ is also very important becasue even if it is quite small it has quite some implications | 13:35 |
| huats | zhangguoqing: thanks | 13:36 |
| huats | liujiong: gpocentek and zhangguoqing can you also review https://review.openstack.org/#/c/406180/ ? | 13:36 |
| huats | it is one of the biggest change we will have on the release so I would really like to have it | 13:37 |
| huats | (since it will have some developments done in horizon after that) | 13:37 |
| liujiong | I'm fine, will have a look and test on it soon. | 13:38 |
| huats | liujiong: thanks ! | 13:38 |
| zhangguoqing | Maxime Cottret have a good work for them, thanks! | 13:38 |
| aolwas | thanks zhangguoqing | 13:39 |
| huats | do you have some specific things you want to address ? | 13:39 |
| huats | becasue I think we start to have a step-by-step for our last weeks | 13:40 |
| zhangguoqing | https://review.openstack.org/#/c/397192/ and https://review.openstack.org/#/c/397198/ should be merged now please. | 13:40 |
| huats | ok | 13:41 |
| zhangguoqing | https://review.openstack.org/#/c/404089/ merging now please. | 13:41 |
| zhangguoqing | https://review.openstack.org/#/c/365575/ also pelase! | 13:42 |
| liujiong | thanks, guoqing! | 13:42 |
| huats | zhangguoqing: I will review the lasat one (documentation) and pabardina will review the 2 related to the dashboard | 13:42 |
| huats | gpocentek: can you take the point to review https://review.openstack.org/#/c/365575/ please ? | 13:42 |
| huats | I think it would be great to have another meeting next week : same time/same day | 13:43 |
| huats | is it ok for you ? so that we can focus on other things | 13:43 |
| huats | (hoping that all that will be done meanwhile) | 13:43 |
| liujiong | OK for me. | 13:44 |
| zhangguoqing | me too. | 13:44 |
| aolwas | good for me too | 13:44 |
| huats | ok then | 13:45 |
| huats | great ! | 13:45 |
| huats | we just have to review review and review in the next week :) | 13:45 |
| liujiong | I have a patch that hangs for a long time. https://review.openstack.org/#/c/329242/ | 13:45 |
| huats | (and improve all our patches accordingly of course) | 13:45 |
| huats | gpocentek: can you please have a look at https://review.openstack.org/#/c/329242/ ? | 13:46 |
| zhangguoqing | liujiong:https://review.openstack.org/#/c/329242/ is very import for RDO package. | 13:46 |
| liujiong | zhangguoqing: Yes. rdo has be updated by you. | 13:47 |
| liujiong | So this one needs to be merged this cycle. | 13:47 |
| huats | (I am asking you since you had some strong knowledget f packaging and did a lot for both RDO and UBuntu) | 13:47 |
| huats | I know that gpocentek is not available right now but he will put a review soon | 13:48 |
| zhangguoqing | but your patch set is not good, I will help you make it better. | 13:48 |
| liujiong | zhangguoqing: Thank you! | 13:48 |
| liujiong | Please let me know. | 13:48 |
| zhangguoqing | liujiong: no problem. | 13:48 |
| huats | zhangguoqing: but please review first the one I showed you from aolwas | 13:49 |
| zhangguoqing | yes, I see. | 13:49 |
| huats | I would be in favor of merging first the biggest | 13:49 |
| huats | I think we can terminate that meeting | 13:50 |
| huats | and hang around if you have things to discuss on specific points ! | 13:50 |
| huats | (meanwhile I'll give each of you a list of small review to have a look) | 13:51 |
| liujiong | huats: All right! We can also check the logs here. http://eavesdrop.openstack.org/irclogs/%23cloudkitty/ | 13:52 |
| huats | yes ! | 13:54 |
| huats | thanks liujiong | 13:54 |
| *** jwcroppe has joined #cloudkitty | 13:56 | |
| *** jwcroppe has quit IRC | 14:08 | |
| *** Guest74290 is now known as zeus` | 14:18 | |
| *** zeus` is now known as zeus | 14:18 | |
| *** zeus has quit IRC | 14:18 | |
| *** zeus has joined #cloudkitty | 14:18 | |
| *** zhangguoqing has quit IRC | 14:19 | |
| *** openstackgerrit has joined #cloudkitty | 14:33 | |
| openstackgerrit | Merged openstack/cloudkitty-dashboard: Drop MANIFEST.in - it's not needed by pbr https://review.openstack.org/397198 | 14:33 |
| openstackgerrit | Merged openstack/cloudkitty: Bring the begin/end checking before the storage module https://review.openstack.org/402117 | 14:35 |
| openstackgerrit | Merged openstack/cloudkitty: The qty's type should be more precision in storage tables https://review.openstack.org/365575 | 14:35 |
| openstackgerrit | Merged openstack/cloudkitty-dashboard: Don't include openstack/common in flake8 exclude list https://review.openstack.org/397192 | 14:40 |
| *** jwcroppe has joined #cloudkitty | 14:51 | |
| *** liujiong has quit IRC | 14:55 | |
| *** jwcroppe has quit IRC | 15:03 | |
| *** jwcroppe has joined #cloudkitty | 15:22 | |
| openstackgerrit | Christophe Sauthier proposed openstack/cloudkitty: Add a note to indicate that the default port of cloudkitty changed https://review.openstack.org/417087 | 16:47 |
| openstackgerrit | Christophe Sauthier proposed openstack/cloudkitty: Add a note to indicate the change of default port https://review.openstack.org/417087 | 16:50 |
| openstackgerrit | Christophe Sauthier proposed openstack/cloudkitty: Add a note to indicate the change of default port https://review.openstack.org/417087 | 17:46 |
| *** jwcroppe has quit IRC | 23:58 | |
| *** jwcroppe has joined #cloudkitty | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!