Monday, 2017-02-06

*** Jeffrey4l has joined #openstack-mogan00:02
*** l4yerffeJ has joined #openstack-mogan00:32
*** litao has joined #openstack-mogan00:43
*** wanghao has joined #openstack-mogan00:43
*** wanghao_ has joined #openstack-mogan01:06
*** wanghao has quit IRC01:09
*** kevinz has joined #openstack-mogan01:35
*** zhangjl has joined #openstack-mogan01:36
*** l4yerffeJ_ has joined #openstack-mogan01:50
*** l4yerffeJ_ has quit IRC01:50
*** l4yerffeJ_ has joined #openstack-mogan01:50
*** Jeffrey4l_ has joined #openstack-mogan01:51
zhenguomorning mogan!01:52
*** l4yerffeJ has quit IRC01:52
*** Jeffrey4l has quit IRC01:53
*** fredli has joined #openstack-mogan02:33
openstackgerritwanghao proposed openstack/mogan master: Support quota in mogan(part two)  https://review.openstack.org/42855702:59
openstackgerritwanghao proposed openstack/mogan master: Quota support in Mogan(part 1)  https://review.openstack.org/41732603:04
openstackgerritZhenguo Niu proposed openstack/mogan master: Specify schema format checker to enable format validation  https://review.openstack.org/42946203:10
* zhenguo brb03:11
openstackgerritTao Li proposed openstack/mogan master: Support multi-instance creating (part 1)  https://review.openstack.org/42106403:34
*** l4yerffeJ_ has quit IRC03:37
*** l4yerffeJ_ has joined #openstack-mogan03:37
zhenguozhangjl: just left some comments on instance lock spec, thanks for the update!03:43
zhangjlzhenguo:thanks03:43
zhenguozhangjl: np03:44
zhenguozhangjl: seems you introduced a new contributor Jolie, lol03:46
zhangjlyes, she is my colleague03:47
zhenguozhangjl: hah, thanks03:47
*** ChanServ sets mode: +o zhenguo03:52
* zhenguo away for lunch03:59
*** l4yerffeJ_ has quit IRC04:06
*** l4yerffeJ_ has joined #openstack-mogan04:06
openstackgerritMerged openstack/mogan master: Specify schema format checker to enable format validation  https://review.openstack.org/42946204:33
*** fredli has quit IRC04:45
*** l4yerffeJ_ has quit IRC05:14
*** wanghao has joined #openstack-mogan05:39
*** wanghao_ has quit IRC05:42
openstackgerritTao Li proposed openstack/mogan master: Support multi-instance creating (part 1)  https://review.openstack.org/42106406:18
*** zhangjl has quit IRC06:41
*** zhangjl has joined #openstack-mogan06:47
openstackgerritzhangjl proposed openstack/mogan-specs master: Adds specification for lock instances  https://review.openstack.org/42048607:10
openstackgerritwanghao proposed openstack/mogan master: Support quota in mogan(part two)  https://review.openstack.org/42855707:22
openstackgerritzhangjl proposed openstack/mogan-specs master: Adds specification for lock instances  https://review.openstack.org/42048608:04
zhenguoliusheng: do you know why we accept specifying uuid when creating instance type?08:47
liushengzhenguo: actually i don't know :(08:49
zhenguoliusheng: it's a bit confused08:49
liushengzhenguo: maybe just a mistake when add the api ?08:50
zhenguoliusheng: I would like to use soft_delete for all tables, then need to add id field to instance_type table08:50
zhenguoliusheng: but find there's a test for uuid duplicate :(08:50
zhenguoliusheng: maybe we can just remove that support, as it's seems useless08:50
liushengzhenguo: why soft_delete need 'id' field ?08:51
zhenguoliusheng: as it just set id to deleted08:51
liushengzhenguo: oh, I remember it08:52
zhenguoliusheng: I will just delete the uuid duplicate test case, and need to remove that from API later08:53
liushengzhenguo: you mean don't support creating type with uuid specified ?08:58
zhenguoliusheng: yes08:58
liushengzhenguo: ok, agree08:58
zhenguoliusheng: thanks08:58
liushengzhenguo: seems Nova don't add deleted field in new comming resources08:58
liushengzhenguo: just asked zhenyu08:58
zhenguoliusheng: how to handle that?08:59
liushengzhenguo: they may think it is unnecessary to support soft_delete09:00
liushengzhenguo: but for old resources, they don't have any plan to remove it09:00
liushengzhenguo: additionally, do we need to add shadow tables like Nova ?09:01
zhenguoliusheng: not sure, I don't find it's useful, but if Nova plans to get rid of it, we should not add it back09:01
zhenguoliusheng: so for quotas table, we should not add deleted09:02
zhenguoKevin_Zheng: are you around?09:03
liushengzhenguo: seems Nova guys also don't have a clear plan for this09:03
Kevin_Zhengyeah, whats up?09:03
zhenguoKevin_Zheng: about soft_delete, does nova plan to get rid of it for new resources?09:03
Kevin_Zhengno09:04
zhenguofor all resouces?09:04
Kevin_Zhenghttps://specs.openstack.org/openstack/nova-specs/specs/mitaka/implemented/no-more-soft-delte.html09:05
zhenguocan't open it09:06
Kevin_Zhengyeah, im checking, I typed it :)09:06
Kevin_Zhenghttps://specs.openstack.org/openstack/nova-specs/specs/mitaka/implemented/no-more-soft-delete.html09:07
zhenguoKevin_Zheng: oh, thanks, so we don't need to add deleted for new resource anymore09:10
Kevin_Zhengyou are welcome09:10
zhenguoliusheng: oh, so I will abandon my patch to add soft_delete for all tables,09:11
liushengzhenguo: lol09:11
liushengzhenguo: we completely follow Nova09:11
zhenguowanghao: for quotas new tables, please remove soft_delete related things09:11
zhenguoliusheng: hah09:12
zhenguoliusheng: but we have already support soft delete for instances09:16
liushengzhenguo: just leave it ?09:17
zhenguoliusheng: not sure, does deleted instances used for usage report?09:17
zhenguoliusheng: mind to have a look at this https://review.openstack.org/#/c/420486/8/specs/ocata/approved/lock-instance-api.rst if you got time? I think it's almost good, need to approve it ASAP, so they can start coding09:20
liushengzhenguo: don't think so, usage report may be addressed by other component, deleted instances seens mainly for audit09:20
liushengzhenguo: ok09:20
zhenguoliusheng: ok, if it's useless, we can just remove it09:21
liushengzhenguo: many projects don't support soft_delete09:22
zhenguoliusheng: yes, ironic is one of them09:22
liushengzhenguo: telemetry is the 2ed, lol09:23
zhenguoliusheng: lol09:23
zhenguoliusheng: I find an interesting nova cmd, ssh. which support for connecting to an instance09:27
liushengzhenguo: it don't have dependency ?09:29
zhenguoliusheng: not sure.09:29
liushengzhenguo: looks like it is only for testing09:32
zhenguoliusheng: seems yes09:33
*** wanghao has quit IRC09:33
*** kevinz has quit IRC10:16
*** zhangjl has quit IRC11:22
openstackgerritZhenguo Niu proposed openstack/mogan master: Add portgroups support  https://review.openstack.org/42967012:06
*** kevinz has joined #openstack-mogan12:27
*** kevinz has quit IRC13:03
*** kevinz has joined #openstack-mogan13:04
*** Jeffrey4l_ has quit IRC13:32
*** Jeffrey4l_ has joined #openstack-mogan13:34
*** Jeffrey4l_ has quit IRC13:58
*** kevinz has quit IRC15:00
openstackgerritZhenguo Niu proposed openstack/mogan master: Add portgroups support  https://review.openstack.org/42967016:48
*** lin_yang has joined #openstack-mogan18:25
*** Jeffrey4l has joined #openstack-mogan20:44
*** Jeffrey4l has quit IRC20:49
*** Jeffrey4l has joined #openstack-mogan20:51
*** Jeffrey4l has quit IRC20:55
*** Jeffrey4l has joined #openstack-mogan20:56
*** Jeffrey4l has quit IRC20:57
*** Jeffrey4l has joined #openstack-mogan20:58
*** Jeffrey4l has quit IRC21:03
*** Jeffrey4l has joined #openstack-mogan21:06
*** Jeffrey4l has quit IRC21:11
*** Jeffrey4l has joined #openstack-mogan21:18
*** Jeffrey4l has quit IRC21:28
*** Jeffrey4l has joined #openstack-mogan21:36
*** shaohe_feng_ has joined #openstack-mogan23:33
*** shaohe_feng has quit IRC23:34
*** shaohe_feng_ is now known as shaohe_feng23:34

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