Thursday, 2017-06-08

*** ricolin has joined #openstack-meeting-502:08
*** gomarivera has joined #openstack-meeting-502:09
*** eanylin has quit IRC02:18
*** gomarivera has quit IRC02:27
*** Rockyg has joined #openstack-meeting-502:37
*** Rockyg has quit IRC02:37
*** gomarivera has joined #openstack-meeting-502:44
*** eanylin has joined #openstack-meeting-503:00
*** gomarivera has quit IRC03:30
*** gomarivera has joined #openstack-meeting-503:31
*** gomarivera has joined #openstack-meeting-503:32
*** ramishra has joined #openstack-meeting-503:43
*** ramishra has left #openstack-meeting-503:45
*** gomarivera has quit IRC03:56
*** gomarivera has joined #openstack-meeting-503:57
*** gomarivera has quit IRC04:01
*** LanceHaig has joined #openstack-meeting-504:30
*** LanceHaig has joined #openstack-meeting-504:30
*** LanceHaig has quit IRC05:14
*** ricolin_ has joined #openstack-meeting-505:15
*** ricolin has quit IRC05:15
*** ricolin_ has quit IRC05:19
*** ricolin has joined #openstack-meeting-505:23
*** ricolin has quit IRC05:26
*** gomarivera has joined #openstack-meeting-505:27
*** ricolin has joined #openstack-meeting-505:28
*** ricolin has quit IRC05:32
*** rarcea has joined #openstack-meeting-505:34
*** LanceHaig has joined #openstack-meeting-505:48
*** makowals has joined #openstack-meeting-506:35
*** gomarivera has quit IRC06:41
*** gomarivera has joined #openstack-meeting-506:42
*** gomarivera has quit IRC06:47
*** LanceHaig has quit IRC07:38
*** ralonsoh has joined #openstack-meeting-507:48
*** ralonsoh has quit IRC07:49
*** ralonsoh has joined #openstack-meeting-507:56
*** LanceHaig has joined #openstack-meeting-507:57
*** LanceHaig has quit IRC07:57
*** LanceHaig has joined #openstack-meeting-507:57
*** LanceHaig has quit IRC08:52
*** tiantian has left #openstack-meeting-509:36
*** markvoelker has quit IRC10:27
*** markvoelker has joined #openstack-meeting-510:27
*** dougbtv_ has quit IRC11:08
*** gomarivera has joined #openstack-meeting-511:43
*** gomarivera has quit IRC11:47
*** krtaylor has quit IRC12:25
*** derekh has joined #openstack-meeting-512:31
*** krtaylor has joined #openstack-meeting-512:38
*** ricolin has joined #openstack-meeting-512:45
*** jroll has left #openstack-meeting-512:51
*** eanylin has quit IRC13:01
*** eanylin has joined #openstack-meeting-513:21
*** gomarivera has joined #openstack-meeting-513:36
*** gomarivera has joined #openstack-meeting-513:37
*** krtaylor has quit IRC14:01
*** felipemonteiro_ has joined #openstack-meeting-514:18
*** felipemonteiro__ has joined #openstack-meeting-514:30
*** felipemonteiro_ has quit IRC14:34
*** makowals has quit IRC15:34
*** makowals has joined #openstack-meeting-515:35
*** bzhao_ has joined #openstack-meeting-515:35
*** tiendc has joined #openstack-meeting-515:53
*** hshiina has joined #openstack-meeting-515:56
*** phuongnh has joined #openstack-meeting-515:59
TheJuliao/16:00
*** gomarivera has quit IRC16:01
hshiinao/16:01
TheJulia#startmeeting ironic_bfv16:01
openstackMeeting started Thu Jun  8 16:01:50 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
openstackThe meeting name has been set to 'ironic_bfv'16:01
TheJuliaGood morning everyone!16:02
derekho/16:02
TheJuliaOur agenda for today16:02
TheJulia#link 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 believe this is the week of Pike-2, so we definitely need to start getting some reviews and some of the code landed soon.16:03
hshiinathat's right16:03
*** mjturek has joined #openstack-meeting-516:04
mjtureko/16:04
mjturekhey everyone :)16:04
TheJuliaI and mjturek rebased/updated some of the patches to try and get that cleaned up.  Hopefully later today well have patches with clean ci runs ready for review16:04
* TheJulia should have said that backwards, but *shrug*16:04
TheJuliaThat being said, that is also current status :)16:05
mjturekyeaaah, screwed up py27/35 and pep8 in a couple places. Will have fixed shortly16:05
TheJulia#info Reviewers, please consult the list of revisions at https://etherpad.openstack.org/p/Ironic-BFV16:05
TheJuliaI guess we should move on to current status unless anyone has anything to announce?16:06
* TheJulia thinks she has crickets, or everyone is watching US congressional testimony16:06
mjturekoh - I probably can't make next week's meeting, but I will try!16:07
TheJuliamjturek: Okay, good to know, thanks!16:07
TheJuliaMoving on!16:07
TheJulia#topic Current Status16:07
*** openstack changes topic to "Current Status (Meeting topic: ironic_bfv)"16:07
TheJuliaOur etherpad is fairly up to date, https://etherpad.openstack.org/p/Ironic-BFV16:07
TheJuliamjturek is going to fix a couple minor breaks above, and we'll get everything rebased so we have verified+1 for CI16:08
TheJuliaI also saw the notes from last week \o/16:08
TheJuliabut we can discuss that more in discussion. :)16:09
TheJuliaAnyone have any status updates to report?16:09
derekhWe were talking about tempest and a new CI job last week, I've proposed a new CI job here https://review.openstack.org/#/c/470307/16:09
mjturekyep it looks good derekh!16:09
hshiinaderekh: thanks16:09
derekhbut its a WIP until the tempest test are ready any we know what ironic-tempest-regex to use16:09
TheJuliaoohhhh16:10
TheJuliaIs that on the ether pad?16:10
derekhyup, line 3816:10
TheJuliaawesome!16:10
TheJuliaWow, things are coming together kind of nicely16:10
TheJuliaAnything else to bring up?16:12
TheJuliaMoving on if there is nothing else16:13
* TheJulia waits a little bit16:13
derekhnothing from me16:13
mjturekI'm good - I'm pretty sure I've handled the comments in the current two patches. If I've missed anything, please ping me16:13
TheJuliaAwesome16:14
TheJulia#topic Planning/Priorities16:14
*** openstack changes topic to "Planning/Priorities (Meeting topic: ironic_bfv)"16:14
TheJuliaI think the priorities at the moment are getting the patchset in order with CI feedback and reviews.16:14
TheJuliaThe skip deployment patch may need a little more work, hshiina I left a reply in review after checking the revisions that mjturek updated.16:15
TheJuliaI'd like to discuss that, if you have time hshiina16:16
* TheJulia feels like she is just kind of stating the obvious today :)16:16
mjturekyeaaah need to dig into the skip patch a bit today, will review16:17
hshiinaTheJulia: sure.16:17
TheJuliaI think we're good to move on to Open Discussion at this point, unless there is anything we should incorporate or that needs to be raised?16:18
mjturekI'm good :) (wow we're blazing through today)16:19
TheJulia*shrugs*16:19
TheJulia#topic Open Discussion16:19
*** openstack changes topic to "Open Discussion (Meeting topic: ironic_bfv)"16:19
TheJuliabare with me a moment, I'm looking for a specific revision to pose a question/thought16:21
mjtureknp16:21
TheJuliaSo, going back to https://review.openstack.org/#/c/454243/ which is the skip patch, and then glancing at https://review.openstack.org/#/c/441944/9/ironic/drivers/modules/agent.py which is the patch hshiina proposed.  Which adds task.driver.boot.prepare_instance(task).  I feel like that is already called by the Agent deploy driver's prepare method.16:23
TheJuliaI think that would cover it, with the wire-in being in the conductor before deploy.prepare, but I could very well be wrong or recalling something incorrectly16:24
TheJuliaso, looking for the link, it looks like I'm wrong16:26
TheJuliawe would need to re-call it it seems, since we are truly only performing attachment on power ops in this case16:26
TheJuliaerr, that is only when active16:27
mjturekso basically prepare_instance wasn't getting called unless the node was already ACTIVE?16:27
TheJuliahttps://review.openstack.org/#/c/406290/34/ironic/drivers/modules/agent.py16:27
TheJuliaso yeah, we do need to still call prepare_instance because we are in DEPLOYING16:28
TheJuliaprepare_instance only gets called because of the conductor takeover code16:28
hshiinaTheJulia: yes, prepare_instance is not called in bfv case.16:29
TheJuliaI'll update the revision this afternoon to reflect that16:29
hshiinaTheJulia: thanks16:30
*** stendulker has joined #openstack-meeting-516:30
mjturekcool16:30
TheJuliahshiina: I think i still need to digest your changes in ironic/drivers/modules/pxe.py.  Do you see them as necessary with the skip patchset?16:31
hshiinai think it is necessary16:32
hshiinasome configuration for pxe is done in prepare_ramdisk16:33
hshiinaif prepare_ramdisk is not called, some work seems necessary in prepare_instnce16:34
TheJuliaOkay, that seems very likely given how complex both of those methods are16:35
TheJuliaI'll update the skip revision later today16:36
TheJuliaAnything else to discuss today?16:37
hshiinai have nothing16:39
TheJuliaExcellent, Thank you everyone!16:39
mjturekthanks TheJulia!16:40
mjturekttyl16:40
derekhty16:40
hshiinathank you!16:40
TheJulia#endmeeting16:41
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings"16:41
openstackMeeting ended Thu Jun  8 16:41:21 2017 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:41
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-06-08-16.01.html16:41
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-06-08-16.01.txt16:41
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-06-08-16.01.log.html16:41
*** phuongnh has quit IRC16:41
*** hshiina has left #openstack-meeting-516:42
*** derekh has quit IRC16:42
*** tiendc has quit IRC16:43
*** stendulker has quit IRC16:59
*** ralonsoh has quit IRC17:03
*** rarcea has quit IRC17:19
*** ricolin has quit IRC17:41
*** bzhao_ has quit IRC17:45
*** zaneb has quit IRC19:24
*** zaneb has joined #openstack-meeting-519:38
*** skazi__ has quit IRC20:33
*** skazi_ has joined #openstack-meeting-520:33
*** mjturek has quit IRC21:31
*** bzhao_ has joined #openstack-meeting-522:06
*** zaneb has quit IRC22:15
*** zaneb has joined #openstack-meeting-522:23
*** felipemonteiro__ has quit IRC23:05
*** markvoelker has quit IRC23:34

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