13:00:07 <ruijie> #startmeeting senlin
13:00:08 <openstack> Meeting started Tue Jan  2 13:00:07 2018 UTC and is due to finish in 60 minutes.  The chair is ruijie. Information about MeetBot at http://wiki.debian.org/MeetBot.
13:00:09 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
13:00:12 <openstack> The meeting name has been set to 'senlin'
13:01:41 <ruijie> \o
13:02:25 <chenyb4> hi
13:02:34 <Qiming> hi
13:02:46 <ruijie> evening :)
13:03:25 <ruijie> sorry I just changed my job.. didn't touch with you these days
13:05:19 <Qiming> ha
13:06:44 <chenyb4> I see you change stackalytics person message
13:07:05 <ruijie> ha?
13:07:32 <Qiming> more closer to openstack or more remote now?
13:08:01 <ruijie> more remote actually, it's about the log tracing
13:10:02 <ruijie> https://review.openstack.org/#/c/530707/
13:10:23 <ruijie> so, we are ready to publish these 2 kinds of policies?
13:13:09 <Qiming> I think it is fine.
13:13:37 <Qiming> actually, they *are* builtin policy types to be documented
13:15:29 <ruijie> okay
13:15:30 <ruijie> https://review.openstack.org/#/c/530433/
13:17:38 <ruijie> we have the same problem for the other tables since they are using StringField
13:19:02 <chenyb4> we have exist same problem for the profile has exist 'ascii' code. view this bug : https://bugs.launchpad.net/senlin/+bug/1710591
13:19:03 <openstack> Launchpad bug 1710591 in senlin "profile-create character error" [Undecided,In progress] - Assigned to chenyb4 (chenyb4)
13:19:36 <Qiming> yes ... it is gonna be a big issue if people always try non-English characters
13:20:07 <Qiming> I'm now sure how important it is
13:20:25 <Qiming> you guys have more experiences in the field
13:22:57 <ruijie> not sure the versionedobjects have a plan to support other languages
13:23:46 <Qiming> yup. that is the reason I was calling it a big issue
13:24:04 <Qiming> there are other oslo libs which may need changes
13:26:26 <ruijie> http://git.openstack.org/cgit/openstack/oslo.versionedobjects/tree/oslo_versionedobjects/fields.py#n258
13:26:48 <ruijie> is it caused by the six. string_types?
13:29:44 <Qiming> looking
13:30:44 <Qiming> I think tingting is looking at the right spot but the change can be improved
13:31:09 <Qiming> we could have use py2-py3 compatible code to get this done, I mean the six package
13:32:18 <Qiming> the way shown in the oslo_versionedobjects lib is the right direction
13:32:21 <ruijie> so we override the String() in versionedobjects too?
13:36:25 <Qiming> I believe we were doing it for other reasons
13:38:42 <Qiming> ah, the schema module is not related to oslo_versionedobjects , :)
13:39:41 <Qiming> we should probably do the conversion as exemplified by ovo
13:42:15 <ruijie> agree
13:42:40 <ruijie> return six.string_types(value)
13:47:37 <chenyb4> I have a question, senlin project not considering too much volume information, example: https://github.com/openstack/senlin/blob/master/senlin/profiles/os/nova/server.py#L1502 if the vm use bootable volume, the rebuild well faild
13:48:22 <Qiming> ya.
13:48:30 <Qiming> any idea on how to improve that?
13:50:13 <chenyb4> if vm exist volume attach and not server.image, so use block driver get bootable volume, get volume image message
13:51:25 <chenyb4> In my environment, I've done it
13:53:28 <Qiming> so image is not a mandatory field
13:53:42 <Qiming> other than that, any changes needed?
13:56:01 <chenyb4> nothing, only in vm use volume environment, add get image message
13:56:22 <Qiming> okay
13:56:36 <Qiming> would be happy to see any changes needed
13:58:15 <ruijie> 2 minutes left :)
13:59:13 <chenyb4> ruijie, event listener need more testing, right?
13:59:28 <ruijie> still under developing :(
13:59:36 <chenyb4> ok
13:59:40 <ruijie> thanks for joining :)
13:59:48 <ruijie> #endmeeting