Thursday, 2017-03-30

*** joanna__ has quit IRC00:03
*** ricolin has joined #openstack-meeting-500:59
*** ricolin has quit IRC01:06
*** luzC has quit IRC01:20
*** luzC has joined #openstack-meeting-501:25
*** liuyuqi has joined #openstack-meeting-501:31
*** mrhillsman has left #openstack-meeting-502:18
*** Rockyg has quit IRC02:43
*** cwolferh has quit IRC03:08
*** dmellado has quit IRC05:14
*** zaneb has quit IRC05:15
*** zaneb has joined #openstack-meeting-505:16
*** dmellado has joined #openstack-meeting-505:16
*** matrohon has joined #openstack-meeting-507:02
*** ricolin has joined #openstack-meeting-507:06
*** matrohon has quit IRC07:07
*** ralonsoh has joined #openstack-meeting-507:39
*** rarcea has joined #openstack-meeting-508:06
*** makowals_ has joined #openstack-meeting-508:46
*** makowals has quit IRC08:48
*** ralonsoh has quit IRC09:04
*** ralonsoh has joined #openstack-meeting-509:10
*** ralonsoh_ has joined #openstack-meeting-509:12
*** ralonsoh has quit IRC09:15
*** matrohon has joined #openstack-meeting-509:21
*** ralonsoh_ is now known as ralonsoh09:26
*** matrohon has quit IRC10:45
*** matrohon has joined #openstack-meeting-511:44
*** krtaylor has joined #openstack-meeting-512:54
*** jlvillal is now known as jlvillal_pto13:35
*** matrohon has quit IRC14:00
*** matrohon has joined #openstack-meeting-514:22
*** mjturek has joined #openstack-meeting-515:18
*** hshiina has joined #openstack-meeting-515:54
*** phuongnh has joined #openstack-meeting-515:58
TheJulia#startmeeting ironic_bfv16:00
TheJuliao/16:00
openstackMeeting started Thu Mar 30 16:00:03 2017 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.16:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
*** openstack changes topic to " (Meeting topic: ironic_bfv)"16:00
openstackThe meeting name has been set to 'ironic_bfv'16:00
hshiinao/16:00
TheJuliaGood morning, afternoon, evening everyone!16:00
TheJuliaOur agenda today has a few discussion items \o/16:00
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic-BFV16:01
TheJulia#topic Announcements/Reminders16:01
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic_bfv)"16:01
TheJuliaI have none at the moment, does anyone have anything they would like to make a note of today?16:01
hshiinai have nothing16:02
TheJuliaOkay then, moving on!16:02
TheJulia#topic Current Status16:02
*** openstack changes topic to "Current Status (Meeting topic: ironic_bfv)"16:02
TheJuliaI see we have a number of notes in the etherpad today \o/16:03
TheJulia#link https://etherpad.openstack.org/p/Ironic-BFV16:03
TheJuliaLooks like the cinder interface has some feedback that might need to be addressed16:04
*** joanna__ has joined #openstack-meeting-516:04
TheJuliaAnd there is some ongoing discussion in https://review.openstack.org/#/c/366197/16:05
joannao/16:05
joannasorry I am late.16:05
TheJuliao/ joanna16:05
TheJuliaNo worries, we were just taking a minute to review the etherpad.16:05
joannaI'll update the interface patch16:06
TheJuliaI updated the status on the etherpad for the cinder common code revision, there is feedback on it.16:06
TheJuliaAwesome, Thank you joanna16:06
TheJuliamjturek: I see you've left some notes, can you give us a quick summary?16:06
joannaI'm wondering if we should handle this in attach, or rather fail and let the caller call detach16:07
*** matrohon has quit IRC16:07
TheJuliaThat might make more sense16:08
TheJuliaAnyway, I guess mjturek is busy right now, so we can move on.16:09
mjturekTheJulia I'm sorry - on a call will be on ASAP16:09
joannaor maybe call abort and do the cleanup in abort, se we return in somewhat clean state. I'll think about it :)16:09
TheJuliamjturek: no worries!16:10
TheJulia#topic Planning/Priorities16:10
*** openstack changes topic to "Planning/Priorities (Meeting topic: ironic_bfv)"16:10
TheJuliaIt appears that mjturek has been looking at devstack stuffs, and continues to do so.  That is a relatively high priority for our velocity.16:10
TheJuliajoanna: Any idea on timing for the interface revision?16:11
joanna~3hours?16:11
TheJuliaawesome16:12
joanna:)16:12
TheJuliaMy one ask is that everyone takes some time out before mid next week to review https://review.openstack.org/#/c/355625/ which is the cinder common code.16:12
TheJulia#link https://review.openstack.org/#/c/355625/16:12
TheJuliaAside from that, I guess we're good to move into discussion, unless someone has questions relating to priority or planning?16:13
hshiinai aggree the plan and have no question16:14
TheJuliaExcellent16:14
TheJuliaMoving on then....16:14
TheJulia#topic Discussion16:14
*** openstack changes topic to "Discussion (Meeting topic: ironic_bfv)"16:14
* mjturek is back16:16
TheJuliaToday we have three items, the first is the question of the MAC address to IP lookup.  This is raised from a question hshiina raised on, I believe the cinder common code since it doesn't support that presently, and I believe we need to address that in ironic since we can't in nova. hshiina I see you also mentioned the nova patch and possible solutions to the16:16
TheJuliaoriginal issue of MAC -> IP lookups?16:16
hshiinaTheJulia, yes16:17
mjturekTheJulia: by cinder common code do you mean os_block?16:17
mjturekor is that separate?16:17
mjturekos-brick*16:17
TheJuliamjturek: the ironic cinder common code16:17
mjturekoh!16:17
mjtureksorry understood16:17
TheJuliamjturek: no worries16:18
joannaTheJulia, hshiina: the comments are in the driver patch16:18
TheJuliahshiina: Looking at the ironic nova driver comments, I would think the attach could only actually take place after the node the node has been spawned in ironic, or at least in process, which makes me think the best thing to do is still let ironic handle the attachments in it's workflow16:21
TheJuliaAhh, I also see the note about it might be difficult to just somehow skip it :\16:22
TheJuliaSo we could possibly need the info in both places. :(16:22
TheJuliaWell, that kind of renders that discussion pointless until we have more clarity.16:22
TheJuliaI guess we should move on then?16:23
joannaTheJulia: I have a question about mac16:23
hshiinaTheJulia, yes. i will add some memo to etherpad later16:23
hshiinaTheJulia, about this issue16:23
joannasince spec says it's valid - what is the actual reason behind mac to ip conversion16:24
TheJuliaif I define a mac to be part of a storage network, I don't know it's IP until I deploy.  It would be a reasonable way to preserve the fact that it is valid in the hardware configuration.16:25
TheJuliahshiina: Awesome, thank you16:25
TheJuliajoanna: does that answer your question?16:26
TheJuliaPossibly the other context is that the connector intent was to be as generic as possible to provide storage as much connector information as possible.16:26
joannaTheJulia: yes. Why does spec include mac support?16:26
TheJuliaI believe it is more-so you can define "this network card is always part of the storage fabric, lean towards using it" concept, but I honestly don't remember at this point.16:27
joannathanks :)!16:28
TheJuliaMoving on, the next item is...16:28
TheJuliaCinder storage interface should_write_image() logic and deployment skip logic addition16:28
TheJuliahshiina pointed out, i believe in the ipxe template patch, that the logic for that had not yet been proposed. I can likely do it fairly quickly in the next week or so. I just think it should be an additional patch on top of the ipxe template change.  If there are no objections, I'll try and put that together before next thursday.16:30
hshiinano objection, thanks16:31
TheJuliaAwesome16:32
TheJuliaIn that case since, our last item is raised by hshiina, wondering if we should split apart the API patches.16:32
TheJuliaFrom an ironic review standpoint, it is rather painful to get two independent API changes in in order since each one results in a micro version bump.16:33
TheJuliahshiina: Is your reasoning to split it just over review velocity?16:33
hshiinaTheJulia, yes. I have heard large patch is difficult to review in some other patches.16:34
joannahshiina: what is the proposed split? Volume and VolumeConnector?16:35
joannaI think we should push it to the priority list for the next week, this can solve the issue of lack of reviews16:35
hshiinajoanna, VolumeConnector and VolumeTarget16:35
TheJuliaI agree with joanna16:36
mjturekseems doable16:36
joannaI volunteer as a reviewer :) will look at that later today or tomorrow16:37
TheJuliaThat way we get some of the later review/nitpicking out of the way, of course, the changes have functionally been up for a long time.  It just seems like we should be able to review and agree if there is more work required in that patch.16:37
TheJuliaA single api change will be much easier to land overall, so I don't think we should split them apart.16:37
TheJulia#action TheJulia will raise it in the ironic meeting on monday.16:39
hshiinaTheJulia, thanks16:39
TheJuliaAnything else on this topic?16:39
TheJulia#undo16:40
openstackRemoving item from minutes: #action TheJulia will raise it in the ironic meeting on monday.16:40
TheJulia#action TheJulia will raise the API patch in the ironic meeting on monday for reviews.16:40
* TheJulia hears crickets :)16:40
TheJulia#topic Open Discussion16:41
*** openstack changes topic to "Open Discussion (Meeting topic: ironic_bfv)"16:41
mjturekso I can give a summary of devstack investigation16:41
TheJuliamjturek: that would be awesome :)16:41
mjturekbasically I've been looking for anything in virtualbmc or the libvirt guests that we'd need to do differently. I don't forsee any changes there16:42
joannajust FYI: I looked at the sequence diagram shown by cinder team, that shows async communication (the one that caused the discussion an -1s on the interface patch). Nova uses the same client, which makes sync REST call, so everything is sync from our perspective16:42
mjturekwhat I think it's going to come down to is just setting the configs for a boot from volume tempest test16:42
mjturekI believe all the config changes are in the cinder common code patch?16:43
joannamjturek: yes, the config will include additional fields16:43
mjturekanyways, I also linked to the tempest test that nova does which we'll probably want to do16:44
joannado we already have devstack with cinder plugin?16:44
mjturekwell ironic and cinder stack together fine16:44
mjturekbut I haven't tried with the patches16:44
mjturekhshiina: it seems like you might've already though?16:44
hshiinamjturek, i tried devstack with the patches.16:45
mjturekcool, and nothing strange there?16:45
hshiinamjturek, stack.sh succeeded16:45
mjtureknice!16:45
mjturekanyway, I want to try that myself this week and see if I can get devstack to be ready to go for something similar to that tempest test16:46
joannahshiina: awesome :)16:46
TheJuliamjturek: I think we're just missing one vital piece to the puzzle, but I'll try to get that hammered out (the missing skip deployment check, which shouldn't be difficult to take care of for me in the next few days)16:47
mjturekcool cool16:47
TheJuliaAwesome, sounds like we're on a good path.  Anything else to discuss or shall we all get ten minutes back?16:48
hshiinai have nothing to discuss16:48
mjturekgood for now16:49
joannai'm good16:49
TheJuliaAwesome!  Thank you everyone, and have a wonderful rest of the day!16:50
hshiinathank you16:50
joannathank you, TheJulia, hshiina, mjturek :)16:50
TheJulia#endmeeting16:50
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings"16:50
openstackMeeting ended Thu Mar 30 16:50:37 2017 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:50
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-03-30-16.00.html16:50
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-03-30-16.00.txt16:50
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-03-30-16.00.log.html16:50
*** ralonsoh has quit IRC17:04
*** phuongnh has quit IRC17:40
*** TheJulia_ has joined #openstack-meeting-517:56
*** hogepodge_ has joined #openstack-meeting-517:57
*** TheJulia has quit IRC17:59
*** clarkb has quit IRC17:59
*** hogepodge has quit IRC17:59
*** hogepodge_ is now known as hogepodge18:00
*** clarkb has joined #openstack-meeting-518:01
*** ricolin has quit IRC18:13
*** hshiina has quit IRC18:53
*** TheJulia_ is now known as TheJulia18:56
*** Rockyg has joined #openstack-meeting-518:56
*** joanna__ has quit IRC18:57
*** joanna__ has joined #openstack-meeting-519:04
*** joanna__ has quit IRC19:08
*** joanna__ has joined #openstack-meeting-519:17
*** joanna__ has quit IRC19:20
*** joanna__ has joined #openstack-meeting-519:31
*** joanna__ has quit IRC19:34
*** joanna__ has joined #openstack-meeting-519:50
*** joanna__ has quit IRC19:55
*** joanna__ has joined #openstack-meeting-519:59
*** rarcea has quit IRC20:46
*** joanna__ has quit IRC21:19
*** joanna__ has joined #openstack-meeting-521:23
*** Rockyg has quit IRC21:50
*** bzhao_ has joined #openstack-meeting-522:40
*** joanna__ has quit IRC23:25
*** openstack has joined #openstack-meeting-523:45
*** ChanServ sets mode: +o openstack23:45

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