17:00:01 <gibi> #startmeeting nova notification
17:00:01 <openstack> Meeting started Tue Dec  6 17:00:01 2016 UTC and is due to finish in 60 minutes.  The chair is gibi. Information about MeetBot at http://wiki.debian.org/MeetBot.
17:00:03 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
17:00:05 <openstack> The meeting name has been set to 'nova_notification'
17:00:08 <gibi> hi!
17:00:31 <sneti> hi gibi
17:01:24 <gibi> lets wait couple of minutes
17:01:31 <gibi> for the others to show up
17:02:13 <sneti> ok
17:02:59 <STEWIE925> hola
17:03:25 <gibi> STEWIE925: hi
17:03:34 <STEWIE925> :)
17:03:40 <STEWIE925> hi gibi
17:03:56 <gibi> let's get started
17:04:11 <gibi> no big news actually
17:04:37 <gibi> notification transformation progresses
17:05:10 <gibi> I saw that the searchlight related notification extension bp also has some code up for review but I haven't looked at it yet
17:06:07 <gibi> I think I have no other imediate info
17:06:11 <gibi> do you have questions?
17:06:15 <sneti> gibi, yes. myself and aunnam started working on additional-notification-fields-for-searchlight
17:07:03 <gibi> sneti: yes, I saw that there is a patch I just havent looked at it yet
17:08:03 <gibi> sneti: if you have questions regarding that just ask it now :)
17:08:15 <gibi> sneti: I will try to review the patch this week
17:08:33 <sneti> gibi, nothing for now. just started working on adding tags field. will submit a patch for that today
17:09:13 <gibi> sneti: good
17:09:30 <STEWIE925> hey gibi
17:09:33 <gibi> STEWIE925: I promised to look at resize_confirm
17:09:37 <gibi> STEWIE925: and I did
17:09:38 <STEWIE925> yes, thank you :)
17:09:42 <gibi> STEWIE925: but no solution yet
17:09:56 <STEWIE925> gibi: thanks much!
17:10:07 <STEWIE925> gibi: no problem :)
17:10:20 <gibi> it seems that the resize_confirm is sent in the test
17:10:39 <gibi> but in some cases the instance in the call and the instance in the test do not match
17:10:59 <STEWIE925> yeah the expected and actual
17:11:40 <gibi> if you replace the instance with mock.ANY then the test passes for resize_confirm action but if it is the instance from the test then it fails in 3 test cases but not in all of them
17:11:49 <gibi> I will continue looking tomorrow
17:12:06 <STEWIE925> thank you sir
17:12:38 <gibi> any other business?
17:12:49 <STEWIE925> I am good for now
17:13:23 <sneti> good for now
17:13:24 <gibi> then this will be a short meeting today
17:13:54 <gibi> btw STEWIE925 if you have time you could look at sneti's patch that extends the instance notification
17:14:15 <gibi> https://review.openstack.org/#/c/407128/
17:14:35 <gibi> but I will look at it as wll
17:14:36 <gibi> well
17:14:40 <gibi> let's close the meeting
17:14:42 <gibi> see you around
17:14:46 <gibi> #endmeeting