Wednesday, 2017-02-15

*** gouthamr has quit IRC02:45
*** gouthamr has joined #openstack-karbor02:46
*** tonyb_ has joined #openstack-karbor03:06
*** tonyb has quit IRC03:07
*** openstackgerrit has joined #openstack-karbor03:11
openstackgerritchenying proposed openstack/karbor-dashboard master: Fix checkpoint restore form  https://review.openstack.org/43404403:11
openstackgerritzhangshuai proposed openstack/karbor-dashboard master: Fix checkpoint restore form  https://review.openstack.org/43404403:12
openstackgerritchenying proposed openstack/karbor master: Use common config in clients  https://review.openstack.org/40840003:17
*** tonyb_ is now known as tonyb03:21
*** Jeffrey4l_ has quit IRC03:43
*** Jeffrey4l_ has joined #openstack-karbor03:43
*** zhangshuai has joined #openstack-karbor03:45
*** gouthamr has quit IRC04:01
openstackgerritxiangxinyong proposed openstack/karbor master: etc/karbor.conf URL error  https://review.openstack.org/43406905:02
*** zhangshuai has quit IRC06:29
*** Jeffrey4l__ has joined #openstack-karbor06:30
*** zhangshuai has joined #openstack-karbor06:32
*** zhangshuai has quit IRC06:33
*** Jeffrey4l_ has quit IRC06:34
openstackgerritchenying proposed openstack/karbor master: Use common config in clients  https://review.openstack.org/40840007:12
openstackgerritxiangxinyong proposed openstack/python-karborclient master: Command `karbor help` info error  https://review.openstack.org/43410407:13
*** tommylikehu_ has joined #openstack-karbor07:30
*** tommylikehu_ has quit IRC07:35
*** yamamoto has quit IRC07:38
openstackgerritxiangxinyong proposed openstack/python-karborclient master: Unified help information style  https://review.openstack.org/43411407:38
*** yuval has joined #openstack-karbor07:57
openstackgerritYuval Brik proposed openstack/karbor master: Add and use Karbor beaver mascot  https://review.openstack.org/43413708:30
chenyinghi yuval08:36
yuvalhey chenying08:36
chenyingyuval Several patches which you give some comments have been adressed. Can you have a look at them.08:38
yuvalchenying: yes, I know that08:38
chenyingyuval  xinyong and I have submitted some small fixes about dashborad and client.08:41
chenyingThe mascot of karbor is very cute. :)08:45
yuvalchenying: edisonxiang: https://review.openstack.org/434114 and https://review.openstack.org/434104 fail CI08:53
chenyingyuval  'Support to execute specific fullstack test' The patch depends on this one https://review.openstack.org/#/c/433408/. Fix a unit test.08:55
yuvalchenying: https://review.openstack.org/#/c/433428/ is a duplicate of https://review.openstack.org/#/c/433042/08:55
yuvalchenying: I regret on merging the patch that is not doing lazy initialization for the client factory08:56
yuvalchenying: highly annoying08:56
yuvalchenying: also, I don't understand this patch: https://review.openstack.org/#/c/434044/2/karbor_dashboard/checkpoints/forms.py  why not mark the password and username as required?08:58
chenyingyuval Now the restore also can be created without  password and username.08:59
chenyingmainly in single site use case. To restore the resources to local site.09:01
yuvalchenying: the UI conveys that poorly09:03
chenyingyuval: If user don't input username and password. It meant the it is a local site restoration.09:04
yuvalchenying: I understand. The UI conveys that poorly09:04
chenyingalso include the auth url09:04
yuvalchenying: I understand. The UI conveys that poorly09:04
chenyingyuval Maybe we can consider add a drop-down box. It could slect local site restore or cross site restore in the restore page.09:05
chenyings/slect/select09:05
chenyingyuval If user slect the cross site restoration, then the url usename password input box are shown in this page.09:06
yuvalchenying: great, we should design that for next release09:07
chenyingyuval Sound good.09:07
chenyingyuval I will abandon this one. https://review.openstack.org/#/c/433428/09:15
openstackgerritMerged openstack/karbor-dashboard master: Fix checkpoint restore form  https://review.openstack.org/43404409:20
openstackgerritMerged openstack/karbor master: Unit test may failed without client_factory.init()  https://review.openstack.org/43340809:21
*** yamamoto has joined #openstack-karbor09:23
*** yamamoto_ has joined #openstack-karbor09:24
*** yamamoto_ has quit IRC09:25
*** yamamoto_ has joined #openstack-karbor09:26
*** yamamoto_ has quit IRC09:26
*** yamamoto has quit IRC09:28
*** zhangshuai has joined #openstack-karbor09:28
openstackgerritMerged openstack/karbor master: Support to execute specific fullstack test  https://review.openstack.org/43216109:35
openstackgerritMerged openstack/karbor master: etc/karbor.conf URL error  https://review.openstack.org/43406909:37
openstackgerritxiangxinyong proposed openstack/karbor-dashboard master: Update restore status in restore page  https://review.openstack.org/43301509:43
yuvalchenying: zhangshuai: here?09:53
chenyingyuval Yes09:53
openstackgerritMerged openstack/karbor-dashboard master: checkpoints: errorneous status_choices -> False  https://review.openstack.org/43304209:53
zhangshuaiyuval: yes09:53
openstackgerritMerged openstack/karbor-dashboard master: Update restore status in restore page  https://review.openstack.org/43301509:54
chenyingyuval But I will come off work soon. I will be online when I come home.09:54
openstackgerritOpenStack Proposal Bot proposed openstack/karbor master: Updated from global requirements  https://review.openstack.org/43187110:19
openstackgerritOpenStack Proposal Bot proposed openstack/karbor-dashboard master: Updated from global requirements  https://review.openstack.org/43187210:19
*** Jeffrey4l__ is now known as Jeffrey4l11:22
*** tommylikehu_ has joined #openstack-karbor11:40
yuvalchenying: did you take a look at: https://review.openstack.org/433124 ?11:41
*** yamamoto has joined #openstack-karbor11:53
chenyingyuval The jenkins of this patch fail.11:55
*** yamamoto has quit IRC12:43
*** tommylikehu_ has quit IRC12:47
*** tommylikehu_ has joined #openstack-karbor12:49
*** yamamoto has joined #openstack-karbor12:50
*** yamamoto has quit IRC12:51
*** tommylikehu_ has quit IRC12:51
*** tommylikehu_ has joined #openstack-karbor12:51
*** gouthamr has joined #openstack-karbor13:14
openstackgerritMerged openstack/karbor master: Add and use Karbor beaver mascot  https://review.openstack.org/43413713:14
*** chenying has quit IRC13:55
*** chenying has joined #openstack-karbor13:56
chenyingyuval Do we need merge the mascot patch to ocata branch?13:59
yuvalchenying: yes13:59
openstackgerritOpenStack Proposal Bot proposed openstack/karbor master: Updated from global requirements  https://review.openstack.org/43187114:05
*** zhonghua2 has joined #openstack-karbor15:10
*** zhonghua has quit IRC15:12
*** yuval has quit IRC16:01
*** tommylikehu_ has quit IRC16:29
*** edisonxiang has quit IRC16:52
*** edisonxiang has joined #openstack-karbor16:52
*** SuprDewd has joined #openstack-karbor19:19
*** SuprDewd has quit IRC19:19
*** SuprDewd has joined #openstack-karbor20:40
*** zhonghua2 has quit IRC21:19
*** zhonghua has joined #openstack-karbor21:20
*** SuprDewd has left #openstack-karbor21:21
*** Jeffrey4l has quit IRC21:34
*** Jeffrey4l has joined #openstack-karbor21:35
*** gouthamr has quit IRC22:22
*** gouthamr has joined #openstack-karbor22:53
*** tommylikehu_ has joined #openstack-karbor23:13
*** tommylikehu_ has quit IRC23:20

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