| *** smile-luobin has quit IRC | 02:36 | |
| *** smile-luobin has joined #openstack-karbor | 02:37 | |
| *** Jeffrey4l has quit IRC | 02:51 | |
| *** yamamoto_ has joined #openstack-karbor | 02:51 | |
| *** zhurong has joined #openstack-karbor | 03:27 | |
| openstackgerrit | Merged openstack/karbor-dashboard: Reinit operation logs panel in Karbor dashboard https://review.openstack.org/375789 | 03:48 |
|---|---|---|
| openstackgerrit | chenying proposed openstack/karbor: Add metadata parameter to checkpoint API https://review.openstack.org/352748 | 03:51 |
| *** zhurong has quit IRC | 03:58 | |
| *** zhurong has joined #openstack-karbor | 03:59 | |
| *** zhurong has quit IRC | 04:26 | |
| *** Jeffrey4l has joined #openstack-karbor | 04:42 | |
| *** Jeffrey4l has quit IRC | 04:43 | |
| *** Jeffrey4l has joined #openstack-karbor | 04:44 | |
| *** zhangshuai_ has joined #openstack-karbor | 05:00 | |
| *** zhurong has joined #openstack-karbor | 05:12 | |
| *** yamamoto_ has quit IRC | 05:30 | |
| *** zhurong has quit IRC | 05:31 | |
| *** zhurong has joined #openstack-karbor | 05:36 | |
| *** yamamoto has joined #openstack-karbor | 06:15 | |
| openstackgerrit | chenying proposed openstack/karbor: Add metadata parameter to checkpoint API https://review.openstack.org/352748 | 06:16 |
| openstackgerrit | chenying proposed openstack/karbor: Add metadata parameter to checkpoint API https://review.openstack.org/352748 | 06:22 |
| *** yamamoto has quit IRC | 06:51 | |
| *** yamamoto has joined #openstack-karbor | 06:54 | |
| *** oshidoshi has joined #openstack-karbor | 06:56 | |
| *** lihi has joined #openstack-karbor | 07:13 | |
| *** yuval has joined #openstack-karbor | 07:14 | |
| *** zhurong has quit IRC | 07:16 | |
| chenying_ | hi yuval About this patch :https://review.openstack.org/#/c/384062/ Fix the singleton pattern in operation engine Could you please review it? I note that zhengcheng have updated this patch. | 07:24 |
| chenying_ | I note that zengchen have updated this patch. | 07:25 |
| *** oshidoshi has quit IRC | 07:31 | |
| *** oshidoshi1 has joined #openstack-karbor | 07:31 | |
| yuval | chenying_: surwe | 07:49 |
| yuval | *sure | 07:49 |
| chenying_ | yuval: About your patch: | 07:55 |
| chenying_ | Stabilize Protection Plugin API. I think the fault and exception processing about protect/delete resource_flow sould be considered. | 07:55 |
| openstackgerrit | bailin.zhang proposed openstack/karbor: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/400082 | 07:56 |
| openstackgerrit | bailin.zhang proposed openstack/karbor: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/400082 | 07:57 |
| openstackgerrit | zengchen proposed openstack/karbor: Fix compute next time in green thread of trigger https://review.openstack.org/400083 | 07:57 |
| *** saggi has joined #openstack-karbor | 08:04 | |
| *** oshidoshi1 has quit IRC | 08:07 | |
| *** dimak has joined #openstack-karbor | 08:30 | |
| yuval | chenying_: I agree and plan to add that | 08:54 |
| chenying_ | yuval: Ok Thanks. | 09:07 |
| chenying_ | ping saggi | 09:07 |
| saggi | chenyingpong | 09:07 |
| saggi | chenying, pong | 09:07 |
| chenying_ | saggi ;) | 09:07 |
| chenying_ | saggi : ) Last irc meeting, we talk about the freezer integration. | 09:08 |
| saggi | chenying, yes but it's lower priority than other things that still don't work. | 09:09 |
| chenying_ | saggi: Can you list some other high priority works need to do in the etherpad of karbor. We can focus on these work. | 09:12 |
| saggi | https://bugs.launchpad.net/karbor | 09:13 |
| saggi | chenying ^^ | 09:14 |
| chenying_ | saggi: Some repoted bugs may have been solved. We may need do some verification works about these. | 09:19 |
| saggi | chenying, if it is solved mark it as solved. | 09:20 |
| saggi | But only if it's committed | 09:20 |
| chenying_ | saggi: I will check one by one. | 09:20 |
| *** oshidoshi has joined #openstack-karbor | 09:23 | |
| *** gouthamr has joined #openstack-karbor | 09:24 | |
| yuval | ping xiangxinyong | 09:34 |
| yuval | ping chenying_ | 09:34 |
| yuval | xiangxinyong: xiangxinyong: https://review.openstack.org/#/c/375789/ broke operation logs dashboard page | 09:34 |
| openstackgerrit | zengchen proposed openstack/karbor: Don't send canceled operation to executor https://review.openstack.org/400146 | 09:35 |
| chenying_ | yuval: I will check this patch. | 09:37 |
| xiangxinyong | yuval: hello | 09:37 |
| xiangxinyong | yuval: I will check it. Thanks | 09:38 |
| yuval | xiangxinyong: :) | 09:38 |
| *** dimak has quit IRC | 09:49 | |
| openstackgerrit | zengchen proposed openstack/karbor: Kill the green thread immediately if it is sleeping https://review.openstack.org/400158 | 10:06 |
| *** dimak has joined #openstack-karbor | 10:42 | |
| openstackgerrit | Merged openstack/karbor: Fix the singleton pattern in operation engine https://review.openstack.org/384062 | 10:52 |
| openstackgerrit | zhangshuai proposed openstack/python-karborclient: Add checkpoint extra info https://review.openstack.org/400196 | 11:17 |
| openstackgerrit | xiangxinyong proposed openstack/karbor-dashboard: Solve operation logs ui broken https://review.openstack.org/400205 | 11:39 |
| *** Jeffrey4l has quit IRC | 11:45 | |
| openstackgerrit | zhangshuai proposed openstack/karbor: Add extra_info when scheduled exec https://review.openstack.org/400222 | 12:17 |
| xiangxinyong | yuval: hi | 12:21 |
| *** Jeffrey4l has joined #openstack-karbor | 12:41 | |
| *** Jeffrey4l has quit IRC | 12:42 | |
| *** Jeffrey4l_ has joined #openstack-karbor | 12:42 | |
| *** Jeffrey4l_ has quit IRC | 12:42 | |
| *** Jeffrey4l_ has joined #openstack-karbor | 12:43 | |
| *** Jeffrey4l_ has quit IRC | 12:43 | |
| *** Jeffrey4l_ has joined #openstack-karbor | 12:43 | |
| *** Jeffrey4l_ has quit IRC | 12:44 | |
| *** Jeffrey4l has joined #openstack-karbor | 12:45 | |
| *** Jeffrey4l has quit IRC | 12:47 | |
| *** Jeffrey4l has joined #openstack-karbor | 12:49 | |
| *** Jeffrey4l has quit IRC | 12:52 | |
| *** Jeffrey4l has joined #openstack-karbor | 12:52 | |
| yuval | xiangxinyong: hey | 13:22 |
| *** Jeffrey4l has quit IRC | 13:26 | |
| chenying_ | yuval: xiangxinyong have submitted a patch to fix the operation_logs error. | 14:04 |
| yuval | chenying_: I saw that, waiting for tests to end | 14:05 |
| chenying_ | yuval: The jekins is very slow today. | 14:07 |
| yuval | chenying_: yeah, I noticed : | 14:07 |
| yuval | :\ | 14:07 |
| chenying_ | saggi yuval Could you review this spec? | 14:13 |
| chenying_ | The spec about adding extra_info field to the response of instances API https://review.openstack.org/#/c/387302/ | 14:13 |
| chenying_ | This spec extend the instances and plan API in karbor. | 14:13 |
| * saggi is looking | 14:13 | |
| chenying_ | I aslo rebase the patch about the extra-info of checkpoint API basing on yuval's patch. | 14:16 |
| *** yamamoto has quit IRC | 14:24 | |
| yuval | chenying_: I'll try to look at it tomorrow | 14:38 |
| *** zhangshuai_ has quit IRC | 15:00 | |
| *** zhangshuai_ has joined #openstack-karbor | 15:01 | |
| *** yamamoto has joined #openstack-karbor | 15:22 | |
| *** yamamoto has quit IRC | 15:38 | |
| openstackgerrit | Merged openstack/karbor: Fix compute next time in green thread of trigger https://review.openstack.org/400083 | 15:43 |
| chenying_ | hi saggi I have replied the comment about the spec. | 15:43 |
| saggi | chenying, I need to think about it. The main issue is that since this is information that needs to be part of the API it will need to have a schema. But now only protection plugins have a schema. | 15:48 |
| saggi | There needs to be some way for the UI to know how to display that information. | 15:48 |
| *** lihi has quit IRC | 15:50 | |
| saggi | There isn't really a way to show it in the tree. | 15:52 |
| chenying_ | saggi: Only add a node about the whole extra-info for every resoure. Or the extra info only can be see when the mouse pointer is on the resource. | 15:57 |
| saggi | How do you know how to display the field. You can't have the field name since this should only be lower case without spaces. | 15:58 |
| saggi | Maybe: {extra_info: [{name: "Availability Zone", value="AZ-001"}, {...}]} | 15:59 |
| saggi | Do we limit to only having scalar types as values? | 15:59 |
| *** yuval has quit IRC | 16:00 | |
| chenying_ | saggi: sound a good idea Yes I think limit to scalar types only should be string. | 16:00 |
| saggi | It also can't be used inside Karbor. It should only be to the user. Since you can't know what fields might exist. | 16:01 |
| saggi | We also don't want to support those fields through future versions | 16:01 |
| chenying_ | saggi: I will discuss the format with xiangxinyong, find a easy way for UI to display. Yes the extra_info about resoure only for tenant, not for karbor sercice. | 16:02 |
| saggi | chenying, good. Since it's information for the user it' | 16:04 |
| saggi | s important for it to be something that works in the UI | 16:04 |
| chenying_ | saggi: The limit about extra-info fields should be write to spec. The fields about resource extra-info can not be parsed and used in karbor service. | 16:06 |
| saggi | Sure, update the spec after you talk to xiangxinyong | 16:07 |
| chenying_ | saggi: OK Thanks. | 16:07 |
| *** oshidoshi has quit IRC | 16:18 | |
| *** yamamoto has joined #openstack-karbor | 16:18 | |
| *** yamamoto has quit IRC | 16:19 | |
| *** yamamoto has joined #openstack-karbor | 16:29 | |
| *** yamamoto has quit IRC | 16:30 | |
| *** yamamoto has joined #openstack-karbor | 16:52 | |
| *** saggi has quit IRC | 17:01 | |
| *** yamamoto has quit IRC | 17:01 | |
| *** dimak has quit IRC | 17:02 | |
| *** dimak has joined #openstack-karbor | 18:11 | |
| *** gouthamr has quit IRC | 19:07 | |
| *** dimak has quit IRC | 20:24 | |
| *** openstack has joined #openstack-karbor | 23:45 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!