Thursday, 2017-03-23

*** ricolin has joined #openstack-meeting-501:35
*** jtaryma has joined #openstack-meeting-501:42
*** bzhao_ has joined #openstack-meeting-502:58
*** markvoelker has quit IRC05:08
*** markvoelker has joined #openstack-meeting-505:09
*** markvoelker has quit IRC05:13
*** irenab has joined #openstack-meeting-506:04
*** tonyb_ has joined #openstack-meeting-506:06
*** markvoelker has joined #openstack-meeting-506:09
*** tonyb has quit IRC06:13
*** markvoelker has quit IRC06:13
*** joanna has quit IRC06:35
*** jtaryma is now known as joanna06:35
*** bzhao_ has quit IRC06:38
*** dmellado has quit IRC07:03
*** dmellado has joined #openstack-meeting-507:08
*** matrohon has joined #openstack-meeting-507:53
*** kbaikov has quit IRC07:59
*** markvoelker has joined #openstack-meeting-508:10
*** kbaikov has joined #openstack-meeting-508:12
*** rarcea has joined #openstack-meeting-508:12
*** markvoelker has quit IRC08:14
*** matrohon has quit IRC08:20
*** matrohon has joined #openstack-meeting-508:35
*** ralonsoh has joined #openstack-meeting-508:43
*** ricolin has quit IRC08:46
*** ricolin has joined #openstack-meeting-508:49
*** makowals_ has quit IRC09:00
*** makowals has joined #openstack-meeting-509:01
*** jrist has quit IRC09:12
*** jrist has joined #openstack-meeting-509:13
*** matrohon has quit IRC09:42
*** matrohon has joined #openstack-meeting-509:43
*** HelenYao has quit IRC09:57
*** markvoelker has joined #openstack-meeting-510:11
*** markvoelker has quit IRC10:16
*** HelenYao has joined #openstack-meeting-510:29
*** makowals has quit IRC10:38
*** makowals has joined #openstack-meeting-510:43
*** matrohon has quit IRC11:16
*** joanna has quit IRC11:36
*** markvoelker has joined #openstack-meeting-512:13
*** markvoelker has quit IRC12:17
*** matrohon has joined #openstack-meeting-512:38
*** markvoelker has joined #openstack-meeting-512:46
*** kbaikov has quit IRC13:42
*** krtaylor has joined #openstack-meeting-513:43
*** matrohon has quit IRC15:14
*** matrohon has joined #openstack-meeting-515:15
*** joanna has joined #openstack-meeting-515:28
*** ttx has quit IRC15:44
*** jtaryma has joined #openstack-meeting-515:45
*** ttx has joined #openstack-meeting-515:46
*** shasha_t_ has joined #openstack-meeting-515:48
*** crushil has joined #openstack-meeting-515:54
*** hshiina has joined #openstack-meeting-515:56
*** dtantsur has joined #openstack-meeting-515:57
TheJulia#startmeeting ironic_bfv16:00
openstackMeeting started Thu Mar 23 16:00:42 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
TheJuliao/16:00
openstackThe meeting name has been set to 'ironic_bfv'16:00
joannao/16:00
hshiinao/16:00
TheJuliaGood morning everyone16:00
crushil\o16:00
*** mjturek has joined #openstack-meeting-516:01
TheJuliaOur agenda for this week, which is minimal (You folks know your welcome to edit it right?) :)16:01
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic-BFV16:01
mjtureko/16:01
TheJulia#topic Announcements/Reminders16:02
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic_bfv)"16:02
TheJuliaAs mentioned last week, I created an etherpad for tracking the items related to BFV.16:02
TheJulia#link https://etherpad.openstack.org/p/Ironic-BFV16:03
TheJuliaAny other items for announcement?16:03
joannathank Julia! it made things easier16:03
dtantsuro/16:03
dtantsurplease make sure the etherpad is linked from the main whiteboard16:04
TheJulia#action TheJulia to check/update links for etherpads16:04
TheJuliadtantsur: I think I already had, but just in case ^^^ :)16:04
TheJuliaMoving on....16:04
TheJulia#topic Current Status16:05
*** openstack changes topic to "Current Status (Meeting topic: ironic_bfv)"16:05
joannadtantsur, TheJulia : it is in subtem status section under BFV16:05
TheJuliaLooks like joanna has updated the cinder common interface revision, but hit an unrelated ci failure.16:06
joannaYes, I requested recheck.16:06
*** matrohon has quit IRC16:06
mjturekhaven't had a lot of time this week to investigate devstack this week :( but planning to make time for it this week, so don't have much of a status16:06
joannaDepending patches will need to be updated too, probably it will take some time due to similar issue in tests as in cinder api16:07
TheJuliamjturek: Awesome, thank you16:07
TheJuliajoanna: sorry :(16:07
TheJuliahshiina: Have you had time to look into moving connection document logic out of nova, due in part to the need for the ip address?16:08
hshiinai read nova code. it seems difficult to move all logic to ironic.16:08
joannaTheJulia: no problem at all, don't worry16:08
TheJulia#info mjturek will try to look at devstack stuffs next week.16:09
TheJuliahshiina: Seems like we should take that topic to discussion then.16:09
hshiinanow, i updated nova patch to get ip address in nova side by calling vif attach API before creating connection information.16:09
TheJuliajoanna: Are you going to begin updating the dependent patches?  If you already said you were going to, I didn't pick up on it16:09
joannaI will :)16:10
TheJuliahshiina: That... might... just... work16:10
joannaI just wanted to give everyone heads up that it won't be within next couple of hours :)16:10
TheJulia#info joanna to continue to update the ironic in-tree revisions.16:10
TheJuliajoanna: Awesome16:10
TheJuliaSo shall we move on?16:11
joanna+16:11
hshiinasure16:12
TheJulia#topic Planning/Priorities16:12
*** openstack changes topic to "Planning/Priorities (Meeting topic: ironic_bfv)"16:12
TheJuliaI don't think we have anything to really discuss here, so if anyone has anything then awesome, otherwise, I'll just move us onward to discussion16:13
* TheJulia hears crickets16:14
TheJulia#topic Discussion16:14
*** openstack changes topic to "Discussion (Meeting topic: ironic_bfv)"16:14
joannaI have one item. Mario left a review yesterday with a question on what is a desired/best behavior in attach_volumes if one attachment fails16:15
TheJuliajoanna: imho, attachment should fail, power-on should be prevented16:16
joannacurrently it will raise an exception and leave the cleanup responsibility to the caller (which is fine as detach_volumes can take the same list as a param)16:16
joannahis concern was, if we're attaching 10 volumes and only 1 fails... I think that it's good to fail, since we shouldn't deliver half-desired outcome16:17
TheJuliaI think it is fine for the caller to do the clean-up, is the thought to have the common code handle all of the failure cleanup?16:17
TheJuliahshiina: It looks like your uploaded patchset doesn't merge, perhaps time to rebase the nova patchset?16:18
mjturekTheJulia: you said the move of connection document logic was only in part needed for the IP problem (which if I'm understanding, was solved by hshiina's current solution?). What else was it required for?16:18
joannaTheJulia: no, caller already does that in other patch, I answered that to Mario in my reply. The thing was rather about failing all attachments in case of 1 failure16:19
TheJuliamjturek: Mac addresses, although I'm not sure that is a hard requirement.16:19
hshiinai guess the nova conflict is caused by Depend-on: ironic api patch, which has a conflict.16:19
joannagiven examples where failing volume is a 1st one and a 10th one. But detach will handle both cases.16:19
mjturekahh16:19
TheJuliamjturek: possibly worth digging into the the common cinder drivers to see what they "really" want for their connection parameters16:19
joannaTheJulia: It looks like we agree on that. I will leave it as-is, with excepion being raised no matter which volume failed16:20
mjturekTheJulia: yeah sounds like a good path16:20
TheJuliaAnyone want to volunteer to skim the cinder drivers to see if they care about/want a mac address?16:20
mjtureko/16:21
mjtureksounds interesting to me16:21
TheJulia#info mjturek volunteers to skim through the cinder drivers to validate if they want/need mac addresses in their connection information.16:21
TheJuliamjturek: awesome16:21
mjturekthanks TheJulia16:21
TheJuliahshiina: going back to the depends-on, that would do it.16:22
TheJuliaSo there is only one other item in my head right now.  If there are no objections, I'm just going merge discussion/open discussion on the agenda.16:22
mjturekyeah sounds good16:23
hshiinai agree16:23
TheJuliaAwesome!16:23
TheJuliaAnything else this week?16:23
mjtureki'm good16:24
joannai'm good too :)16:25
hshiinai'm good16:25
TheJuliaExcellent!  Thank you everyone!16:25
TheJuliaHave a wonderful rest of the week and talk to you next week!16:25
TheJulia#endmeeting16:25
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings"16:25
openstackMeeting ended Thu Mar 23 16:25:39 2017 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:25
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-03-23-16.00.html16:25
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-03-23-16.00.txt16:25
hshiinathanks16:25
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-03-23-16.00.log.html16:25
*** mjturek has left #openstack-meeting-516:25
*** dtantsur has left #openstack-meeting-516:25
*** crushil has left #openstack-meeting-516:26
*** matrohon has joined #openstack-meeting-516:26
*** hshiina has left #openstack-meeting-516:26
*** krtaylor has quit IRC17:42
*** ralonsoh has quit IRC18:04
*** matrohon has quit IRC18:08
*** joanna has quit IRC19:03
*** jtaryma is now known as joanna19:03
*** krtaylor has joined #openstack-meeting-519:15
*** matrohon has joined #openstack-meeting-519:32
*** rarcea has quit IRC20:08
*** jtaryma has joined #openstack-meeting-520:23
*** krtaylor has quit IRC21:08
*** matrohon has quit IRC21:27
*** jtaryma has quit IRC21:37
*** jtaryma has joined #openstack-meeting-521:47
*** jtaryma is now known as joanna_mac22:34
*** joanna_mac has quit IRC23:19

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