04:03:23 <rkmrHonjo> #startmeeting masakari
04:03:24 <openstack> Meeting started Tue Jun  6 04:03:23 2017 UTC and is due to finish in 60 minutes.  The chair is rkmrHonjo. Information about MeetBot at http://wiki.debian.org/MeetBot.
04:03:25 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
04:03:27 <openstack> The meeting name has been set to 'masakari'
04:04:16 <rkmrHonjo> #topic critical bugs
04:04:34 <rkmrHonjo> Do you have any bugs for discussion?
04:05:58 <tpatil> Looks like patches are up for review for most of the bugs
04:06:06 <tpatil> I will review these patches in this week
04:06:27 <rkmrHonjo> tpatil: Great.
04:06:36 <sagara> thanks
04:08:32 <rkmrHonjo> Dinesh_Bhor: Do you have any updates about '"ERROR" instances will be unexpectedly changed to "ACTIVE"' and '"resized" instance that's state was "stopped" before resizing, it will be "active"' ?
04:09:25 <Dinesh_Bhor> rkmrHonjo: This patch solves those issue: https://review.openstack.org/#/c/469029/
04:11:10 <rkmrHonjo> Dinesh_Bhor: Thanks. Sampath promised to create doc for that patch last week. But It is probably not completed.
04:12:41 <abhishekk> rkmrHonjo: thank you for update
04:12:59 <rkmrHonjo> np
04:13:12 <rkmrHonjo> Can I go to next bug?
04:13:21 <Dinesh_Bhor> yes
04:13:27 <rkmrHonjo> thanks.
04:13:29 <rkmrHonjo> Fix os_privileged_user_auth_url
04:13:38 <rkmrHonjo> #link https://review.openstack.org/#/c/465800/
04:14:20 <rkmrHonjo> devstack plugin will be failed if this patch is not merged. I want review comments.
04:15:22 <tpatil> PROTOCOL://HOST/identity_admin, is this not supported by Keystone anymore?
04:16:15 <rkmrHonjo> tpatil: This patch is related to following merged patch:
04:16:20 <rkmrHonjo> Try to remove /identity_admin https://review.openstack.org/#/c/458226/
04:16:54 <abhishekk> rkmrHonjo: ok
04:17:20 <rkmrHonjo> "Try to remove /identity_admin" removed "/identity_admin" paths from devstack. So we should also remove it from masakari plugin.
04:17:34 <sagara> I understood
04:18:21 <rkmrHonjo> thanks. Could you review this patch?
04:18:27 <tpatil> I got it. I think it's better to mention why we are changing from /identity_admin to /identity in the commit message
04:19:07 <sagara> I posted W+1
04:19:34 <sagara> tpatil: sorry..
04:20:14 <tpatil> Sagara : Np, I have understood the reason behind it
04:21:56 <rkmrHonjo> tpatil,sagara: I had to point it out at first. sorry. But I don't think we shouldn't revert and re-commit.
04:22:17 <rkmrHonjo> s/I don't think/I think/g
04:23:38 <tpatil> there are couple of unit tests where we are using /identity_admin, we will need to fix all these cases
04:24:08 <Dinesh_Bhor> #link https://github.com/openstack/masakari/blob/master/masakari/tests/unit/compute/test_nova.py#L43
04:25:08 <tpatil> The above one is just one example, there are also other places where /identity_admin is used
04:25:29 <sagara> Do we revert that?
04:25:45 <tpatil> Let's do it
04:26:06 <rkmrHonjo> tpatil: Thank you for pointing out.
04:26:48 <tpatil> Dinesh pointed out this problem. THank you, Dinesh
04:27:08 <Dinesh_Bhor> np
04:27:19 <rkmrHonjo> I posted revert commit.
04:27:26 <rkmrHonjo> #link https://review.openstack.org/#/c/471199/
04:27:34 <sagara> Dinesh_Bhor: thank you
04:29:15 <rkmrHonjo> I'll grep "/identity_admin" and fix all. And I'll write the reason on commit log.
04:29:32 <tpatil> Thank you
04:30:36 <rkmrHonjo> sagara: Could you merge revert patch? or, Can I merge it?
04:31:40 <sagara> rkmrHonjo: Please merge it
04:33:01 <rkmrHonjo> OK, I merge after completing jenkins jobs. (Oh, revert patches are duplicated...I'll abandon mine.)
04:34:14 <rkmrHonjo> ok, can we go to next item?
04:34:52 <rkmrHonjo> Do you have any other bugs for discussion?
04:35:18 <tpatil> No
04:36:01 <abhishekk> rkmrHonjo: yes
04:36:26 <abhishekk> rkmrHonjo: yes we can move to next item
04:36:47 <rkmrHonjo> abhishekk, tpatil: thanks.
04:36:55 <rkmrHonjo> #topic Discussion points
04:37:39 <rkmrHonjo> There are three items in this topic. Do you have any updates?
04:37:51 <abhishekk> need review on Recovery method customization
04:38:29 <rkmrHonjo> #link https://review.openstack.org/#/c/458023/
04:39:43 <abhishekk> We want to discuss about db purge thing
04:40:33 <rkmrHonjo> abhishekk: what is the point for discussion?
04:40:59 <tpatil> Presently, there is no way to delete notifications
04:41:27 <tpatil> if there are large numberof notifications in the db, it's going to slow down db search
04:42:04 <tpatil> it's better to support db purge of notifcations which are in finished, ignored and failed status
04:42:47 <tpatil> db purge can be extended to other db tables which are in deleted state
04:44:21 <rkmrHonjo> tpatil: Thank you for explaining. I agree your opinion.
04:44:51 <tpatil> We will add a lite spec for this feature
04:45:59 <sagara> tpatil: Me, too. Do you think what client is good to implement db purge? like nova-manage, should we create new cli tool?
04:46:29 <sagara> s/Me, too./I agree too/
04:46:30 <abhishekk> sagara: yes
04:47:02 <tpatil> we already have masakari-manage to sync db, so we can add this purge command to it
04:47:36 <abhishekk> it will be like masakari-manage db purge
04:47:45 <sagara> tpatil: Thanks, I see.
04:49:11 <rkmrHonjo> abhishekk: No objection.
04:50:58 <rkmrHonjo> OK, do you have any other discussion points?
04:51:24 <abhishekk> rkmrHonjo: no
04:52:24 <rkmrHonjo> #topic rpm packaging
04:52:41 <rkmrHonjo> All items were merged now. I'll write "Done!" for https://review.openstack.org/#/c/460239/ after that.
04:53:11 <rkmrHonjo> I'll write it on wiki page.
04:54:40 <rkmrHonjo> ah...Do you want to talk about this topic?
04:55:26 <rkmrHonjo> ok, let's go to AOB.
04:55:33 <rkmrHonjo> #topic AOB
04:56:46 <rkmrHonjo> Do you have topics?
04:56:55 <abhishekk> rkmrHonjo: no
04:58:02 <sagara> rkmrHonjo: no
04:58:14 <rkmrHonjo> OK, time is up.
04:58:14 <rkmrHonjo> Please talk at ML or #openstack-masakari if you have topics.
04:58:35 <abhishekk> rkmrHonjo: sure
04:58:35 <rkmrHonjo> thank you all.
04:58:45 <abhishekk> Thank you.
04:58:58 <sagara> thank you.
04:58:59 <rkmrHonjo> #endmeeting