17:00:10 #startmeeting nova notification 17:00:10 Meeting started Tue Jul 5 17:00:10 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:12 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 17:00:14 The meeting name has been set to 'nova_notification' 17:00:15 hi! 17:00:16 o/ 17:00:22 * rlrossit has better internet today 17:00:58 I hope you had lunch not just internet 17:01:00 :) 17:01:07 Hi ! :) 17:01:21 macsz: o/ 17:01:43 Not many folks will join today 17:02:00 I think it's just us 17:02:02 we've already got more than last week :) 17:02:06 ok 17:02:08 then lets start 17:02:09 #topic outstanding reviews 17:02:11 :D 17:02:34 in the last minute rush we merged the must have commits of the transformation patches 17:03:09 That's great 17:03:16 I sent a mail to the ML asking for ffe for the other instance action transformation that is close 17:03:39 mriedem added an item to the ffe tracking etherpad as well 17:03:54 I was occupied with other stuff and failed to improve functional tests that were failing on reboot 17:04:03 Notification 17:04:28 no decision yet, but I hope for an ffe 17:04:47 macsz: then that will be done in ocata timeframe 17:05:33 wasn't the meeting for that this morning? 17:05:42 * rlrossit realizes morning is relative 17:05:58 there was, but they didn't talked about the clean up like tasks 17:06:09 I guess that will happen later today, or tomorrow 17:07:10 yeah, it is late afternoon here in the EU :) 17:07:34 beside the transformations 17:08:09 there is the json schema generation. I think not much happend in the last couple days with that 17:09:11 I have one bugfix regarding the hash generation of the payload object. I will poke oslo cores to get the fix merged in ovo 17:09:39 rlrossit: I saw you have a bug about object compatibility test 17:09:54 yeah, and I kept wondering why it wasn't failing with new notifications... 17:09:56 rlrossit: I left a comment in it today. The notification object tests are moved 17:10:07 but then I realized we don't register them, so they aren't picked up :) 17:10:14 exactly :) 17:11:02 I'm not 100% sure that we need these compatibility tests for notifications as we are not planning to backport the notification ovos 17:11:11 or at least we haven't planned to do that yet 17:11:48 yeah I think the bug is of less importance now that we don't register the notification objects 17:12:05 when we registered them, things would break on the first version bump, but now we don't have to worry anymore 17:12:31 agree 17:12:56 syjulian proposed a followup patch about the exception notification 17:13:08 to remove sensitive data properly from the exception text 17:13:32 https://review.openstack.org/#/c/336284/ 17:13:53 there are other followups on the wiki. mostly testing and doc generation related tasks 17:13:59 I will get to them later this week 17:14:46 that is all I had in my mind 17:14:55 any other review we need to talk about? 17:14:59 I have my follow-up patch as well 17:15:09 for moving the enums over to the compute module 17:15:14 right 17:15:15 you stuck a comment on that, addressing it now 17:15:22 thanks 17:15:28 It is a nice cleanup 17:15:49 I feel bad I didn't do it right in the first place 17:17:30 any other review? 17:18:23 Unfortunately no 17:18:44 #topic open discussion\ 17:18:53 #topic open discussion 17:19:39 rlrossit: the usual request, could you report on the thursday's nova meeting about the subteam? I think this week it is in US timezone 17:19:59 I should be able to 17:20:08 thanks a lot 17:20:13 it looks like my reminder for that on my calendar expired, I should probably put that back on there 17:20:32 Just a newbie question - is there any speed up for tox in general? Executing single functional test ( or rather creating env doing so) takes 10 minutes 17:20:55 for making venv's, there's no real speed up 17:20:55 I double checked it will be July 7th 2016 2100 UTC, #openstack-meeting 17:21:06 unless you have a local mirror you can download from 17:21:20 but if you're running lots of tests, more cores helps (if you do things in VMs) 17:21:21 I guess pip uses local cache already 17:21:45 you do the venv download once, and then after that you don't need to do it anymore 17:22:01 so the first run will always be the slowest, after that you will only run tests 17:22:10 (unless you accidentally have the --recreate flag on tox) 17:22:32 or delete the .tox dir with git clean xdff as I usually do 17:22:46 Ok I was unsure if tox does delete venv after tests by default 17:23:27 you can check what happens in your setup, the venv shoudl be under .tox/ dir in the git repo 17:24:31 but I agree rlrossit, more core helps a lot in test execution 17:24:54 I mean more cpu cores :) 17:25:18 you can also run individual tests with regex's 17:25:43 like tox -e py27 nova.tests.unit.objects is a frequent one for me 17:26:16 I am running single test, the venv and time to create is the only problem 17:26:46 Will check .tox next time I run tests 17:27:36 OK, anything else? 17:27:37 macsz: feel free to ask questions in nova too :) don't be scared! 17:27:43 +! 17:27:44 #openstack-nova that is 17:27:46 +1 17:27:59 Oh I am! But will be brave ! 17:28:02 :) 17:29:29 rlrossit: I'm +1 on the vm_state clean up now. thanks 17:29:39 woo :) 17:29:52 I feel we ran out of topic for today :) 17:30:09 can I close? 17:30:16 I'm done! 17:30:39 Done :) 17:30:53 then thank you for the meeting. See you around! 17:31:06 #endmeeting