*** joanna__ has quit IRC | 00:03 | |
*** ricolin has joined #openstack-meeting-5 | 00:59 | |
*** ricolin has quit IRC | 01:06 | |
*** luzC has quit IRC | 01:20 | |
*** luzC has joined #openstack-meeting-5 | 01:25 | |
*** liuyuqi has joined #openstack-meeting-5 | 01:31 | |
*** mrhillsman has left #openstack-meeting-5 | 02:18 | |
*** Rockyg has quit IRC | 02:43 | |
*** cwolferh has quit IRC | 03:08 | |
*** dmellado has quit IRC | 05:14 | |
*** zaneb has quit IRC | 05:15 | |
*** zaneb has joined #openstack-meeting-5 | 05:16 | |
*** dmellado has joined #openstack-meeting-5 | 05:16 | |
*** matrohon has joined #openstack-meeting-5 | 07:02 | |
*** ricolin has joined #openstack-meeting-5 | 07:06 | |
*** matrohon has quit IRC | 07:07 | |
*** ralonsoh has joined #openstack-meeting-5 | 07:39 | |
*** rarcea has joined #openstack-meeting-5 | 08:06 | |
*** makowals_ has joined #openstack-meeting-5 | 08:46 | |
*** makowals has quit IRC | 08:48 | |
*** ralonsoh has quit IRC | 09:04 | |
*** ralonsoh has joined #openstack-meeting-5 | 09:10 | |
*** ralonsoh_ has joined #openstack-meeting-5 | 09:12 | |
*** ralonsoh has quit IRC | 09:15 | |
*** matrohon has joined #openstack-meeting-5 | 09:21 | |
*** ralonsoh_ is now known as ralonsoh | 09:26 | |
*** matrohon has quit IRC | 10:45 | |
*** matrohon has joined #openstack-meeting-5 | 11:44 | |
*** krtaylor has joined #openstack-meeting-5 | 12:54 | |
*** jlvillal is now known as jlvillal_pto | 13:35 | |
*** matrohon has quit IRC | 14:00 | |
*** matrohon has joined #openstack-meeting-5 | 14:22 | |
*** mjturek has joined #openstack-meeting-5 | 15:18 | |
*** hshiina has joined #openstack-meeting-5 | 15:54 | |
*** phuongnh has joined #openstack-meeting-5 | 15:58 | |
TheJulia | #startmeeting ironic_bfv | 16:00 |
---|---|---|
TheJulia | o/ | 16:00 |
openstack | Meeting 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 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
*** openstack changes topic to " (Meeting topic: ironic_bfv)" | 16:00 | |
openstack | The meeting name has been set to 'ironic_bfv' | 16:00 |
hshiina | o/ | 16:00 |
TheJulia | Good morning, afternoon, evening everyone! | 16:00 |
TheJulia | Our agenda today has a few discussion items \o/ | 16:00 |
TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic-BFV | 16:01 |
TheJulia | #topic Announcements/Reminders | 16:01 |
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic_bfv)" | 16:01 | |
TheJulia | I have none at the moment, does anyone have anything they would like to make a note of today? | 16:01 |
hshiina | i have nothing | 16:02 |
TheJulia | Okay then, moving on! | 16:02 |
TheJulia | #topic Current Status | 16:02 |
*** openstack changes topic to "Current Status (Meeting topic: ironic_bfv)" | 16:02 | |
TheJulia | I see we have a number of notes in the etherpad today \o/ | 16:03 |
TheJulia | #link https://etherpad.openstack.org/p/Ironic-BFV | 16:03 |
TheJulia | Looks like the cinder interface has some feedback that might need to be addressed | 16:04 |
*** joanna__ has joined #openstack-meeting-5 | 16:04 | |
TheJulia | And there is some ongoing discussion in https://review.openstack.org/#/c/366197/ | 16:05 |
joanna | o/ | 16:05 |
joanna | sorry I am late. | 16:05 |
TheJulia | o/ joanna | 16:05 |
TheJulia | No worries, we were just taking a minute to review the etherpad. | 16:05 |
joanna | I'll update the interface patch | 16:06 |
TheJulia | I updated the status on the etherpad for the cinder common code revision, there is feedback on it. | 16:06 |
TheJulia | Awesome, Thank you joanna | 16:06 |
TheJulia | mjturek: I see you've left some notes, can you give us a quick summary? | 16:06 |
joanna | I'm wondering if we should handle this in attach, or rather fail and let the caller call detach | 16:07 |
*** matrohon has quit IRC | 16:07 | |
TheJulia | That might make more sense | 16:08 |
TheJulia | Anyway, I guess mjturek is busy right now, so we can move on. | 16:09 |
mjturek | TheJulia I'm sorry - on a call will be on ASAP | 16:09 |
joanna | or maybe call abort and do the cleanup in abort, se we return in somewhat clean state. I'll think about it :) | 16:09 |
TheJulia | mjturek: no worries! | 16:10 |
TheJulia | #topic Planning/Priorities | 16:10 |
*** openstack changes topic to "Planning/Priorities (Meeting topic: ironic_bfv)" | 16:10 | |
TheJulia | It 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 |
TheJulia | joanna: Any idea on timing for the interface revision? | 16:11 |
joanna | ~3hours? | 16:11 |
TheJulia | awesome | 16:12 |
joanna | :) | 16:12 |
TheJulia | My 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 |
TheJulia | Aside from that, I guess we're good to move into discussion, unless someone has questions relating to priority or planning? | 16:13 |
hshiina | i aggree the plan and have no question | 16:14 |
TheJulia | Excellent | 16:14 |
TheJulia | Moving on then.... | 16:14 |
TheJulia | #topic Discussion | 16:14 |
*** openstack changes topic to "Discussion (Meeting topic: ironic_bfv)" | 16:14 | |
* mjturek is back | 16:16 | |
TheJulia | Today 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 the | 16:16 |
TheJulia | original issue of MAC -> IP lookups? | 16:16 |
hshiina | TheJulia, yes | 16:17 |
mjturek | TheJulia: by cinder common code do you mean os_block? | 16:17 |
mjturek | or is that separate? | 16:17 |
mjturek | os-brick* | 16:17 |
TheJulia | mjturek: the ironic cinder common code | 16:17 |
mjturek | oh! | 16:17 |
mjturek | sorry understood | 16:17 |
TheJulia | mjturek: no worries | 16:18 |
joanna | TheJulia, hshiina: the comments are in the driver patch | 16:18 |
TheJulia | hshiina: 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 workflow | 16:21 |
TheJulia | Ahh, I also see the note about it might be difficult to just somehow skip it :\ | 16:22 |
TheJulia | So we could possibly need the info in both places. :( | 16:22 |
TheJulia | Well, that kind of renders that discussion pointless until we have more clarity. | 16:22 |
TheJulia | I guess we should move on then? | 16:23 |
joanna | TheJulia: I have a question about mac | 16:23 |
hshiina | TheJulia, yes. i will add some memo to etherpad later | 16:23 |
hshiina | TheJulia, about this issue | 16:23 |
joanna | since spec says it's valid - what is the actual reason behind mac to ip conversion | 16:24 |
TheJulia | if 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 |
TheJulia | hshiina: Awesome, thank you | 16:25 |
TheJulia | joanna: does that answer your question? | 16:26 |
TheJulia | Possibly 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 |
joanna | TheJulia: yes. Why does spec include mac support? | 16:26 |
TheJulia | I 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 |
joanna | thanks :)! | 16:28 |
TheJulia | Moving on, the next item is... | 16:28 |
TheJulia | Cinder storage interface should_write_image() logic and deployment skip logic addition | 16:28 |
TheJulia | hshiina 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 |
hshiina | no objection, thanks | 16:31 |
TheJulia | Awesome | 16:32 |
TheJulia | In that case since, our last item is raised by hshiina, wondering if we should split apart the API patches. | 16:32 |
TheJulia | From 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 |
TheJulia | hshiina: Is your reasoning to split it just over review velocity? | 16:33 |
hshiina | TheJulia, yes. I have heard large patch is difficult to review in some other patches. | 16:34 |
joanna | hshiina: what is the proposed split? Volume and VolumeConnector? | 16:35 |
joanna | I think we should push it to the priority list for the next week, this can solve the issue of lack of reviews | 16:35 |
hshiina | joanna, VolumeConnector and VolumeTarget | 16:35 |
TheJulia | I agree with joanna | 16:36 |
mjturek | seems doable | 16:36 |
joanna | I volunteer as a reviewer :) will look at that later today or tomorrow | 16:37 |
TheJulia | That 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 |
TheJulia | A 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 |
hshiina | TheJulia, thanks | 16:39 |
TheJulia | Anything else on this topic? | 16:39 |
TheJulia | #undo | 16:40 |
openstack | Removing 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 Discussion | 16:41 |
*** openstack changes topic to "Open Discussion (Meeting topic: ironic_bfv)" | 16:41 | |
mjturek | so I can give a summary of devstack investigation | 16:41 |
TheJulia | mjturek: that would be awesome :) | 16:41 |
mjturek | basically 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 there | 16:42 |
joanna | just 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 perspective | 16:42 |
mjturek | what I think it's going to come down to is just setting the configs for a boot from volume tempest test | 16:42 |
mjturek | I believe all the config changes are in the cinder common code patch? | 16:43 |
joanna | mjturek: yes, the config will include additional fields | 16:43 |
mjturek | anyways, I also linked to the tempest test that nova does which we'll probably want to do | 16:44 |
joanna | do we already have devstack with cinder plugin? | 16:44 |
mjturek | well ironic and cinder stack together fine | 16:44 |
mjturek | but I haven't tried with the patches | 16:44 |
mjturek | hshiina: it seems like you might've already though? | 16:44 |
hshiina | mjturek, i tried devstack with the patches. | 16:45 |
mjturek | cool, and nothing strange there? | 16:45 |
hshiina | mjturek, stack.sh succeeded | 16:45 |
mjturek | nice! | 16:45 |
mjturek | anyway, 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 test | 16:46 |
joanna | hshiina: awesome :) | 16:46 |
TheJulia | mjturek: 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 |
mjturek | cool cool | 16:47 |
TheJulia | Awesome, sounds like we're on a good path. Anything else to discuss or shall we all get ten minutes back? | 16:48 |
hshiina | i have nothing to discuss | 16:48 |
mjturek | good for now | 16:49 |
joanna | i'm good | 16:49 |
TheJulia | Awesome! Thank you everyone, and have a wonderful rest of the day! | 16:50 |
hshiina | thank you | 16:50 |
joanna | thank you, TheJulia, hshiina, mjturek :) | 16:50 |
TheJulia | #endmeeting | 16:50 |
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings" | 16:50 | |
openstack | Meeting ended Thu Mar 30 16:50:37 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:50 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-03-30-16.00.html | 16:50 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-03-30-16.00.txt | 16:50 |
openstack | Log: http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-03-30-16.00.log.html | 16:50 |
*** ralonsoh has quit IRC | 17:04 | |
*** phuongnh has quit IRC | 17:40 | |
*** TheJulia_ has joined #openstack-meeting-5 | 17:56 | |
*** hogepodge_ has joined #openstack-meeting-5 | 17:57 | |
*** TheJulia has quit IRC | 17:59 | |
*** clarkb has quit IRC | 17:59 | |
*** hogepodge has quit IRC | 17:59 | |
*** hogepodge_ is now known as hogepodge | 18:00 | |
*** clarkb has joined #openstack-meeting-5 | 18:01 | |
*** ricolin has quit IRC | 18:13 | |
*** hshiina has quit IRC | 18:53 | |
*** TheJulia_ is now known as TheJulia | 18:56 | |
*** Rockyg has joined #openstack-meeting-5 | 18:56 | |
*** joanna__ has quit IRC | 18:57 | |
*** joanna__ has joined #openstack-meeting-5 | 19:04 | |
*** joanna__ has quit IRC | 19:08 | |
*** joanna__ has joined #openstack-meeting-5 | 19:17 | |
*** joanna__ has quit IRC | 19:20 | |
*** joanna__ has joined #openstack-meeting-5 | 19:31 | |
*** joanna__ has quit IRC | 19:34 | |
*** joanna__ has joined #openstack-meeting-5 | 19:50 | |
*** joanna__ has quit IRC | 19:55 | |
*** joanna__ has joined #openstack-meeting-5 | 19:59 | |
*** rarcea has quit IRC | 20:46 | |
*** joanna__ has quit IRC | 21:19 | |
*** joanna__ has joined #openstack-meeting-5 | 21:23 | |
*** Rockyg has quit IRC | 21:50 | |
*** bzhao_ has joined #openstack-meeting-5 | 22:40 | |
*** joanna__ has quit IRC | 23:25 | |
*** openstack has joined #openstack-meeting-5 | 23:45 | |
*** ChanServ sets mode: +o openstack | 23:45 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!