Monday, 2016-11-07

*** yizhihui has joined #openstack-karbor00:47
openstackgerritlichunyu proposed openstack/karbor: add eisoo oracle plugin entry point  https://review.openstack.org/39421501:02
*** LiChunyu has joined #openstack-karbor01:03
*** yizhihui has quit IRC01:16
*** yizhihui has joined #openstack-karbor01:21
*** zhurong has joined #openstack-karbor01:27
*** zhurong has quit IRC01:34
*** zhurong has joined #openstack-karbor01:34
*** zengyingzhe__ has quit IRC01:47
*** zengyingzhe has joined #openstack-karbor01:51
openstackgerritMerged openstack/karbor-dashboard: Remove the extra required=True  https://review.openstack.org/37998602:05
openstackgerritMerged openstack/karbor-dashboard: Don't include openstack/common in flake8 exclude list  https://review.openstack.org/39170402:07
openstackgerritMerged openstack/karbor-dashboard: Updated from global requirements  https://review.openstack.org/38989702:07
openstackgerritlichunyu proposed openstack/karbor: add eisoo oracle plugin entry point  https://review.openstack.org/39421502:10
openstackgerritzengchen proposed openstack/karbor: Optimize time trigger: Move the context to be held in green thread  https://review.openstack.org/37205102:36
openstackgerritzengchen proposed openstack/karbor: Optimize time trigger: remove the interval variable '_next_run_time'  https://review.openstack.org/37209102:50
openstackgerritzengchen proposed openstack/karbor: Optimize time trigger: use instance of time format instead of class itself  https://review.openstack.org/35282603:12
openstackgerritzengchen proposed openstack/karbor: Fix the singleton pattern in operation engine  https://review.openstack.org/38406203:49
openstackgerritpawnesh kumar proposed openstack/karbor: Enable translations  https://review.openstack.org/39194404:13
openstackgerritzengchen proposed openstack/karbor: Optimize time trigger: Move the context to be held in green thread  https://review.openstack.org/37205106:05
openstackgerritzengchen proposed openstack/karbor: Optimize time trigger: fix swapping out execution control of green thread  https://review.openstack.org/37219106:05
openstackgerritzengchen proposed openstack/karbor: Optimize time trigger: remove the interval variable '_next_run_time'  https://review.openstack.org/37209106:05
openstackgerritzengchen proposed openstack/karbor: Optimize time trigger: fix swapping out execution control of green thread  https://review.openstack.org/37219106:06
openstackgerritzengchen proposed openstack/karbor: Optimize time trigger: use instance of time format instead of class itself  https://review.openstack.org/35282606:06
*** yinwei_computer has joined #openstack-karbor06:21
*** Jeffrey4l has joined #openstack-karbor06:30
Jeffrey4lxiangxinyong, hi.06:31
xiangxinyongHello, welcome Jeffrey4106:32
Jeffrey4lthanks ;)06:33
xiangxinyongJeffrey41 is kolla project's core member.06:34
xiangxinyongAbout adding karbor into kolla and implementing container deployment, He is very Professional.06:37
*** c00281451 is now known as zengchen06:42
zengchenwelcome.06:43
*** Jeffrey4l has quit IRC07:18
yinwei_computerwelcome07:21
*** yuval has joined #openstack-karbor07:32
chenyinghi Jeffrey41 welcome07:55
chenyinghi lichunyu I note that you have splited the eisoo plugins to several patches. If you need any help, you can call me.07:57
openstackgerritzhangshuai proposed openstack/karbor: Support icalendar time format for time trigger  https://review.openstack.org/35971108:00
LiChunyuI meet some problem when commit these patches.  I abandon all these patches, and I'll recommit them in order08:01
LiChunyusetup.cfg -> eisoo.conf -> ClientFactory -> protectable plugin -> protection plugin .I will commit patches in this order .Is this OK?08:04
LiChunyuping chenying08:04
chenyingLiChunyu hi08:04
chenyingLiChunyu : The order about the patches should keep that all the unit test and fullstack test is OK08:06
LiChunyuand I have change abclient version in openstack-requirements, this may cost some time08:07
chenyingLiChunyu: IMO  ClientFactory -> protectable plugin -> protection plugin -> setup.cfg -> eisoo.conf  All the tests is OK with this order.08:07
LiChunyuOK, I'll commit in this order08:09
LiChunyuafter change on openstack-requirements is merged08:10
chenyinglichunyu When you commit every patch, you need note the unit tests.08:10
chenyingLiChunyu openstack-requirements you can commit it with ClientFactory patch.08:11
LiChunyuDo u mean commit unit test are committed together with the code ?08:14
LiChunyuDo u mean unit test are committed together with the code ?08:14
chenyinglicunyu YEs08:16
chenyinglicunyu : every patch include the unit test about these code in this ptach.08:17
*** Jeffrey4l has joined #openstack-karbor08:17
LiChunyuchenying ok08:20
yuvalhey chenying08:33
chenyingyuval: hi08:34
yuvalchenying: how are you?08:34
chenyingyuval:I am fine Good morning.08:35
yuvalchenying: good afternoon :)08:35
yuvalchenying: is anyone working on the server protection plugin on your side?08:35
yuvalchenying: in Barcelona we spoke about a few problems with the current implementation08:35
chenyingchenying: You mean the server plugins base on your new workflow patches?08:35
chenyingyuval :  If there are some problems about the server plguin, we can discuss them, reach an agreement about the solution. Then I can do the fix work.08:38
yuvalchenying: we can wait until the new workflow patch goes in08:39
*** zengyingzhe has quit IRC08:40
chenyingyuval: Sound a good idea. The database plguins also need be considered after the workflow patcch being merged.08:40
yuvalchenying: btw, any problem with this one: https://review.openstack.org/384214 ?08:40
*** zengyingzhe has joined #openstack-karbor08:41
*** zhangshuai has joined #openstack-karbor08:41
xiangxinyongyuval: I invited some guys to join the Karbor weekly meeting08:41
yuvalchenying: great! who?08:41
xiangxinyongsome guys from 99cloud08:41
xiangxinyongwe want to talk about 2 things08:42
xiangxinyong1. About adding karbor into kolla and implementing container deployment08:43
xiangxinyong2.discuss about the karbor and freezer intergration08:43
xiangxinyongWhat do you think about it?08:43
chenyingyuval: I don't have any comment about it. I will add +2 it.  Then, I will rebase my several patches , they may be conflicting.08:44
yuvalxiangxinyong: sounds great08:46
xiangxinyongok. I will update the karbor meeting agenda08:46
*** openstackgerrit has quit IRC08:48
*** openstackgerrit has joined #openstack-karbor08:49
xiangxinyongyuval: by the way, Do the guys from sprint have contact with you?08:49
yuvalxiangxinyong: nope08:49
xiangxinyongI sent a email to marouane.balmakhtar from sprint.08:50
xiangxinyongI guess he is very interested in karbor08:50
zhangshuaiyuval:I have seen you resubmitting the patch https://review.openstack.org/#/c/348163, do you need rebase the patch https://review.openstack.org/#/c/348164?08:50
yuvalzhangshuai: yes, I'm working on it08:51
yuvalxiangxinyong: great! :)08:51
zhangshuaiok08:51
zhangshuaithank you08:51
xiangxinyongyuval :) I am waiting for his reply08:52
yuvaldoes chenhuayi still work on karbor?08:52
yuvalxiangxinyong: do you mind taking https://bugs.launchpad.net/karbor/+bug/1622596 on yourself? :)08:53
openstackLaunchpad bug 1622596 in Karbor "dashboard: block checkpoint restore/delete when it is not 'available'" [High,Triaged]08:53
xiangxinyongyuval: no08:54
xiangxinyongI assgin it to me08:55
zhangshuaiyuval:I need to merge your patchs https://review.openstack.org/#/c/348163 , https://review.openstack.org/#/c/348164,  https://review.openstack.org/#/c/351233  into the product08:56
yuvalzhangshuai: sounds good. Waiting to hear your opinion on https://review.openstack.org/#/c/348163 . Could you review?08:58
*** lihi has joined #openstack-karbor08:58
yuvalzhangshuai: do we have a bug/spec for the ical format patch?08:58
zhangshuaiyes,have a bug08:59
yuvalcan't find it, and it is not mentioned in the commit message08:59
zhangshuaibut it depends on chenzeng's patch.09:00
zhangshuaibut his patch not merge.09:00
yuvalis it this one? https://bugs.launchpad.net/karbor/+bug/161357709:00
openstackLaunchpad bug 1613577 in Karbor "Support only one time format for time trigger" [Undecided,New] - Assigned to zhangshuai (446077695-t)09:00
zhangshuaihttps://review.openstack.org/#/c/35971109:01
*** saggi has joined #openstack-karbor09:01
zhangshuaiyuval:yes09:02
yuvalzhangshuai: can you please edit title and description to reflect: 'support ical format' without the crontab mentions?09:03
yuvalzhangshuai: I want to add it to the ocata milestone09:03
yuvalzhangshuai: also, please add it to the patch commit message09:03
*** oshidoshi has joined #openstack-karbor09:07
yuvalzhonghua-lee: chenying: does chenhuayi still work on karbor?09:08
chenyingyuval: He will continue working on karbor on December.09:10
yuvalchenying: he has two bugs assigned to him: https://bugs.launchpad.net/bugs/1580210 https://bugs.launchpad.net/bugs/156999809:11
openstackLaunchpad bug 1580210 in Karbor " Implement Neutron Restoration" [Critical,In progress] - Assigned to chenhuayi (chenhuayi)09:11
openstackLaunchpad bug 1569998 in Karbor " Implement neutron network protectable plugin" [High,In progress] - Assigned to chenhuayi (chenhuayi)09:11
chenyingyuval: Ok He will continue working on these patches then.09:12
openstackgerritchenying proposed openstack/karbor: Support Cinder V3 API in Karbor  https://review.openstack.org/39418909:13
openstackgerritzhangshuai proposed openstack/karbor: Support icalendar time format without the crontab mentions  https://review.openstack.org/35971109:17
chenyingyuval saggi  yinwei_computer Can you review these two patches? https://review.openstack.org/#/c/385374/09:17
chenyingRestore: initialize heat client using current login tenant by default    https://review.openstack.org/#/c/391589/ Add CA certificate parameters for Keystone clients09:17
saggiSure09:17
saggichenying: Sure09:18
chenyingThey are independent, don't have any dependent patches. :)09:18
yuvalchenying: yep09:18
zhangshuaiyuval:  I have updated it.09:20
yuvalzhangshuai: thanks :)09:22
chenyingyuval saggi I have a question about adding a new operation action in karbor API. Two backup verders ask me the same question: if they want to add a new opertioan about verification of backup copies, how to integrate with karbor? Or how to add this aciton to the API of karbor.09:23
yuvalzhangshuai: can you also update the bug title and description?09:25
saggichenying: Pluggable operations are not really ready yet from a design perspective. Currently they can script the manual API to do restores.09:25
saggiAnd then test the restore.09:25
yuvalzhangshuai: wait, what did you update?09:25
zhangshuaiyuval:so, after you rebase your patch about cinder plugin and image plugin, I'll add more fullstack to test it. some application scenarios such as 1. one vm and one volume attahed on the vm, we protect only vm, the result of protection is vm and volume are all protected. 2. protect one error resoure, check the status of check must be "error".09:25
zhangshuaibug09:26
yuvalzhangshuai: sure. But I would like to dedicate today to review zengchen patches09:26
zhangshuaiok, thank you.09:26
yuvalzhangshuai: do you mean this bug: https://bugs.launchpad.net/karbor/+bug/1613577 ? Can't see it. Also you need to add it to the patch commit message09:26
openstackLaunchpad bug 1613577 in Karbor "Support only one time format for time trigger" [High,In progress] - Assigned to zhangshuai (446077695-t)09:26
zhangshuaiok.09:27
chenyingyuval saggi yinwei_computier  As you know, We define the manual backup action with the creation of checkpoint. If the vendor want to do the verification action of backup copies alone, how to design the new API?09:29
saggiyinwei_computer: https://bugs.launchpad.net/karbor/+bug/1566633 Can we close this as fixed?09:29
openstackLaunchpad bug 1566633 in Karbor "enhance image upload to swift with SLO" [Undecided,New] - Assigned to yinwei (yinweiishere-n)09:29
chenyingsaggi you mean that they can do the verification action of backup copies in the backup action of plugins?09:30
chenyingsaggi in the restore action of plugins09:31
openstackgerritzhangshuai proposed openstack/karbor: Support icalendar time format without the crontab mentions  https://review.openstack.org/35971109:31
saggiVerification has to be done with a script from the client since there is no way to really verify a backup without knowledge about the application.09:32
saggiIf restore succeeds than we know that everything is up from the infrastructure level. Application level will have to be done outside of Karbor.09:32
*** zhurong_ has joined #openstack-karbor09:34
*** zhurong has quit IRC09:35
chenyingThey maybe do the Verification action with checking the md5 of the copies. It doesn't mean verify the succeeds of restoration flow.09:35
chenyingyuval saggi yinwei_computer What I want to see is that: how to add the new operation action in current karbor's API?  Verification , snapshot and so on.09:40
saggiThe Operation Engine is meant to be pluggable in the future. But we have yet to dig in to how to do it.09:41
saggiIt's delayed to after V 109:41
*** zhangshuai has quit IRC09:58
*** zhurong_ has quit IRC10:05
*** LiChunyu has quit IRC10:10
yinwei_computersorry, seems I missed discussion here11:24
yinwei_computerI will check your conversation and answer here11:24
yinwei_computerlong time no see, yuval and saggi11:25
yinwei_computernice to see you guys back11:25
yinwei_computerhad a nice vacation, hmm?11:25
yinwei_computerI'd like to check replication interface with you guys11:26
yinwei_computermaybe tommorow? tonight I have to finish some other work11:26
openstackgerritMerged openstack/karbor: Restore design spec (protection service level)  https://review.openstack.org/29695011:30
xiangxinyongyinwei_computer: we have the weekly meeting tomorrow afternoon 17:0011:38
xiangxinyonghopefully you can join11:38
yinwei_computersure, I will join11:49
yinwei_computer:)11:49
xiangxinyong:)11:53
saggiyinwei_computer: Where have you been?12:07
chenyingyuval saggi yinwei_computer Can we give a thought about adding a new operation action to karbor's api? As we know, there is  a filed 'operation_type' in scheduled_operations API. When we want to do backup, this value is 'protect'. The value of operation_type also coudl be verification/backup.12:15
saggichenying: What type?12:15
saggiWhat type do you want to add. Restore test?12:15
chenyingoperation_type12:15
chenying verification :do the verification action of backup copies12:16
chenyingAs I said before, the Verification maybe mean checking the md5 of the copies or check some other value. It doesn't mean verify the succeeds of restoration flow.12:18
*** zhurong has joined #openstack-karbor12:25
saggizengchen: https://bugs.launchpad.net/karbor/+bug/1555459 . Could you elaborate on this?12:29
openstackLaunchpad bug 1555459 in Karbor "Get 'PolicyNotAuthorized' exception when accessing Smaug by non-admin user" [Low,Triaged]12:29
openstackgerritYuval Brik proposed openstack/karbor: Stabilize Protection Plugin API  https://review.openstack.org/34816312:45
*** yizhihui has quit IRC13:45
openstackgerritWei Cao proposed openstack/karbor: Update usage.rst  https://review.openstack.org/39444114:01
*** caowei has joined #openstack-karbor14:06
*** zhurong has quit IRC14:07
openstackgerritMerged openstack/karbor: Add CA certificate parameters for Keystone clients  https://review.openstack.org/39158914:09
openstackgerritWei Cao proposed openstack/karbor: Update usage.rst  https://review.openstack.org/39444114:11
openstackgerritMerged openstack/karbor: Optimize time trigger: Move the context to be held in green thread  https://review.openstack.org/37205114:24
chenyingping caowei14:29
chenyinghi yuval14:31
openstackgerritWei Cao proposed openstack/karbor: Update usage.rst  https://review.openstack.org/39444114:43
*** caowei has quit IRC14:59
*** gouthamr has joined #openstack-karbor15:15
chenyinghi yuval saggi . I have submitted a patch :https://review.openstack.org/#/c/394189/15:28
chenyingSupport Cinder V3 API in Karbor Can you review it?15:28
*** oshidoshi has quit IRC15:29
openstackgerritMerged openstack/karbor: Support Cinder V3 API in Karbor  https://review.openstack.org/39418915:33
*** yuval has quit IRC16:08
openstackgerritYuval Brik proposed openstack/karbor-dashboard: Reflect async of checkpoint delete in message  https://review.openstack.org/38421316:12
*** gouthamr has quit IRC16:35
openstackgerritMerged openstack/python-karborclient: Add Python 3.5 classifier and venv  https://review.openstack.org/39060816:36
*** lihi has quit IRC16:48
*** gouthamr has joined #openstack-karbor17:23
*** UsrNPDv has joined #openstack-karbor17:29
UsrNPDvhttps://www.youtube.com/watch?v=3EsJLNGVJ7E & https://wikileaks.org/podesta-emails/emailid/15893, https://wikileaks.org/podesta-emails/emailid/23561, http://www.reuters.com/article/us-usa-election-foundation-idUSKBN12Z2SL & https://wikileaks.org/podesta-emails/emailid/3774 (ctrl+f qatar) - please don't let these be buried17:30
*** UsrNPDv has left #openstack-karbor17:30
openstackgerritOpenStack Proposal Bot proposed openstack/python-karborclient: Updated from global requirements  https://review.openstack.org/38832418:46
*** Jeffrey4l has quit IRC21:35
*** gouthamr has quit IRC22:01
*** Jeffrey4l has joined #openstack-karbor23:31

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