Thursday, 2016-03-17

*** c00281451_ has quit IRC00:19
*** c00281451 has joined #openstack-smaug00:20
*** c00281451 is now known as chenzeng00:51
*** yinwei_computer has quit IRC02:31
*** yinwei_computer has joined #openstack-smaug02:32
openstackgerritwangliuan proposed openstack/smaug: Implement Provider and ProviderRegistry  https://review.openstack.org/28161802:41
*** yinwei_computer has quit IRC02:41
*** yinwei_computer has joined #openstack-smaug02:42
*** yinwei_computer has quit IRC02:49
*** yinwei_computer has joined #openstack-smaug02:50
openstackgerritzengchen proposed openstack/smaug: implement time trigger  https://review.openstack.org/27119603:40
openstackgerritzengchen proposed openstack/smaug: Implement operation manager  https://review.openstack.org/27433903:47
openstackgerritzengchen proposed openstack/smaug: Implement executor of OperationEngine  https://review.openstack.org/28226303:49
*** WANG_Feng has quit IRC05:11
*** WANG_Feng has joined #openstack-smaug05:12
*** WANG_Feng has quit IRC05:12
*** chenying_ has joined #openstack-smaug05:49
openstackgerritchenying proposed openstack/python-smaugclient: Skeleton for the smaug cli client  https://review.openstack.org/28643706:08
*** wangliuan has joined #openstack-smaug06:15
openstackgerritchenying proposed openstack/python-smaugclient: Implement the triggers cli of smaugclient  https://review.openstack.org/29162106:26
chenying_https://review.openstack.org/#/c/286424/ Why not this patch is not merged? It have been Workflow+1.06:30
*** wangliuan has quit IRC06:54
*** yuval has joined #openstack-smaug07:05
*** zengyingzhe has quit IRC07:11
*** zengyingzhe has joined #openstack-smaug07:12
yuvalping zengyingzhe07:14
zengyingzhehi07:18
zengyingzhewhat's up, yuval07:18
yuvalGreat, how are you?07:18
zengyingzhei'm good07:19
yuvalIs there anything new with the ClientFactory we spoke about?07:20
yuvalor functions07:21
zengyingzheyes, i've finished the code, about to submit.07:24
yuvalGreat! :)07:24
openstackgerritYingzhe Zeng proposed openstack/smaug: Implement the common logic for service client  https://review.openstack.org/29152807:29
zengyingzheyuval, please check the patch07:30
zengyingzheif no problem, this patch could be merged first.07:30
yuvalping chenzeng07:58
chenzengyuval:yes08:02
yuvalchenzeng: how are you? did you had the chance to look on Eran's comment about rebasing the patches?08:03
yuvalchenzeng: we would like to merge them (ScheduleOperation and Trigger REST API)08:03
chenzengyuval:ok, wait a moment. i will do it right now.08:04
chenzengyuval:do you see the changes about the executor?08:04
yuvalchenzeng: yes, but I think it requires more work. Can you use the thread executor without the log executor right now?08:05
chenzengyuval:no08:07
chenzengyuval:right now. we must record logs in the db before execute the operation.08:08
yuvalchenzeng: sure, I understand, and I think we can find a way for it to be modular08:09
chenzengyuval:hope for your good ideas. thank you.08:10
yuvalzengyingzhe: did you try the client factory?08:26
openstackgerritzengchen proposed openstack/smaug: Implement executor of OperationEngine  https://review.openstack.org/28226308:27
gampelthanks zengchen will try to merge the API layer08:29
gampelPlease vote on the two API patches  https://review.openstack.org/282263 trigger and schedule  operation08:30
chenzenggampel:i am rebasing now. wait a moment.08:31
gampelOK08:31
gampelthx08:31
yuvalzengyingzhe: please make the factory's create_client a static method if possible, or make the factory a singleton08:32
openstackgerritzengchen proposed openstack/smaug: Implement rpc interfaces of OperationEngine  https://review.openstack.org/27608108:32
gampelchenying: hi08:33
openstackgerritzengchen proposed openstack/smaug: Implement RestAPIs of trigger.  https://review.openstack.org/28640608:35
gampelchenying are you there08:36
xiangxinyonggampel: chenying is on holiday08:37
gampelok thx08:37
xiangxinyonggampel: he wiil be back on march 2808:37
gampelwill someone be able to check why the CI pep8 does not catch /r/n and the local tox do catch it08:37
openstackgerritMerged openstack/python-smaugclient: Sync common code from oslo-incubator  https://review.openstack.org/28642408:38
openstackgerritMerged openstack/python-smaugclient: Skeleton for the smaug cli client  https://review.openstack.org/28643708:38
gampelthere are many patches from windows with the wrong line ending , but the CI does not catch taht08:38
xiangxinyonggampel: you means the local "tox -e pep8 command" could catch /r/n08:39
openstackgerritzengchen proposed openstack/smaug: Implement RestAPIs of Scheduled Operation.  https://review.openstack.org/28703608:39
xiangxinyonggampel: but the openstack ci did not catch it08:40
xiangxinyonggampel:is this your problem?08:40
chenzenggampel:rebase done.08:41
xiangxinyonggampel: i will try to find a answer about this question.08:44
gampelthx08:46
gampelhurong : can you please rebase https://review.openstack.org/#/c/286458/08:48
gampelplease review https://review.openstack.org/#/c/286406/08:49
yuvalzengyingzhe: also, we co/n08:53
yuvaln/08:53
zengyingzheyuval, i'm working on it.08:56
yuvalzengyingzhe: also, the client factory didn't work for me. We can use os.listdir instead of os.walk and import using relative module path08:57
yuvalzengyingzhe: thanks08:57
yinweiisherehi, yuval09:31
yinweiishereis saggi there?09:32
yuvalNo, can I help?09:32
yinweiishereit's about his checkpoint patch09:32
yinweiisherenow we have two patches dependent on checkpoint implementation09:32
yinweiishereif you could help on updating this patch once you guys algined, that would be nice09:33
yinweiishereit's checkpoint should add a pair of functions to update/get resource_def in memory. we need to persist protection result through the update function and sync checkpoint status through the get function09:34
yinweiishereyuval, the patch is this one:https://review.openstack.org/#/c/280325/09:37
yinweiishereI have given some comments there, but I see no response09:37
yuvalyinweiishere: I'll look at it09:41
yuvalping chenzeng09:41
chenzengyuval:yes09:42
yinweiishereyuval, thanks!09:53
yinweiishereyuval, nice to have you always here :)09:53
openstackgerritYingzhe Zeng proposed openstack/smaug: Implement the common logic for service client  https://review.openstack.org/29152810:03
openstackgerritYingzhe Zeng proposed openstack/smaug: Implement Nova service client  https://review.openstack.org/29153210:11
openstackgerritYingzhe Zeng proposed openstack/smaug: Implement Cinder service client  https://review.openstack.org/29153510:17
*** wangliuan2 has joined #openstack-smaug10:54
openstackgerritchenying proposed openstack/python-smaugclient: Implement the plans cli of smaug  https://review.openstack.org/28771811:02
wangliuan2https://review.openstack.org/#/c/281618/     Eran,I have address your comments in new patch11:02
openstackgerritchenying proposed openstack/python-smaugclient: Implement the plans cli of smaug  https://review.openstack.org/28771811:11
openstackgerritchenying proposed openstack/python-smaugclient: Implement the restore cli of smaugclient  https://review.openstack.org/28786411:25
openstackgerritchenying proposed openstack/python-smaugclient: Implement the plans cli of smaug  https://review.openstack.org/28771811:29
openstackgerritchenying proposed openstack/python-smaugclient: Implement the restore cli of smaugclient  https://review.openstack.org/28786411:45
*** openstackgerrit has quit IRC11:48
*** openstackgerrit has joined #openstack-smaug11:48
openstackgerritchenying proposed openstack/python-smaugclient: Implement the protectable cli of smaugclient  https://review.openstack.org/28978412:26
zengyingzheping yuval12:34
yuvalzengyingzhe: hey12:34
openstackgerritzengchen proposed openstack/smaug: Implement executor of OperationEngine  https://review.openstack.org/28226312:44
openstackgerritchenying proposed openstack/python-smaugclient: Implement the providers cli of smaugclient  https://review.openstack.org/29103713:03
*** gampel has quit IRC13:28
*** saggi has quit IRC13:29
*** yuval has quit IRC13:29
openstackgerritchenying proposed openstack/python-smaugclient: Implement the providers cli of smaugclient  https://review.openstack.org/29103713:34
*** yuval has joined #openstack-smaug13:41
*** saggi has joined #openstack-smaug13:43
*** gampel has joined #openstack-smaug13:43
*** gampel has quit IRC13:55
openstackgerritchenying proposed openstack/python-smaugclient: Implement the checkpoints cli of smaugclient  https://review.openstack.org/29104414:31
openstackgerritMerged openstack/python-smaugclient: Implement the plans cli of smaug  https://review.openstack.org/28771814:40
*** gampel has joined #openstack-smaug15:45
openstackgerritchenying proposed openstack/python-smaugclient: Implement the triggers cli of smaugclient  https://review.openstack.org/29162116:28
*** yuval has quit IRC16:31
openstackgerritMerged openstack/python-smaugclient: Implement the restore cli of smaugclient  https://review.openstack.org/28786419:19
openstackgerritMerged openstack/smaug: docs: fix operation_engine_design rst syntax  https://review.openstack.org/29289819:19
openstackgerritMerged openstack/python-smaugclient: Implement the protectable cli of smaugclient  https://review.openstack.org/28978419:19

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!