19:00:02 #startmeeting Ironic 19:00:02 #chair devananda 19:00:02 Meeting started Mon Jan 20 19:00:02 2014 UTC and is due to finish in 60 minutes. The chair is NobodyCam. Information about MeetBot at http://wiki.debian.org/MeetBot. 19:00:03 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 19:00:04 Welcome everyone to the Ironic meeting. 19:00:06 The meeting name has been set to 'ironic' 19:00:07 Current chairs: NobodyCam devananda 19:00:15 o/ 19:00:20 who here for the meeting 19:00:22 o/ 19:00:24 \o 19:00:25 o/ 19:00:26 who's even 19:00:29 o/ 19:00:29 \o 19:00:35 o/ 19:00:35 me 19:00:41 Of course the agenda can be found at: 19:00:42 #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting 19:00:44 + 19:01:10 o/ 19:01:10 #topic Greetings, and announcements 19:01:13 welcome all 19:01:37 ok let start with a quick announcement 19:01:48 wel 19:02:04 devananda: sent a really good email to the list yesterdat 19:02:06 http://lists.openstack.org/pipermail/openstack-dev/2014-January/024823.html 19:02:10 #link http://lists.openstack.org/pipermail/openstack-dev/2014-January/024823.html 19:02:21 def worth the read 19:02:49 NobodyCam: thanks for the link 19:03:17 got it 19:03:30 its a holiday here so I may be a bit slow 19:03:33 :-p 19:04:02 hehe 19:04:12 lets jump into 19:04:13 #topic Outstanding, in-progress or Action Item updates 19:04:37 Icehouse release progress // graduation? 19:04:43 :) 19:04:45 this is the week 19:04:56 we are so close! 19:05:09 yep 19:05:13 dkehn: are you around? 19:05:16 what do you mean, this is the week? 19:05:23 so i'm going to be working with ttx this week to get the i2 release tested and tagged 19:05:35 nice 19:05:43 s/release/milestone 19:05:56 so if we don't make i2, we won't graduate? 19:06:10 NobodyCam: just got here 19:06:16 :) 19:06:17 rloo: not exactly 19:06:26 rloo: this is a checkpoint of sorts 19:06:30 NobodyCam: eating lunch 19:06:45 :) 19:06:48 devananda: ok. (phew). 19:06:51 the TC evaluates our progress as a project towards becoming part of the release process here 19:07:21 Guys, I have a few questions about migrations from Nova 19:07:36 This is the thing we also need to have as soon as possible 19:08:04 so it's more about me working with them to get the milestone tagged, tested, etc. 19:08:08 romcheg: that would be nice to have... but is not required as I understand it 19:08:24 we are so close to having deploy() working, though, and I was hoping we'd have it done this week 19:08:26 Ah, cool then 19:08:34 NobodyCam, I thought it was required 19:08:43 but I still have the questions :) 19:08:52 migration from nova is required, but not this week 19:09:01 we need that by icehouse release 19:09:07 * NobodyCam notes holiday status and use of the word understanding 19:09:21 but not the mile stone 19:09:32 correct 19:09:36 by april something 19:09:43 devananda, do you have a checklist points of what is missing for deploy(). Neutron integration? 19:10:02 so what is needed, if anything for icehouse-2 this week? (ie, what do you need if anything, from us, todo?) 19:10:19 lucasagomes: not off hand, no. i'm still recovering my test env after LCA 19:10:29 I've tested the deployment with some variables off (neutron and nova), and at least the ironic part seems to be stable 19:10:44 lucasagomes: the main is setting the dhcp options for pxe boot 19:10:51 lucasagomes: with what's in trunk, or with patches taht are in flight? 19:10:57 which has to come from the conductor incharge 19:11:00 devananda, with the patches 19:11:10 lucasagomes: ok. so lett's land those 19:11:15 but none of them are big patches 19:11:23 lucasagomes: can you (after the meeting) call those out to me specificaly? 19:11:28 for those interested, there's a guideline I wrote (and has being improved by others) 19:11:34 i'll prioritize reviewing/fixing/landing that stuff today 19:11:36 #link https://etherpad.openstack.org/p/IronicDeployDevstack 19:11:39 devananda, sure 19:11:42 thanks 19:12:34 NobodyCam, yea, I hear ya, I didn't test the dhcp part integrated with neutron yet 19:12:42 ok to bring the meeting back under control 19:13:09 do we want to go over the bp/ bugs? 19:13:28 lucasagomes: that really about it. 19:13:33 let's see what's targeted to i2 19:13:38 and either close it or bump it 19:13:45 #link https://launchpad.net/ironic/+milestone/icehouse-2 19:14:01 BP's first 19:14:03 #link https://blueprints.launchpad.net/ironic/+spec/pxe-mount-and-dd 19:14:08 I have thought about a hack that would allow supporting a single conductor, but its not the right way 19:14:12 we know this works now, so we can close it, yes? 19:14:23 yes 19:14:43 it was more an historic bp 19:14:44 +1 19:14:54 agreed 19:14:57 #link https://blueprints.launchpad.net/ironic/+spec/add-neutron-support 19:15:05 this one is still stuck. let's move it to i3 19:15:07 yes? 19:15:24 +1 19:15:30 we kinda have too 19:15:37 makes sense to me 19:15:46 k 19:15:51 #link https://blueprints.launchpad.net/ironic/+spec/instance-mapping-by-consistent-hash 19:16:00 what about that one? 19:16:30 we have almost pretty much everything for this one in place already right? 19:16:38 most of it has landed no? 19:16:39 list of conductors, routing the rpc calls 19:16:42 yep 19:16:51 rebalance / takeover hasn't landed yet 19:16:57 but i can do that separeately 19:17:01 ya 19:17:08 core function are there 19:17:15 +1 19:17:25 there's something about update mapping in neutron? 19:17:39 yea, that's aprt of the takeover 19:17:42 rloo, takeover? 19:17:46 and depends on the neutron work in teh prior BP 19:17:52 i should add that as a dependency 19:17:57 well 19:18:19 #action deva to file a new bp for takeover, including the neutron update part, and remove this from https://blueprints.launchpad.net/ironic/+spec/instance-mapping-by-consistent-hash 19:18:23 we will need a initial way of setting it for the options for neutron 19:18:31 lastly 19:18:32 #link https://blueprints.launchpad.net/ironic/+spec/serial-console-access 19:18:42 sjing has posted code for this but i dont know if anyone of us have tested 19:18:46 (i haven't) 19:18:53 * NobodyCam has not :( 19:19:02 * GheRivero not me 19:19:18 me neither 19:19:18 ok, i'll bump 19:19:29 to i-3? 19:19:31 it needs ipmi to test it 19:19:53 #action nobodycam to test serial-console 19:20:04 devananda: it is in code review status. need more comments 19:20:10 yes 19:20:10 we can do it ipmi native 19:20:50 for us to say we have it will we NEED it in ipmitool? 19:20:57 there are many open bugs targeted to i2 19:22:26 anyone blocked on anything in particular there? 19:22:45 me 19:22:50 deva can we bump anything below high ? 19:22:51 (reviewers) 19:22:52 I can't reproduce https://bugs.launchpad.net/ironic/+bug/1244747 19:23:10 NobodyCam: we can bump anything we need to, but i'd rather get them fixed :) 19:24:00 max_lobur: it looks like lucasagomes referenced a bug in pecan/wsme as a possible cause 19:24:10 rloo: smack /me upside the head when that is the case 19:24:10 deva 19:24:17 that's a separate bug 19:24:24 actually, there's two different approach 19:24:29 just another way to implement the hook 19:24:29 NobodyCam. Sure, a virtual smack? :-) 19:24:35 max_lobur, approach doesn't need wsme to be fixed 19:25:10 devananda, lucasagomes yes, we're able to fix on current wsme already 19:25:30 I mean it's already seems fixed to me ;) 19:26:16 devananda: this seems kinda big-ish 19:26:21 #link https://bugs.launchpad.net/ironic/+bug/1187209 19:27:21 sorry guys, gotta step away for 2 mintues. brb 19:28:00 * NobodyCam turns on the wheel-of-fourtune music 19:28:18 so mostly of the bugs seems to be marked as in-progress 19:28:22 bugs for i2 19:28:27 ya 19:28:38 I think we'doing good on them 19:29:24 NobodyCam, stevedore bug doesn't seems hard to fix 19:29:34 also it's low priority 19:29:47 max_lobur: ya after re-rereading it I was just about to say that 19:29:59 hehe :) 19:30:22 shall we move on to the call outs 19:30:46 #Topic Callouts 19:30:49 back 19:30:59 WB 19:31:34 nova driver while still lacking any real tests can deploy a nova 19:32:04 woot! good job! 19:32:06 it is current stuck at deploy as the node never attempts to pxe boot 19:32:38 its just that the pxe options are not set. (for neutron) 19:32:45 does that gets fixed after passing the dhcp options to neutron 19:32:47 via pxe driver? 19:33:00 ok that asnwer 19:33:01 yes 19:33:13 :) 19:33:14 good stuff NobodyCam :D 19:33:37 #topic testing 19:34:03 romcheg: devananda any thing to say on testing? 19:34:39 NobodyCam: agordeev2 is working on basic infrastructure for intergation testing 19:34:51 agordeev2: are you around? 19:34:58 yep, i'm here 19:35:15 Hey agordeev2 :) welcome 19:35:22 * lucasagomes brb 19:35:26 AFAIR there were several questions about how to perform initial configuration 19:35:49 waiting for this to land 19:35:51 #link https://review.openstack.org/#/c/65845/ 19:35:54 on last thirsday i'd visited QA meeing just to highlight testing scheme and get more attention to it. 19:36:07 which will move tempest API tests from the experimental pipeline into check & gate 19:36:20 But I don't think that creating VMs in tempest is a good idea because it's unlikely to be accepted 19:36:57 romcheg: our discussion on the ML seemed to settle on that being the right approach 19:36:57 agreed 19:37:28 romcheg, agordeev2 - what do you suggest intead of creating the VMs via tempest? 19:37:50 I would do that in devstack. 19:38:14 ok. that was my initial suggestion :) 19:38:37 create the VMs and the bridge in devstack as part of the bring-up 19:38:44 dump the MACs to a text file 19:38:49 Or even in nodepool. That might be more efficient but then regular users will have to create vms by hands 19:39:02 then we can do the enrol/deploy/undeploy/delete in the tempest tests 19:39:07 back 19:39:13 not nodepool 19:39:48 The good thing in nodepool AFAIK is that it can pre-create test nodes. So that might reduce the time required for running a job 19:39:51 we should be able to run these tests with tempest + devstack. it shouldn't need more infrastructure 19:40:15 devananda: agreeded :) 19:40:17 I vote for devstack as well 19:40:32 I just had to mention that thing about nodepool :) 19:40:44 :) 19:40:44 :) 19:40:57 devstack making some quick calls to libvirt to create NN vm's shouldn't take taht long 19:40:57 ok anything else on testing 19:41:00 it's not installing anything 19:41:05 just making blank VMs 19:41:20 on testing, just to mention again my email about third-party testing 19:41:29 but if anyone has comments, please discuss on the ML, not here today 19:41:43 :) 19:42:04 No more news from me 19:42:10 ok :) 19:42:15 what to move to 19:42:41 agenda says: 19:42:42 #topic Food for Thought / Open Discussion 19:43:05 I wanted to update you guys on SeaMicro driver status 19:43:06 Guys, I have a few questions on migrating from Nova 19:43:11 shot 19:43:19 k4n01: great! please do 19:43:20 I'd like to discuss a race bug today 19:43:25 but since it may be long 19:43:35 we may proceed on our chart 19:43:39 -chat 19:43:59 So I wrote a script that converts nova's db into Ironic db 19:44:01 We have finished coding for the Power and VendorPassThru blueprints, we doing internal reviews and will be ready for community before end of this week 19:44:04 max_lobur: if thats ok we can do in channel after meeting? 19:44:24 max_lobur: you're referring to the need for an intent lock between API cals, yes? yea, let's do in channel. i'll be around for a while 19:44:31 However Ironic's DB will change in the future 19:44:34 k4n01: awesome 19:44:34 max_lobur, I remember devananda raised some q about python threads (os threads) and green threads 19:44:42 have you looked into it? 19:44:44 devananda: did you see my update to SeaMicro VendorPassthru blueprint? 19:44:53 k4n01: no. lemme take a look now 19:44:59 So I think whether it's reasonable to use existing migrations somehow? 19:45:00 k4n01: please take look at devananda's email to the list 19:45:38 romcheg: i can think of two solutions 19:45:40 NobodyCam: Yes, already read it, i agree with those 5 points in email, and I will come up with SeaMicro's plan to implement them till next time we have meeting 19:45:43 devananda, lucasagomes, NobodyCam yes, lets discuss in ironic chat. I have an answer to green threads question. I'm armed and dangerous :) 19:45:52 :D 19:46:20 k4n01: :) Ty 19:46:25 romcheg: 1. we keep the novadb->ironicdb migration review open and maintain it until icehouse-3 milestone, at which point feature freeze should prevent (most) db changes after taht 19:46:30 romcheg: and we land it at that point 19:46:47 romcheg: or, 2) we land it now, and maintain it via additional small patches until icehouse release 19:46:56 If someone wants to migrate after i3? 19:47:15 We might want to change the DB in future releases 19:47:30 romcheg: we'll need the migration script to be applicable on the icehouse release itself 19:47:37 romcheg: not after taht 19:47:51 makes sense 19:47:53 noav-bm will be marked deprecated if ironic graduates this cycle 19:47:59 devananda: ++ 19:48:03 so no new changes to nova_bm schema will be allowed 19:48:05 after taht 19:48:14 after then uses should uising Ironic 19:48:28 and migration path can be defined as Icehouse Noav_bm -> Icehouse Ironic -> "J" Ironic -> "K" Ironic ... etc 19:48:31 Sounds like a dictatorship but I like it :) 19:48:32 users 19:49:09 :) 19:49:16 k4n01: i dont see a reply on either BP 19:49:17 The second question was how should be store this utility, but now I understand that it should be the part of Ironic distribution, shouldn't it?? 19:49:28 romcheg: yes, part of ironic 19:49:41 romcheg: something like our ironic-dbsync utility. 19:49:49 romcheg: eg, ironic-import-db-from-nova, or something 19:49:50 devananda: i have updated the main description of https://blueprints.launchpad.net/ironic/+spec/seamicro-vendor-passthru-interface-implementation 19:49:56 yes, makes sense 19:50:03 in https://github.com/openstack/ironic/tree/master/tools 19:50:13 devananda: I am not sure if that triggers an update mail to people involved , sorry 19:50:19 and what people thinks about: dict-like behavior on the ironic objects should be removed or stay? (https://review.openstack.org/#/c/64336) 19:50:24 Should that be one single migrate_from_nova script or a series of scripts? 19:50:26 k4n01: ah! i see now, thanks. I will review the more detailed BP after meeting 19:50:28 it seems to me that from user's point of view, icehouse nova_bm->icehouse ironic, icehouse nova_bm->J ironic, etc would make more sense. 19:50:34 #action devananda to review https://blueprints.launchpad.net/ironic/+spec/seamicro-vendor-passthru-interface-implementation 19:50:47 devananda: ok, i will hang around #openstack-ironic 19:50:48 like migrate_nova_db build_nova_tftproots 19:51:30 lucasagomes, I'd vote for removing dict like behavior 19:51:32 rloo: in my past discussions with the TC and other PTLs, taht didnt' seem necessary. take cinder and neutron as examples. they didn't skip releases like that 19:51:43 I don't see any profit on it 19:51:51 devananda: ok, if there's a precedence, that's fine. 19:51:52 romcheg: I like the chain of scripts (with a wrapper to make it easy to use ofc) 19:51:52 (dict like behavior) 19:52:04 right, yea I kinda like the idea of having only one consistent way to access the the objects attributes, so I would remove the dict behavior as well 19:52:17 rloo: but if, during the J release, someone wants to add a "migrate from icehouse nova to J ironic" script, and that is even possible given how different our arch might be by then.... i dont think there'll be a problem with it 19:52:24 i would remove dict behaviour too 19:52:48 lucasagomes: i'm fine with removing dict usage of objects 19:52:56 great 19:52:56 wrt remove dict behaviour, i think that makes sense, but the review only removes the [] part of dict like behaviour. 19:52:58 lucasagomes: however that object code in noa stil lhas it 19:53:14 devananda, yea, I commented that on patch one 19:53:19 lucasagomes: and when it moves to oslo (which I think is in progress???) it will still be there, too, i believe 19:53:19 I don't think we should diverge from nova 19:53:19 I would go along as well :-p 19:53:30 rloo and what else? 19:53:37 lucasagomes: so we can't actualy REMOVE that support fromour code. we can jsut -1 any patch that tries to use it 19:53:54 devananda, right, so shouldnot remove it from the base object code 19:53:58 right 19:54:07 max_lobur. there are a bunch of methods that support dict-like behaviour, not just the setitem/getitem stuff I think. 19:54:17 I see 19:54:40 devananda, cool, right that looks reasonable for me 19:54:48 I will review that series of patches as well 19:54:52 so maybe a blueprint with whats required ? 19:55:06 (to remove dict_*) 19:55:08 also, removing dict-like object access is pretty low on my priority list 19:55:14 so we'll just remove usages right? 19:55:30 devananda, yea, just brought it up beucase it's on the meeting calendar 19:55:36 lucasagomes: ah. thanks 19:55:53 we can also have an intermediate parent class between oslo's object and our, which will throw exception in dict methods 19:56:01 to be sure we're not using them 19:56:12 if lucasagomes is looking at the actual agenda 19:56:16 seems like a lot of work to do for ... ?? 19:56:17 or it will be to much? :) 19:56:22 can we talk about Using an intent/two-phase lock or a thread to solve the race condition in four moinutes 19:56:27 minutes even 19:56:35 NobodyCam, +1, in the ironic channel 19:56:36 :) 19:56:51 :) 19:56:57 there's a point about tempest coverage 19:57:03 I started to expand it 19:57:23 but it turns out to be just porting all the tests from our API 19:57:53 max_lobur: our API unit tests aren't actually exercizing all the pathways (RPC, DBAPI, etc) 19:58:17 for RPC - true 19:58:22 so tempest API tests are functional tests. yes, they may cover the same CRUD operations 19:58:27 that's definitely should be in tempest 19:58:33 for orthers 19:58:34 but i dont' think they're duplicate effort 19:58:54 do you think we need to expand coverage in tempest or just add some tests to the api? 19:59:07 what's missing? 19:59:09 in some cases yes, in some no, grenade is a ersion upgrade testing 19:59:26 because tempest will require a separate patch to update tests, it may be not convenient 19:59:38 soem of the others rely on the devstack exercises etc. 19:59:39 * NobodyCam note times up 19:59:47 note 19:59:51 max_lobur: not convenient to do what? 19:59:52 :) 20:00:03 to update tempest for each API change 20:00:13 dkehn: grenade is a great effort. but ironic doesn't have a prior release to upgrade from (yet) 20:00:19 we should move back to channel 20:00:25 https://review.openstack.org/#/c/67854/1 expanding tempest coverage 20:00:26 assuming it will 20:00:40 yep, times up -- let's continue these great discussions back in channel 20:00:47 thanks everyone! 20:00:52 bye 20:00:54 thanks 20:00:58 thank you all see un in channel 20:01:01 thk 20:01:01 #endmeeting