17:00:02 #startmeeting nova notification 17:00:06 Meeting started Tue Oct 4 17:00:02 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:08 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 17:00:11 The meeting name has been set to 'nova_notification' 17:00:15 hi 17:01:57 hello 17:02:01 syjulian: hello 17:02:20 hello gibi_ 17:02:39 hi gibi_ 17:03:18 Let's get started 17:03:36 I didn't do too much in the last weeks 17:03:46 I saw that new patches are comming in for the transformation 17:03:51 thanks for that 17:03:56 I'm trying to review them 17:04:18 I was asked to create a burndown chart for the remaining work 17:04:29 similar to the api-ref work 17:04:39 I'm just finishing that up 17:04:52 that's different from the wiki? 17:04:53 it currently sits there http://vogon.no-ip.org/gibizer/vnt-burndown/ 17:05:13 it is like the wiki tables but it is generated automatically based on gerrit 17:05:20 nice 17:05:26 I will replace the wiki tables with this 17:05:45 I couldnt access the site behind co firewall 17:05:51 me too :( 17:05:55 o/ 17:05:55 mee too :-( 17:05:58 ohh 17:06:21 then I have to find a public provider you can access. This now servered from my raspbery pi :) 17:06:44 cool! 17:06:54 +1 17:06:55 :) 17:07:11 that was all I did 17:07:21 do you have anything to discuss? 17:07:45 hi gibi_ yes I do - having issues with functional test with assertion errors 17:08:14 I guess I can post this in paste.openstack.org and attach it to my commit https://review.openstack.org/#/c/375784 17:09:10 anyway assertionerror in self.assertEqual(2, len(fake_notifier.VERSIONED_NOTIFICATIONS)) where 2!=4 17:10:22 most probably your suspend-resume action pair trigger more notification than the resume 17:11:14 try to look at what other notifications you gett 17:12:26 and either assert those as well 17:12:29 or ignore them 17:12:30 so there should only be 2 not 4, is that right? 17:13:01 the test tells you that you got 4 notification but you expected only 2 17:13:03 its' probably the suspend notifications + resume notifications 17:13:53 syjulian: yeas 17:13:59 yeah - latest patch: https://review.openstack.org/#/c/375784/4/nova/tests/functional/notification_sample_tests/test_instance.py 17:14:40 line 357 I had to change to do post_server_action to set "suspend" status instead of deleting instance 17:15:19 but yeah, I'll look into it further, thanks guys 17:15:57 stewie925: I will look at it after the meeting and comment the review 17:16:13 thank you sir 17:16:23 any other thing to discuss? 17:19:21 two weeks from now I will be on the summit so there won't be IRC meeting 17:19:29 ohh not 17:19:35 4 + 14 =18 17:19:46 then there will be meeting on 18th of October 17:20:27 but you don't have to wait until if you get stuck with the transformation 17:20:32 just ping me on irc 17:20:50 gibi_: appreciate it 17:21:15 OK. Let's close this for today. See you around! 17:21:23 #endmeeting