16:01:08 <Sukhdev> #startmeeting ironic_neutron
16:01:09 <openstack> Meeting started Mon Aug  1 16:01:08 2016 UTC and is due to finish in 60 minutes.  The chair is Sukhdev. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:01:10 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:01:12 <openstack> The meeting name has been set to 'ironic_neutron'
16:01:15 <mjturek1> o/
16:01:26 <TheJulia> o/
16:01:27 <hshiina> o/
16:01:43 <Sukhdev> #topic: Agenda
16:01:58 <Sukhdev> #link: https://wiki.openstack.org/wiki/Meetings/Ironic-neutron#Meeting_August_1.2C_2016
16:02:19 <Sukhdev> Anybody would like to add anything to the agenda?
16:02:57 <Sukhdev> #topic: Announcements
16:03:19 <Sukhdev> I do not have any announcements to make - do any of you have any?
16:03:28 <Sukhdev> jroll ?
16:03:37 <jroll> oh, it's that time already eh
16:03:41 <jroll> hi :)
16:04:01 <jroll> so just one thing:
16:04:03 <Sukhdev> Happy August!!!
16:04:10 <Sukhdev> time is flying by :-)
16:04:28 <jroll> we ran into an issue on the nova patch: https://review.openstack.org/#/c/297895/9/nova/virt/ironic/driver.py@1170
16:04:41 <jroll> we're fixing it in this ironic chain https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:master+topic:bug/1608511
16:05:02 <jroll> then the nova thing will be good to go
16:05:12 <jroll> still not sure if we can get an FFE on that but I'm working on it
16:05:46 <Sukhdev> we'll keep our fingers crossed
16:06:48 <Sukhdev> I saw rloo's comment over the weekend
16:07:18 <Sukhdev> #topic: Update on testing latest code
16:07:32 <Sukhdev> So, I have an update to give to the team on my testing activities -
16:08:01 <Sukhdev> I pulled all of the latest code (all patches) and this is the first time, everything went reasonably smooth
16:08:13 <Sukhdev> except for one change in behavior -
16:08:36 <Sukhdev> my script used to create the node and then modify to add the driver information
16:09:09 <Sukhdev> conductor did not like it well - but, when I changed the script to add driver information during node-create
16:09:15 <Sukhdev> everything went smooth
16:09:46 <Sukhdev> did not know if this change in behavior was a bug or intentional - I thought I bring to team's attention
16:09:56 <jroll> can we see the script and the error?
16:10:18 <jroll> guess not :(
16:10:28 <mjturek1> lol
16:10:49 <devananda> sorry I'm late, folks.
16:11:03 <Sukhdev> sorry I got disconnected - back now
16:11:16 <jroll> wb Sukhdev :P
16:11:31 <jroll> Sukhdev: can we see the script and the error? hard to tell if code problem or user error without it
16:11:50 <Sukhdev> jroll : understood
16:12:26 <Sukhdev> I have to log into different system to pull the information - will post shortly
16:12:33 <jroll> ok, even after meeting is fine
16:12:37 <Sukhdev> other than that - all works fine
16:12:56 <jroll> great
16:14:08 <Sukhdev> jroll - http://paste.openstack.org/show/545161/
16:14:29 <Sukhdev> jroll - that script used to work
16:15:09 <jroll> Sukhdev: what's the rror?
16:15:50 <Sukhdev> jroll : did not capture for pasting, but, something like - can not change from flat to neutron - unknown interface type
16:15:55 <Sukhdev> or something like that
16:16:10 <jroll> Sukhdev: oh, you probably need to make sure enabled_network_interfaces has neutron
16:16:13 <jroll> it does not by default
16:16:17 <jroll> I do not remember why
16:16:49 <Sukhdev> right - even after doing that, I saw that error
16:17:20 <Sukhdev> but, once I switch to providing all information during node-create, all works fine
16:18:20 <jroll> did you restart API and conductor after doing that?
16:19:17 <Sukhdev> jroll: I suspected that may be the reason - I am not good at restarting api ( I usually restart conductor) :-)
16:19:37 <jroll> yeah, api uses that config for validation now
16:19:45 <Sukhdev> I will try to reproduce it later in the day and, if I am able to, will capture the past and post it
16:20:34 <Sukhdev> Other than that I do not have anything else to report - which is a good thing :-)
16:20:49 <mjturek1> Sukhdev: just want to clarify, are you testing against master now?
16:20:57 <mjturek1> or do you need to pull in other patches
16:20:58 <Sukhdev> mjturek1 : yes
16:21:01 <mjturek1> cool, thanks
16:21:28 <Sukhdev> mjturek1 : no patches
16:21:43 <mjturek1> got it, thanks :)
16:21:53 <Sukhdev> moving along -
16:22:10 <Sukhdev> #topic: Remaning work items
16:22:22 <Sukhdev> I split the work items into two categories
16:22:50 <Sukhdev> Desired for Newton cycle and O-cycle items
16:23:11 <Sukhdev> 1) port groups
16:23:20 <Sukhdev> 2) inspector, and
16:23:26 <Sukhdev> 3) security groups
16:23:42 <mjturek1> In regards to (1) - I did a bit of breaking up on the portgroups API patch (per a discussion a few meetings ago). Not sure if people would rather just keep the API patch the way it is though https://review.openstack.org/#/c/347549 Any thoughts?
16:23:43 <Sukhdev> All are doable in this cycle -
16:25:51 <devananda> mjturek1: have you posted the broken-up patch(es) somewhere?
16:26:06 <mjturek1> devananda: https://review.openstack.org/#/c/347549
16:26:16 <mjturek1> it's just this one currently, still pretty big :-\
16:26:43 <mjturek1> haven't removed the code from the original patch yet either, wanted to see what people thought first
16:27:41 <mjturek1> the original being https://review.openstack.org/#/c/332177
16:28:28 <devananda> 1400 LOC is better than 1900 ... but still a lot
16:28:46 <mjturek1> devananda: I can probably break it up a bit more
16:29:22 <devananda> mjturek1: looking at the two patches, this one is at least targeted to just the API changes, which does make it easier to review
16:29:45 <devananda> I think it's fine
16:30:16 <devananda> good job breaking part the addition of portgroups from the other changes :)
16:30:26 <mjturek1> :) thanks
16:31:19 <Sukhdev> A quick browse through the patch - looks very clean
16:31:46 <mjturek1> great, thanks Sukhdev
16:32:52 <Sukhdev> sambetts|afk : Sam is that you?
16:33:05 <jroll> Sukhdev: he won't be here for the meeting
16:33:17 <Sukhdev> ah I see
16:33:46 <Sukhdev> anything else on this?
16:34:28 <mjturek1> nothing here
16:35:17 <Sukhdev> So, the other two items API changes for interface attach/detach and vlan aware servers to be moved to the O-cycle
16:35:58 <Sukhdev> hopefully neutron will land vlan aware VMs in Neuton and then we can build upon that
16:36:35 <Sukhdev> neutron/newton always messes me up :-)
16:37:06 <mjturek1> :)
16:37:06 <Sukhdev> moving along -
16:37:17 <Sukhdev> #topic: Patches
16:37:32 <Sukhdev> patches in merge conflict - https://review.openstack.org/#/q/project:openstack/ironic+branch:master+topic:bug/1526403+status:open
16:38:00 <Sukhdev> these are still in conflict state
16:39:57 <Sukhdev> other than these and Nova patch, for which jroll already mentioned, everything else seems good
16:40:44 <jroll> I feel like those got replaced and need to be abandoned, but not totally sure
16:42:08 <Sukhdev> jroll - this one too - https://review.openstack.org/#/c/228496/
16:42:34 <jroll> I'm not sure about that one
16:43:39 <Sukhdev> last update was more than a month ago - you may be right
16:44:43 <Sukhdev> Other than this, I do not have anything else to cover
16:44:50 <Sukhdev> #topic: Open Discussion
16:45:04 <Sukhdev> Any open discussion?
16:45:28 <Sukhdev> There is VmWorld in Las Vegas later this week - if anybody cares
16:46:17 <Sukhdev> And, also Future:net, which runs in parallel to VmWorld in Las Vegas as well
16:46:34 <Sukhdev> sorry I mean later this month - not week
16:46:38 <Sukhdev> meant
16:47:11 <Sukhdev> Any body wants to discuss anything
16:47:18 <Sukhdev> if not, we are done -
16:47:27 <Sukhdev> I like shorter meetings :-)
16:47:46 <Sukhdev> Thanks everybody -
16:47:50 <jroll> thanks Sukhdev :)
16:47:52 <Sukhdev> bye
16:48:00 <Sukhdev> #endmeeting