15:03:15 <korzen> #startmeeting neutron_upgrades
15:03:16 <openstack> Meeting started Mon Jul 11 15:03:15 2016 UTC and is due to finish in 60 minutes.  The chair is korzen. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:03:17 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:03:20 <openstack> The meeting name has been set to 'neutron_upgrades'
15:03:26 <slunkad_> hello
15:03:30 <rossella_s> hey
15:03:33 <korzen> hello everybody
15:03:45 <korzen> so today Ihar is busy
15:04:05 <korzen> and I will be chairing
15:04:30 <korzen> ok, so #topic Announcements
15:04:42 <korzen> #topic Announcements
15:05:04 <korzen> I guess there has been a quite week
15:05:32 <korzen> besides midcycle in August I do not have anything interesting
15:05:42 <korzen> I have put the upgrades on agenda there
15:06:28 <korzen> if there is nothing important
15:07:22 <korzen> #topic Partial Multinode Grenade
15:08:19 <korzen> so I have seen some patch for enabling the mutinode dvr as voting
15:08:24 <diga_> Yingxin: http://paste.openstack.org/show/529767/
15:08:25 <korzen> #link https://review.openstack.org/#/c/336116
15:08:51 * sc68cal is back from pto
15:09:01 <korzen> oh, hi sc68cal
15:09:13 <korzen> I've seen your +1
15:09:58 <sc68cal> yeah, I guess I just need to update the commit message on 336116 to remove the WIP and we're good to go
15:10:16 <sc68cal> Ihar contacted me late last week about it and said it lgtm
15:11:02 <korzen> ok, sc68cal anything else you want to add?
15:11:50 <korzen> We also have LinuxBridge experimental job
15:11:51 <sc68cal> https://review.openstack.org/#/c/336793/ looks good too
15:11:58 <korzen> #link https://review.openstack.org/#/c/336793/
15:12:26 <korzen> yes, that one just needs +w
15:12:57 <slunkad_> I think I will need to leave..can join back in about 20 mins.
15:13:08 <korzen> ok, lets move on
15:13:18 <korzen> #topic Object implementation
15:13:54 <korzen> slunkad_, just went and I had a question about SG ovo ;)
15:14:35 <korzen> I have sent the bi-weekly report to ML just an hour ago
15:14:39 <korzen> #link: http://lists.openstack.org/pipermail/openstack-dev/2016-July/099039.html
15:15:06 <korzen> it seems that bi-weekly means once a three weeks for us but...
15:15:55 <korzen> from my perspective, I have finished splitting the patches for subnet object adjustment
15:16:57 <korzen> still waiting for main patch with get_objects ported to OVO till other dependant patches will land
15:17:24 <korzen> rossella_s, what about port object?
15:17:29 <korzen> any progress there?
15:19:27 <korzen> jlibosva, did you have time to look at ny port-related patches?
15:19:31 <rossella_s> korzen, I announced 2 weeks ago I think that I have no time for it...I think Ihar volunteered but he's probably too busy too
15:20:09 <jlibosva> korzen: I can try to find some time
15:20:30 * jlibosva feels bad for not being active on objects
15:20:40 <korzen> I guess that we need someone to take care of port integration
15:21:08 <korzen> I'm busy with subnet and I would like to continue on network after that
15:21:40 <korzen> jlibosva, do you know when Ihar will be back in business?
15:21:59 <jlibosva> I think next week
15:22:59 <korzen> ok, so we will see in next week, if you will not find any free cycles this week
15:23:56 <korzen> ok so anything else with object implementation?
15:25:06 <korzen> I've seen some bp on refactor model definitions
15:25:09 <korzen> #link https://bugs.launchpad.net/neutron/+bug/1597913
15:25:09 <openstack> Launchpad bug 1597913 in neutron "refactor model definitions" [Wishlist,New]
15:25:42 <korzen> rossella_s, I;ve seen you had put it in wishlist
15:26:06 <korzen> rossella_s, anything actionable from that BP should be done?
15:27:51 <korzen> I guess the involved person should publish the patch and we will see
15:27:58 <rossella_s> korzen, I was just triaging it
15:28:06 <rossella_s> korzen, I don't have much info
15:28:15 <korzen> rossella_s, ok
15:28:43 <korzen> ok, moving on
15:28:55 <korzen> #topic Open discussion
15:29:37 <korzen> We are progressing but slowly
15:29:50 <korzen> I hope to see at least some objects ported in Newton
15:30:25 <korzen> The midcycle discussion would be mainly on Ocata planning I guess
15:30:28 <rossella_s> korzen, we need more hope ;)
15:30:46 <rossella_s> korzen, why?
15:31:07 <korzen> the release of N-3 will be just a week after Cork
15:31:25 <korzen> and I'm on PTO after cork so :)
15:31:43 <korzen> #link http://releases.openstack.org/newton/schedule.html
15:32:11 <korzen> we are meeting in aug 15-19 and the n3 is in 2 weeks after
15:32:18 <rossella_s> korzen, just in time
15:33:23 <korzen> so we have some time to deliver the objects but not so many as I thought we would have to finish all of the objects
15:34:17 <rossella_s> korzen, right...all objects was a bit too hopeful
15:34:24 <slunkad> hello, I'm back
15:34:42 <korzen> so slunkad it is your last call to speak
15:35:08 <slunkad> well just want to ask about https://review.openstack.org/#/c/284738/
15:35:23 <slunkad> I am not sure what other tests I should add to it
15:36:48 <slunkad> probably rossella_s can answer that..
15:36:58 <korzen> if it is regarding the IPNetwork,  then I guess it can be added in follow-up patch
15:37:33 <rossella_s> korzen, slunkad I commented regarding adding a test for the is_default field
15:37:46 <korzen> ok, that one
15:38:22 <korzen> so the id-default field is a bit strange to me
15:38:22 <slunkad> rossella_s: right, but what test do you want for it?
15:38:46 <rossella_s> slunkad, something like you create a sec group and see if it's default is set
15:39:18 <slunkad> rossella_s: ok, makes sense
15:39:20 <korzen> rossella_s, slunkad and that is the issue here, we do not set the is-default on get_objects method
15:39:35 <korzen> if I understand correctly
15:39:54 <korzen> the is-default is handled only in create()
15:40:12 <korzen> but the get_objects will return not-set field
15:40:28 <slunkad> korzen: right it is handled only in create
15:40:35 <korzen> since we do not have any method loading the is-default for synthetic fields
15:41:34 <korzen> so we are not able to create an object with is-default set and check via get_objects if it was correctly set
15:42:02 <slunkad> korzen: oh, is there no way to get the set fields?
15:42:20 <korzen> in my understanding no
15:42:30 <korzen> since is-default is sythnetic fields
15:42:54 <rossella_s> if you create the unit test it will be easy to check that
15:43:05 <korzen> and synthetic fields need to have an separate object or loading method
15:43:07 <rossella_s> anyway I think korzen is right
15:43:21 <korzen> like I have it on subnets.shared attr
15:43:30 <korzen> let me show you what I mean
15:43:41 <slunkad> okay
15:44:18 <korzen> #link https://review.openstack.org/#/c/337634/1/neutron/objects/subnet.py@187 loading shared attribute by dedicated method
15:45:08 <korzen> in my case, I'm setting the shared attr by load_shared, and in order to use it in get_objects I have overridden the from_db_objects method
15:45:15 <korzen> https://review.openstack.org/#/c/337634/1/neutron/objects/subnet.py@206
15:45:48 <korzen> slunkad, you need to follow similar approach for is-default
15:46:18 <slunkad> korzen: alright, I'll take a look
15:46:35 <korzen> please remember also for addint is-default as extra filtering argument, https://review.openstack.org/#/c/337634/1/neutron/objects/subnet.py@180
15:46:51 <korzen> s/addint/adding
15:47:12 <korzen> if the API is going to filter by is-default
15:47:27 <slunkad> korzen: I don't think that is possible
15:47:37 <slunkad> is-default is not exposed to the API
15:47:50 <korzen> ok
15:48:32 <korzen> slunkad, do you have anything more?
15:48:44 <slunkad> korzen: no that's all! thanks
15:49:58 <korzen> ok, so if there is nothing more...
15:50:25 <korzen> #endmeeting