Tuesday, 2017-08-15

*** Guest6666 has joined #openstack-ironic00:00
*** SpamapS has joined #openstack-ironic00:00
*** honza has joined #openstack-ironic00:00
*** patriciadomin has joined #openstack-ironic00:00
*** igordc has joined #openstack-ironic00:00
*** bcornec has joined #openstack-ironic00:00
*** mtreinish has joined #openstack-ironic00:00
*** ianychoi has joined #openstack-ironic00:00
*** cinerama` has joined #openstack-ironic00:00
*** jaosorior has joined #openstack-ironic00:00
*** actatux has joined #openstack-ironic00:00
*** jcook has joined #openstack-ironic00:00
*** PixieBoots has joined #openstack-ironic00:00
*** alezil has joined #openstack-ironic00:00
*** yolanda has joined #openstack-ironic00:00
*** srart has joined #openstack-ironic00:00
*** zzzeek has joined #openstack-ironic00:00
*** openstackgerrit has joined #openstack-ironic00:00
*** francoblanco has joined #openstack-ironic00:00
*** mordred has joined #openstack-ironic00:00
*** jkilpatr has joined #openstack-ironic00:00
*** early has joined #openstack-ironic00:00
*** lennyb has joined #openstack-ironic00:00
*** jroll has joined #openstack-ironic00:00
*** afazekas has joined #openstack-ironic00:00
*** bfournie has joined #openstack-ironic00:00
*** karthiks has joined #openstack-ironic00:00
*** racedo has joined #openstack-ironic00:00
*** libin_ has joined #openstack-ironic00:00
*** dyasny__ has joined #openstack-ironic00:00
*** d0ugal has joined #openstack-ironic00:00
*** wong has joined #openstack-ironic00:00
*** rybridges has joined #openstack-ironic00:00
*** sasha2 has joined #openstack-ironic00:00
*** sauloaislan has joined #openstack-ironic00:00
*** dsneddon has joined #openstack-ironic00:00
*** chlong_ has joined #openstack-ironic00:00
*** MattMan has joined #openstack-ironic00:00
*** fxpester has joined #openstack-ironic00:00
*** yangyapeng has joined #openstack-ironic00:00
*** rloo has joined #openstack-ironic00:00
*** jcoufal has joined #openstack-ironic00:00
*** mjturek has joined #openstack-ironic00:00
*** rpioso has joined #openstack-ironic00:00
*** krtaylor has joined #openstack-ironic00:00
*** slagle has joined #openstack-ironic00:00
*** nicodemos has joined #openstack-ironic00:00
*** etingof has joined #openstack-ironic00:00
*** hoonetorg has joined #openstack-ironic00:00
*** rwsu has joined #openstack-ironic00:00
*** crushil has joined #openstack-ironic00:00
*** Dinesh_Bhor has joined #openstack-ironic00:00
*** jrist has joined #openstack-ironic00:00
*** exodusftw has joined #openstack-ironic00:00
*** makowals has joined #openstack-ironic00:00
*** dr_gogeta86 has joined #openstack-ironic00:00
*** xavierr has joined #openstack-ironic00:00
*** efried_zzz has joined #openstack-ironic00:00
*** lihi has joined #openstack-ironic00:00
*** oanson has joined #openstack-ironic00:00
*** leyal has joined #openstack-ironic00:00
*** betherly has joined #openstack-ironic00:00
*** TheJulia has joined #openstack-ironic00:00
*** fdegir has joined #openstack-ironic00:00
*** hamzy has joined #openstack-ironic00:00
*** kzaitsev1pi has joined #openstack-ironic00:00
*** jistr|off has joined #openstack-ironic00:00
*** lifeless_ has joined #openstack-ironic00:00
*** huats has joined #openstack-ironic00:00
*** ppiela_ has joined #openstack-ironic00:00
*** portdirect has joined #openstack-ironic00:00
*** rodrigofm has joined #openstack-ironic00:00
*** junbo has joined #openstack-ironic00:00
*** ricardoas has joined #openstack-ironic00:00
*** lhinds has joined #openstack-ironic00:00
*** xnox has joined #openstack-ironic00:00
*** ianw has joined #openstack-ironic00:00
*** logan- has joined #openstack-ironic00:00
*** jmorgan1 has joined #openstack-ironic00:00
*** vishwana_ has joined #openstack-ironic00:00
*** aarefiev_afk has joined #openstack-ironic00:00
*** fbouliane has joined #openstack-ironic00:00
*** patchbot has joined #openstack-ironic00:00
*** skazi has joined #openstack-ironic00:00
*** aolwas has joined #openstack-ironic00:00
*** serverascode has joined #openstack-ironic00:00
*** weshay has joined #openstack-ironic00:00
*** asettle has joined #openstack-ironic00:00
*** dutsmoc has joined #openstack-ironic00:00
*** sdake has joined #openstack-ironic00:00
*** chihhsin has joined #openstack-ironic00:00
*** mguz__ has joined #openstack-ironic00:00
*** sambetts|afk has joined #openstack-ironic00:00
*** anupn has joined #openstack-ironic00:00
*** oomichi has joined #openstack-ironic00:00
*** masber has joined #openstack-ironic00:00
*** sergek_ has joined #openstack-ironic00:00
*** kiseok7 has joined #openstack-ironic00:00
*** dims has joined #openstack-ironic00:00
*** arnewiebalck has joined #openstack-ironic00:00
*** robcresswell has joined #openstack-ironic00:00
*** ptx0 has joined #openstack-ironic00:00
*** andymccr has joined #openstack-ironic00:00
*** jralph has joined #openstack-ironic00:00
*** andyjones2001 has joined #openstack-ironic00:00
*** eglute has joined #openstack-ironic00:00
*** yuanying has joined #openstack-ironic00:00
*** jmccrory has joined #openstack-ironic00:00
*** lucas-afk has joined #openstack-ironic00:00
*** sai has joined #openstack-ironic00:00
*** jtanner has joined #openstack-ironic00:00
*** vsaienk0 has joined #openstack-ironic00:00
*** alex_xu has joined #openstack-ironic00:00
*** greghaynes has joined #openstack-ironic00:00
*** antonym has joined #openstack-ironic00:00
*** hogepodge has joined #openstack-ironic00:00
*** ildikov has joined #openstack-ironic00:00
*** patrickeast has joined #openstack-ironic00:00
*** samueldmq has joined #openstack-ironic00:00
*** mgkwill has joined #openstack-ironic00:00
*** dtantsur|afk has joined #openstack-ironic00:00
*** pas-ha has joined #openstack-ironic00:00
*** UForgotten has joined #openstack-ironic00:00
*** rfolco has joined #openstack-ironic00:00
*** melwitt has joined #openstack-ironic00:00
*** zhenguo has joined #openstack-ironic00:00
*** ionutbalutoiu has joined #openstack-ironic00:00
*** johnthetubaguy has joined #openstack-ironic00:00
*** devananda has joined #openstack-ironic00:00
*** aignatov has joined #openstack-ironic00:00
*** mariojv has joined #openstack-ironic00:00
*** BadCub has joined #openstack-ironic00:00
*** mmedvede has joined #openstack-ironic00:00
*** mgagne has joined #openstack-ironic00:00
*** EmilienM has joined #openstack-ironic00:00
*** aimeeu has joined #openstack-ironic00:00
*** zigo has joined #openstack-ironic00:00
*** JayF has joined #openstack-ironic00:00
*** zer0c00l_ has joined #openstack-ironic00:00
*** lukl has joined #openstack-ironic00:00
*** fungi has joined #openstack-ironic00:00
*** kencjohnston_ has joined #openstack-ironic00:00
*** rajinir has joined #openstack-ironic00:00
*** leifmadsen has joined #openstack-ironic00:00
*** davidlenwell has joined #openstack-ironic00:00
*** keekz has joined #openstack-ironic00:00
*** natorious has joined #openstack-ironic00:00
*** bradjones has joined #openstack-ironic00:00
*** ddurst has joined #openstack-ironic00:00
*** zul has joined #openstack-ironic00:00
*** vdrok has joined #openstack-ironic00:00
*** fyxim has joined #openstack-ironic00:00
*** NobodyCam has joined #openstack-ironic00:00
*** bl0m1 has joined #openstack-ironic00:00
*** aweeks has joined #openstack-ironic00:00
*** jxiaobin has joined #openstack-ironic00:00
*** osdev has joined #openstack-ironic00:00
*** rook has joined #openstack-ironic00:00
*** rook has quit IRC00:00
*** rook has joined #openstack-ironic00:00
*** pc_m has joined #openstack-ironic00:00
*** marlinc has joined #openstack-ironic00:03
*** jroll is now known as Guest1949700:03
*** rook is now known as Guest5218900:03
*** trown has joined #openstack-ironic00:04
*** raginbajin has joined #openstack-ironic00:08
*** baoli has joined #openstack-ironic00:11
*** raginbajin has quit IRC00:11
*** mkrai has joined #openstack-ironic00:13
*** baoli has quit IRC00:15
*** yangyapeng has quit IRC00:15
*** raginbajin has joined #openstack-ironic00:16
*** shobhar_ has joined #openstack-ironic00:18
*** pc_m has quit IRC00:20
*** shobhar_ has quit IRC00:25
*** pc_m has joined #openstack-ironic00:30
*** retr0h has joined #openstack-ironic00:40
*** csmart has joined #openstack-ironic00:40
*** rama_y has joined #openstack-ironic00:45
*** mat128 has joined #openstack-ironic00:50
*** rama_y has quit IRC00:54
*** zhangfei has joined #openstack-ironic00:55
*** hw_wutianwei has joined #openstack-ironic01:01
*** phuongnh has joined #openstack-ironic01:02
*** zhangfei has quit IRC01:09
*** mjturek has quit IRC01:12
*** litao__ has joined #openstack-ironic01:13
*** zzzeek has quit IRC01:14
*** zzzeek has joined #openstack-ironic01:15
*** rama_y has joined #openstack-ironic01:18
*** baoli has joined #openstack-ironic01:19
*** bfournie has quit IRC01:19
*** yangyapeng has joined #openstack-ironic01:19
*** yangyapeng has quit IRC01:20
*** yangyapeng has joined #openstack-ironic01:20
*** rwsu has quit IRC01:21
*** rloo has quit IRC01:22
*** zhangfei has joined #openstack-ironic01:22
*** yangyape_ has joined #openstack-ironic01:29
*** yangyapeng has quit IRC01:30
openstackgerritHoang Trung Hieu proposed openstack/ironic master: Follow up Secure boot support for irmc-pxe driver.  https://review.openstack.org/49339401:45
openstackgerritAnup Navare proposed openstack/ironic-ui master: Allow selecting interfaces while enrolling nodes  https://review.openstack.org/47395701:49
*** trungnv has joined #openstack-ironic01:49
*** baoli has quit IRC01:57
*** chlong_ has quit IRC02:12
*** yangyape_ has quit IRC02:16
*** yangyapeng has joined #openstack-ironic02:17
*** trungnv has quit IRC02:27
*** trungnv has joined #openstack-ironic02:31
*** rama_y has quit IRC02:45
*** jxiaobin has quit IRC02:51
*** jcoufal has quit IRC03:05
*** rama_y has joined #openstack-ironic03:29
*** links has joined #openstack-ironic03:34
*** trungnv has quit IRC03:41
*** trungnv has joined #openstack-ironic04:03
*** d0ugal has quit IRC04:06
*** Guest19497 has quit IRC04:07
*** andymccr has quit IRC04:07
*** Guest19497 has joined #openstack-ironic04:07
*** mat128 has quit IRC04:08
*** andymccr has joined #openstack-ironic04:12
*** rcernin has joined #openstack-ironic04:19
*** d0ugal has joined #openstack-ironic04:19
*** jaganathan has joined #openstack-ironic04:20
*** sasha2 has quit IRC04:49
*** rcernin has quit IRC04:52
*** sivat has joined #openstack-ironic04:57
*** sivat has quit IRC04:57
*** karthiks has quit IRC05:04
*** karthiks has joined #openstack-ironic05:05
openstackgerritVasyl Saienko proposed openstack/ironic master: Wait for cleaning is completed after base smoke tests  https://review.openstack.org/48941805:36
*** rama_y has quit IRC05:41
*** marios has joined #openstack-ironic05:48
*** libin_ has quit IRC05:54
*** libin_ has joined #openstack-ironic05:54
*** leifmadsen has quit IRC05:57
*** leifmadsen has joined #openstack-ironic05:58
*** jtomasek has joined #openstack-ironic06:14
pas-hamorning ironic :)06:16
charcolmorning pas-ha :)06:19
*** mjura has joined #openstack-ironic06:19
openstackgerritVasyl Saienko proposed openstack/ironic master: Wait for cleaning is completed after base smoke tests  https://review.openstack.org/48941806:21
*** rbartal_ has joined #openstack-ironic06:27
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Update the documentation links - install guide  https://review.openstack.org/48379706:49
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Update the documentation links - code comments  https://review.openstack.org/49376006:49
*** mrch has joined #openstack-ironic06:50
*** e0ne has joined #openstack-ironic06:51
*** rcernin has joined #openstack-ironic07:01
openstackgerritVasyl Saienko proposed openstack/ironic master: WIP: wait for resources after destroy  https://review.openstack.org/49376707:03
*** pcaruana has joined #openstack-ironic07:06
*** e0ne has quit IRC07:11
*** jaypipes has joined #openstack-ironic07:12
*** charcol has quit IRC07:18
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Clean up current release notes  https://review.openstack.org/49248907:18
*** racedo has quit IRC07:21
*** aarefiev_afk is now known as aarefiev07:22
*** yangyapeng has quit IRC07:23
openstackgerritVasyl Saienko proposed openstack/ironic master: Wait for cleaning is completed after base smoke tests  https://review.openstack.org/48941807:25
openstackgerritVasyl Saienko proposed openstack/ironic master: WIP: wait for resources after destroy  https://review.openstack.org/49376707:25
*** etingof has quit IRC07:27
*** tesseract has joined #openstack-ironic07:27
openstackgerritD G Lee proposed openstack/ironic master: ironic-conductor can't report heartbeat to database  https://review.openstack.org/47521007:40
*** etingof has joined #openstack-ironic07:41
*** yangyapeng has joined #openstack-ironic07:41
openstackgerritMerged openstack/ironic master: Update the documentation links - install guide  https://review.openstack.org/48379707:42
*** etingof has quit IRC07:47
*** adreznec has quit IRC07:52
*** adreznec has joined #openstack-ironic07:58
*** etingof has joined #openstack-ironic07:59
*** athomas has joined #openstack-ironic08:13
*** ccamacho has joined #openstack-ironic08:20
*** amarao has joined #openstack-ironic08:22
*** priteau has joined #openstack-ironic08:23
*** mgoddard has joined #openstack-ironic08:25
*** lucas-afk is now known as lucasagomes08:26
vdrokmorning ironic, pas-ha !08:26
*** milan has joined #openstack-ironic08:26
vdrokand good morning milan :)08:27
milanmorning Ironic and vdrok! :D08:30
milan#pixiesay -mflexing could use a cup of coffee08:30
PixieBootsᕙʕ⇀ᴥ⇀ʔᕗ: could use a cup of coffee08:30
aarefievmilan: morning08:33
milanmorning aarefiev! :)08:33
aarefievmilan: we've landed pxe filtering drivers concept, right?08:34
milanaarefiev, yup08:34
aarefievmilan: what about release notes :)08:34
milanhehe :D08:34
lucasagomesvdrok, aarefiev mikal morning :D08:35
milanaarefiev, it's not possible to use before the filter reform for iptables O:-)08:35
milanso no releasenotes :-/08:35
milanlucasagomes, morning! :)08:35
vdrokmorning aarefiev and lucasagomes !08:35
aarefievmorning lucasagomes, vdrok :)08:35
milanaarefiev, but I can follow up with a note, no prob, that08:35
milanbut it's going to be an FYI one :D08:36
aarefievmilan: there is new entry point and docs for them08:38
milanhmmmmm08:38
milanfair enough08:38
milanaarefiev, so should that be an other note? cause one can't really configure it08:38
milanyet08:38
aarefievI guess so08:39
milanack08:41
*** amarao has quit IRC08:42
openstackgerritVasyl Saienko proposed openstack/ironic master: Wait for cleaning is completed after base smoke tests  https://review.openstack.org/48941808:44
*** kromanenko has joined #openstack-ironic08:46
openstackgerritVasyl Saienko proposed openstack/ironic master: Wait for cleaning is completed after base smoke tests  https://review.openstack.org/48941808:47
openstackgerritVasyl Saienko proposed openstack/ironic master: WIP: wait for resources after destroy  https://review.openstack.org/49376708:47
*** racedo has joined #openstack-ironic08:48
openstackgerritOpenStack Release Bot proposed openstack/bifrost stable/pike: Update .gitreview for stable/pike  https://review.openstack.org/49379008:50
openstackgerritOpenStack Release Bot proposed openstack/bifrost stable/pike: Update UPPER_CONSTRAINTS_FILE for stable/pike  https://review.openstack.org/49379108:50
openstackgerritOpenStack Release Bot proposed openstack/bifrost master: Update reno for stable/pike  https://review.openstack.org/49379208:50
openstackgerritHoang Trung Hieu proposed openstack/ironic master: Follow up Secure boot support for irmc-pxe driver.  https://review.openstack.org/49339408:50
*** tuanluong has joined #openstack-ironic08:50
*** jaganathan has quit IRC08:51
*** lifeless_ is now known as lifeless08:56
*** gcb has joined #openstack-ironic09:02
*** Nisha_Agarwal has joined #openstack-ironic09:25
openstackgerritVasyl Saienko proposed openstack/ironic master: Do not restart n-cpu during upgrade  https://review.openstack.org/49380709:30
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: Test OPNFV CI verify job. Please ignore.  https://review.openstack.org/36760809:30
*** dsneddon has quit IRC09:33
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Add a relnote for the PXE filter driver  https://review.openstack.org/49381009:34
milanaarefiev, ^09:34
*** e0ne has joined #openstack-ironic09:34
aarefievthanks milan09:36
milannp ;)09:38
*** ChubYann has joined #openstack-ironic09:42
openstackgerritVladyslav Drok proposed openstack/ironic master: [DNM] test multinode grenade failures  https://review.openstack.org/49381209:46
*** Nisha_Agarwal has quit IRC09:53
*** Nisha_Agarwal has joined #openstack-ironic09:54
Nisha_Agarwalafternoon Ironic09:55
Nisha_Agarwalmilan, hi09:59
milanhi Nisha_Agarwal! :)10:00
Nisha_Agarwali addressed ur comments on the patch https://review.openstack.org/42290210:00
patchbotpatch 422902 - ironic - Add the new capabilities to the iLO InspectInterface10:00
milanNisha_Agarwal, me looks, thx!10:00
Nisha_Agarwalthe Jenkins is failing because of grenade job failing10:00
Nisha_AgarwalThanks milan10:01
*** dtantsur|afk is now known as dtantsur10:15
dtantsurmorning ironic10:15
Nisha_Agarwaldtantsur, o/10:16
*** yangyapeng has quit IRC10:22
*** chas has joined #openstack-ironic10:22
openstackgerritSam Betts proposed openstack/ironic master: Switch API ref to use versionadded syntax throughout  https://review.openstack.org/49000810:29
vdrokgood morning dtantsur10:31
*** jkilpatr has quit IRC10:34
dtantsuro/10:34
vdrokdtantsur: https://review.openstack.org/489418 might now fix the issue thanks to vsaienk010:41
patchbotpatch 489418 - ironic - Wait for cleaning is completed after base smoke tests10:41
vdrokhe added the wait for nova resources instead of just waiting for available state for nodes10:41
dtantsurnice!10:42
dtantsurnote that we'll have to change how wait_for_nova_resources work in Queens..10:43
Nisha_Agarwalhi milan10:49
*** amarao has joined #openstack-ironic10:49
*** sambetts|afk is now known as sambetts10:52
sambettsMorning all10:52
*** jkilpatr has joined #openstack-ironic10:52
*** amarao has quit IRC10:54
Nisha_Agarwali just replied to ur comments on the patch10:54
Nisha_Agarwalmilan, ^^^^10:54
*** early has quit IRC10:56
*** Nisha_Agarwal has quit IRC10:56
*** Nisha_Agarwal has joined #openstack-ironic10:57
openstackgerritMerged openstack/ironic-inspector master: Clean up current release notes  https://review.openstack.org/49248910:58
openstackgerritNisha Agarwal proposed openstack/ironic master: Add the new capabilities to the iLO InspectInterface  https://review.openstack.org/42290211:00
openstackgerritVladyslav Drok proposed openstack/ironic master: Set resource class during upgrade  https://review.openstack.org/49307811:02
*** Nisha_ has joined #openstack-ironic11:02
*** Nisha_ has quit IRC11:03
vdrokmorning sambetts11:03
*** Nisha_Agarwal has quit IRC11:04
*** zhangfei has quit IRC11:05
*** Nisha_Agarwal has joined #openstack-ironic11:07
Nisha_Agarwalmilan, hi11:07
Nisha_Agarwalmilan, i updated the https://review.openstack.org/422902 and replied to ur comments on the previous patch set11:08
patchbotpatch 422902 - ironic - Add the new capabilities to the iLO InspectInterface11:08
*** mgoddard has quit IRC11:10
*** early has joined #openstack-ironic11:20
*** Nisha_ has joined #openstack-ironic11:20
*** Nisha_Agarwal has quit IRC11:20
*** phuongnh has quit IRC11:21
openstackgerritAnton Arefiev proposed openstack/python-ironic-inspector-client master: Update Inspector docs links in README  https://review.openstack.org/49383511:22
rpiosoGood morning, ironicers :)11:27
dtantsurmorning rpioso11:28
Nisha_vdrok, Hi11:29
Nisha_vdrok, thanks for the review on https://review.openstack.org/#/c/476895/711:29
patchbotpatch 476895 - ironic - Documetation for 'ilo' hardware type11:29
*** etingof has quit IRC11:29
vdrokmorning rpioso and Nisha_11:29
rpiosoGood morning, dtantsur and vdrok.11:30
Nisha_vdrok, you didnt approve the patch, so are you expecting a patchset here. TheJulia said she is fine with follow up patch for the same comment11:30
vdrokNisha_: I am too, kind of. that's why it's not -111:30
rpiosodtantsur: Thank you for the rechecks.11:31
Nisha_vdrok, :)11:31
rpiosodtantsur: I'm considering tweaking the commit message and release note in that change, https://review.openstack.org/#/c/493421/, and rebasing it while I'm at it.  Gate has continued to fail against it.  Would that be bad form, since it already has a couple of +2s and workflow approved?11:34
patchbotpatch 493421 - ironic - Fix DRAC classic driver double manage/provide11:34
Nisha_vdrok, i/stendulker will place a follow up patch for the same, could you approve the above if its ok?11:34
vdrokok11:35
rpiosodtantsur: I figured I'd improve it while we fix the gate, and rebase the new iDRAC hardware type on it, so it's dependent on that bug fix.11:35
dtantsurrpioso: why do you need rebasing it? we've approved a fix for the CI, so it may come back in shape soon. clean rebases do not remove +2 (though they remove +A), but commit message updates do11:35
dtantsurbut it's up to you in the end11:35
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui stable/pike: Imported Translations from Zanata  https://review.openstack.org/49383911:36
rpiosodtantsur: Thank you.11:38
*** fellypefca has joined #openstack-ironic11:38
*** aparnav has joined #openstack-ironic11:41
*** lucasagomes is now known as lucas-hungry11:41
openstackgerritRichard G. Pioso proposed openstack/ironic master: Fix DRAC classic driver double manage/provide  https://review.openstack.org/49342111:44
TheJuliaGood morning everyone11:45
fellypefcaGood morning, TheJulia :)11:46
dtantsurmorning TheJulia11:47
openstackgerritMerged openstack/ironic master: Add hardware types to support Cisco UCS Servers  https://review.openstack.org/48610011:50
*** litao__ has quit IRC11:51
dtantsurso, folks, I'm planning to request releases for IPA, networking-baremetal, sushy-tools and virtualbmc11:54
dtantsurironic and ironic-inspector will follow tomorrow11:54
dtantsursounds good?11:54
dtantsurTheJulia: ^^11:54
openstackgerritVladyslav Drok proposed openstack/ironic master: [DNM] test multinode grenade failures  https://review.openstack.org/49381211:55
TheJuliasounds good to me11:56
* TheJulia goes off and double checks the IPA queue11:56
fellypefcaNice! :D11:57
TheJulialgtm11:57
dtantsurcool, lemme finish one review first11:59
*** dprince has joined #openstack-ironic12:00
*** Guest19497 is now known as jroll12:03
*** bfournie has joined #openstack-ironic12:03
openstackgerritMerged openstack/bifrost stable/pike: Update .gitreview for stable/pike  https://review.openstack.org/49379012:07
*** tuanluong has quit IRC12:07
*** bfournie has quit IRC12:08
TheJuliaQuestion: https://review.openstack.org/#/c/493597/1/doc/source/admin/upgrade-guide.rst line 176. Does the subject of the nodes remain what you think of as you read that paragraph?12:13
patchbotpatch 493597 - ironic - Follow-up to rolling upgrade docs12:13
dtantsurTheJulia: well, it makes me pause for a second12:15
* TheJulia can't decide if that is good or bad12:15
TheJuliachaotic neutra?12:15
TheJulianeutral12:16
*** mgoddard has joined #openstack-ironic12:16
TheJuliagood morning mgagne12:16
TheJuliaerr12:18
TheJuliamgoddard:12:18
* TheJulia gives up typing12:18
*** libin_ has quit IRC12:22
*** libin_ has joined #openstack-ironic12:22
*** mat128 has joined #openstack-ironic12:25
*** lucas-hungry is now known as lucasagomes12:26
rodrigofmMorning12:28
*** gcb has quit IRC12:30
fellypefcaGood Morning, rodrigofm12:30
*** hw_wutianwei has quit IRC12:34
*** bfournie has joined #openstack-ironic12:36
openstackgerritRichard G. Pioso proposed openstack/ironic master: DNM/WIP: Add 'hardware type' for Dell EMC iDRACs  https://review.openstack.org/49126312:39
openstackgerritVladyslav Drok proposed openstack/ironic master: Set resource class during upgrade  https://review.openstack.org/49307812:41
*** rloo has joined #openstack-ironic12:45
*** jlabarre has joined #openstack-ironic12:47
milanaarefiev, dtantsur, w/r https://review.openstack.org/#/c/493810/1 so I've got the impression better to abandon introducing new relnote; rather fix the conf.py and regenerate the config sample?12:49
openstackgerritJulia Kreger proposed openstack/ironic master: Follow-up to rolling upgrade docs  https://review.openstack.org/49359712:49
patchbotpatch 493810 - ironic-inspector - Add a relnote for the PXE filter driver12:49
dtantsurmilan: yeah (and the docs, if any)12:49
milandtantsur, ack12:50
*** rpioso is now known as rpioso|eat12:50
*** makowals has quit IRC12:52
openstackgerritShivanand Tendulker proposed openstack/ironic master: Documetation for 'ilo' hardware type  https://review.openstack.org/47689512:52
*** makowals has joined #openstack-ironic12:53
*** Nisha_ has quit IRC12:53
*** aparnav has quit IRC12:54
*** makowals has quit IRC12:55
rloohowdy and good morning all yee good folks milan, dtantsur, TheJulia12:56
TheJuliagood morning!12:56
milanmorning rloo, TheJulia and all :D12:57
rloodtantsur: I just -1'd this but realized it is the bot. Sigh. what should we do wrt the title? https://review.openstack.org/#/c/493792/12:57
patchbotpatch 493792 - bifrost - Update reno for stable/pike12:57
*** sasha21 has joined #openstack-ironic12:57
vdrokdtantsur: tested, works fine12:57
dtantsurvdrok: awesome!12:57
milanrloo, you put a smile on my face every time we greet! :)12:57
*** makowals has joined #openstack-ironic12:57
dtantsurrloo: feel free to just fix it here or in a follow-up12:57
rloodtantsur: ok12:58
rloomilan: :)12:58
*** jcoufal has joined #openstack-ironic13:00
*** Nisha_Agarwal has joined #openstack-ironic13:00
*** chas has quit IRC13:01
dtantsurvdrok, vsaienk0: the patch to fix the multinode grenade failed the multinode grenade :(13:02
dtantsuris it worth rechecking even?13:02
vsaienk0dtantsur: we still need to make sure resources are free before start new tests13:02
dtantsurok13:03
openstackgerritNisha Agarwal proposed openstack/ironic master: Add the new capabilities to the iLO InspectInterface  https://review.openstack.org/42290213:05
Nisha_Agarwalmilan, ^^^ fixed the comments13:06
milanNisha_Agarwal, thanks13:06
Nisha_Agarwalmilan, thanks for review13:06
milanNisha_Agarwal, no prob! :)13:07
dtantsurTheJulia, rloo, requested releases for everything but ironic and ironic-inspector13:08
TheJuliaAwesome13:08
rlooyay!13:08
vdrokgood morning rloo13:10
*** mrtenio has joined #openstack-ironic13:10
*** makowals has quit IRC13:10
rloohi vdrok!13:10
Nisha_Agarwaldtantsur, +2 on nova patch .... :)13:11
dtantsuryep, finally :)13:11
Nisha_Agarwalyep13:11
Nisha_Agarwal:)13:11
Nisha_Agarwalvdrok, https://review.openstack.org/476895 patch is updated13:12
patchbotpatch 476895 - ironic - Documetation for 'ilo' hardware type13:12
Nisha_Agarwal:)13:12
openstackgerritRuby Loo proposed openstack/bifrost master: [releasenotes] update  https://review.openstack.org/49387013:13
*** etingof has joined #openstack-ironic13:14
rlooTheJulia: ^^ Do we care to backport releasenotes stuff? stable/pike's version is missing ocata in the index.13:15
dtantsurrloo: release notes are built from master13:16
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: PXE filter options have no effect yet  https://review.openstack.org/49387113:16
milanaarefiev, dtantsur ^13:16
dtantsurthanks13:16
rloodtantsur: good to know! (i can't remember all these details)13:16
*** makowals has joined #openstack-ironic13:17
*** persia has joined #openstack-ironic13:17
* dtantsur brb13:18
openstackgerritMerged openstack/bifrost stable/pike: Update UPPER_CONSTRAINTS_FILE for stable/pike  https://review.openstack.org/49379113:18
openstackgerritMerged openstack/bifrost master: Update reno for stable/pike  https://review.openstack.org/49379213:18
*** etingof has quit IRC13:20
*** aparnav has joined #openstack-ironic13:20
*** mrtenio has quit IRC13:22
*** baoli has joined #openstack-ironic13:24
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Syncing example.conf  https://review.openstack.org/49387213:25
dtantsurvdrok, pas-ha, vsaienk0: I've suddently realized that we have zero release notes for networking-baremetal :(13:25
dtantsurcould you please write a prelude-type release note, introducing the project?13:25
dtantsurI'd include it in the release13:25
milanaarefiev, dtantsur unsure whether it makes sense to fix right now but seems example.conf is out of sync for inspector ^^^13:26
*** jlabarre has quit IRC13:26
openstackgerritVladyslav Drok proposed openstack/ironic master: Do not restart n-cpu during upgrade  https://review.openstack.org/49380713:26
dtantsurthanks milan, let's fix it13:26
dtantsurI suspect we need the same for ironic13:26
vdrokdtantsur: okie13:27
*** jlabarre has joined #openstack-ironic13:28
dtantsurTheJulia, rloo, any of you up for writing a "prelude" for our release notes, noting our biggest achievements?13:28
dtantsurs/our/ironic/ largely13:28
*** makowals has quit IRC13:28
rloodtantsur: oh, i thought that was an honour for the PTL to do? :)13:29
rloodtantsur, TheJulia: I can do it if neither of you want to, just let me know :)13:29
*** makowals has joined #openstack-ironic13:29
*** baoli has quit IRC13:29
dtantsurrloo: this way we can save a few iterations of finishing my English13:29
*** Guest52189 is now known as rook13:29
dtantsurs/finishing/fixing/ (daaaaamn)13:29
* dtantsur has no idea what he is typing13:30
rloodtantsur: ha ha. Your English or my omission of things ;)13:30
TheJuliarloo: Why don't you start it, since after a while things become word soup for me13:30
dtantsur++13:30
TheJuliawhich is how I get missing words in stuff I write13:30
rlooTheJulia, dtantsur: ok, will do after I finish reviewing a patch.13:30
TheJuliarloo: thanks!13:30
dtantsurthanks a lot!13:31
* dtantsur brb for real13:31
milanaarefiev, w/r https://review.openstack.org/#/c/493872/ any suggestions where to?13:33
patchbotpatch 493872 - ironic-inspector - Syncing example.conf13:33
milanoops13:34
milanhttps://review.openstack.org/#/c/493872/13:34
patchbotpatch 493872 - ironic-inspector - Syncing example.conf13:34
milaninteresting https://review.openstack.org/#/c/493871/13:34
patchbotpatch 493871 - ironic-inspector - PXE filter options have no effect yet13:34
*** etingof has joined #openstack-ironic13:35
milancopy paste no longer works on milan 's laptop13:35
milanaarefiev, ^^13:35
*** mjturek has joined #openstack-ironic13:35
aarefievmilan: here ? https://github.com/openstack/ironic-inspector/blob/master/CONTRIBUTING.rst#implementing-pxe-filter-drivers13:38
milanaaah13:38
milanOK13:38
* milan updates13:38
*** chlong_ has joined #openstack-ironic13:38
*** Nisha_Agarwal has quit IRC13:39
vdrokdtantsur: hrm, I see https://docs.openstack.org/networking-baremetal/latest/ instead of http://docs-draft.openstack.org/14/450714/24/check/gate-networking-baremetal-docs-ubuntu-xenial/de4daae//doc/build/html/13:40
openstackgerritDebayan Ray proposed openstack/ironic master: Update proliantutils version for Pike release  https://review.openstack.org/47874113:41
*** deray has joined #openstack-ironic13:44
*** mrtenio has joined #openstack-ironic13:45
*** baoli has joined #openstack-ironic13:46
derayHello and good morning Ironic'ers !13:46
fellypefcaMorning deray13:47
*** Nisha_Agarwal has joined #openstack-ironic13:47
derayfellypefca, g'morning o/13:48
*** makowals has quit IRC13:51
derayTheJulia, hi13:52
deraythis is regarding the comment: https://review.openstack.org/#/c/346683/9/releasenotes/notes/ilo-refresh-firmware-version-on-firmware-update-d6fd02d6b292c906.yaml@613:53
patchbotpatch 346683 - ironic - Refresh firmware versions on firmware update13:53
TheJuliaderay: it might make sense to reply in the revision or in the filed bug13:53
*** crushil has quit IRC13:53
*** mrch has quit IRC13:54
TheJuliaas in adding additional contextual detail13:54
derayTheJulia, sure..13:54
*** makowals has joined #openstack-ironic13:56
*** cdearborn has joined #openstack-ironic13:56
rlooTheJulia: is there a doc (admin?) that describes how to boot from volume?13:57
TheJuliamjturek: ^^^13:58
mjturekrloo: TheJulia: I still haven't finished the draft (other work obligations have been taking priority), I'm hoping to dedicate some time to it today. Sorry :(13:59
mjturekrloo ^^13:59
rloomjturek: oh. cuz i'm writing the prelude to release notes, and wanted to link to that. ok, would be good to have that done, or at least *something* in, even if it isn't complete.14:00
mjturekgot it, if I got something up today would that be alright?14:00
TheJuliarloo: contributor docs on devstack have details14:00
rloomjturek: yes, we'll have to see what state it is in and decide from there. thx!14:01
mjturekwill  ping you when it's up thanks14:01
vsaienk0dtantsur: https://review.openstack.org/#/c/49387914:02
patchbotpatch 493879 - networking-baremetal - Add initial release note14:02
dtantsurvdrok: both links look identical at first glance14:02
dtantsurvsaienk0: thanks, will check after the meeting14:02
vsaienk0thanks!14:02
vdrokdtantsur: doh, the old page was cached in browser :(14:03
dtantsuror the post job was not finished when you checked14:03
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: PXE filter options have no effect yet  https://review.openstack.org/49387114:04
rlooTheJulia: hmm. not ideal, i don't see it (devstack contributor doc) at first glance, and not sure we want to point ot that. i'll put in a placeholder for now...14:04
milanaarefiev, ^^14:04
TheJuliarloo: okay14:04
*** links has quit IRC14:06
*** strigazi is now known as strigazi_off14:07
*** crushil has joined #openstack-ironic14:07
*** strigazi_off is now known as strigazi_OFF14:08
*** sauloaislan has quit IRC14:10
*** amotoki__away is now known as amotoki14:12
*** rama_y has joined #openstack-ironic14:13
*** aparnav has quit IRC14:25
openstackgerritVladyslav Drok proposed openstack/ironic master: [DNM] test multinode grenade failures  https://review.openstack.org/49381214:36
*** efried_zzz is now known as efried14:37
*** pc_m has quit IRC14:42
*** rbartal_ has quit IRC14:43
openstackgerritRuby Loo proposed openstack/ironic master: [reno] Add prelude for Pike release  https://review.openstack.org/49390614:44
rloodtantsur, TheJulia: ^^. There are some TODOs there, folks should vote as to what ought to go in there or not be in there. We did A LOT this cycle!14:45
dtantsurthanks!14:46
rlooTheJulia: wrt https://review.openstack.org/#/c/473717/, do you know if Nguyen will update soon? Is that something we want in? Should someone else update? or should i change it all to nits and approve?14:46
patchbotpatch 473717 - ironic - Add functional API tests for volume connector and ...14:46
*** rama_y has quit IRC14:47
openstackgerritMerged openstack/ironic master: Remove translator assignments from i18n  https://review.openstack.org/44819614:48
openstackgerritMerged openstack/ironic master: Update the documentation links - code comments  https://review.openstack.org/49376014:48
*** rodrigofm has quit IRC14:49
*** pc_m has joined #openstack-ironic14:50
vdrokdtantsur: woohoo http://logs.openstack.org/78/493078/4/check/gate-grenade-dsvm-ironic-ubuntu-xenial/882be1e/logs/new/screen-n-cpu.txt.gz#_2017-08-15_14_18_38_38214:56
dtantsursweet :)14:56
dtantsurthis resource class thing starts converging to something working, apparently14:59
vdrokdtantsur: I also added a depends-on on this one https://review.openstack.org/493898 in the patch testing the multinode grenade issue. as the possible reason is that the weighter actually returns the same value for all nodes, so during concurrent test runs, the same nodes get selected, and then they fall into reschedule15:01
patchbotpatch 493898 - nova (stable/ocata) - [DNM] add random weigher15:01
*** rcernin has quit IRC15:01
vdrokor we could try to increase number of reschedules15:02
vdrokwhat placement reports seems to be correct15:02
*** baoli has quit IRC15:03
vdrokso it does not seem to be the issue with the lack of resources15:03
*** baoli has joined #openstack-ironic15:03
*** jaypipes has quit IRC15:04
TheJuliarloo: I doubt 473717 will be updated today in time for dtantsur to cut the release tomorrow morning. Realistically, it can land when it lands since the release occurs by commit hash.15:06
rlooTheJulia, dtantsur, anyone. We cannot update old renos, right? is it after we branch, or after we do a release?15:07
TheJuliaonly with-in the current release15:07
rlooTheJulia: ok, although that is really too bad.15:07
TheJuliarloo: it is life15:07
rlooTheJulia: if we can land a barebones page, the link will be there in the reno and we can update that actual page later.15:08
TheJuliaYeah, that makes sense15:08
TheJuliaand just link it to master in case someone does look at the file in the past15:09
rloowrt updating renos, it is cuz i was just looking at https://review.openstack.org/#/c/492876/ and i think there are older renos there15:09
patchbotpatch 492876 - ironic - Fix wrong links15:09
*** makowals has quit IRC15:10
TheJuliaour hands are tied15:10
rlooTheJulia: right, that's what i was thinking. Unless all the doc links change again :-(15:10
TheJuliawe can't15:10
*** makowals has joined #openstack-ironic15:11
rlooTheJulia: right. So can we change renos for releases (but not branched yet), or change *only* renos for unrelease, do you know?15:11
TheJuliaones not on another branch I believe15:12
rlooTheJulia: cuz we released 8.0.0 but we haven't branched yet.15:12
TheJuliaI think branch because it looks at the data on the branches when building the index15:12
rlooTheJulia: ok. on my list to update our docs with that info... thx!15:12
dtantsurrloo: initially no release borders could be crossed, but now it looks like crossing numbered releases is ok. I'm not sure about named releases, dhellmann may know more15:14
rloodtantsur: named releases I'm sure are bad, cuz that's where the branching happens. I wish the renos were in folders, per release. would make it easier. oh well.15:14
*** jaypipes has joined #openstack-ironic15:17
dtantsurfolks, I recommend checking https://review.openstack.org/#/c/492964/8/releasenotes/notes/fix-ironic-inventory-d565c77af83c710d.yaml just for your information, in case you see people having problems with Pike15:19
patchbotpatch 492964 - nova - Fix reporting inventory for provisioned nodes in t...15:19
*** baoli has quit IRC15:21
dtantsurrloo: reno in folders would not work well with backports, so dunno.. no perfect solution here, I guess15:21
*** baoli has joined #openstack-ironic15:21
*** baoli has quit IRC15:22
*** stendulker has joined #openstack-ironic15:23
rloodtantsur: yup, it just means we have to get the renos RIGHT the first time :D15:24
*** pcaruana has quit IRC15:24
*** MattMan has quit IRC15:24
dtantsuryeah, maybe15:24
*** MattMan has joined #openstack-ironic15:24
dtantsurmaybe we should actually publish some guidelines on reno15:24
dtantsurlike, using links, fomatting, which section is for what, etc15:24
rloodtantsur: that's a good idea, in fact, one guideline for all openstack projects!15:25
*** mgoddard has quit IRC15:31
*** mjura has quit IRC15:31
*** baoli has joined #openstack-ironic15:32
TheJuliaWe already have some light guidelines I think, but everyone is different, and the reality is that those who are consuming the named releases for productization may be re-packaging release noes15:32
*** links has joined #openstack-ironic15:32
* jlvillal was happy to learn yesterday he has been approved to go to the PTG :)15:33
* jlvillal was also happy that rloo was approved too :)15:34
rloo:)15:34
vdrokmorning jlvillal15:35
jlvillalHi vdrok :)15:35
TheJulia\o/15:35
rlooTheJulia: there are things like do you use ``word``? Honestly, I hope they change the theme or whatever, because it is well known that highlighting with RED is bad.15:35
rloos/well known/known to some I guess/15:36
dtantsurmorning jlvillal, congrats jlvillal and rloo :)15:39
jlvillalHi dtantsur TheJulia milan vsaienk0 rloo mjturek and the rest of Ironic15:40
openstackgerritRichard G. Pioso proposed openstack/ironic master: DNM/WIP: Add 'hardware type' for Dell EMC iDRACs  https://review.openstack.org/49126315:40
*** Nisha_ has joined #openstack-ironic15:41
*** Nisha_Agarwal has quit IRC15:41
milanmorning jlvillal! :)15:41
pas-harpioso|eat: you've forgot to remove `DNM/WIP:` from commit message ;)15:43
TheJuliapas-ha: maybe he won't notice if you update the commit message ? ;)15:44
*** rama_y has joined #openstack-ironic15:44
pas-hahehe :)15:44
*** tesseract has quit IRC15:44
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Add 'hardware type' for Dell EMC iDRACs  https://review.openstack.org/49126315:45
pas-hadone15:45
TheJulia:)15:45
openstackgerritMerged openstack/ironic master: Documetation for 'ilo' hardware type  https://review.openstack.org/47689515:51
*** Nisha_ has quit IRC15:52
*** marios has quit IRC15:52
*** jaypipes has quit IRC15:54
openstackgerritRuby Loo proposed openstack/ironic master: [reno] Add prelude for Pike release  https://review.openstack.org/49390615:54
openstackgerritDebayan Ray proposed openstack/ironic master: Update proliantutils version for Pike release  https://review.openstack.org/47874115:56
NobodyCamGood Morning Ironic'ers :)16:01
*** links has quit IRC16:01
dtantsurmorning NobodyCam16:02
rpioso|eatpas-ha: I hadn't forgotten.16:02
rpioso|eatpas-ha: Please reintstate it.16:02
NobodyCamGood Morning dtantsur16:02
rpioso|eatpas-ha: It needs to pass a custom made job in our 3d party CI before it merges.16:03
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: DNM/WIP: Add 'hardware type' for Dell EMC iDRACs  https://review.openstack.org/49126316:03
rpioso|eatpas-ha: Thank you!16:04
dtantsurrpioso|eat: please keep in mind that you can mark your patches with Workflow - 116:04
dtantsurthis is handy when you're waiting for a CI to pass, but the code seems done16:04
pas-harpioso|eat: done, my bad..16:04
*** rpioso|eat is now known as rpioso16:04
rpiosodtantsur: Thank you for informing me that I have that power :)16:05
rpiosopas-ha: Now worries at all.  My intent wasn't clear.16:05
rpiosos/Now/No/16:05
*** stendulker has quit IRC16:05
*** krtaylor has quit IRC16:06
rpiosodtantsur, pas-ha: The good news is that the custom job run is executing.  We should know soon.16:06
*** e0ne has quit IRC16:07
vdrokmorning NobodyCam !16:07
NobodyCamMorning vdrok :)16:07
*** lucasagomes is now known as lucas-afk16:10
derayNobodyCam, good morning16:11
*** yangyapeng has joined #openstack-ironic16:17
openstackgerritIlya Etingof proposed openstack/ironic-specs master: IPA live logs spec  https://review.openstack.org/49270216:18
NobodyCamGood Morning deray :)16:24
derayNobodyCam, o/16:25
NobodyCam:) how are you today deray16:25
derayNobodyCam, going good, as usual :)16:27
derayNobodyCam, hows life for you?16:28
NobodyCam\o/ Very good :)16:28
NobodyCamdoing good, today :)16:28
deray.. just that for today.. lurking on my couple of open patches. waiting to see them getting blessed ;)16:30
NobodyCamNice :)16:30
deraydo you mind if you have some time? https://review.openstack.org/#/c/346683/  and   https://review.openstack.org/#/c/478741/16:31
patchbotpatch 346683 - ironic - Refresh firmware versions on firmware update16:31
patchbotpatch 478741 - ironic - Update proliantutils version for Pike release16:31
NobodyCamI can Try :)16:32
NobodyCambut may be a little later this after noon16:32
derayNobodyCam, np .. tyt16:32
NobodyCam:)16:32
* milan calls it a day! Night Ironic!16:35
NobodyCamnight milan :)16:35
milanand morning NobodyCam! :D16:36
NobodyCam:)16:36
openstackgerritRuby Loo proposed openstack/ironic master: [reno] update  https://review.openstack.org/49394716:36
rloo^^ I really tried to avoid looking at the release notes...16:37
*** rpioso has quit IRC16:39
*** milan has quit IRC16:40
*** trown is now known as trown|lunch16:44
vdrokdtantsur: ok, so this passed https://review.openstack.org/49381216:44
patchbotpatch 493812 - ironic - [DNM] test multinode grenade failures16:44
vdrokand I see a much more healthy usage of resources16:45
vdrok5 vms on compute 1 and 4 vms on compute 216:45
vdrokinstead of 2 and 316:45
rloogood morning NobodyCam!16:45
rloovdrok: that is great news16:45
vdrokrloo: not really, as I expect nova folks won't be happy about https://review.openstack.org/493898 :)16:46
patchbotpatch 493898 - nova (stable/ocata) - [DNM] add random weigher16:46
NobodyCamGood Morning rloo :)16:46
openstackgerritRuby Loo proposed openstack/ironic master: [reno] Add prelude for Pike release  https://review.openstack.org/49390616:46
*** krtaylor has joined #openstack-ironic16:46
rloovdrok: OH!16:46
dtantsurvdrok: on a related note: I would love a cowsay driver :D16:47
vdrok:) no ponysay please16:47
NobodyCam++++ on cowsay driver16:47
dtantsuryeah, the weigher driver does not look easy for sure..16:47
vdrokI mean, it is simple, but I think the default is all weighers enabled, so if we add this one, it will get enabled eg for libvirt, which doesn not seem good16:48
dtantsurok folks, beer won't drink itself, I have to run now :)16:48
dtantsursee you tomorrow, hopefully being able to release something16:48
vdrokhave a good evening dtantsur :)16:49
dtantsurplease update the whiteboard with whatever still blocks the release16:49
NobodyCamhave a good night dtantsur16:49
dtantsur... and keep rechecking, sigh..16:49
deraydtantsur, good night16:49
*** dtantsur is now known as dtantsur|afk16:49
rloobye dtantsur|afk16:50
*** kromanenko has quit IRC16:53
*** jlabarre has quit IRC16:56
*** yosshy2 has joined #openstack-ironic16:58
*** vishwana_ has quit IRC16:58
*** vishwanathj has joined #openstack-ironic16:58
*** jlabarre has joined #openstack-ironic16:59
*** baoli has quit IRC17:01
*** baoli has joined #openstack-ironic17:02
*** baoli has quit IRC17:02
*** yosshy2 has quit IRC17:07
*** Nisha_ has joined #openstack-ironic17:08
Nisha_vdrok, hi17:08
*** jlabarre has quit IRC17:08
*** jkilpatr has quit IRC17:08
*** jlabarre has joined #openstack-ironic17:10
*** baoli has joined #openstack-ironic17:12
*** kzaitsev1pi has quit IRC17:13
*** fellypefca has quit IRC17:17
*** nicodemos has quit IRC17:18
*** kzaitsev_pi has joined #openstack-ironic17:18
*** fellypefca has joined #openstack-ironic17:19
*** nicodemos has joined #openstack-ironic17:19
openstackgerritVladyslav Drok proposed openstack/ironic stable/ocata: Make nova randomize the results of scheduling  https://review.openstack.org/49396017:20
*** jcoufal has quit IRC17:20
*** ricardoas has quit IRC17:20
*** ricardoas has joined #openstack-ironic17:21
*** jcoufal has joined #openstack-ironic17:21
Nisha_vdrok, deray answered the query on patch https://review.openstack.org/#/c/478741/17:22
patchbotpatch 478741 - ironic - Update proliantutils version for Pike release17:22
vdrokok17:24
openstackgerritVladyslav Drok proposed openstack/ironic master: [DNM] test multinode grenade failures  https://review.openstack.org/49381217:25
openstackgerritRuby Loo proposed openstack/ironic master: [reno] update  https://review.openstack.org/49394717:26
vdrokgood night everyone!17:27
NobodyCamhave a good night vdrok17:27
rloobye vdrok!17:27
derayvdrok, good night :)17:30
*** baoli has quit IRC17:32
*** baoli has joined #openstack-ironic17:33
openstackgerritNisha Agarwal proposed openstack/ironic master: Add the new capabilities to the iLO InspectInterface  https://review.openstack.org/42290217:37
openstackgerritIlya Etingof proposed openstack/ironic-specs master: IPA live logs spec  https://review.openstack.org/49270217:38
openstackgerritMerged openstack/ironic-inspector master: Syncing example.conf  https://review.openstack.org/49387217:38
Nisha_vdrok, i updated the patch17:39
Nisha_oh bye17:39
rloojlvillal: if you still have your handy recheck script, you can add this: https://review.openstack.org/#/c/493421/17:40
patchbotpatch 493421 - ironic - Fix DRAC classic driver double manage/provide17:40
jlvillalrloo: sure I'll start it back up17:41
rloojlvillal: thx, and then maybe we won't need it :)17:41
derayTheJulia, hi17:41
*** mgoddard has joined #openstack-ironic17:42
TheJuliaGreetings17:42
jlvillalrloo: Okay, it is watching17:42
derayTheJulia, just a quick query on the comment: https://review.openstack.org/#/c/478741/5/doc/source/admin/drivers/ilo.rst@1317:42
patchbotpatch 478741 - ironic - Update proliantutils version for Pike release17:42
rloojlvillal: all eyes... :)17:42
* sambetts --> home 17:42
sambettsNight all!17:42
rloonight sambetts!17:43
*** e0ne has joined #openstack-ironic17:43
*** sambetts is now known as sambetts|afk17:43
anupnnight sambetts17:43
Nisha_good night sambetts17:43
sambetts|afko/ Nisha_ anupn rloo17:43
derayTheJulia, should I remove that conformance even when it works for generic redfish driver?17:43
*** trown|lunch is now known as trown17:44
TheJuliaderay: I think so yes, it becomes noise, unless your wanting to push users to the redfish reference driver as an alternative17:44
*** jkilpatr has joined #openstack-ironic17:44
TheJuliaI mean, in theory, you would have a ilo5 hardware type that includes redfish like the existing type that imports ipmi for some features.17:45
TheJuliaAt which point, it makes total sense to delineate and introduce a reference.17:46
TheJuliaderay: Alternatively, I guess you could explicitly state that the redfish driver is an option for ilo5 gear, but that it does not support the ilo specific features.17:47
TheJuliaderay: Does that make sense?17:47
derayTheJulia, okay .. we are **not** planning to have a new ilo5 h/w type unless if we plan to introduce new features based on iLO5. The existing ilo h/w type will be sufficient even for supporting Gen1017:48
derayTheJulia, "Alternatively, I guess you could explicitly state that the redfish driver is an option for ilo5 gear" -> this seems fine to me17:49
*** srobert has joined #openstack-ironic17:52
etingofhey, what if I encountered an out-of-date fact noted in an implemented spec? should I submit a patch against existing spec or come up with a new, updated spec?17:52
etingofin other words: what is the proper way to update a spec?17:53
TheJuliaderay: Sounds good to me17:53
TheJuliaetingof: just propose a fix to the repo17:53
etingofok, thanks!17:53
TheJuliaetingof: if it is really bad, please feel free to get our attention to review it, otherwise we will review it when we have cycles to do spec reviews17:53
derayTheJulia, thanks! will update the patch soon-ish17:53
TheJuliaderay: awesome17:54
etingofTheJulia, no, it is not that bad but may be misleading17:54
TheJuliaetingof: sadly that happens with specs more than anyone would like since they are the document proposing and setting forth the overall idea. Actual implementation sometimes differs in ways that are not caught and fixed in the spec after the fact17:56
TheJuliaI guess it is a perfect example of theory not always being practice :)17:57
etingofTheJulia, the other way to look at that is as if it's the agile dev process at work ;)17:58
TheJuliaetingof: Indeed! Except we don't handle releases that way :(17:58
*** baoli has quit IRC17:59
*** baoli has joined #openstack-ironic18:01
*** baoli has quit IRC18:01
*** [1]rpioso has joined #openstack-ironic18:02
anupnTheJulia, are we having a meeting today?18:05
TheJuliaanupn: I don't think so. I indicated during the last meeting that we would meet again again the week before the PTG.18:07
anupnTheJulia, cool18:07
*** [1]rpioso is now known as rpioso18:09
*** baoli has joined #openstack-ironic18:10
openstackgerritMerged openstack/ironic-inspector master: PXE filter options have no effect yet  https://review.openstack.org/49387118:13
rlooTheJulia: did I ask you this already? the tempest tests for volume are the only 'main' outstanding (apart from nova ones) blocker: https://review.openstack.org/#/c/473717/18:13
patchbotpatch 473717 - ironic - Add functional API tests for volume connector and ...18:13
rlooTheJulia: should I update, or change my vote, or ?18:13
openstackgerritDebayan Ray proposed openstack/ironic master: Update proliantutils version for Pike release  https://review.openstack.org/47874118:17
TheJuliarloo: I don't think we're going to see an update today, so if we create a follow-up, and push that one through, I think we are better off.  You can update if you want though.  Btw, I think we can set uuid for any object we offer up for creation.18:19
rlooTheJulia: ok. if I have time, will fix it today; otherwise will +A.18:20
TheJuliaI need to run an errand, I'll check with you when I get back, I might be able to do it then.18:21
rlooTheJulia: okey dokey18:21
*** baoli has quit IRC18:23
*** baoli has joined #openstack-ironic18:23
*** fellypefca has quit IRC18:27
openstackgerritNisha Agarwal proposed openstack/ironic master: Add the new capabilities to the iLO InspectInterface  https://review.openstack.org/42290218:28
*** e0ne has quit IRC18:29
openstackgerritIlya Etingof proposed openstack/ironic-specs master: Updates to the snmp-power-driver spec  https://review.openstack.org/49397818:36
openstackgerritNisha Agarwal proposed openstack/ironic master: Add the new capabilities to the iLO InspectInterface  https://review.openstack.org/42290218:36
*** mgoddard has quit IRC18:36
openstackgerritNisha Agarwal proposed openstack/ironic master: Add the new capabilities to the iLO InspectInterface  https://review.openstack.org/42290218:38
openstackgerritFellype Cavalcante proposed openstack/ironic master: [WIP/DNM] Updating the minimum version of python-oneviewclient  https://review.openstack.org/49398318:52
openstackgerritIlya Etingof proposed openstack/ironic-specs master: Updates to the snmp-power-driver spec  https://review.openstack.org/49397818:53
rpiosorloo: Thank you for reviewing https://review.openstack.org/#/c/491263/.18:54
patchbotpatch 491263 - ironic - DNM/WIP: Add 'hardware type' for Dell EMC iDRACs18:54
rloorpioso: just get rid of the DNM/WIP :)18:54
rpiosorloo: Are you available to discuss your first comment?18:55
rloorpioso: in a meeting now. in a few min ok?18:55
rpiosorloo: lol re: commit msg18:55
rpiosorloo: Absolutely!18:55
rpiosorloo: I'll be in a meeting then, but that'll be okay.  I'll "multitask".18:56
*** Nisha_ has quit IRC18:58
rloorpioso: ok, my meeting is over18:59
*** ricardoas has quit IRC19:00
rloorpioso: did you still have questions?19:00
rpiosorloo: I do.19:00
rpiosorloo: The "nit" one is not clear to me.19:00
rloorpioso: oh. the test is to override the agent. based on the test name. but you are overriding agent + inspect19:01
rpiosorloo: A couple of other hardware types include a similar test, ilo and irmc.19:01
rloorpioso: and the previous test overrides inspector. so i wondered why you were doing that.19:01
rloorpioso: oh, copy/paste issue then.19:01
rloorpioso: well, it is a nit.you can think about it.19:01
rpiosorloo: Perhaps, but ...19:01
rloorpioso: and people wonder why i am so picky when i review. cuz afterwards we cut/paste :-(19:02
*** baoli has quit IRC19:03
*** srobert has quit IRC19:03
*** jkilpatr has quit IRC19:03
*** nicodemos has quit IRC19:03
*** vishwanathj has quit IRC19:03
*** mrtenio has quit IRC19:03
*** chlong_ has quit IRC19:03
*** leifmadsen has quit IRC19:03
*** d0ugal has quit IRC19:03
*** andymccr has quit IRC19:03
*** trungnv has quit IRC19:03
rpiosorloo: :)19:03
rloorpioso: anyway, as i indicate, it is a nit. you're intelligent; just think about it regardless of where you copy/pasted from.19:03
*** trungnv has joined #openstack-ironic19:03
*** andymccr_ has joined #openstack-ironic19:03
*** mrtenio has joined #openstack-ironic19:03
*** vishwanathj has joined #openstack-ironic19:03
*** baoli has joined #openstack-ironic19:03
*** d0ugal has joined #openstack-ironic19:04
*** leifmadsen has joined #openstack-ironic19:04
*** srobert has joined #openstack-ironic19:04
*** chlong_ has joined #openstack-ironic19:04
rpiosorloo: In our downstream solution, an operator can choose between OOB and in-band introspection.  OOB is the default.  Deploy remains iSCSI.19:04
*** jkilpatr has joined #openstack-ironic19:04
*** pc_m has quit IRC19:04
rpiosoThat's the test_override_with_inspector() test.19:04
rpiosorloo: test_override_with_agent() goes all in with the agent, except RAID, which we don't presently support.19:05
rloorpioso: oh. interesting. so what happens if someone sets deploy to direct and raid to idrac?19:07
rpiosorloo: I didn't just copy and past it :)19:07
rloorpioso: sorry, it was cuz you said the other drivers did it. which usually means someone copy/pasted.19:07
openstackgerritVasyl Saienko proposed openstack/ironic master: WIP: wait for resources after destroy  https://review.openstack.org/49376719:08
rpiosorloo: No worries at all.  Just saying :)19:08
rpiosorloo: I don't believe we've tried that against real h/w.19:09
rpiosorloo: Do you believe there would be an issue?19:09
rpiosorloo: idrac raid is for manual cleaning.19:10
rloorpioso: i don't know, you said above 'except RAID which we don't presently support' :)19:10
*** dsneddon has joined #openstack-ironic19:10
*** dsneddon has quit IRC19:10
rpiosorloo: We don't currently support in-band RAID via the agent.19:11
rpiosorloo: RAID is supported out-of-band via the drac.19:11
openstackgerritNisha Agarwal proposed openstack/ironic master: Add the new capabilities to the iLO InspectInterface  https://review.openstack.org/42290219:12
rloorpioso: so you meant 'except in-band RAID which we don't presently support' ?19:13
rloorpioso: anyway, i think i answered your question?19:13
rpiosorloo: When manual out-of-band RAID cleaning is requested, ironic-conductor PXE boots the agent even if there's nothing for it to do.  If I remember correctly, it shuts down and oob RAID proceeds.19:13
*** dsneddon has joined #openstack-ironic19:13
rpiosorloo: I believe so.  Thanks, again.19:14
rloorpioso: :)19:14
*** Nisha_Agarwal has joined #openstack-ironic19:15
rloorpioso: btw, I would have given a +2 if that commit msg had been updated :)19:17
Nisha_AgarwalTheJulia, Hi19:18
derayvdrok, mind putting +2 vote again on this: https://review.openstack.org/#/c/478741/19:25
patchbotpatch 478741 - ironic - Update proliantutils version for Pike release19:25
derayrloo, TheJulia is you have some time.. then ^^^19:27
openstackgerritMerged openstack/ironic master: iRMC: Support volume boot for iRMC virtual media boot interface  https://review.openstack.org/46791119:27
rpiosorloo: LOL!19:27
rpiosorloo: You'll have another opportunity soon-ish.19:27
rpiosorloo: :)19:27
*** bfournie has quit IRC19:27
rloorpioso: waitin'   :)19:28
*** bfournie has joined #openstack-ironic19:33
*** bfournie has quit IRC19:36
*** dyasny__ has quit IRC19:52
*** Nisha_Agarwal has quit IRC19:58
*** deray has quit IRC19:59
*** dyasny has joined #openstack-ironic20:00
openstackgerritRuby Loo proposed openstack/ironic master: [releasenotes] update irmc's boot-from-volume support  https://review.openstack.org/49400720:02
*** bfournie has joined #openstack-ironic20:06
*** bfournie has quit IRC20:07
*** dyasny has quit IRC20:08
*** mrtenio has quit IRC20:08
*** dyasny has joined #openstack-ironic20:08
*** krtaylor has quit IRC20:08
*** bfournie has joined #openstack-ironic20:09
*** e0ne has joined #openstack-ironic20:24
*** srobert has quit IRC20:27
*** e0ne has quit IRC20:37
rpiosoHas the node show CLI command been updated to include hardware type interfaces?20:40
rpiosos/include/display/20:40
rloorpioso: yes20:42
rloomake sure you have the right microversion20:42
rpiosorloo: How is that done?20:43
rloowhat's your request like?20:43
rloorpioso: https://docs.openstack.org/ironic/latest/contributor/webapi-version-history.html20:44
rlooi just use 'latest' though.20:44
openstackgerritVladyslav Drok proposed openstack/ironic master: [DNM] test multinode grenade failures  https://review.openstack.org/49381220:44
rloorpioso: are you using ironic CLI, openstack baremetal CLI, or sending REST API request?20:44
rpiosorloo: Either CLI works for me.20:45
rloorpioso: export IRONIC_API_VERSION=latest20:45
rpiosorloo: Literally "latest"?20:46
rloorpioso: environment var ^^. can also set it in the request itself but i would have to look that up20:46
rloorpioso: yup, 'latest' :)20:46
rpiosorloo: Awesome!20:46
rpiosorloo: And set that in the shell environment prior to starting the conductor?20:46
rloorpioso: that is for ironic CLI. it is diff for openstack baremetal cli20:46
rloorpioso: has nothing to do with the conductor. you set that envir var wherever you are invoking the request20:47
rloorpioso: used by python-ironicclient20:47
rpiosorloo: Got it.  Thank you so much.20:47
rloorpioso: yw20:47
*** pc_m has joined #openstack-ironic20:50
* TheJulia needs an injection of coffee badly20:51
*** trown is now known as trown|outtypewww20:54
*** rcernin has joined #openstack-ironic20:54
openstackgerritMichael Turek proposed openstack/ironic master: [WIP] Add admin guide for boot from volume  https://review.openstack.org/49401620:55
*** jcoufal has quit IRC20:56
* rloo hands freshly-brewed coffee to TheJulia20:56
openstackgerritMichael Turek proposed openstack/ironic master: [WIP] Add admin guide for boot from volume  https://review.openstack.org/49401620:58
*** baoli has quit IRC21:00
*** baoli has joined #openstack-ironic21:01
*** crushil has quit IRC21:01
*** baoli has quit IRC21:06
openstackgerritJulia Kreger proposed openstack/ironic master: Follow-up to rolling upgrade docs  https://review.openstack.org/49359721:07
TheJuliarloo: ^^^ just fixed the two items you noted21:07
rlooTheJulia: ok21:08
TheJuliajlvillal: ^^^ it is a follow-up to the first doc, if you have a few minutes21:08
jlvillalTheJulia: I need a break from my devstack stuff :)21:09
* rloo hears that jlvillal wants a break... easy one: https://review.openstack.org/#/c/493947/21:10
patchbotpatch 493947 - ironic - [reno] update21:10
rlooTheJulia: I started updating 473717. why did i have so many comments?21:12
*** dprince has quit IRC21:14
*** jcoufal has joined #openstack-ironic21:16
rloojlvillal: you were a bit hasty with 493947. can you remove the +A?21:17
jlvillalrloo: Sure. It looked good when I was reading the release notes and comparing to the changes.21:18
rloojlvillal: yeah, but we probably still want two +2 on it?21:18
jlvillalrloo: Oh yeah, duh21:18
rloojlvillal: :D21:18
jlvillalrloo: I was reading the first patch that had your +2, then went and reviewed your patch.21:19
jlvillalrloo: And was still thinking already had a +2.21:19
rloojlvillal: heh21:19
openstackgerritMichael Turek proposed openstack/ironic master: [WIP] Add admin guide for boot from volume  https://review.openstack.org/49401621:19
TheJuliamjturek: great start, I left a few notes on rev 2. I think a few quick changes and we could land a first revision of it and then iterrate as time goes on.21:24
*** jcoufal_ has joined #openstack-ironic21:25
TheJuliarloo: re 473717, no idea.....21:25
rlooTheJulia: A lot of times I think it would be better (for me) if I didn't review :)21:26
openstackgerritIlya Etingof proposed openstack/ironic-specs master: Updates to the snmp-power-driver spec  https://review.openstack.org/49397821:27
*** mordred has quit IRC21:27
*** mordred has joined #openstack-ironic21:27
TheJuliarloo: I think this is where I pass coffee to you :)21:27
rlooTheJulia: tea. Thanks! :D21:28
TheJuliaright!21:28
* TheJulia makes a nice mint tea and slides it over to rloo 21:28
*** jcoufal has quit IRC21:28
* rloo enjoys the mint tea while editing tests...21:29
*** crushil has joined #openstack-ironic21:29
*** jcoufal_ has quit IRC21:29
*** priteau has quit IRC21:32
*** etingof has quit IRC21:33
openstackgerritAnup Navare proposed openstack/ironic-ui master: Allow selecting interfaces while enrolling nodes  https://review.openstack.org/47395721:37
*** ccamacho has quit IRC21:49
*** etingof has joined #openstack-ironic21:58
*** dsneddon_ has joined #openstack-ironic22:00
*** dsneddon has quit IRC22:03
TheJuliarloo: w/r/t https://review.openstack.org/#/c/494016/2 If you generally concur with my review, I guess I could edit it and we can use it moving forward as admin doc page for bfv and expand upon it, that way the link can go into the prelude22:04
patchbotpatch 494016 - ironic - [WIP] Add admin guide for boot from volume22:04
rlooTheJulia: looking...22:05
*** jlabarre has quit IRC22:07
*** chlong_ has quit IRC22:08
*** rama_y has quit IRC22:08
TheJuliarpioso: Looks like the idrac hw type job failed  :( But I can't access the logs to see why :(22:08
TheJuliarpioso: https://stash.opencrowbar.org/logs/63/491263/4/check/dell-hw14G-tempest-dsvm-ironic-idrac/c20f20822:09
*** dyasny has quit IRC22:19
*** dyasny_ has joined #openstack-ironic22:19
*** rloo has quit IRC22:19
*** rloo has joined #openstack-ironic22:19
*** mjturek has quit IRC22:19
*** mjturek has joined #openstack-ironic22:20
*** rama_y has joined #openstack-ironic22:25
*** Nisha_Agarwal has joined #openstack-ironic22:44
Nisha_AgarwalTheJulia, hi22:44
TheJuliaHi Nisha_Agarwal22:44
Nisha_AgarwalTheJulia, please could you spare time to review https://review.openstack.org/478741 and https://review.openstack.org/42290222:44
patchbotpatch 478741 - ironic - Update proliantutils version for Pike release22:44
patchbotpatch 422902 - ironic - Add the new capabilities to the iLO InspectInterface22:44
openstackgerritRuby Loo proposed openstack/ironic master: Add functional API tests for volume connector and volume target  https://review.openstack.org/47371722:44
Nisha_AgarwalThe jenkins fail because of grenade test failure in both22:44
rlooTheJulia: ^^ updated 473717; I didn't deal with all my comments cuz... :)22:44
rlooTheJulia: will check on it later this evening22:44
TheJulia:)22:44
TheJuliaNisha_Agarwal: the first lgtm, the second needs its reno revised22:50
Nisha_Agarwalcould u comment on it22:53
Nisha_Agarwali will update the patch now22:53
openstackgerritNisha Agarwal proposed openstack/ironic master: Add the new capabilities to the iLO InspectInterface  https://review.openstack.org/42290223:01
Nisha_AgarwalTheJulia, thanks for the review23:01
Nisha_AgarwalTheJulia, addressed the comment and posted the patch23:01
*** rcernin has quit IRC23:12
rpiosoIf memory serves, we had a discussion about vendor extensions to Redfish.  Did something come of that, such as a spec?23:21
Nisha_AgarwalTheJulia, Even this one needs review. If you have some time please review this too...https://review.openstack.org/42257223:26
patchbotpatch 422572 - ironic - Support SUM based firmware update as clean step fo...23:26
Nisha_AgarwalTheJulia, i updated the patch https://review.openstack.org/422902 as per your comment23:26
patchbotpatch 422902 - ironic - Add the new capabilities to the iLO InspectInterface23:26
TheJuliarpioso: I don't think so, because extensions vary by vendor.  I think the idea is to grow upon basic support23:27
Nisha_AgarwalTheJulia, any comments i will update both the patches23:28
rpiosoTheJulia: ty23:29
Nisha_AgarwalTheJulia, Thanks23:30
Nisha_Agarwalrpioso, i kind of agree with TheJulia here...Every vendor has their own extensions23:31
Nisha_Agarwaland maintain them in sushy will be too much for reference driver23:32
*** rama_y has quit IRC23:33
rpiosoNisha_Agarwal: I was under the impression that someone was working on an ironic spec about it.  I seem to recall it was going to be analogous to vendor passthru.23:37
Nisha_Agarwalrpioso, ok if thats the case i am not aware23:37
rpiosoNisha_Agarwal: Likewise, since I don't recall the details :)23:39
Nisha_Agarwalrpioso, i remember that it was discussed in the IRC weekly meeting that vendor extensions willnot b part of sushy23:40
Nisha_Agarwalit has to be different library23:41
rpiosoNisha_Agarwal: Yes, I found that in the meeting logs.  But that's about it.23:41
Nisha_Agarwalrpioso, which can extend sushy for vendor extensions and hence different hardware type also23:42
rpiosoNisha_Agarwal: Has anyone put something like that up for review?23:44
rpiosoAn extension of sushy for vendor-specific extensions?23:44
*** mjturek has quit IRC23:45
Nisha_Agarwalrpioso, like proliantutils extends sushy (imports Sushy) for hpe hardware23:45
rpiosojlvillal: You're a faster model than I -- https://review.openstack.org/493421 ;)23:46
patchbotpatch 493421 - ironic - Fix DRAC classic driver double manage/provide23:46
rpiosojlvillal: Thank you for rechecking that.23:46
Nisha_Agarwalso now all ilo drivers work with redfish hardware23:46
rpiosoNisha_Agarwal: Yes.  I need to take a look at it.23:48
openstackgerritMerged openstack/ironic master: Follow-up to rolling upgrade docs  https://review.openstack.org/49359723:48
rpiosoNisha_Agarwal: I noticed that the ilo h/w type doesn't support the ironic Redfish power and management interfaces.  Does it support Redfish via its IloPower, IloManagement, etc. interfaces?23:50
Nisha_Agarwalyes as of now yes23:50
rpiosoCool23:51
TheJuliaNisha_Agarwal: re: the sum patch, I didn't look at the tests in detail because I've not reviewed it previously and don't have complete context, but I looked at the docs proposed, and raised a few different things.23:52
Nisha_AgarwalTheJulia, thanks a lot23:53
TheJuliarpioso: jlvillal wrote a bot....23:53
rpiosoTheJulia: LOL.  That's awesome.  No wonder it beat me :)23:54
Nisha_AgarwalTheJulia, will address the comments asap.23:56
*** dsneddon_ is now known as dsneddon23:57
TheJuliarpioso: So thinking vendor extensions, I think if something is defined in the agreed upon dmtf standard, and a vendor wants to implement it, I suspect that is okay with sushy, but I also know the standard left quite a bit of room for vendors to add their own things on.23:57

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