17:00:02 <rlrossit> #startmeeting nova notification
17:00:03 <openstack> Meeting started Tue May 24 17:00:02 2016 UTC and is due to finish in 60 minutes.  The chair is rlrossit. Information about MeetBot at http://wiki.debian.org/MeetBot.
17:00:04 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
17:00:07 <openstack> The meeting name has been set to 'nova_notification'
17:00:14 * rlrossit is new to this
17:00:19 <rlrossit> hey everyone!
17:00:26 <syjulian> o/
17:00:48 <rlrossit> uh oh might be a quick one today syjulian
17:01:00 <rlrossit> gibi's out, so you guys are stuck with me
17:01:00 <syjulian> lol
17:01:06 <syjulian> alright
17:01:11 <rlrossit> #topic specs
17:01:17 <rlrossit> looks like both specs are merged now
17:01:23 <rlrossit> https://review.openstack.org/#/c/311194/
17:01:28 <rlrossit> https://review.openstack.org/#/c/286675/
17:01:36 <rlrossit> nova one went in yesterday
17:01:43 <rlrossit> any other spec-related things?
17:02:05 * rlrossit hears crickets
17:02:13 <rlrossit> #topic open reviews
17:02:13 <syjulian> the implementation of the json schema one
17:02:30 <rlrossit> https://review.openstack.org/#/q/topic:bp/json-schema-for-versioned-object
17:02:43 <rlrossit> yep it looks like that's the only review up for that one, and you own it syjulian
17:02:48 <rlrossit> I haven't gotten a chance to look at it yet
17:03:06 <rlrossit> looks like gibi gave an initial pass on it at least
17:03:45 <rlrossit> https://review.openstack.org/#/q/topic:bp/versioned-notification-transformation-newton
17:03:50 <rlrossit> here's the nova side of the transformations
17:03:57 <rlrossit> looks like gibi owns most of those
17:04:37 <rlrossit> I think the priority on those right now are getting the testing infra down for the versioning checks
17:04:42 <syjulian> which is the one that has the instance delete and update?
17:04:58 <rlrossit> I think it starts here: https://review.openstack.org/#/c/309454/
17:05:23 <syjulian> ok
17:05:51 <rlrossit> I think the plan is to not register the objects, but still be able to test the versioning on them
17:06:01 <rlrossit> (which in current implementation doesn't work out well)
17:06:32 <rlrossit> it's mostly going to be don't register them, but register them at least for tests so we can check versioning
17:06:55 <syjulian> so register them in the unit tests?
17:07:04 <rlrossit> yep
17:07:27 <rlrossit> I had an example for it at one point, but I don't want to go digging it up again
17:07:52 <rlrossit> plus we want to make it general enough, so if you add a new notification, you don't need to manually add anything, it'll automatically pick it up because you're in the notification space
17:08:06 <rlrossit> that way reviewers don't have to be checking for it, it's always enforced
17:09:07 <rlrossit> syjulian: anything else to discuss on reviews? (besides you want some more on your change I'm sure)
17:09:34 <syjulian> nope. that's it
17:09:46 <rlrossit> alright. well I suppose for formality...
17:09:49 <rlrossit> #topic open discussion
17:09:55 <rlrossit> anything else you want to talk about?
17:10:43 <syjulian> nothing really. but some people in my team are interested in helping in the notifications transformation
17:10:58 <rlrossit> that's good
17:11:09 <rlrossit> let me or gibi know if we can make things easier on their way in
17:11:22 <rlrossit> we took a pretty small chunk for newton on purpose
17:11:36 <rlrossit> because there wasn't much interest in Mitaka, and we don't want to overload the cores with reviews
17:11:46 <rlrossit> but hopefully that changes in O if we get enough of us
17:12:04 <syjulian> will do. thanks!
17:12:33 <rlrossit> well, I don't have the record for shortest meeting, but I probably got close
17:12:46 <rlrossit> hopefully we get more people next week with gibi back :)
17:12:52 <rlrossit> #endmeeting