Monday, 2018-08-13

*** mjturek has quit IRC00:47
*** mjturek has joined #openstack-ironic00:49
*** mjturek has quit IRC00:50
*** mjturek has joined #openstack-ironic00:50
openstackgerritKaifeng Wang proposed openstack/ironic master: Reformat instructions related with various OS  https://review.openstack.org/59010901:12
*** phuongnh has joined #openstack-ironic01:17
*** mjturek has quit IRC01:19
*** rh-jelabarre has quit IRC02:45
*** Bhujay has joined #openstack-ironic04:50
*** rcernin has quit IRC04:54
*** rcernin has joined #openstack-ironic04:54
*** jaganathan has joined #openstack-ironic05:04
*** openstackgerrit has quit IRC05:18
*** pcaruana has joined #openstack-ironic05:59
*** pcaruana has quit IRC06:05
*** pcaruana has joined #openstack-ironic06:19
*** serlex has joined #openstack-ironic06:40
*** rcernin has quit IRC07:03
*** yolanda has joined #openstack-ironic07:10
*** milan has joined #openstack-ironic07:12
*** arne_wiebalck has joined #openstack-ironic07:13
*** openstackgerrit has joined #openstack-ironic07:35
openstackgerrityolanda.robla proposed openstack/ironic master: Add automated_clean field  https://review.openstack.org/58579507:35
*** rpittau has joined #openstack-ironic07:43
*** pcaruana has quit IRC07:43
*** olivierbourdon38 has joined #openstack-ironic07:44
*** pcaruana has joined #openstack-ironic07:57
*** w-miller has joined #openstack-ironic07:59
openstackgerrityolanda.robla proposed openstack/ironic master: Add automated_clean field  https://review.openstack.org/58579508:14
openstackgerrityolanda.robla proposed openstack/ironic master: Add automated_clean field to the node object and API  https://review.openstack.org/58599108:14
openstackgerrityolanda.robla proposed openstack/ironic master: Add functionality for individual cleanup on nodes  https://review.openstack.org/58627708:17
*** lucasagomes has joined #openstack-ironic08:18
*** e0ne has joined #openstack-ironic08:20
*** etingof has joined #openstack-ironic08:20
*** mgoddard has joined #openstack-ironic08:25
*** ajya has joined #openstack-ironic08:27
openstackgerritDebayan Ray proposed openstack/sushy master: Add storage and simple_storage attr to system  https://review.openstack.org/56305108:29
*** tssurya has joined #openstack-ironic08:34
*** derekh has joined #openstack-ironic08:36
*** dougsz_ has joined #openstack-ironic08:45
*** MattMan_1 has quit IRC08:48
*** MattMan_1 has joined #openstack-ironic08:49
*** dtantsur|afk is now known as dtantsur08:49
dtantsurmorning ironic08:49
etingofdtantsur, o/ how is the weather today?08:51
dtantsuretingof: somewhat warmer again, but acceptable. and in Brno?08:51
etingofdtantsur, it is not bad at all so far! last night it was even chilly in the field (watching Perseids)08:52
dtantsurah, cool :) yeah, nights are fresh now, that's awesome.08:52
phuongnhmorning dtantsur09:10
dtantsuro/09:11
phuongnhHi all, I am testing scheduling based on trails follow guidance here: https://docs.openstack.org/ironic/queens/install/configure-nova-flavors.html09:21
phuongnhI added a custom trait CUSTOM_TRAIT1 to a node and to a flavor but I think system is still deploying based on properties. How can I check which schedule method is used?09:21
phuongnhI setup system by devstack and get the latest source code on 15-Jul09:21
*** S4ren has joined #openstack-ironic09:35
*** rpittau has quit IRC09:36
*** rpittau has joined #openstack-ironic09:36
*** ajya has quit IRC09:37
*** ajya has joined #openstack-ironic09:38
*** serlex has quit IRC09:40
arne_wiebalckphuongnh: The nova logs should tell you. IIRC, nova reports how many nodes it has before and after each filter. And with debugging enabled (?) it even tells you why it dropped a specific node. Alternatively, you can change your traits/properties to see at which state scheduling doesn’t find your host :)09:41
*** serlex has joined #openstack-ironic09:43
phuongnharne_wiebalck: Thanks, could you please tell me how to enable debugging?09:44
arne_wiebalckphuongnh: debug=True in nova.conf, then restart nova09:51
*** S4ren has quit IRC09:56
*** hjensas has joined #openstack-ironic10:02
dtantsurdoes anyone still remember how VIF attach/detach worked before the proper API?10:06
openstackgerrityolanda.robla proposed openstack/ironic master: Add functionality for individual cleanup on nodes  https://review.openstack.org/58627710:07
phuongnharne_wiebalck: Thanks10:17
*** milan has quit IRC10:21
openstackgerrityolanda.robla proposed openstack/python-ironicclient master: Add management of automated_clean field  https://review.openstack.org/58650810:38
*** dmellado has quit IRC10:46
*** sambetts has joined #openstack-ironic11:03
*** sambetts is now known as sambetts|afk11:04
*** slagle has joined #openstack-ironic11:22
*** phuongnh has quit IRC11:38
jrolldtantsur: port.extra[vif_port_id] or something11:38
jrollmorning \o11:39
*** rh-jelabarre has joined #openstack-ironic11:43
* sambetts|afk ears purk up and the mentions of vif_port_id 11:44
sambetts|afkat the *11:44
sambetts|afkalso lost all my scroll back because ZNC got kicked into openstack-unregistered :(11:44
*** sambetts|afk is now known as sambetts11:45
TheJuliaGood morning11:54
etingofgood Monday morning jroll sambetts and TheJulia ;)12:00
TheJuliaSeems irc spammers have redoubled efforts12:03
*** rpittau is now known as rpittau|afk12:10
*** rpittau|afk is now known as rpittau12:10
*** bfournie has quit IRC12:15
dtantsurjroll: yep, seems so.12:25
dtantsurmorning TheJulia, sambetts, jroll12:25
jroll\o12:27
*** jcoufal has joined #openstack-ironic12:30
*** jistr is now known as jistr|call12:32
*** trown|outtypewww is now known as trown12:32
*** jcoufal has quit IRC12:41
*** jcoufal has joined #openstack-ironic12:42
* TheJulia updates agendas and stuffs12:45
*** rloo has joined #openstack-ironic12:48
TheJuliagood morning rloo12:49
rlooGood morning TheJulia and all the ironic'ers out there :)12:49
*** dmellado has joined #openstack-ironic12:50
TheJuliaerr, looks like pike might be broken, http://logs.openstack.org/42/588942/1/gate/ironic-tempest-dsvm-ipa-partition-redfish-tinyipa/7bbeba6/logs/devstacklog.txt.gz :\12:51
TheJuliaat least for redfish12:51
dtantsurmorning rloo12:55
rloohi dtantsur!12:55
dtantsurTheJulia: why wouldn't it indeed.. there is a sushy patch to backport for that.12:55
dtantsurTheJulia: this https://github.com/openstack/sushy/commit/ae06c028d41f4e284745b1c14988ef267127cf7712:56
TheJuliadtantsur: got a link handy?12:56
*** jroll has quit IRC12:59
*** jroll has joined #openstack-ironic13:00
*** jistr|call is now known as jistr13:04
*** bfournie has joined #openstack-ironic13:05
dtantsurTheJulia: wanna cherry-pick it ^^^ or should I? I suspect there may be conflict, since AccountService did not exist in sushy in Pike.13:14
jrollTIL we're supposed to cut branches for the tempest plugin13:15
dtantsurjroll: WUT?13:16
TheJuliadtantsur: yeah, I don't think it did, I'm going to circl eback to that13:16
dtantsurjroll: the whole point of separate plugins is not having branches. are you sure they're not talking about releases?13:16
jrolldtantsur: cycle-with-intermediary http://lists.openstack.org/pipermail/openstack-dev/2018-August/133189.html13:17
jrollthere's a reference to doing coordinated releases of the plugins there13:17
TheJuliadtantsur: they decided they needed to be considered and tracked with the release, at least that is what I understand... I don't really agree13:17
TheJuliaAnyway, they moved the commit data for me into the proper place since a bunch of us missed it13:18
jrollright, they did it for the end user people13:18
TheJuliaoh, but they didn't tag it13:18
TheJuliafor the branch13:18
TheJulialovely13:18
* TheJulia adds to do list13:19
jrollyeah, I think it's silly13:19
* etingof can probably backport that sushy fix if TheJulia is overwhelmed13:19
TheJuliaetingof: that would be awesome, since I'm going through all the status items and double checking things13:19
etingofTheJulia, sure, tonight13:21
*** SteelyDan is now known as dansmith13:25
*** jaypipes has joined #openstack-ironic13:26
TheJuliadtantsur: https://etherpad.openstack.org/p/IronicWhiteBoard line 246, is that still valid?13:26
*** etingof is now known as etingof|brb13:27
dtantsurTheJulia: let us not create the branch for ironic-tempest-plugin. if people insist, we just move the tests back in-tree.13:28
* TheJulia has many conflicted feelings13:29
dtantsurwell, making them branchless was the only reason to go through the pain for separating them13:30
dtantsurif somebody just backed this decision so lightly, I'm going to have some ugly rant13:30
dtantsurseveral of us wasted a couple of weeks on it13:30
TheJuliamore than a couple13:30
dtantsurbut I hope we're confusing something: heat also released without a branch: https://review.openstack.org/#/c/591244/1/deliverables/rocky/heat-tempest-plugin.yaml13:31
patchbotpatch 591244 - releases - Release heat tempest plugin 0.2.0 - 0h 7m 17s spent in CI13:31
jrolldtantsur: and we'll need to waste more time adjusting CI jobs to always use master now :(13:32
dtantsurjroll: well, we won't, because plugins are supposed to be branched now \o/13:32
jrolldtantsur: I seem to remember from that thread that upstream is still supposed to test with master13:33
jrolland branches are only for end users13:33
jrollwhich is even more insane13:33
dtantsurThis is insane, and I'm going to -1 the hell out of this effort until whoever is doing this gets their mind together13:33
dtantsurFor the record, I did raise this issue when that decision was made, and it was dismissed13:34
jrollyes, I remember :)13:34
* jroll is reading the old thread right now13:34
jrollI suspect the final decision was made elsewhere though13:34
*** mjturek has joined #openstack-ironic13:35
dtantsurwell, of course. 4 opens and all the bs like that.13:36
jrollI want to move this discussion to -qa but I'm not sure I can be friendly13:36
dtantsurI cannot and likely will not.13:37
* sambetts is also -1 to branching tempest plugins... 13:37
TheJuliaI -1'ed the change13:37
TheJuliawell, WF-113:37
dtantsurI wonder if somebody realizes that the second we create stable/rocky, it will be used for rocky testing, not master13:37
TheJuliaand basically put that I'm tired of the back and forth on this subject13:37
TheJuliaI put the pro for it in the message13:37
dtantsurand to overcome it we'll have to screw with zuul13:38
TheJuliabut anyway, other things are more important13:38
dtantsur++13:38
jrollright, though I'd like us to not suddenly be testing rocky with the rocky branch if someone else branches this for us13:38
dtantsurwell, we will, this is how zuul works by default13:39
sambettswe can force it on our end to always be master13:39
jrollexactly13:39
jrolldtantsur: which is why i want to ask questions now :)13:39
dtantsursambetts: right, but I'm not sure how it will behave with the plugin's own CI13:39
dtantsuron master it will have master overriding with pike (overriding plugin with master)13:40
sambettswe can set a project specific override_checkout to checkout master for ironic-tempest-plugin, it should be fine across the board zuul handles the merging well in that regard13:40
dtantsuron pike it will have pike testing with pike through a few overrides, but nothing else we use it13:40
dtantsursambetts: another round of stable patches across all the projects.. but yes, it will work.13:40
jrollor or or orrrrrr13:41
jrollwe can not branch things ಠ_ಠ13:41
sambetts1000%13:41
sambettsjust thinking of how to avoid everything blowing up if someone does it for is13:42
sambettsus13:42
dtantsurI wish we just had in-tree tests running on the same branch..13:42
* jroll drops a message in qa13:42
jrollI suspect there's still the argument of not installing the whole project (e.g. ironic) to run tempest for end users13:44
TheJuliaThere is13:44
TheJuliaThere is also the desire to take Queens tests and run them against a rocky deploy13:44
TheJuliaand separate versioned releases enables that13:44
jrollwat13:45
jrollwhy13:45
openstackgerritSam Betts proposed openstack/ironic master: Force ironic-tempest-plugin branch to master for all ironic jobs  https://review.openstack.org/59139113:45
jrollbranchless also supports that, as branchless must work with all supported versions :)13:46
jrollno13:46
sambettsin case we need it ^13:46
jrollstop13:46
* jroll cries13:46
jrollI guess we aren't branching, yay13:48
sambetts\o/13:48
openstackgerritSam Betts proposed openstack/ironic master: [devstack] Allow setting TFTP max blocksize  https://review.openstack.org/52873713:49
TheJuliasambetts: you can abandon 591391, we all misunderstood the email13:51
dtantsur\o/13:52
sambetts\o/13:52
*** baha has joined #openstack-ironic13:54
openstackgerritDmitry Tantsur proposed openstack/sushy stable/pike: Mark Systems/Managers/SessionService optional  https://review.openstack.org/59139514:00
dtantsurTheJulia: ^^^14:00
TheJuliathanks dtantsur14:06
dtantsurTheJulia: oh, I need to remove the release note14:07
TheJulia++14:07
openstackgerritDmitry Tantsur proposed openstack/sushy stable/pike: Mark Systems/Managers/SessionService optional  https://review.openstack.org/59139514:08
dtantsurnow ^^^14:08
openstackgerritAija Jaunteva proposed openstack/sushy master: Cleanup docstring for removed etag param  https://review.openstack.org/59139614:08
*** bnemec-pto is now known as bnemec14:10
TheJuliamgoddard: do you feel https://review.openstack.org/#/c/579583/2/specs/approved/virtual-bare-metal-cluster-manager.rst is up for some review activity ?14:23
patchbotpatch 579583 - ironic-specs - [WIP]: Add virtual Bare Metal Clusters spec - 0h 12m 42s spent in CI14:23
*** exodusftw has quit IRC14:24
*** Bhujay has quit IRC14:27
jrollTheJulia: so I want to install an out-of-tree driver in bifrost. I also want to add a second server and additional endpoints to the nginx config there. are those things I should contribute to bifrost itself, or would you recommend running additional ansible playbooks after running bifrost, or something else?14:39
TheJuliajroll: I would be happy with the contribution... I think additional playbooks would be fine, although you may want to mix some of that together14:40
TheJuliajroll: some of that groundwork was already layed with changing the data structures to be way more complex14:40
TheJuliabut we didn't do multiple I don't think, so you might have some awesome variation14:41
TheJuliamix/match, whatever makes sense :)14:41
* TheJulia thinks it is time to make pre-meeting coffee14:42
jrollTheJulia: ok, thanks. I'll poke around14:43
jrolldefinitely want to contribute TLS config to the nginx config regardless :)14:43
TheJuliaThat would be awesome14:46
TheJuliaI'm really hoping to get back to UEFI stuffs soonish, to make the world a better place... and maybe get an arm job at some point for us14:46
*** etingof|brb is now known as etingof14:54
mgoddardTheJulia: it's still WIP, but would certainly be interested to hear people's initial thoughts14:56
*** rpioso|afk is now known as rpioso14:57
mgoddardw-miller is just getting to the end of the investigation phase, after which he'll be looking at the spec and implementation14:57
rpiosoGood morning, ironicers14:57
TheJuliagood morning14:58
*** stendulker has joined #openstack-ironic15:00
TheJulia#startmeeting ironic15:00
openstackMeeting started Mon Aug 13 15:00:09 2018 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
TheJuliao/15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
openstackThe meeting name has been set to 'ironic'15:00
etingofo/15:00
TheJuliaGood morning everyone!15:00
bdoddo/15:00
bahao/15:00
jroll\o15:00
stendulkero/15:00
w-millero/15:01
TheJuliaOur agenda can be found on the wiki!15:01
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:01
rlooo/15:01
TheJulia#topic Announcements / Reminder15:01
TheJulia#info The Stein PTG is in 29 days!15:01
rpiosoo/15:01
TheJulia#info Please remember to register and book your hotel room. The room block is expected to sell out.15:02
TheJuliaOur planning etherpad continues to grow with disussion. If there is something you'd like to see or discuss, please feel free to add or comment. I'll work on creating the schedule after the ?23rd?15:03
TheJulia#link https://etherpad.openstack.org/p/ironic-stein-ptg15:03
*** kaifeng has joined #openstack-ironic15:03
dtantsuro/15:03
kaifengo/15:03
mgoddardo/15:03
TheJulia#info Ironic 11.1 and all of our related projects were released last week. A giant thanks goes to everyone for their effort during the Rocky cycle.15:04
TheJulia#info stable/rocky branches have been cut as a result of the rocky release, and we can begin accepting patches for inclusion in Stein to master.15:04
rlooawesome, great work everyone!15:04
TheJulia#info If there is a patch you feel requires being backported to rocky, please raise awareness.15:05
TheJulia#info Next week during our weekly meeting we will have a discussion about ironic-staging-drivers. All cores please make every effort to attend, as I anticipate a vote being called.15:06
rlooTheJulia: is the discussion a vote to keep it?15:06
TheJuliaThat is it for Announcements/Reminders from my point of view. Does anyone have anything to raise or that they wish to announce?15:07
dtantsurI think before we put up a vote, we should post something to the ML15:07
rloo++ (since I don't even know what the discussion is about, and I don't think we/ironic own that)15:07
TheJuliadtantsur: I can post to the ML today15:07
jrolldtantsur: +115:08
TheJuliaIf there are no objections, I'll post to the mailing list after the meeting.15:09
dtantsurrloo: that's precise the question: should we own that15:09
rloodtantsur: ah, ok, i'll get out the popcorn :D15:09
TheJuliarloo: good plan!15:09
dtantsur:D15:09
TheJulia#action TheJulia to send an email to the mailing list regarding ironic-staging-drivers.15:10
TheJuliaAnything else before we move on?15:10
* TheJulia hears crickets in the background again ;)15:11
rloocongrats to TheJulia for being our fearless leader again!15:11
TheJuliarloo: thanks!15:11
TheJulias/fearless/crazy/? ;)15:11
rloowe were all afraid to run against her ;)15:11
TheJulialol15:11
mjtureko/15:11
TheJulia\o15:11
TheJuliaOkay, I think we can move on15:12
*** serlex has quit IRC15:12
TheJuliaSince there were no action items last week, we can skip directly to subteam status updates15:12
TheJulia#topic Review subteam status reports15:12
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:13
TheJuliaStarting around line 18015:13
openstackgerritJim Rollenhagen proposed openstack/bifrost master: Add doc/requirements.txt to releasenotes tox environment  https://review.openstack.org/59141615:14
openstackgerritJim Rollenhagen proposed openstack/bifrost master: Allow extra packages to be installed with ironic  https://review.openstack.org/59141715:14
TheJuliaI didn't mention deploy steps because I didn't see a way to use it in the released client for cycle highlights, at least at a fast glance, so it feels like we have some more work to do there15:14
dtantsuryeah, we need API to choose steps, more steps to choose from and finally deploy templates15:15
dtantsursounds like 1-2 cycles more15:15
jrollyep, we just merged the backbone15:15
rlooright, there isn't anything you can do wrt the deploy steps. not yet anyway.15:15
rlooso i think deploy steps framework is done15:16
TheJuliaThat was my refreshed recollection. I'm not suggesting we do that right now, I did put deploy templates spec on the list things that could use some eyeballs/braincells this week. :)15:16
TheJuliarloo: ack, guess I concur sans api definition of what to do, but I'm sure we can get to that during Stein15:17
rlooTheJulia: ah. ok. hmm, we should discuss whether we need another spec to decompose the one big deploy step into smaller steps.15:18
TheJuliaYeah, that part is a little difficult, but maybe looking at deploy templates spec might create some bubbling juices in everyone's brains  ;)15:19
TheJuliaI think we're functionally done with bios interfaces for now. Would be nice to get the redfish one merged once people are happy with it.15:19
TheJuliaRefarch docs need work :(15:19
TheJuliaI proposed an admin-only doc, so we can at least hopefully make some more progress on that, it is currently in review.15:21
rloowrt bios interface, i think it is done. the redfish work is a separate RFE.15:22
rloojroll: conductor location awareness is done after the doc merges?15:23
jroll++15:23
jrollyes15:23
rloonice, 3 done in a row :)15:23
TheJuliaGraphical console was basically "done" in terms of what we had for our goal for the cycle15:23
jrolluntil sambetts pushes on his giant affinity craziness thing :D15:24
sambetts;)15:24
TheJuliamkrai has started working on patches for graphical console, so we should be good to just let that proceed into stein15:24
openstackgerritMerged openstack/ironic master: Add conductor_group docs  https://review.openstack.org/58927715:24
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: [DNM] Switch to Ansible 2.6  https://review.openstack.org/59142015:25
TheJuliaNeutron event processing and cleaning steps never made it out of spec review15:25
etingofonce we merge bios support in redfish driver, that would make the whole bios thing functional in ironic15:25
rlooI'm good with the status reports15:25
rloowe focus now on bug fixes, clean up?15:25
TheJuliaSpecs and bug fixes15:26
TheJuliaetingof: \o/ that is great motivation for us to get that done.15:26
TheJuliaI expect our meetings until the PTG will mainly focus on spec review/discussion since we've largely wrapped the rocky cycle at this point.15:27
rloobut we've branched, so we can start adding stuff into master, for the stein release, right?15:28
TheJuliarloo: yes15:28
TheJuliaI also encourage picking up neat things that look abandoned or that have not had any attention and hacking it out... like splitting ipxe/pxe into separate interfaces, or at least starting that process ;)15:28
dtantsur:)15:29
*** ajya has quit IRC15:29
TheJuliaAnyway, we should look at priorities for the coming week!15:29
*** ajya has joined #openstack-ironic15:29
TheJulia#topic Deciding on priorities for the coming week15:30
TheJulia(since we were already heading in that direction of discussion15:30
* TheJulia goes and cleans up merged things15:30
TheJuliaAnd so, I have a list of docs and specs, and some minor housekeeping changes aligning to the python3-first goal that I feel would be good to have on the list for the week, any objections?15:31
jrollI'm fine with that, I'll be mostly downstream this week anyway, I think15:32
TheJuliaAnyway, given the time in the cycle, specs are naturally important for us going into this next cycle.15:33
TheJuliaanyway, list updtaed15:33
TheJuliaupdated15:33
rlooI'm fine, although that is a big list of specs :)15:33
dtantsurlgtm15:34
TheJuliarloo: I swap them around for next weke ;)15:34
TheJuliaweek15:34
rlooTheJulia: how do you decide which specs to list?15:34
TheJuliarloo: I exclude my own and mostly just go down the list (although slightly randomly15:34
TheJulia)15:34
TheJuliastatus:open15:34
rloofrom newer specs? cuz we have over 100 I think?15:35
rloook, not 100. maybe 20 or so.15:35
TheJuliathese are mostly in review ready or conceptually in a good place, I didn't want to list 20-something specs15:36
rloogotcha. and sigh, it is more than 20.15:36
*** pcaruana has quit IRC15:37
TheJuliathem being review ready or recently updated means there is at least people and interest behind them. I figure it might be good to go through older orphaned items soonish15:37
rloojust skimming; lots seem valid still. but another discussion :)15:38
rlooI'm ready to move on15:38
TheJuliayeah, they need eyes15:38
TheJuliaand button clicks, or git magic :)15:38
TheJuliaAnyway, since we have no discussion topics, I think we're good to move to open discussion15:39
TheJulia#topic Open Discussion15:39
TheJuliaI feel like we need an ironic comic book.... ;)15:40
* TheJulia is just being silly15:40
jrollsupercricket15:40
* TheJulia wonders if there is a supercricket station on pandora15:40
jrollI was thinking along the lines of a superhero :P15:41
TheJuliaInteresting!15:41
dtantsur:D15:41
* TheJulia should look for cricket stickers online....15:41
jrollsecond mascot?15:41
TheJuliadunno, but it could just be amusing15:41
TheJuliaAnyway, Thanks everyone!15:42
TheJuliaHave a wonderful week!15:42
TheJuliao/15:43
TheJulia#endmeeting15:43
openstackMeeting ended Mon Aug 13 15:43:40 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:43
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-08-13-15.00.html15:43
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-08-13-15.00.txt15:43
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-08-13-15.00.log.html15:43
jrollthanks!15:44
*** olivierbourdon38 has quit IRC15:44
*** kaifeng has quit IRC15:44
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: [DNM] Switch to Ansible 2.6  https://review.openstack.org/59142015:47
NobodyCamGood Morning Ironic'ers15:59
*** e0ne has quit IRC16:03
*** stendulker has quit IRC16:14
*** gyee has joined #openstack-ironic16:23
rlooTheJulia: if you have a few minutes. I forgot about this patch; I didn't know what you wanted me to change: https://review.openstack.org/#/c/569166/16:25
patchbotpatch 569166 - ironic - Disable periodic tasks if interval set to 0 - 1h 58m 18s spent in CI16:25
*** sambetts is now known as sambetts|afk16:26
*** namnh has joined #openstack-ironic16:26
*** olivierbourdon38 has joined #openstack-ironic16:26
* TheJulia raises an eyebrow16:26
TheJuliaat patchbot16:26
jrollhuh, that "spent in CI" thing is new16:26
jrolljinx?16:26
TheJuliajroll: *shrugs*16:28
*** e0ne has joined #openstack-ironic16:28
openstackgerritJim Rollenhagen proposed openstack/bifrost master: Allow extra packages to be installed with ironic  https://review.openstack.org/59141716:32
TheJuliarloo: I think I just wanted the space in the commit message so it is tracked and closed out accordingly and I believe I was hoping for you to bool() the result of the compare explicitly (from a readability standpoint at least). there were a couple other things I'll reply to16:33
openstackgerritMerged openstack/ironic-python-agent master: Update reno for stable/rocky  https://review.openstack.org/59032316:33
rloothx TheJulia, I had forgotten about that patch. will try to get it done this week.16:34
rloodtantsur, TheJulia: do we need to backport to rocky? https://review.openstack.org/#/c/590675/216:34
patchbotpatch 590675 - ironic-inspector - Imported Translations from Zanata - 1h 58m 29s spent in CI16:34
rlooi don't like/care to know how long it spent in CI16:35
TheJuliarloo: I think the tech debt concern was the -1 to 0 migrations that may be needed, at least looking at it from a 10,000 foot view16:36
*** ajya has quit IRC16:36
TheJuliarloo: re backport, eh... not really, the bot should propose it if necessary I think.16:37
rlooTheJulia: ok, i will look at it again. I've forgotten what I did, didn't think I made any changes that required any migration.16:37
*** ajya has joined #openstack-ironic16:37
TheJuliarloo: I was more concerned about you complicating your own work later :)16:37
rlooTheJulia: ha ha, thx for thinking of me! :)16:38
TheJuliaanyway, I think it is time to find some food16:38
* rloo is eating now...16:38
* TheJulia looks longingly at the grill next to the campsite....16:40
dtantsurrloo: yeah, zanata stuff should be proposed by the bot16:40
TheJuliaand the rain16:40
*** pcaruana has joined #openstack-ironic16:40
rloodtantsur, TheJulia: wrt translation patches, it isn't clear from this email: http://lists.openstack.org/pipermail/openstack-dev/2018-August/133092.html16:41
rlooguess we can backport it later if it isn't done automagically16:42
dtantsurrloo: translations themselves are branched and proposed against the corresponding source branch16:42
rloodtantsur: oh, interesting. (so much to know/remember...) thx!16:43
dtantsurnp :)16:43
openstackgerritMerged openstack/ironic-specs master: Moving UCS specs to backlog  https://review.openstack.org/58947416:44
openstackgerritMerged openstack/bifrost master: Update reno for stable/rocky  https://review.openstack.org/59109616:53
*** dtantsur is now known as dtantsur|afk16:59
dtantsur|afko/16:59
TheJuliagoodnight dtantsur|afk16:59
*** derekh has quit IRC17:01
*** w-miller has quit IRC17:01
*** e0ne has quit IRC17:01
*** dougsz_ has quit IRC17:01
*** mgoddard has quit IRC17:04
*** trown is now known as trown|lunch17:07
openstackgerritMerged openstack/bifrost stable/rocky: Update .gitreview for stable/rocky  https://review.openstack.org/59109417:08
*** namnh has quit IRC17:09
*** baha has quit IRC17:12
*** namnh has joined #openstack-ironic17:14
*** openstackgerrit has quit IRC17:19
*** NobodyCam has quit IRC17:19
*** NobodyCam has joined #openstack-ironic17:19
*** namnh has quit IRC17:22
*** eandersson has quit IRC17:32
*** tssurya has quit IRC17:36
*** ianychoi_ has joined #openstack-ironic17:38
*** openstackgerrit has joined #openstack-ironic17:38
openstackgerritPaul Belanger proposed openstack/bifrost master: Refactor bifrost-integration-dhcp jobs  https://review.openstack.org/59147117:38
*** ianychoi has quit IRC17:41
openstackgerritMerged openstack/ironic-python-agent stable/rocky: Update .gitreview for stable/rocky  https://review.openstack.org/59032117:43
openstackgerritMerged openstack/ironic-inspector master: Imported Translations from Zanata  https://review.openstack.org/59067517:43
rpiosoTheJulia: Is stable/ocata still open for bug fix backports?17:44
jrollrpioso: I believe the rules have changed a bit and we now leave branches open forever17:48
* jroll tries to find a source17:49
jrollhttps://governance.openstack.org/tc/resolutions/20180301-stable-branch-eol.html#end-of-life17:49
rpiosojroll: Interesting ... I ask, because dtantsur|afk had some reservations about approving https://review.openstack.org/#/c/588949/17:51
patchbotpatch 588949 - ironic (stable/ocata) - Fix iDRAC hardware type does not work with UEFI - 0h 52m 36s spent in CI17:51
*** olivierbourdon38 has quit IRC17:52
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: Switch to Ansible 2.6  https://review.openstack.org/59142017:52
jrollrpioso: interesting, I think maybe dtantsur|afk is hoping some other people maintain that branch17:53
jrollI think at that point it's up to the (ironic) team to decide if/how to maintain that17:53
jrollso I guess we'll have to talk about it?17:53
rpiosojroll: We're working on addressing the failed Dell EMC 3d party CI tests of that change.17:56
rpiosojroll: My read of that doc is that even if no further releases are made, bug fixes can be merged.17:56
rpioso"Contributors can continue to push fixes and they can be merged but they will not be released upstream."17:57
jrollrpioso: yes, bug fixes can be merged, not must :) the review, overall policy, and optional delegation is up to the core team17:57
rpiosojroll: Ack :) Perhaps the word 'may' should have been used, instead of 'can'. Just saying ;-)17:58
jrollindeed17:59
*** pcaruana has quit IRC17:59
*** cdearborn has joined #openstack-ironic18:00
rpiosojroll: On a related, but different, change, the backport to stable/pike was approved and workflowed; however, the Zuul gate job failed -- https://review.openstack.org/#/c/588942/18:00
patchbotpatch 588942 - ironic (stable/pike) - Fix iDRAC hardware type does not work with UEFI - 3h 41m 44s spent in CI18:00
jrollrpioso: yep, see whiteboard, we have a known issue18:01
rpiosojroll: I see. Redfish!18:03
rpiosoty18:03
jroll:)18:03
TheJuliajroll: Yeah, it is up to us to decide, but even having the mechanism to submit the patch and for it to be someplace enables downstream consumers to pull it in if needed.18:16
jrollTheJulia: yep, agree18:17
TheJuliaif third party ci passes the patch, and its isolated to that driver, and a vendor is standing behind it saying "yes, this fixes" I'm all for merging it18:17
jrollyeah, same, it sounds like dmitry might disagree18:18
* TheJulia re-reads the small book of an email she typed up18:18
jrollis why I suggested we discuss it :)18:18
TheJuliaare we not now :P18:18
TheJuliaWe can put it on the meeting agenda for next week18:19
jroll¯\_(ツ)_/¯18:19
TheJulia:)18:19
TheJuliadoes the shrug come with beer? I feel like what would have helped typing this silly long email18:20
jrollheh18:21
jrollshrugs come with jroll, and jroll comes with beer, so yes?18:21
TheJulia\o/18:21
*** trown|lunch is now known as trown18:22
*** ajya has quit IRC18:25
*** e0ne has joined #openstack-ironic18:30
*** baha has joined #openstack-ironic18:31
openstackgerritMerged openstack/virtualbmc master: Update name for virtualbmc CI job  https://review.openstack.org/58929418:35
*** namnh has joined #openstack-ironic18:37
*** tssurya has joined #openstack-ironic18:39
*** namnh has quit IRC18:42
rloojroll, TheJulia: my take on old branches is that as long as two cores are willing to +2/approve, it should be fine. Having said that, only a few cores (4 of us?) can do that currently.18:43
rlooI wonder how other projects are dealing with it18:43
TheJuliaI think at least one of the project teams granted the ability to all cores for their older branches18:43
rlooTheJulia: if we can do that, I would prefer that. vs an ironic policy.18:44
TheJuliaWhich I would be good with actually, that minimizes the winning the lottary factor18:44
TheJuliarloo: TC stance is that projects can make that call on their own as to what best fits them18:45
rlooTheJulia: sounds like an email and vote! :)18:46
openstackgerritMerged openstack/ironic-inspector master: Remove deprecated option node_status_keep_time  https://review.openstack.org/58981818:59
openstackgerritMerged openstack/ironic master: Imported Translations from Zanata  https://review.openstack.org/59112618:59
openstackgerritMerged openstack/networking-baremetal master: Changing CI job templates for python3-first  https://review.openstack.org/59034919:06
openstackgerritwaleed mousa proposed openstack/ironic-python-agent master: Configuring mellanox interfaces using ironic python agent  https://review.openstack.org/56654419:13
*** bfournie has quit IRC19:22
*** mgagne_ has joined #openstack-ironic19:24
*** rajinir has quit IRC19:25
*** davidlenwell has quit IRC19:25
*** mnasiadka has quit IRC19:25
*** robcresswell has quit IRC19:25
*** serverascode_ has quit IRC19:25
*** mgagne has quit IRC19:25
openstackgerritMerged openstack/ironic master: Switch ironic-tempest-dsvm-bfv too zuulv3  https://review.openstack.org/58849419:27
*** amotoki has quit IRC19:27
openstackgerritMerged openstack/ironic master: Switch ironic-tempest-dsvm-ironic-inspector too zuulv3  https://review.openstack.org/58849519:28
*** amotoki has joined #openstack-ironic19:30
openstackgerritwaleed mousa proposed openstack/ironic-python-agent master: Configuring mellanox interfaces using ironic python agent  https://review.openstack.org/56654419:44
openstackgerritwaleed mousa proposed openstack/ironic-python-agent master: Configuring mellanox interfaces using ironic python agent  https://review.openstack.org/56654419:46
*** e0ne has quit IRC19:52
*** bfournie has joined #openstack-ironic20:02
*** rajinir has joined #openstack-ironic20:05
*** tssurya has quit IRC20:07
*** e0ne has joined #openstack-ironic20:18
* rpioso finished reading the backscroll20:20
rpiosojroll, TheJulia: Thanks re: stable/ocata backport consideration.20:22
NobodyCamoh Mellanox support \o/20:25
TheJuliaNobodyCam: well.... we've had it, this is a new thing20:29
NobodyCam:) i have to look at it20:30
NobodyCamjust saw the title20:30
TheJuliano setting up infiniband across the desert20:30
NobodyCamheehehe20:30
NobodyCamWiMax20:30
TheJuliaSpeaking of which, it occurs to me that we might be able to do a line of site PTP link...20:31
TheJuliaor near line of site20:31
TheJuliasight20:31
* TheJulia can't spell20:31
NobodyCamyea20:31
NobodyCamright thru the windmills20:31
TheJuliahmmm... packet loss20:32
NobodyCam:>20:32
*** e0ne has quit IRC20:33
*** jcoufal has quit IRC20:34
*** trown is now known as trown|outtypewww20:55
*** markguz_ has joined #openstack-ironic21:07
*** markguz_ has quit IRC21:15
*** hwoarang has joined #openstack-ironic21:18
*** baha has quit IRC21:22
*** slagle has quit IRC21:26
*** Nisha_Agarwal has joined #openstack-ironic21:30
*** e0ne has joined #openstack-ironic21:50
*** e0ne has quit IRC21:56
*** rcernin has joined #openstack-ironic22:01
openstackgerritwaleed mousa proposed openstack/ironic-python-agent master: Configuring mellanox interfaces using ironic python agent  https://review.openstack.org/56654422:05
*** rpioso is now known as rpioso|afk22:15
*** cdearborn has quit IRC22:18
*** pabelanger has joined #openstack-ironic22:20
pabelangergreetings, could I get a few reviews on https://review.openstack.org/591471/22:21
patchbotpatch 591471 - bifrost - Refactor bifrost-integration-dhcp jobs - 0h 16m 0s spent in CI22:21
pabelangerremoves some legacy nodesets from ironic tests and duplicated code22:21
pabelangerspecifically trying to remove the legacy-opensuse-423 nodeset from openstack-zuul-jobs22:22
pabelangerthanks!22:22
TheJuliapabelanger: ok, thanks!22:31
*** namnh has joined #openstack-ironic22:38
*** namnh has quit IRC22:44
*** Nisha_Agarwal has quit IRC23:17
*** namnh has joined #openstack-ironic23:51

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