| *** gouthamr has quit IRC | 01:25 | |
| *** liujiong has joined #openstack-karbor | 01:31 | |
| *** yushb has joined #openstack-karbor | 01:33 | |
| *** jiaopengju has joined #openstack-karbor | 01:42 | |
| jiaopengju | Hi chenying, please help to review this patch https://review.openstack.org/#/c/484319/. Not only fix the fulll listing issue, and fix the updating of chunk num in image_protection_plugin | 01:48 |
|---|---|---|
| chenying | jiaopengju sure | 01:51 |
| jiaopengju | chenying thanks | 01:51 |
| *** yamamoto has joined #openstack-karbor | 02:17 | |
| *** zengchen has quit IRC | 02:18 | |
| *** gouthamr has joined #openstack-karbor | 02:21 | |
| *** yamamoto has quit IRC | 02:38 | |
| *** yamamoto has joined #openstack-karbor | 02:48 | |
| *** yamamoto has quit IRC | 03:57 | |
| *** yamamoto has joined #openstack-karbor | 03:59 | |
| *** yamamoto has quit IRC | 04:04 | |
| *** yamamoto has joined #openstack-karbor | 04:05 | |
| *** gouthamr has quit IRC | 04:14 | |
| *** yamamoto has quit IRC | 04:14 | |
| *** yushb has quit IRC | 04:15 | |
| *** yamamoto has joined #openstack-karbor | 04:15 | |
| *** yamamoto has quit IRC | 04:21 | |
| *** yushb has joined #openstack-karbor | 04:24 | |
| *** yushb has quit IRC | 04:36 | |
| *** gouthamr has joined #openstack-karbor | 04:42 | |
| *** yamamoto has joined #openstack-karbor | 04:44 | |
| *** gouthamr has quit IRC | 05:07 | |
| *** yushb has joined #openstack-karbor | 05:50 | |
| openstackgerrit | Pengju Jiao proposed openstack/karbor master: Add option to enable or disable providers https://review.openstack.org/485046 | 05:55 |
| openstackgerrit | Pengju Jiao proposed openstack/karbor master: Add option to enable or disable providers https://review.openstack.org/485046 | 05:58 |
| *** yushb has quit IRC | 06:31 | |
| *** yushb has joined #openstack-karbor | 07:04 | |
| chenying | ping yuval | 07:21 |
| yuval | ping chenying | 07:43 |
| openstackgerrit | Yuval Brik proposed openstack/karbor master: Operation Engine: Trigger Multi Node Support (WIP) https://review.openstack.org/481106 | 07:43 |
| openstackgerrit | Merged openstack/karbor master: Increase s3 bank plugin https://review.openstack.org/482060 | 07:57 |
| chenying | hi yuval | 08:07 |
| chenying | Refactor dsvm gate prepare https://review.openstack.org/#/c/484383/ Amrith give a comment about this patch, it seem that he don' want to introduce a new cmd for other project. | 08:07 |
| yuval | chenying: saw that, I'll answer him | 08:07 |
| *** liujiong has quit IRC | 08:09 | |
| openstackgerrit | chenying proposed openstack/karbor master: Fix a error about network ports resources restore(don't merge) https://review.openstack.org/485084 | 08:09 |
| openstackgerrit | Yuval Brik proposed openstack/karbor master: Operation Engine: Trigger Multi Node Support (WIP) https://review.openstack.org/481106 | 08:10 |
| yuval | chenying: https://review.openstack.org/#/c/484383 | 08:13 |
| yuval | chenying: answered him | 08:13 |
| *** yushb has quit IRC | 08:24 | |
| *** yushb has joined #openstack-karbor | 08:39 | |
| *** openstackgerrit has quit IRC | 08:49 | |
| jiaopengju | ping yuval | 08:50 |
| yuval | hey jiaopengju | 08:50 |
| jiaopengju | hi, I'am fixing the bug of 'checking protectable plugin', do you think it is a good way to do this check when the client doing resource query? | 08:52 |
| jiaopengju | I try to do the check when ProtectablePlugin init, but failed. It is because the context that contains auth info may only have value when a request arrive | 08:53 |
| chenying | yuval jiaopengju Seam that all the protectable plugins are loaded here even the service is not enable. https://github.com/openstack/karbor/blob/master/karbor/services/protection/protectable_registry.py#L41-L44 | 08:54 |
| chenying | yuval jiaopengju: Only use protectable plugins to query resources, then raise a exception about can not find a endpoint which service is not enable. | 08:56 |
| yuval | jiaopengju: not sure I understand. The problem is when the protectable plugin class is instantized? | 09:02 |
| *** yushb has quit IRC | 09:05 | |
| *** yushb has joined #openstack-karbor | 09:05 | |
| *** yamamoto has quit IRC | 09:06 | |
| jiaopengju | yuval: Sorry about my bad english. I tried to do the protectable check in 'ProtectablePlugin init' (Create client in the init), but failed. It is because that if no request arrive, we can not get the auth info to create a client. So it seems that we can only do the check when a query request come. | 09:07 |
| chenying | yuval: jiaopengju want to add a check about initing the self._client in init method of protectable plugin. So that the plugins will raise a exception when the service is not enable. But the context is None. | 09:08 |
| jiaopengju | chenying: good explanation | 09:10 |
| *** yamamoto has joined #openstack-karbor | 09:10 | |
| *** yamamoto has quit IRC | 09:10 | |
| chenying | yuval We thought that service is not enable which protectable plugin will raise a e, but it doesn't. It also is loaded here. https://github.com/openstack/karbor/blob/master/karbor/services/protection/protectable_registry.py#L41-L44 | 09:11 |
| *** yamamoto has joined #openstack-karbor | 09:14 | |
| *** NatanBro has joined #openstack-karbor | 09:16 | |
| *** openstackgerrit has joined #openstack-karbor | 09:33 | |
| openstackgerrit | Pengju Jiao proposed openstack/karbor master: Add option to enable or disable providers https://review.openstack.org/485046 | 09:33 |
| *** yamamoto has quit IRC | 09:36 | |
| *** yamamoto has joined #openstack-karbor | 09:36 | |
| *** yushb has quit IRC | 09:49 | |
| *** yushb has joined #openstack-karbor | 09:54 | |
| *** yushb has quit IRC | 10:09 | |
| openstackgerrit | Yuval Brik proposed openstack/karbor master: Operation Engine: Trigger Multi Node Support (WIP) https://review.openstack.org/481106 | 10:54 |
| *** yamamoto has quit IRC | 10:55 | |
| *** yamamoto has joined #openstack-karbor | 11:40 | |
| *** saggi has quit IRC | 11:59 | |
| *** yamamoto has quit IRC | 12:03 | |
| *** saggi has joined #openstack-karbor | 12:05 | |
| *** yamamoto has joined #openstack-karbor | 12:21 | |
| *** dimak has quit IRC | 12:38 | |
| *** dimak has joined #openstack-karbor | 12:39 | |
| openstackgerrit | Akihiro Motoki proposed openstack/python-karborclient master: Fix wrong entry points which breaks OSC gate https://review.openstack.org/485193 | 12:43 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/karbor master: Updated from global requirements https://review.openstack.org/484552 | 12:50 |
| *** gouthamr has joined #openstack-karbor | 12:58 | |
| openstackgerrit | Pengju Jiao proposed openstack/karbor master: Use full listing in bank plugin https://review.openstack.org/484319 | 13:02 |
| openstackgerrit | chenying proposed openstack/karbor master: Add the spec for trove database proection plugin https://review.openstack.org/480943 | 13:04 |
| *** jiaopengju has quit IRC | 13:05 | |
| *** jiaopengju has joined #openstack-karbor | 13:05 | |
| *** dimak has quit IRC | 13:31 | |
| *** dimak has joined #openstack-karbor | 13:31 | |
| openstackgerrit | Merged openstack/python-karborclient master: Fix wrong entry points which breaks OSC gate https://review.openstack.org/485193 | 13:39 |
| *** lihi has quit IRC | 13:47 | |
| *** lihi has joined #openstack-karbor | 13:48 | |
| openstackgerrit | Pengju Jiao proposed openstack/karbor master: Add option to enable or disable providers https://review.openstack.org/485046 | 13:52 |
| *** jiaopengju has quit IRC | 14:02 | |
| chenying | hi yuval Could you help review the patches about spec operation log API? https://review.openstack.org/#/c/477075/ https://review.openstack.org/#/c/478459/ | 14:03 |
| *** yamamoto has quit IRC | 14:09 | |
| *** jiaopengju has joined #openstack-karbor | 14:16 | |
| *** jiaopengju has quit IRC | 14:18 | |
| *** NatanBro has quit IRC | 14:19 | |
| openstackgerrit | Merged openstack/karbor master: Add the spec for trove database proection plugin https://review.openstack.org/480943 | 14:29 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/karbor master: Updated from global requirements https://review.openstack.org/484552 | 14:43 |
| *** yamamoto has joined #openstack-karbor | 15:09 | |
| *** jiaopengju has joined #openstack-karbor | 15:12 | |
| *** jiaopengju has quit IRC | 15:16 | |
| *** yamamoto has quit IRC | 15:19 | |
| *** gouthamr has quit IRC | 21:49 | |
| *** gouthamr has joined #openstack-karbor | 22:19 | |
| *** yamamoto has joined #openstack-karbor | 23:04 | |
| *** yamamoto has quit IRC | 23:18 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!