Monday, 2018-01-15

*** etingof has joined #openstack-ironic00:00
*** superdan is now known as dansmith00:11
*** cjloader has joined #openstack-ironic00:13
*** fragatina has joined #openstack-ironic00:13
*** cjloader has quit IRC00:17
*** Goneri has quit IRC00:18
*** yangyapeng has quit IRC00:28
*** yangyapeng has joined #openstack-ironic00:29
*** fragatina has quit IRC00:31
*** yangyapeng has quit IRC00:33
*** ijw has joined #openstack-ironic00:36
*** ijw has quit IRC00:41
*** ccollett has joined #openstack-ironic01:02
*** charcol has quit IRC01:05
*** phuongnh has joined #openstack-ironic01:07
*** yangyapeng has joined #openstack-ironic01:13
*** yangyapeng has quit IRC01:21
*** yangyapeng has joined #openstack-ironic01:21
*** threestrands_ has joined #openstack-ironic01:40
*** threestrands has quit IRC01:43
*** tiendc has joined #openstack-ironic01:43
*** rbudden has quit IRC01:46
*** rbudden has joined #openstack-ironic01:49
*** namnh has joined #openstack-ironic01:56
*** charcol has joined #openstack-ironic02:01
*** ccollett has quit IRC02:02
*** cjloader has joined #openstack-ironic02:13
*** cjloader has quit IRC02:17
*** hshiina has joined #openstack-ironic02:45
openstackgerritlei zhang proposed openstack/ironic master: Remove the deprecated "giturl" option  https://review.openstack.org/53346502:50
*** jappleii__ has joined #openstack-ironic03:12
*** jappleii__ has quit IRC03:13
*** jappleii__ has joined #openstack-ironic03:13
*** threestrands_ has quit IRC03:15
*** charcol has quit IRC03:21
*** charcol has joined #openstack-ironic03:23
*** namnh has quit IRC03:34
*** namnh has joined #openstack-ironic03:35
openstackgerritDao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode tempest tests  https://review.openstack.org/52869903:37
openstackgerritlei zhang proposed openstack/ironic master: Remove the deprecated "giturl" option  https://review.openstack.org/53346503:45
*** sdake has quit IRC03:46
*** sdake has joined #openstack-ironic03:47
*** sdake has quit IRC03:47
*** sdake has joined #openstack-ironic03:47
*** bfournie has joined #openstack-ironic03:48
*** rbudden has quit IRC04:16
-openstackstatus- NOTICE: The logs.openstack.org filesystem has been restored to full health. We are attempting to keep logs uploaded between the prior alert and this one, however if your job logs are missing please issue a recheck.04:48
*** ChanServ changes topic to "The logs.openstack.org filesystem has been restored to full health. We are attempting to keep logs uploaded between the prior alert and this one, however if your job logs are missing please issue a recheck."04:48
*** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic"04:53
*** links has joined #openstack-ironic05:11
*** pcaruana has joined #openstack-ironic05:23
*** pcaruana has quit IRC05:32
*** rbartal has joined #openstack-ironic05:37
*** rbartal has quit IRC05:48
*** zhangfei has joined #openstack-ironic06:13
*** zhangfei has quit IRC06:19
*** jaganathan has joined #openstack-ironic06:31
*** zhangfei has joined #openstack-ironic06:31
*** liuzz has joined #openstack-ironic06:37
*** absubram has joined #openstack-ironic06:38
*** hieulq has quit IRC06:41
*** rbartal has joined #openstack-ironic06:43
*** rbartal has quit IRC06:51
*** zhangfei has quit IRC06:51
openstackgerritAnkit Kumar proposed openstack/ironic master: DNM Patch : Testing the HPEProliant CI  https://review.openstack.org/53351306:57
*** liuzz has quit IRC06:59
*** liuzz has joined #openstack-ironic07:00
*** jappleii__ has quit IRC07:02
*** rbartal has joined #openstack-ironic07:05
pas-hamorning ironic :)07:07
*** zhangfei has joined #openstack-ironic07:08
*** marios has joined #openstack-ironic07:09
*** marios has quit IRC07:14
*** sapd_ has quit IRC07:16
*** r-daneel has quit IRC07:17
*** r-daneel has joined #openstack-ironic07:17
*** marios has joined #openstack-ironic07:19
*** sapd_ has joined #openstack-ironic07:20
openstackgerritShivanand Tendulker proposed openstack/ironic master: Adds RPC calls for rescue interface  https://review.openstack.org/50933607:23
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add rescue related methods to network interface  https://review.openstack.org/50934207:23
*** hjensas has quit IRC07:33
*** hjensas has joined #openstack-ironic07:34
*** zhangfei has quit IRC07:42
*** makowals has joined #openstack-ironic07:42
*** AlexeyAbashkin has joined #openstack-ironic07:54
*** zhangfei has joined #openstack-ironic07:54
*** hoonetorg has quit IRC07:58
*** rcernin has quit IRC07:59
*** zhangfei has quit IRC08:07
*** hoonetorg has joined #openstack-ironic08:11
openstackgerritAnkit Kumar proposed openstack/ironic master: DNM Patch : Testing the HPEProliant CI  https://review.openstack.org/53355108:13
*** pmannidi has quit IRC08:18
*** zhangfei has joined #openstack-ironic08:20
openstackgerritDao Cong Tien proposed openstack/ironic master: devstack: add support for rescue mode  https://review.openstack.org/52411808:20
openstackgerritDao Cong Tien proposed openstack/ironic master: CI job for rescue mode  https://review.openstack.org/52870408:23
openstackgerritDao Cong Tien proposed openstack/ironic master: DNM: test rescue mode  https://review.openstack.org/53318708:25
*** fragatina has joined #openstack-ironic08:26
*** racedo has joined #openstack-ironic08:29
*** e0ne has joined #openstack-ironic08:29
*** athomas has joined #openstack-ironic08:35
*** fragatina has quit IRC08:50
*** e0ne has quit IRC08:53
*** jaianshu has joined #openstack-ironic08:59
*** amarao has joined #openstack-ironic09:03
*** jtomasek has joined #openstack-ironic09:05
*** stendulker has joined #openstack-ironic09:05
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add rescue related methods to network interface  https://review.openstack.org/50934209:12
*** lucas-afk is now known as lucasagomes09:15
*** hshiina has quit IRC09:15
openstackgerritvinay kumar muddu proposed openstack/ironic master: Fixes issue with UEFI bootable ISO creation  https://review.openstack.org/53170009:31
*** derekh has joined #openstack-ironic09:36
*** athomas has quit IRC09:40
*** absubram has quit IRC09:44
dtantsurmorning ironic09:46
*** athomas has joined #openstack-ironic09:52
stendulkerGood morning dtantsur09:52
dtantsurhi stendulker09:52
stendulkerdtantsur: The patch related iLO power retry has git merged into Proliantutils https://review.openstack.org/#/c/519967/09:53
patchbotpatch 519967 - proliantutils - Retry power on operation for Blade servers (MERGED)09:53
dtantsuryes, I've seen it. thanks!09:53
stendulkerdtantsur: Proliantutils has dependency on sushy library. I think both libraries have Jan 19 as release deadline09:55
stendulkerProliantutils would have to use newer version of sushy. What is the likely day for sushy release?09:56
dtantsurstendulker: I think we need to finish the redfish sessions patch before09:57
stendulkerdtantsur: ok09:57
dtantsurI hope to release on Wed09:57
stendulkerdtantsur: ok. Thank you09:57
dtantsurmaybe earlier, if possible09:57
stendulkerdtantsur: ok09:58
*** hieulq has joined #openstack-ironic09:59
*** makowals has quit IRC10:04
*** makowals has joined #openstack-ironic10:05
*** gmonteiro|home has joined #openstack-ironic10:10
*** namnh has quit IRC10:11
*** cjloader has joined #openstack-ironic10:13
*** cjloader has quit IRC10:17
gmonteiro|homeGood Morning Ironic :)10:18
*** sambetts|afk is now known as sambetts10:26
sambettsMorning all10:26
*** e0ne has joined #openstack-ironic10:27
openstackgerritAnshul Jain proposed openstack/ironic-specs master: Update OOB RAID for iLO5 based HPE Proliant Servers  https://review.openstack.org/53083810:30
stendulkerGood morning gmonteiro|home, sambetts10:33
gmonteiro|homestendulker: o/10:33
openstackgerritparesh sao proposed openstack/ironic master: Implementation for UEFI iSCSI boot for ILO  https://review.openstack.org/46828810:35
*** rbartal has quit IRC10:36
*** stendulker_ has joined #openstack-ironic10:47
*** dtantsur is now known as dtantsur|bbl10:48
*** yolanda has joined #openstack-ironic10:50
*** gmonteiro|home has quit IRC10:50
yolandahi, good morning10:50
yolandai'm trying deployment on queens, and i'm having trouble with some deprecated commands10:50
yolandanow trying this one10:50
*** stendulker has quit IRC10:50
yolandaopenstack baremetal introspection bulk status -f yaml10:50
*** phuongnh has quit IRC10:51
yolandawhat is the alternative for it on queens?10:51
*** stendulker_ has quit IRC10:52
*** zhangfei has quit IRC10:52
sambettsyolanda: those commands come from TripleO not from ironic-inspector, so you'll need to check thier docs, https://docs.openstack.org/tripleo-docs/latest/install/basic_deployment/basic_deployment_cli.html#introspect-nodes10:53
*** gmonteiro|home has joined #openstack-ironic10:54
sambettsyolanda: the reason those commands where removed is because they pretended to be ironic-inspector commands when they come from a different project10:54
yolandayep, they look as ironic commands10:54
sambettsyolanda: you can do: openstack baremetal introspection list  to list all node status in inspector now though I think10:55
yolandai tried, i'm just getting an empty list10:55
yolandamaybe is the expected status at that point10:57
sambettsyolanda: that is likely an API versioning thing, like in Ironic. You can specify --inspector-api-version 1.810:57
sambettsto bump the default API version10:57
yolandamm, it's likely that the introspection was not even started11:00
openstackgerritparesh sao proposed openstack/ironic master: Implementation for UEFI iSCSI boot for ILO  https://review.openstack.org/46828811:13
*** cjloader has joined #openstack-ironic11:13
*** milan_ has joined #openstack-ironic11:15
*** cjloader has quit IRC11:17
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Setup ansible interface in devstack  https://review.openstack.org/52938311:34
openstackgerritMerged openstack/ironic master: Update links  https://review.openstack.org/53315911:40
*** tiendc has quit IRC11:41
*** pcaruana has joined #openstack-ironic11:44
*** gmonteiro|home has quit IRC11:49
*** gmonteiro|home has joined #openstack-ironic11:57
*** liuzz has quit IRC11:57
*** liuzz has joined #openstack-ironic11:57
*** etingof has quit IRC11:58
*** liuzz has quit IRC12:03
nicodemosmorning, ironic12:07
*** cjloader has joined #openstack-ironic12:12
*** lucasagomes is now known as lucas-hungry12:13
gmonteiro|homeo/ nicodemos12:15
*** jkilpatr has joined #openstack-ironic12:15
*** tesseract has joined #openstack-ironic12:15
*** cjloader has quit IRC12:17
*** jkilpatr has quit IRC12:19
*** jkilpatr has joined #openstack-ironic12:19
*** r-daneel_ has joined #openstack-ironic12:42
*** r-daneel has quit IRC12:43
*** r-daneel_ is now known as r-daneel12:43
*** mrtenio has quit IRC12:47
*** ricardoas-away has quit IRC12:47
*** andreaf has quit IRC12:53
*** andreaf has joined #openstack-ironic12:53
*** pc_m has quit IRC12:56
*** pc_m has joined #openstack-ironic12:58
*** dtantsur|bbl is now known as dtantsur13:01
dtantsuryolanda: it's expected to be empty if no introspections have ever started13:01
openstackgerritDongcan Ye proposed openstack/ironic master: Add meaningful exception in Neutron port show  https://review.openstack.org/53364013:04
yolandadtantsur, yep, introspection list seems to be the right command13:04
*** jaypipes has joined #openstack-ironic13:08
*** dprince has joined #openstack-ironic13:08
*** trown|outtypewww is now known as trown13:08
*** crushil has joined #openstack-ironic13:08
*** etingof has joined #openstack-ironic13:10
*** cjloader has joined #openstack-ironic13:13
*** lucas-hungry is now known as lucasagomes13:13
*** cjloader has quit IRC13:17
*** etingof has quit IRC13:18
*** openstackgerrit has quit IRC13:18
*** etingof has joined #openstack-ironic13:18
*** MattMan_1 has quit IRC13:20
*** MattMan_1 has joined #openstack-ironic13:21
*** r-daneel has quit IRC13:21
*** yangyapeng has quit IRC13:26
TheJuliagood morning everyone13:26
*** yangyapeng has joined #openstack-ironic13:27
dtantsurmorning TheJulia13:27
gmonteiro|homeo/ TheJulia13:29
*** tzumainn has joined #openstack-ironic13:29
* TheJulia sips coffee and tries to wake up13:29
*** serlex has joined #openstack-ironic13:30
*** jaianshu has quit IRC13:31
*** yangyapeng has quit IRC13:32
*** krtaylor has quit IRC13:33
*** lennyb_ has joined #openstack-ironic13:33
*** lennyb_ has quit IRC13:34
*** jcoufal has joined #openstack-ironic13:34
*** rbartal has joined #openstack-ironic13:37
*** r-daneel has joined #openstack-ironic13:38
*** mrtenio has joined #openstack-ironic13:40
*** ricardoas has joined #openstack-ironic13:40
*** jlabarre has joined #openstack-ironic13:44
*** openstackgerrit has joined #openstack-ironic13:48
openstackgerritMerged openstack/ironic master: Remove ironic_tempest_plugin/ directory  https://review.openstack.org/53258513:48
sambettso/ dtantsur, TheJulia, gmonteiro|home13:48
TheJuliagood morning sambetts13:51
openstackgerritMerged openstack/ironic master: Do not validate root partition size for whole disk images in iscsi deploy  https://review.openstack.org/53254813:51
*** mjturek has joined #openstack-ironic13:53
*** gmonteiro|home has quit IRC13:57
*** krtaylor has joined #openstack-ironic13:57
*** yangyapeng has joined #openstack-ironic13:59
*** sasha2 has joined #openstack-ironic14:01
pas-hasambetts: are you the contact point for Cisco CI?14:02
sambettspas-ha: yeah, whats up?14:03
pas-hacan be optimized a bit - it votes on docs-only changes :) e.g. https://review.openstack.org/#/c/525501/914:03
patchbotpatch 525501 - ironic - Add docs for ansible deploy interface14:03
*** jkilpatr has quit IRC14:03
pas-hajust keep the planet greener :)14:04
dtantsurheh14:04
sambettspas-ha: heh, yup good point /me looks into iut14:04
dtantsursambetts and/or TheJulia, could you check https://review.openstack.org/#/c/529383/ please? I still hope to fit the ansible deploy CI in the release14:04
patchbotpatch 529383 - ironic - Setup ansible interface in devstack14:04
*** rbudden has joined #openstack-ironic14:06
*** jkilpatr has joined #openstack-ironic14:06
pas-hadtantsur: I've nudged Vasyl on that :)14:06
dtantsurokay :)14:07
pas-haand then for completeness there are docs for ansible interface https://review.openstack.org/#/c/525501/14:07
patchbotpatch 525501 - ironic - Add docs for ansible deploy interface14:07
*** bfournie has quit IRC14:12
*** hw_wutianwei has joined #openstack-ironic14:12
*** bfournie has joined #openstack-ironic14:12
*** rloo has joined #openstack-ironic14:13
*** athomas has quit IRC14:16
*** athomas has joined #openstack-ironic14:17
*** bfournie has quit IRC14:17
dtantsurI don't have a lot of review throughtput today - Irish visa paperwork :(14:23
sambetts:(14:23
*** r-daneel has quit IRC14:23
* dtantsur -> food14:23
*** makowals has quit IRC14:27
*** makowals has joined #openstack-ironic14:29
pas-hadtantsur: same :(14:43
*** gcb has joined #openstack-ironic14:44
*** bfournie has joined #openstack-ironic14:51
*** jkilpatr has quit IRC14:54
*** jkilpatr has joined #openstack-ironic14:55
*** mjturek has quit IRC15:01
*** mjturek has joined #openstack-ironic15:02
*** racedo_ has joined #openstack-ironic15:02
*** mjturek has quit IRC15:03
*** racedo has quit IRC15:04
*** makowals has quit IRC15:07
*** makowals has joined #openstack-ironic15:09
*** racedo has joined #openstack-ironic15:14
*** racedo_ has quit IRC15:15
*** cjloader has joined #openstack-ironic15:16
*** r-daneel has joined #openstack-ironic15:17
openstackgerritRuby Loo proposed openstack/ironic master: node_tag_exists(): raise exception if bad node  https://review.openstack.org/53370015:19
*** mjturek has joined #openstack-ironic15:20
*** links has quit IRC15:20
*** cjloader has quit IRC15:21
*** cjloader has joined #openstack-ironic15:21
*** cjloader has quit IRC15:21
*** cjloader has joined #openstack-ironic15:21
*** rbartal has quit IRC15:24
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Switch from MechanismDriver to SimpleAgentMechanismDriverBase  https://review.openstack.org/52183815:29
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Devstack - Add ironic-neutron-agent  https://review.openstack.org/53011715:29
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Add support to bind type vlan networks  https://review.openstack.org/53234915:29
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Add dsvm job  https://review.openstack.org/53040915:29
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: start_flag = True, only first time, or conf change  https://review.openstack.org/53370715:29
*** r-daneel has quit IRC15:34
*** r-daneel has joined #openstack-ironic15:35
openstackgerritRuby Loo proposed openstack/ironic master: Add additional context to contribution guide  https://review.openstack.org/52968115:40
pas-hafolks, please review this patch switching ironic-appi to uwsgi https://review.openstack.org/#/c/507067/15:43
patchbotpatch 507067 - ironic - [devstack] Switch ironic to uWSGI15:43
pas-haI know it has -1 from Vlad, but would like to have more opinions / -1s :)15:44
*** rbudden has quit IRC15:45
*** gcb has quit IRC15:54
*** amarao has quit IRC15:54
openstackgerritHugo Nicodemos proposed openstack/ironic master: Migrate oneview hardware type to use python-hpOneView  https://review.openstack.org/52431015:58
rloopas-ha: dumb questiions wrt 507067. is our devstack meant to provide support for 1. mod_wsgi, 2. wsgi; 3. none?15:59
pas-hauwsgi16:00
rloopas-ha: then why does the patch have an IRONIC_USE_MOD_WSGI variable? https://review.openstack.org/#/c/507067/14/devstack/lib/ironic16:01
patchbotpatch 507067 - ironic - [devstack] Switch ironic to uWSGI16:01
rloopas-ha: L7416:01
dtantsurmorning rloo16:01
rloooh yes, good Monday morning everyone including dtantsur, pas-ha!16:01
pas-hayeah, mostly historic reasons I suppose. could be renamed probably tho16:01
* dtantsur gets back to paperwork..16:01
rloopas-ha: if it has to use wsgi, we should just delete that variable. If there is a choice to use or not use, then we'd want a variable.16:02
rloopas-ha: when we deprecate something in ironic, do we need to continue to support it in devstack?16:03
rloopas-ha: am wondering whether it is too early to remove support for mod_wsgi16:03
rloopas-ha: i haven't paid any attention to this stuff, so i think whatever most of the other openstack services do, we should follow16:03
pas-haIMO with devstack we can remove it. actually running it under Apache+mod_wsgi is still supported16:04
rloopas-ha: 'do' -- wrt how they transition from mod-wsgi (if any) to uwsgi16:04
pas-hamost services try to use uwsgi - it makes logs much cleaner, and you don't have to restart the whole apache and all the APIs it already serve to refresh a single service16:05
rloopas-ha: right, i'm not questioning the use of uwsgi. just wondering how much support (if any) we need for mod_wsgi. cuz otherwise, i can't really review that devstack patch. i'm not sure what behaviour we want there.16:06
*** hoonetorg has quit IRC16:09
openstackgerritMerged openstack/ironic master: Setup ansible interface in devstack  https://review.openstack.org/52938316:13
openstackgerritMerged openstack/ironic-tempest-plugin master: Add ansible deploy interface test  https://review.openstack.org/52964016:13
dtantsurwith this ^^^ are we ready for a release?16:14
*** mjturek has quit IRC16:14
TheJuliadid the mapping land?16:20
TheJuliadtantsur: release mapping is still in the gate16:22
*** dsneddon has joined #openstack-ironic16:31
*** hshiina has joined #openstack-ironic16:34
openstackgerritRushil Chugh proposed openstack/ironic master: Add XClarity Driver  https://review.openstack.org/51942716:35
*** makowals has quit IRC16:37
*** ElCoyote_ has joined #openstack-ironic16:39
*** AlexeyAbashkin has quit IRC16:39
*** AlexeyAbashkin has joined #openstack-ironic16:39
*** tesseract has quit IRC16:40
*** AlexeyAbashkin has quit IRC16:47
dtantsur* dtantsur has finished collecting paper work for the Irish visa and feels like getting drunk ASAP16:50
dtantsurbut our meeting is going to prevent it! so don't make me sad, please fill in your statuses16:50
dtantsurTheJulia: yeah, release mapping. good call.16:51
*** stendulker has joined #openstack-ironic16:51
TheJuliadtantsur: on a plus side, we don't really need to wait for the release, since we can just request release of the merge commit for the release mapping patch.16:52
dtantsurTheJulia: the releases repo CI will fail though16:54
*** srart has joined #openstack-ironic16:54
dtantsurso I'll have to watch the patch and recheck it after it merges16:54
TheJuliadtantsur: you would have to wait for it to merge to get the merge commit hash16:55
TheJuliathen the releases repo should become happy16:55
*** rbudden has joined #openstack-ironic16:56
dtantsurthe meeting starts soon!16:59
dtantsur#startmeeting ironic17:00
openstackMeeting started Mon Jan 15 17:00:13 2018 UTC and is due to finish in 60 minutes.  The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot.17:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.17:00
*** openstack changes topic to " (Meeting topic: ironic)"17:00
openstackThe meeting name has been set to 'ironic'17:00
*** mjturek has joined #openstack-ironic17:00
dtantsurhi all!17:00
rpiosoo/17:00
rlooo/17:00
mjtureko/17:00
crushilo/17:00
stendulkero/17:00
TheJuliao/17:00
hshiinao/17:00
dtantsur#link https://wiki.openstack.org/wiki/Meetings/Ironic our agenda17:01
etingofo/17:01
*** baha has joined #openstack-ironic17:01
dtantsur#topic Announcements / Reminder17:01
*** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)"17:01
dtantsur#info Jan 18 - Non-client library freeze17:02
dtantsurthis week!17:02
jlvillalo/ US Holiday for some of us :)17:02
*** marios has quit IRC17:02
dtantsur#link https://review.openstack.org/#/c/529325/ ironic-lib patch to consider finishing17:02
patchbotpatch 529325 - ironic-lib - Add wait_for_disk to destroy_disk_metadata function17:02
dtantsur#link https://review.openstack.org/#/c/471942/ sushy patch to consider finishing17:03
patchbotpatch 471942 - sushy - Implement Redfish Sessions17:03
dtantsurthere are more outstanding sushy patches, all with -1's for quite some time17:03
dtantsur#info Jan 25 - Client release deadline and Feature Freeze17:03
TheJuliaI should have a new version of 529325 up soon.. just fighting the fun of fuser17:03
dtantsurthanks TheJulia17:04
openstackgerritHugo Nicodemos proposed openstack/ironic master: Remove python-oneviewclient from oneview hardware type.  https://review.openstack.org/52472917:04
dtantsurso, by next week we need ironicclient done17:04
rpiosodtantsur: What's the freeze date for vendor drivers?17:04
dtantsurthis particularly includes TheJulia's version negotiation work, esp. its first patch17:04
dtantsurrpioso: the same - feature freeze. we can talk about exceptions, but the date is the same for everyone.17:04
dtantsurJan 2517:04
rpiosodtantsur: ty17:05
dtantsurnp17:05
dtantsurTheJulia: I haven't checked the last version of that ironicclient patch, are we still in disagreement re upper cap on latest?17:05
TheJuliadtantsur: no, we are in agreement now.17:05
dtantsurcool! :)17:05
TheJuliaconceptulaly they match the proposed api-sig spec minus no min/max versions17:06
TheJuliaas arguments17:06
dtantsurTheJulia: okay, we can discuss these separately. I'm more worried about absence of an internal upper limit17:06
dtantsur#info with feature freeze coming, start thinking about potential exceptions you may submit17:07
TheJuliadtantsur: added that as well17:07
dtantsurcool17:07
dtantsurnote that big changes, breaking changes and API additions are unlikely candidates for a FFE17:07
rpiosodtantsur: What's the process for submitting FFE?17:08
dtantsurFFEs will be accepted and discussed on the ML and will have a very short time to finish17:08
dtantsurpost a request to the ML, explain pros, cons and expected risks. mention the ETA for landing, if all goes well.17:09
*** trown is now known as trown|lunch17:09
dtantsuranything else?17:09
rloodtantsur: wrt FFEs, I seem to recall that nova asks for 2 cores to step up to review them. is that something we should do too?17:09
dtantsurrloo: good idea, I like it17:09
dtantsurotherwise it's useless anyway17:10
dtantsurany objections to ^^^?17:10
rlooyup17:10
TheJulianone17:10
stendulkerNone17:10
*** Nisha_Agarwal has joined #openstack-ironic17:10
rloowait... err, oh it was my idea. none :)17:10
dtantsur#info FFEs are requested on the ML. Make sure to include pros, cons, risks and find at least 2 cores agreeing to review it.17:11
dtantsurnote the bribery is illegal in most countries ;)17:11
dtantsuranything else?17:11
rloodtantsur: you still planning on doing releases?17:12
rloodtantsur: ironic and ??17:12
dtantsurironic for now. maybe something else, not sure17:12
dtantsurthe final release will come with a stable/queens branch. IIRC around RC117:12
rloodtantsur: k. this week. maybe today, tomorrow?17:12
dtantsurironic is today or tomorrow, yes17:12
dtantsurmoving on?17:13
rloo+117:13
dtantsur#topic Review action items from previous meeting17:13
*** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)"17:13
dtantsur#link http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-01-08-17.00.html17:13
dtantsurso.. I did not review TheJulia's patches, but I did submit a guideline to API-SIG on how to expose versions in SDKs17:13
dtantsur#link https://review.openstack.org/532814 guideline on how to expose microversions in SDKs17:14
patchbotpatch 532814 - api-wg - Add guideline on exposing microversions in SDKs17:14
*** hoonetorg has joined #openstack-ironic17:14
dtantsurTheJulia: anything new from the bug triage?17:14
dtantsurmjturek: any news re IBM CI with the tempest plugin removed from tree?17:14
TheJuliadtantsur: I last looked at things ?thursday?, and don't remember anything new17:14
dtantsurk thanks TheJulia17:14
mjturekdtantsur yep, it's been restored (thanks mmedvede)17:15
mjturekshould be green since Friday17:15
dtantsurnice! thanks mmedvede and mjturek17:15
dtantsurany comments?17:15
rloothe tempest code was removed on wed, right?17:15
dtantsurit was, or around that17:15
rloosomeone should send email to that email thread, for closure on it.17:15
dtantsurjlvillal: wanna ^^^?17:16
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Add baremetal neutron agent  https://review.openstack.org/45623517:16
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: start_flag = True, only first time, or conf change  https://review.openstack.org/53370717:16
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Switch from MechanismDriver to SimpleAgentMechanismDriverBase  https://review.openstack.org/52183817:16
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Devstack - Add ironic-neutron-agent  https://review.openstack.org/53011717:16
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Add support to bind type vlan networks  https://review.openstack.org/53234917:16
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Add dsvm job  https://review.openstack.org/53040917:16
jlvillaldtantsur, Sure!17:16
mjtureknothing I can think of looking at the diff it was a pretty straight foward change to our pipeline17:16
dtantsur#action jlvillal to announce the final removal of tempest plugins from ironic and inspector trees17:16
dtantsur#topic Review subteam status reports (capped at ten minutes)17:17
*** openstack changes topic to "Review subteam status reports (capped at ten minutes) (Meeting topic: ironic)"17:17
dtantsur#link https://etherpad.openstack.org/p/IronicWhiteBoard starting with line 14717:17
rloodtantsur: the classic driver deprecation. does that need to get in before FFE?17:18
dtantsurwas rescue status really last updated on 18th Dec?17:18
dtantsurrloo: ideally. it's not great to deprecate after FF :)17:18
dtantsurI'll post the patches this week17:19
dtantsur#action dtantsur to post classic driver deprecation patches this week17:19
rloodtantsur: ok. lots to review later this week/next week :-(17:19
dtantsuryeaaahhh17:19
dtantsurtime around FF is never fun17:19
rloowrt traits and rescue. they both have nova components. how much of them need to land before the nova part will be reviewed?17:19
dtantsurrloo: API and ironicclient17:20
dtantsurwhich is essentially everything17:20
rloois it fair to say that if they don't land this week, then no need to hurry to get them into Queens?17:20
rloounless someone asks for FFE on nova side. sigh.17:20
dtantsurAPI has to land this week, for sure17:21
dtantsurwell, I think it's fair to say that the nova part is not making it17:21
*** links has joined #openstack-ironic17:21
dtantsurtraits - maybe, but not rescue, I guess17:21
rloodtantsur: i'm trying to figure out what our/my priorities might be :)17:21
dtantsurI don't think they'll appreciate us coming with a bulk of code 3 days before FF or something17:21
rlooexactlly17:22
dtantsurI'd still prefer to work on rescue and traits this week17:22
*** jralph has quit IRC17:22
dtantsurjust not to lose the pace17:22
dtantsurbut if they don't land this week, the nova part probably won't go to queens17:22
rloook, will see how far we get this week...17:22
*** e0ne has quit IRC17:23
dtantsur#link https://review.openstack.org/#/c/531398/ easy win for your stats17:23
patchbotpatch 531398 - ironic - Make the Python 3 job voting17:23
*** sasha2 has quit IRC17:23
rlooi haven't been paying attention to routed network support (L253). Is there stuff there that needs to get in this week and/or before end of next week?17:23
dtantsurIIRC it has some blocker on nova side. hjensas?17:24
*** sasha2 has joined #openstack-ironic17:24
*** sasha2 has joined #openstack-ironic17:24
rloofor rescue, do we need CI to land too, before nova accepts it?17:25
dtantsurrloo: not sure. but judging from BFV experience, yes17:25
hjensasdtantsur: I am not sure if its a blocker. Placement does not work due to the way they do it in neutron. But the patches solve the problem with ipam.17:25
dtantsurhjensas: let's ask it this way: do we have something to realistically finish before FF in 2 weeks?17:25
dtantsurI mean, something users may benefit from?17:25
hjensasyes, if the patches that are there now land, we can use routed networks (neutron segments plug-in) but we wont have smart placement/scheduling.17:26
hjensasdtantsur: without it we cannot bind ports on segments when routed networks is enabled in neutron.17:27
dtantsurif we cannot bind ports, what exactly can we use?17:27
*** athomas has quit IRC17:28
hjensasdtantsur: with the patches there we can bind.17:28
dtantsurah17:28
dtantsurI'm inclined to exclude networking-baremetal for feature freeze. wdyt?17:29
dtantsurs/for/from/17:29
rloodtantsur: i think we have to ask for a ffe17:29
TheJuliaWithout more cores looking at it, I don't think we have a choice...17:29
*** lucasagomes is now known as lucas-afk17:30
rlooright. so we ask for a ffe, and two cores step up :)17:30
dtantsurokay, let's do it this way17:30
TheJuliaI've already been reviewing, but we basically need a non-redhat core to step up17:30
rloocuz i don't think we/i have time this week for that. but mybe next week...17:30
dtantsurhjensas: I think it's fair to request an FFE for it, if you want to continue17:30
dtantsurI'm in favor of granting such, and I guess sambetts may want to back it17:30
hjensasdtantsur: I will continue working on it.17:31
rloothx hjensas17:31
dtantsurhjensas: please feel free to request an FFE already then17:31
dtantsureveryone ready to move on?17:32
hjensasand reviews are welcome. :) thanks TheJulia, sambetts, jlvillal for the previous ones. :)17:32
rloo+1 (move on)17:32
dtantsur#topic Deciding on priorities for the coming week17:32
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"17:32
mjturekI'd like to propose https://review.openstack.org/#/c/501799/ as a vendor priority if possible. It's had eyes on it, but I'd like a final push to get it merged17:32
patchbotpatch 501799 - ironic-python-agent - Use lshw in place of dmidecode for the default har...17:32
dtantsurso, I think we should start concentrating on things that are making it17:33
dtantsurmjturek: +1 (but please fix the CI)17:33
dtantsuror recheck or whatever17:33
mjturekit's passing CI but except for the non-voting job17:33
*** athomas has joined #openstack-ironic17:34
dtantsurmjturek: meaning, it's not passing ;)17:34
dtantsurI mean, the inspector job is what actually checks your patch17:34
*** athomas has quit IRC17:34
mjturekahhh - okay I'll fix that then17:34
rloodtantsur: ironic-lib patch. i'm not sure about sushy -- someone else will have to vouch for those as priorities. traits and rescue. and julia's client patch(es)?17:34
dtantsurrloo: yep17:34
dtantsurI'd prefer to remove BIOS for now, as it's clearly not making it17:35
rloodtantsur: we also need your migration-to-hardware-types etc patch; i'm guessing maybe that may have to wait til next week to review17:35
jlvillaldtantsur, Should we consider making the inspector job voting for IPA?17:35
rlooyes, unfortunately, bios spec can wait17:35
*** athomas has joined #openstack-ironic17:36
TheJuliaThat looks good to me17:37
*** athomas has quit IRC17:37
mjturekjlvillal: +117:37
TheJuliaheh, changing more :)17:37
dtantsurjlvillal: that's a question. previously we did not want to couple them, right now I'd prefer we do17:37
*** athomas has joined #openstack-ironic17:37
dtantsurlet's wait for the open discussion17:37
dtantsurokay, the list is there. how does it look?17:38
rloodtantsur: lots of juicy things to review :D17:38
TheJuliadtantsur: looks good to me17:38
dtantsur:)17:38
dtantsur#topic Appointing a bug triaging lead for the coming week17:39
*** openstack changes topic to "Appointing a bug triaging lead for the coming week (Meeting topic: ironic)"17:39
dtantsurshould not be a lot of work, unless you want to review old bugs17:39
dtantsuranyone?17:39
mjturekbeen awhile since I've done it, I'd like to volunteer!17:39
dtantsurthanks mjturek :)17:40
dtantsur#action mjturek to lead the bug triaging this week17:40
dtantsur#topic Open discussion17:40
*** openstack changes topic to "Open discussion (Meeting topic: ironic)"17:40
dtantsurshould we discuss making the inspector job voting on ironic and IPA?17:40
rloodtantsur: no. you can decide.17:40
dtantsurpros: covering inspection in ironic and inventory in IPA. cons: moar voting jobs17:41
dtantsurc'mon, aren't we having a voting on voting? :D17:41
jlvillalIf only certain parts of IPA get tested by inspector, it seems like it should17:41
rlooi'm worried about more voting jobs.17:41
rloohave we encountered issues in the past, cuz of lack of these jobs?17:41
rlooi mean ideally, i'd say yes.17:42
TheJuliarloo: I was going to ask jus that.17:42
dtantsuraccording to http://ci-watch.tintri.com/project?project=ironic&time=7+days it's quite stable recently..17:42
rloojust want to be a bit pragmatic. due to situation(s) with jobs/gate/...17:42
*** Nisha_ has joined #openstack-ironic17:42
*** Nisha_Agarwal has quit IRC17:42
dtantsurrloo: well, mjturek's patch may be breaking inspection. I've seen similar cases before as well.17:43
dtantsurI don't recall someone breaking ironic side though17:43
rloodtantsur: it is non-voting now.17:44
rloodtantsur: can we punt this til after queens is released?17:44
dtantsurtotally17:44
dtantsurthen just as a reminder:17:44
dtantsur#info the inspector job on ironic and IPA is relatively stable. please watch its results, especially when a patch modifies the inventory17:45
TheJuliaWarning: 15 minutes17:45
dtantsuranything else to discuss?17:45
rloocrick17:46
* dtantsur curl https://crickets-as-a-service.example.com17:46
rlooets17:46
dtantsurthanks all :D17:46
TheJuliadtantsur: cats-chasing-crickets-as-a-service?17:46
dtantsurOMG17:46
dtantsuryes, totally :D17:46
dtantsur#endmeeting17:46
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic"17:46
openstackMeeting ended Mon Jan 15 17:46:43 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:46
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-01-15-17.00.html17:46
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-01-15-17.00.txt17:46
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-01-15-17.00.log.html17:46
rloodtantsur: you OK if I sort this before I +A?17:47
rloohttps://review.openstack.org/#/c/531398/1/zuul.d/project.yaml17:48
patchbotpatch 531398 - ironic - Make the Python 3 job voting17:48
dtantsurrloo: totally17:48
rloodtantsur: thx :)17:48
openstackgerritRuby Loo proposed openstack/ironic master: Make the Python 3 job voting  https://review.openstack.org/53139817:49
*** links has quit IRC17:50
*** mjturek has quit IRC17:53
TheJuliadtantsur: okay, because I'm also trying to fix a misunderstanding in the ironic-lib patch, looks like it will be later today, at least then we will have unit tests for the utility17:54
dtantsurawesome! anyway I'm going to leave soon. I need to double-check my documentation package for the embassy and get drunk.17:55
TheJuliadtantsur: enjoy the drunkening :)17:56
*** Nisha_ has quit IRC18:01
*** hshiina has quit IRC18:02
*** derekh has quit IRC18:02
*** mjturek has joined #openstack-ironic18:05
*** jkilpatr has quit IRC18:05
rloo%#@#$@#% POST_FAILUREs18:05
dtantsur....18:06
*** serlex has quit IRC18:06
rloodtantsur: https://review.openstack.org/#/c/530723/ and the release mappings patch. thought that they fixed the issue18:06
patchbotpatch 530723 - ironic - Join nodes with traits18:06
rloothis is just a waste of resources. we should have a manual red-button override to merge things18:07
dtantsurwell.. I've seen POST_FAILUREs on different projects18:07
dtantsuror at least a way to recheck only one job18:07
rloodtantsur: oh, how?18:07
dtantsurrloo: how would it work? Not sure, but it does not sound too hard to me to run only one job and overwrite only its result..18:08
rloodtantsur: oh, i meant a way to do that from the patch. yeah, i would think on the infra side, but i dunno.18:09
*** srart has quit IRC18:09
*** sambetts is now known as sambetts|afk18:14
sambetts|afknight all18:15
*** dtantsur is now known as dtantsur|afk18:16
dtantsur|afko/18:16
*** jkilpatr has joined #openstack-ironic18:18
*** milan_ has quit IRC18:19
*** trown|lunch is now known as trown18:19
rloonight sambetts|afk, dtantsur|afk18:19
*** stendulker has quit IRC18:20
-openstackstatus- NOTICE: Zuul has been restarted and has lost queue contents; changes in progress will need to be rechecked.18:23
jlvillal:(18:24
*** srart has joined #openstack-ironic18:24
* jlvillal goes to enjoy his holiday...18:25
rlooenjoy jlvillal !18:25
jlvillalrloo, Thanks. And :( on the Zuul restart above18:25
rloojlvillal: yeah, although i rechecked a few recently. am hoping it'll be faster now cuz there isn't much in the queue.18:26
openstackgerritHarald Jensås proposed openstack/ironic master: Devstack - use neutron segments (routed provider networks)  https://review.openstack.org/53127518:29
openstackgerritHarald Jensås proposed openstack/ironic master: Wait for ironic-neutron-agent to report state  https://review.openstack.org/53163718:29
*** Goneri has joined #openstack-ironic18:31
openstackgerritMerged openstack/ironic-specs master: Update 'Nova rescue/unrescue in Ironic'  https://review.openstack.org/53221118:47
*** e0ne has joined #openstack-ironic18:47
*** mjturek has quit IRC18:47
*** AlexeyAbashkin has joined #openstack-ironic18:50
*** crushil has quit IRC18:53
*** mjturek has joined #openstack-ironic18:53
*** crushil has joined #openstack-ironic18:55
*** AlexeyAbashkin has quit IRC18:55
*** rmcall has joined #openstack-ironic18:55
*** edleafe- has joined #openstack-ironic19:17
*** edleafe has quit IRC19:19
*** edleafe- is now known as edleafe19:19
*** crushil has quit IRC19:19
*** athomas has quit IRC19:30
*** fragatina has joined #openstack-ironic19:33
*** devananda has quit IRC19:34
*** e0ne has quit IRC19:38
*** e0ne has joined #openstack-ironic19:45
*** etingof has quit IRC19:49
*** milan has joined #openstack-ironic19:49
*** AlexeyAbashkin has joined #openstack-ironic19:51
*** jkilpatr has quit IRC19:55
*** AlexeyAbashkin has quit IRC19:55
*** mjturek has quit IRC19:55
rlooTheJulia and anyone else that might be around. What do you think about these '<No description provided>' strings in our .sample? https://docs.openstack.org/ironic/latest/configuration/sample-config.html19:59
*** mjturek has joined #openstack-ironic20:01
TheJuliathat feels new....20:02
rlooTheJulia: it is :-(20:02
rlooTheJulia: i'm trying to figure out hwat to do.20:02
rlooTheJulia: I can update our code to provide descriptions. BUT that change is in oslo.config 5.2.020:03
rlooTheJulia: and our requirements has a lower bound of 5.1.020:03
rlooTheJulia: i suspect if I make the change and someone uses 5.1.0, it won't work20:03
rlooTheJulia: and even if i make the change, any configs from other libraries won't be updated and will have that 'no description' stuff20:04
TheJuliarloo: I would say not worry about it right now, we can always fill it in if the minimum oslo.config version gets bumped to 5.2.0 through g-r20:04
rlooTheJulia: it looks gross.20:05
TheJuliac'est la vie20:05
rlooTheJulia: I noticed it here: https://review.openstack.org/#/c/509342/26/etc/ironic/ironic.conf.sample20:05
patchbotpatch 509342 - ironic - Add rescue related methods to network interface20:05
rlooTheJulia: couldn't see what the actual change was in that patch due to that crap20:05
rloothe good news is that it looks like cisco CI is working20:07
TheJuliarloo: I guess the btter news is that g-r should auto-increment soon20:10
TheJuliabeyond cisco ci20:11
rlooTheJulia: patience is a virtue I guess. I am tempted to send email on the dev list but whatever.20:11
*** e0ne has quit IRC20:21
*** dansmith has quit IRC20:22
*** e0ne has joined #openstack-ironic20:27
*** hw_wutianwei has quit IRC20:31
*** hw_wutianwei1 has joined #openstack-ironic20:31
*** hw_wutianwei1 has quit IRC20:34
*** dansmith has joined #openstack-ironic20:35
*** dansmith is now known as Guest4952220:35
*** fragatina has quit IRC20:35
*** Guest49522 is now known as dansmith20:40
*** AlexeyAbashkin has joined #openstack-ironic20:50
*** jkilpatr has joined #openstack-ironic20:53
*** AlexeyAbashkin has quit IRC20:54
*** deadnull has joined #openstack-ironic20:55
openstackgerritYusef Shaban proposed openstack/sushy master: Implement Redfish Sessions  https://review.openstack.org/47194221:00
*** ElCoyote_ has quit IRC21:05
*** dprince has quit IRC21:06
*** e0ne_ has joined #openstack-ironic21:07
*** e0ne has quit IRC21:09
*** fragatina has joined #openstack-ironic21:13
*** e0ne has joined #openstack-ironic21:14
*** e0ne_ has quit IRC21:15
*** fragatina has quit IRC21:26
*** pcaruana has quit IRC21:30
*** deadnull has quit IRC21:30
*** jappleii__ has joined #openstack-ironic21:32
*** baha has quit IRC21:37
*** VisBits has joined #openstack-ironic21:37
VisBitsWhy are API functions such as port create limited to API version 1.34?   These functions are current but requesting API version 1.9 does not work.   What do I not understand here?21:38
*** mjturek has quit IRC21:41
*** milan has quit IRC21:43
rlooVisBits: cuz we have this concept of versions, here's the version history: https://docs.openstack.org/ironic/latest/contributor/webapi-version-history.html21:44
*** lbragstad has quit IRC21:44
rlooVisBits: features that were added are not available in older versions; they are only avail starting from the API version in which they were added.21:44
VisBitsIs 1.36 the "latest" version?  Why does the tool recommend 1.921:44
rlooVisBits: it depends on what version of ironic you are using. because we didn't want to break existing uses, we made the default be the lowest version.21:45
VisBitsI get that21:45
rlooVisBits: but we're actually changing that in this release, so that it'll use the latest version21:45
VisBitsThat is fantastic news!21:45
rlooVisBits: cuz it caused too much confusion or something21:45
rlooVisBits: or maybe we're changing it so that you have to specify it. i don't recall the details. sec.21:46
rlooVisBits: here you go: https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/cli-default-api-version.html21:46
VisBitsBeautiful, that is great!  Right now 1.34 is  pike correct?21:47
rlooVisBits: and the change has already merged: https://docs.openstack.org/releasenotes/python-ironicclient/unreleased.html21:47
*** lbragstad has joined #openstack-ironic21:47
* VisBits gets on knees and praises21:48
rlooVisBits: yup, 1.34 is pike. this doc helps: https://docs.openstack.org/ironic/latest/contributor/webapi-version-history.html#pike-9-0-021:48
rlooVisBits: i guess your prayers are/will be answered :)21:48
rlooVisBits: but wait... you need to do community service first! :D21:49
VisBitsThank you guys so much for the hard work.  This helps a ton from a config and automation standpoint.  I've been fighting with this for days to silence the outputs21:49
*** AlexeyAbashkin has joined #openstack-ironic21:50
rlooVisBits: you're welcome, glad we didn't do that for nothing :)21:50
rloodtantsur|afk: ^^ you'll be happy to hear a happy user21:50
*** Goneri has quit IRC21:52
*** e0ne has quit IRC21:54
*** AlexeyAbashkin has quit IRC21:54
*** e0ne has joined #openstack-ironic21:59
*** e0ne has quit IRC22:01
*** trown is now known as trown|outtypewww22:05
*** lbragstad has quit IRC22:10
*** fragatina has joined #openstack-ironic22:16
*** lbragstad has joined #openstack-ironic22:19
*** rcernin has joined #openstack-ironic22:21
*** cjloader has quit IRC22:27
*** crushil has joined #openstack-ironic22:40
*** fragatina has quit IRC22:45
openstackgerritMerged openstack/ironic master: Add release 10.0 to release mappings  https://review.openstack.org/53284522:47
openstackgerritMerged openstack/ironic master: Add documentation covering storage multi-attach  https://review.openstack.org/52653122:47
*** jcoufal has quit IRC22:48
*** etingof has joined #openstack-ironic22:51
*** lbragstad has quit IRC23:00
*** cjloader has joined #openstack-ironic23:02
*** hieulq has quit IRC23:02
*** hieulq has joined #openstack-ironic23:03
*** cjloader has quit IRC23:07
*** bfournie has quit IRC23:23
*** crushil has quit IRC23:24
openstackgerritJulia Kreger proposed openstack/ironic-lib master: Add wait_for_disk to destroy_disk_metadata function  https://review.openstack.org/52932523:26
TheJuliaI want liquor because of ^^^23:26
TheJuliajlvillal: please let me know if you feel the unit tests need anything, I think the ones I've added have everything covered23:28
* rloo passes some scotch to TheJulia23:30
rloothx TheJulia, I'll review it tomorrow!23:30
TheJuliathanks rloo!23:31
*** r-daneel has quit IRC23:33
*** lbragstad has joined #openstack-ironic23:53
jlvillalTheJulia, Thanks. I left a review.23:54

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