Thursday, 2017-04-27

*** ramishra has quit IRC00:09
*** ramishra has joined #openstack-meeting-500:11
*** ramishra has quit IRC00:25
*** ramishra has joined #openstack-meeting-500:27
*** ramishra has quit IRC00:35
*** ramishra has joined #openstack-meeting-500:40
*** ramishra has quit IRC00:45
*** ramishra has joined #openstack-meeting-500:57
*** dixiaoli has joined #openstack-meeting-501:01
*** HelenYao_ has joined #openstack-meeting-501:03
*** ramishra has quit IRC01:04
*** HelenYao_ has quit IRC01:06
*** ramishra has joined #openstack-meeting-501:12
*** ramishra has quit IRC01:17
*** ramishra has joined #openstack-meeting-501:18
*** ramishra has quit IRC01:35
*** ramishra has joined #openstack-meeting-501:42
*** ramishra has quit IRC01:49
*** ramishra has joined #openstack-meeting-501:50
*** ramishra has quit IRC02:00
*** ramishra has joined #openstack-meeting-502:01
*** ramishra has quit IRC02:11
*** wxy has joined #openstack-meeting-502:14
*** ramishra has joined #openstack-meeting-502:19
*** ramishra has quit IRC02:23
*** iceyao has joined #openstack-meeting-502:26
*** ramishra has joined #openstack-meeting-502:30
*** ramishra has quit IRC02:42
*** ramishra has joined #openstack-meeting-502:44
*** ricolin has joined #openstack-meeting-502:49
*** ramishra has quit IRC02:50
*** ramishra has joined #openstack-meeting-502:52
*** ramishra has quit IRC02:56
*** ricolin_ has joined #openstack-meeting-502:58
*** ramishra has joined #openstack-meeting-502:59
*** ricolin has quit IRC03:01
*** ricolin_ has quit IRC03:20
*** ricolin has joined #openstack-meeting-503:21
*** HelenYao has joined #openstack-meeting-503:36
*** ramishra has quit IRC03:51
*** ramishra has joined #openstack-meeting-503:54
*** ramishra has quit IRC04:07
*** eeiden has quit IRC04:08
*** ramishra has joined #openstack-meeting-504:08
*** ramishra has quit IRC04:16
*** eeiden has joined #openstack-meeting-504:33
*** iceyao has quit IRC05:05
*** iceyao has joined #openstack-meeting-505:14
*** iceyao has quit IRC05:21
*** iceyao has joined #openstack-meeting-505:34
*** ricolin_ has joined #openstack-meeting-506:26
*** ricolin has quit IRC06:27
*** ricolin__ has joined #openstack-meeting-506:31
*** ricolin_ has quit IRC06:34
*** matrohon has joined #openstack-meeting-506:35
*** ralonsoh has joined #openstack-meeting-507:34
*** matrohon has quit IRC07:46
*** makowals_ has quit IRC08:39
*** makowals has joined #openstack-meeting-508:52
*** makowals has quit IRC09:04
*** aarefiev_afk is now known as aarefiev09:16
*** eeiden has quit IRC09:18
*** eeiden has joined #openstack-meeting-509:35
*** dixiaoli has quit IRC09:57
*** ricolin__ has quit IRC10:01
*** ricolin has joined #openstack-meeting-510:02
*** iceyao has quit IRC10:05
*** iceyao has joined #openstack-meeting-510:10
*** iceyao has quit IRC10:13
*** iceyao has joined #openstack-meeting-510:30
*** iceyao has quit IRC10:35
*** markvoelker_ has joined #openstack-meeting-511:11
*** markvoelker has quit IRC11:12
*** makowals has joined #openstack-meeting-511:15
*** iceyao has joined #openstack-meeting-511:30
*** markvoelker_ has quit IRC11:38
*** matrohon has joined #openstack-meeting-512:29
*** markvoelker has joined #openstack-meeting-512:35
*** Guest94155 has quit IRC13:10
*** iceyao has quit IRC13:30
*** iceyao has joined #openstack-meeting-513:31
*** iceyao has quit IRC13:31
*** derekh has joined #openstack-meeting-513:46
*** iceyao has joined #openstack-meeting-514:19
*** zenirc369 has joined #openstack-meeting-514:37
*** ralonsoh_ has joined #openstack-meeting-514:55
*** ralonsoh has quit IRC14:57
*** zenirc369 has quit IRC14:57
*** ralonsoh has joined #openstack-meeting-515:02
*** ralonsoh_ has quit IRC15:02
*** iceyao has quit IRC15:05
*** zigo has quit IRC15:17
*** iceyao has joined #openstack-meeting-515:25
*** iceyao has quit IRC15:30
*** markvoelker has quit IRC15:30
*** markvoelker has joined #openstack-meeting-515:31
*** aarefiev is now known as aarefiev_afk15:35
*** tiendc has joined #openstack-meeting-515:46
*** makowals has quit IRC15:50
*** mjturek has joined #openstack-meeting-515:52
*** hshiina has joined #openstack-meeting-516:00
*** phuongnh has joined #openstack-meeting-516:00
TheJulia#startmeeting ironic_bfv16:01
openstackMeeting started Thu Apr 27 16:01:00 2017 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.16:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:01
*** openstack changes topic to " (Meeting topic: ironic_bfv)"16:01
mjtureko/16:01
openstackThe meeting name has been set to 'ironic_bfv'16:01
TheJuliao/16:01
derekho/16:01
mjturekwelcome back TheJulia16:01
hshiinao/16:01
TheJuliaThanks!16:01
TheJulia#link Our Agenda, as always https://wiki.openstack.org/wiki/Meetings/Ironic-BFV16:02
TheJulia#topic Announcements/Reminders16:02
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic_bfv)"16:02
TheJuliaI'm back from vacation and two business trips \o/16:02
TheJuliaAnd somehow I'm still alive16:02
TheJuliaDoes anyone have anything to announce this week?16:03
mjturekseems like a no :)16:04
TheJuliaYup16:04
derekhnot sure if its an announce, but I'm hoping to help out here if I can16:04
TheJuliaderekh: Welcome!16:04
mjturekwelcome!16:04
TheJuliaMoving on!16:05
derekhthanks16:05
TheJulia#topic Current Status16:05
*** openstack changes topic to "Current Status (Meeting topic: ironic_bfv)"16:05
TheJuliaI'm a little worried because I've not had time to digest the ether pad or check the review sets this morning.16:05
TheJulia#link https://etherpad.openstack.org/p/Ironic-BFV16:05
*** makowals has joined #openstack-meeting-516:05
mjturekCinder driver - didn't get a ton done but I fixed a problem with failing docs tests this morning. I also introduced a patch in hopes of handling one of the TODOs there. I have it in discussion for later on16:06
TheJuliaAwesome16:06
TheJuliaI guess your the light gold color in the ether pad?16:06
mjturekand orange16:06
mjturekit seems to fade to gold after about a week :)16:07
TheJulia:)16:07
TheJuliaOkay, looks like the ether pad is in decent shape, I just updated the notes on two of the revisions to be more up to date.  Looks like https://review.openstack.org/#/c/406290 needs a rebase too16:09
TheJuliaI guess that is it for status, there really is nothing else to cover it seems.16:10
TheJuliaSo if there are no objections, we can move on to discuss priorities16:10
* TheJulia takes the silence as agreement to proceed16:10
mjturek+116:10
TheJulia#topic Planning/Priorities16:10
*** openstack changes topic to "Planning/Priorities (Meeting topic: ironic_bfv)"16:10
TheJuliaLooks like hshiina found that there was some extra validation logic inside cinder that I didn't find when I dug through the code originally.  Which seems kind of silly since it is asserting things that some drivers don't actually even use :(16:13
TheJuliaBut it is enforcing the contract as it were16:13
mjturekgood finds hshiina16:13
TheJuliaSo that seems like it is going to take some refactoring on our part, unless we just want to catch the error somehow and make it visible.  I think he pointed this out in the validation checks on our side, so it makes sense to store the data and just double check on our end16:14
*** makowals has quit IRC16:14
TheJuliaThe task at hand seems fairly clear actually, Anyone up for volunteering to tackle it?16:15
*** makowals has joined #openstack-meeting-516:15
mjturekTheJulia: is it just cinder driver changes?16:15
mjturekto fix it I mean16:16
TheJuliaYes16:16
mjturekI'll volunteer then, but I'm planning on just catching the errors (unless someone has a better option)16:17
*** makowals has quit IRC16:17
TheJuliaMy brain needs to digest it a little more, to be honest16:17
mjturekfair enough, we can discuss off meeting16:17
TheJuliaI think catching the errors and returning them to the "operator" makes more sense16:17
TheJuliakeeps ironic lighter weight16:17
mjturekgotcha16:18
TheJuliaThat would be a topic my brain goes back and forth on though :)16:18
TheJulia#info mjturek to look at addressing hshiina's finds on the cinder driver with regards to required parameters for connections.16:18
TheJuliaI'll try to rebase the storage interface calls wire-in before the next meeting.  With the summit coming up, I have a few things on my plate so I'm not sure I'll get to it.16:19
TheJulia#info TheJulia to rebase the storage_interface wire-in change.16:20
hshiinamjturek, TheJulia thanks.16:20
TheJuliaI think that is good unless someone has plans or priorities to convey?16:20
mjturek+1 for moving along16:21
hshiinai agree that cinder driver is the priority16:21
*** matrohon has quit IRC16:22
TheJuliaMoving along then!16:22
TheJulia#topic Discussion16:22
*** openstack changes topic to "Discussion (Meeting topic: ironic_bfv)"16:22
mjturekSo my first item was the TODO on line 271 here https://review.openstack.org/#/c/366197/39/ironic/drivers/modules/storage/cinder.py16:23
mjturekTheJulia mentions that retrieving the target from volume_id is preferred16:24
mjturekso I proposed this https://review.openstack.org/#/c/460250/16:24
TheJulia#link https://review.openstack.org/#/c/366197/39/ironic/drivers/modules/storage/cinder.py#27116:24
mjturek#link https://review.openstack.org/#/c/460250/16:24
TheJulia#undo16:25
openstackRemoving item from minutes: #link https://review.openstack.org/#/c/460250/16:25
TheJulia#undo16:25
openstackRemoving item from minutes: #link https://review.openstack.org/#/c/366197/39/ironic/drivers/modules/storage/cinder.py#27116:25
TheJuliaWell, when did that stop working :(16:26
TheJulia#link https://review.openstack.org/#/c/366197/40/ironic/drivers/modules/storage/cinder.py16:26
TheJulia#link https://review.openstack.org/#/c/460250/16:26
mjtureknot sure :(16:26
TheJuliaAnyway, looking at it16:27
TheJuliaBeing able to query directly helps get rid of the loop on 270, which makes sense16:28
mjturekcool, what about the fact that it returns a list of targets? The volume_id doesn't have a unique constraint so I figured it needs to be that way16:29
TheJuliaI have no objection to adding the db api call and updating that code accordingly, or it gets rid of the loop on 269.  Regardless, it becomes simpler and less looping16:29
mjturek\o/ alright I'll rebase the driver on top of it and use it16:30
TheJuliaYeah, that I need to think about though16:30
TheJuliaSo as long as the volumes are limited to the node the task is executing on, I think it is okay16:31
TheJuliaI mean targets, not volumes16:31
TheJuliaif the targets returned are on different nodes, then that is problematic16:31
mjturekif there's one where the node_uuid doesn't match, should it throw an exception?16:31
mjturekokay I think that answers my question lol16:32
TheJuliaheh16:32
TheJuliaYou could have multiple nodes connected to the same target though16:32
mjturekhm16:32
TheJuliabut they should have distinct records for each node in the targets16:32
mjturekhmm, alright I think I'm confused then. I'll need to think that over16:35
mjturekoh I think I get it16:35
TheJulia:)16:35
*** matrohon has joined #openstack-meeting-516:35
mjturekanyway, probably a topic for the code review :)16:36
TheJuliaYeah, Anyway, I guess we're good for set discussion topics.  Everyone ready to move along?16:36
mjtureksure16:36
hshiinayes16:37
TheJuliaExcellent!16:37
TheJulia#topic Open Discussion16:38
*** openstack changes topic to "Open Discussion (Meeting topic: ironic_bfv)"16:38
TheJuliaI have nothing else, other than I'm glad to be home :)16:38
derekhbefore I can help out on things I need to get myself familiar with whats going on,16:38
derekhif I get all the correct patches from the etherpad should I be able to boot a instance from a volume or is there more work yet to happen?16:38
derekhits always nice to be home ;-)16:38
derekhOnce I am familiar it, I'm happy to take suggestions on how to help out16:38
hshiinaderekh, i guess we need more work. there is a wip patch in ehterpad16:42
TheJuliaderekh: A little more work, mjturek was looking at getting things up and running in devstack. There is the tip patch that I posted to get feedback that needs to be finished16:42
mjturekderekh: hshiina also detailed how to try out the patches with devstack. Take a look in the devstack section\16:42
TheJuliaAlso, the nova changes, but hshiina and mjturek have been looking at that16:42
TheJuliaI feel like someone else was seeking background/context for BFV earlier in the week that could also be helpful for you derekh, but I'm completely drawing a blank on who pinged me.16:44
derekhok, I will take a look through the devstack section, to see how far I get, I'll ping should I hit any problems/questions, I only started setting up a env to try it out today so still feeling my way around16:45
* TheJulia gets out ye olde jet lag sign and just holds it above her head16:45
derekhTheJulia: it was me, I mentioned in the virt sprint that I was hoping to help out16:45
TheJuliaderekh: Awesome!16:45
TheJuliaOh, right!16:45
TheJuliaI think I was walking into a datacenter during that conversation too :)16:46
TheJuliaderekh: well, as with anything, we are just a ping away in #openstack-ironic :)16:46
derekhlol, anyways I can take a look at things and try them out, then I'll hopefully have something sensible to say next week16:46
TheJuliaExcellent!16:46
TheJuliaOut of curiosity, is anyone going to be in boston for the Summit?16:47
derekhnot me16:47
hshiinai'm not going16:47
mjturekTheJulia: I'll be there :)16:48
TheJuliaAwesome, I was just kind of curious since I'll be there as well.16:50
TheJuliaAnyway, if we have nothing else to discuss, I say we call the meeting done, and get back to our various tasks for the day.16:50
mjturekttyl all16:51
derekhttyl, thanks16:51
hshiinathanks16:51
*** ralonsoh has quit IRC16:51
TheJuliaThank you everyone!16:52
TheJulia#endmeeting16:52
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings"16:52
openstackMeeting ended Thu Apr 27 16:52:28 2017 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:52
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-04-27-16.01.html16:52
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-04-27-16.01.txt16:52
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-04-27-16.01.log.html16:52
*** derekh has quit IRC16:52
*** tiendc has quit IRC16:55
*** hshiina has left #openstack-meeting-516:56
*** ricolin has quit IRC17:26
*** iceyao has joined #openstack-meeting-517:31
*** iceyao has quit IRC17:35
*** tongli has joined #openstack-meeting-517:41
*** tongli has quit IRC18:06
*** phuongnh has quit IRC18:08
*** Daviey_ has joined #openstack-meeting-518:38
*** Daviey has quit IRC18:38
*** makowals has joined #openstack-meeting-520:11
*** iceyao has joined #openstack-meeting-520:18
*** iceyao has quit IRC20:23
*** makowals has quit IRC20:33
*** tongli has joined #openstack-meeting-520:59
*** iceyao has joined #openstack-meeting-521:19
*** tongli has quit IRC21:22
*** iceyao has quit IRC21:24
*** zigo has joined #openstack-meeting-521:32
*** iceyao has joined #openstack-meeting-521:47
*** iceyao has quit IRC21:52
*** izaakk has quit IRC22:18
*** izaakk has joined #openstack-meeting-522:21
*** iceyao has joined #openstack-meeting-522:32
*** iceyao has quit IRC22:37
*** markvoelker has quit IRC23:40

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!