*** gouthamr has quit IRC | 02:45 | |
*** gouthamr has joined #openstack-karbor | 02:46 | |
*** tonyb_ has joined #openstack-karbor | 03:06 | |
*** tonyb has quit IRC | 03:07 | |
*** openstackgerrit has joined #openstack-karbor | 03:11 | |
openstackgerrit | chenying proposed openstack/karbor-dashboard master: Fix checkpoint restore form https://review.openstack.org/434044 | 03:11 |
---|---|---|
openstackgerrit | zhangshuai proposed openstack/karbor-dashboard master: Fix checkpoint restore form https://review.openstack.org/434044 | 03:12 |
openstackgerrit | chenying proposed openstack/karbor master: Use common config in clients https://review.openstack.org/408400 | 03:17 |
*** tonyb_ is now known as tonyb | 03:21 | |
*** Jeffrey4l_ has quit IRC | 03:43 | |
*** Jeffrey4l_ has joined #openstack-karbor | 03:43 | |
*** zhangshuai has joined #openstack-karbor | 03:45 | |
*** gouthamr has quit IRC | 04:01 | |
openstackgerrit | xiangxinyong proposed openstack/karbor master: etc/karbor.conf URL error https://review.openstack.org/434069 | 05:02 |
*** zhangshuai has quit IRC | 06:29 | |
*** Jeffrey4l__ has joined #openstack-karbor | 06:30 | |
*** zhangshuai has joined #openstack-karbor | 06:32 | |
*** zhangshuai has quit IRC | 06:33 | |
*** Jeffrey4l_ has quit IRC | 06:34 | |
openstackgerrit | chenying proposed openstack/karbor master: Use common config in clients https://review.openstack.org/408400 | 07:12 |
openstackgerrit | xiangxinyong proposed openstack/python-karborclient master: Command `karbor help` info error https://review.openstack.org/434104 | 07:13 |
*** tommylikehu_ has joined #openstack-karbor | 07:30 | |
*** tommylikehu_ has quit IRC | 07:35 | |
*** yamamoto has quit IRC | 07:38 | |
openstackgerrit | xiangxinyong proposed openstack/python-karborclient master: Unified help information style https://review.openstack.org/434114 | 07:38 |
*** yuval has joined #openstack-karbor | 07:57 | |
openstackgerrit | Yuval Brik proposed openstack/karbor master: Add and use Karbor beaver mascot https://review.openstack.org/434137 | 08:30 |
chenying | hi yuval | 08:36 |
yuval | hey chenying | 08:36 |
chenying | yuval Several patches which you give some comments have been adressed. Can you have a look at them. | 08:38 |
yuval | chenying: yes, I know that | 08:38 |
chenying | yuval xinyong and I have submitted some small fixes about dashborad and client. | 08:41 |
chenying | The mascot of karbor is very cute. :) | 08:45 |
yuval | chenying: edisonxiang: https://review.openstack.org/434114 and https://review.openstack.org/434104 fail CI | 08:53 |
chenying | yuval 'Support to execute specific fullstack test' The patch depends on this one https://review.openstack.org/#/c/433408/. Fix a unit test. | 08:55 |
yuval | chenying: https://review.openstack.org/#/c/433428/ is a duplicate of https://review.openstack.org/#/c/433042/ | 08:55 |
yuval | chenying: I regret on merging the patch that is not doing lazy initialization for the client factory | 08:56 |
yuval | chenying: highly annoying | 08:56 |
yuval | chenying: 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 |
chenying | yuval Now the restore also can be created without password and username. | 08:59 |
chenying | mainly in single site use case. To restore the resources to local site. | 09:01 |
yuval | chenying: the UI conveys that poorly | 09:03 |
chenying | yuval: If user don't input username and password. It meant the it is a local site restoration. | 09:04 |
yuval | chenying: I understand. The UI conveys that poorly | 09:04 |
chenying | also include the auth url | 09:04 |
yuval | chenying: I understand. The UI conveys that poorly | 09:04 |
chenying | yuval 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 |
chenying | s/slect/select | 09:05 |
chenying | yuval If user slect the cross site restoration, then the url usename password input box are shown in this page. | 09:06 |
yuval | chenying: great, we should design that for next release | 09:07 |
chenying | yuval Sound good. | 09:07 |
chenying | yuval I will abandon this one. https://review.openstack.org/#/c/433428/ | 09:15 |
openstackgerrit | Merged openstack/karbor-dashboard master: Fix checkpoint restore form https://review.openstack.org/434044 | 09:20 |
openstackgerrit | Merged openstack/karbor master: Unit test may failed without client_factory.init() https://review.openstack.org/433408 | 09:21 |
*** yamamoto has joined #openstack-karbor | 09:23 | |
*** yamamoto_ has joined #openstack-karbor | 09:24 | |
*** yamamoto_ has quit IRC | 09:25 | |
*** yamamoto_ has joined #openstack-karbor | 09:26 | |
*** yamamoto_ has quit IRC | 09:26 | |
*** yamamoto has quit IRC | 09:28 | |
*** zhangshuai has joined #openstack-karbor | 09:28 | |
openstackgerrit | Merged openstack/karbor master: Support to execute specific fullstack test https://review.openstack.org/432161 | 09:35 |
openstackgerrit | Merged openstack/karbor master: etc/karbor.conf URL error https://review.openstack.org/434069 | 09:37 |
openstackgerrit | xiangxinyong proposed openstack/karbor-dashboard master: Update restore status in restore page https://review.openstack.org/433015 | 09:43 |
yuval | chenying: zhangshuai: here? | 09:53 |
chenying | yuval Yes | 09:53 |
openstackgerrit | Merged openstack/karbor-dashboard master: checkpoints: errorneous status_choices -> False https://review.openstack.org/433042 | 09:53 |
zhangshuai | yuval: yes | 09:53 |
openstackgerrit | Merged openstack/karbor-dashboard master: Update restore status in restore page https://review.openstack.org/433015 | 09:54 |
chenying | yuval But I will come off work soon. I will be online when I come home. | 09:54 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/karbor master: Updated from global requirements https://review.openstack.org/431871 | 10:19 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/karbor-dashboard master: Updated from global requirements https://review.openstack.org/431872 | 10:19 |
*** Jeffrey4l__ is now known as Jeffrey4l | 11:22 | |
*** tommylikehu_ has joined #openstack-karbor | 11:40 | |
yuval | chenying: did you take a look at: https://review.openstack.org/433124 ? | 11:41 |
*** yamamoto has joined #openstack-karbor | 11:53 | |
chenying | yuval The jenkins of this patch fail. | 11:55 |
*** yamamoto has quit IRC | 12:43 | |
*** tommylikehu_ has quit IRC | 12:47 | |
*** tommylikehu_ has joined #openstack-karbor | 12:49 | |
*** yamamoto has joined #openstack-karbor | 12:50 | |
*** yamamoto has quit IRC | 12:51 | |
*** tommylikehu_ has quit IRC | 12:51 | |
*** tommylikehu_ has joined #openstack-karbor | 12:51 | |
*** gouthamr has joined #openstack-karbor | 13:14 | |
openstackgerrit | Merged openstack/karbor master: Add and use Karbor beaver mascot https://review.openstack.org/434137 | 13:14 |
*** chenying has quit IRC | 13:55 | |
*** chenying has joined #openstack-karbor | 13:56 | |
chenying | yuval Do we need merge the mascot patch to ocata branch? | 13:59 |
yuval | chenying: yes | 13:59 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/karbor master: Updated from global requirements https://review.openstack.org/431871 | 14:05 |
*** zhonghua2 has joined #openstack-karbor | 15:10 | |
*** zhonghua has quit IRC | 15:12 | |
*** yuval has quit IRC | 16:01 | |
*** tommylikehu_ has quit IRC | 16:29 | |
*** edisonxiang has quit IRC | 16:52 | |
*** edisonxiang has joined #openstack-karbor | 16:52 | |
*** SuprDewd has joined #openstack-karbor | 19:19 | |
*** SuprDewd has quit IRC | 19:19 | |
*** SuprDewd has joined #openstack-karbor | 20:40 | |
*** zhonghua2 has quit IRC | 21:19 | |
*** zhonghua has joined #openstack-karbor | 21:20 | |
*** SuprDewd has left #openstack-karbor | 21:21 | |
*** Jeffrey4l has quit IRC | 21:34 | |
*** Jeffrey4l has joined #openstack-karbor | 21:35 | |
*** gouthamr has quit IRC | 22:22 | |
*** gouthamr has joined #openstack-karbor | 22:53 | |
*** tommylikehu_ has joined #openstack-karbor | 23:13 | |
*** tommylikehu_ has quit IRC | 23:20 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!