Thursday, 2016-03-10

*** openstack has joined #openstack-ironic00:10
*** davideag_ has quit IRC00:14
*** davideagnello has joined #openstack-ironic00:14
*** suro-patz has quit IRC00:15
openstackgerritMerged openstack/bifrost: Make install-ironic skip IPA download if create_ipa_image == true  https://review.openstack.org/29004500:16
*** openstackgerrit has quit IRC00:17
*** openstackgerrit has joined #openstack-ironic00:18
*** suro-patz has joined #openstack-ironic00:20
*** suro-patz has quit IRC00:20
*** rbudden has quit IRC00:20
*** spandhe has quit IRC00:25
*** jaybeale has quit IRC00:25
*** dims has quit IRC00:35
*** thrash is now known as thrash|g0ne00:37
*** chenke_ has joined #openstack-ironic00:44
*** ppiela_ has joined #openstack-ironic00:44
*** lekha_ has joined #openstack-ironic00:46
*** frontrunner2 has joined #openstack-ironic00:46
*** evgenyl_ has joined #openstack-ironic00:46
*** Nakato_ has joined #openstack-ironic00:48
*** kwazar_ has joined #openstack-ironic00:48
*** eglute has quit IRC00:49
*** evgenyl has quit IRC00:49
*** ppiela has quit IRC00:49
*** mkoderer__ has quit IRC00:49
*** kwazar has quit IRC00:49
*** harlowja_at_home has quit IRC00:49
*** zzzeek has quit IRC00:49
*** frontrunner has quit IRC00:49
*** chenke has quit IRC00:49
*** lekha has quit IRC00:49
*** Nakato has quit IRC00:49
*** mkoderer___ has joined #openstack-ironic00:49
*** eglute_s has joined #openstack-ironic00:49
*** evgenyl_ is now known as evgenyl00:49
*** harlowja_at_home has joined #openstack-ironic00:50
*** zzzeek has joined #openstack-ironic00:50
*** lekha_ is now known as lekha00:51
*** harshs has quit IRC00:56
*** garthb_ has quit IRC01:03
*** yonglihe has joined #openstack-ironic01:15
*** izaakk has quit IRC01:17
*** Sukhdev has joined #openstack-ironic01:20
*** Sukhdev has quit IRC01:22
*** Sukhdev has joined #openstack-ironic01:24
SukhdevHey ironic folks, I want to delete a node which is in error state and in maintenance mode - how do I do it?01:25
*** Sukhdev has quit IRC01:27
*** Sukhdev has joined #openstack-ironic01:28
devanandaSukhdev: should be just "ironic node-delete"01:30
Sukhdevdevananda : no it does not let me do it01:30
devanandaSukhdev: what error do you get?01:30
*** dims has joined #openstack-ironic01:31
Sukhdevdevananda : here is the error - http://paste.openstack.org/show/489941/01:32
*** persia has quit IRC01:35
*** persia has joined #openstack-ironic01:38
*** Sukhdev has quit IRC01:39
*** Sukhdev has joined #openstack-ironic01:39
HaomengSukhdev: you can try to node-set-maintenance first, then delete again01:41
*** Sukhdev has quit IRC01:41
*** Sukhdev has joined #openstack-ironic01:42
*** kan_ has joined #openstack-ironic01:42
*** mtanino has quit IRC01:43
*** jaybeale has joined #openstack-ironic01:43
*** Sukhdev has quit IRC01:46
*** Sukhdev_ has joined #openstack-ironic01:46
*** Sukhdev has joined #openstack-ironic01:49
*** Sukhdev_ has quit IRC01:49
*** Sukhdev has quit IRC01:51
*** Sukhdev has joined #openstack-ironic01:52
*** Sukhdev has quit IRC01:53
*** Sukhdev has joined #openstack-ironic01:54
*** Sukhdev has quit IRC01:56
*** Sukhdev has joined #openstack-ironic01:56
*** Sukhdev has quit IRC01:58
*** Sukhdev has joined #openstack-ironic01:59
*** Sukhdev has quit IRC02:01
*** Sukhdev has joined #openstack-ironic02:01
devanandaSukhdev: I can not reproduce that error02:15
devanandaSukhdev: http://paste.openstack.org/show/kIlA6DfXR6MNPzYTaaeV/02:16
devanandaSukhdev: so I need more information to help you figure this out02:16
Sukhdevdevananda : I tried it again and it worked02:16
devananda:)02:16
Sukhdevdevananda: its like when you go to Dr. you start to feel better - same syndrome :-)02:16
devanandahehehe02:18
*** hoangcx has joined #openstack-ironic02:18
*** Sukhdev_ has joined #openstack-ironic02:24
*** Sukhdev has quit IRC02:24
*** baoli has joined #openstack-ironic02:27
*** Sukhdev_ has quit IRC02:30
*** Sukhdev has joined #openstack-ironic02:31
openstackgerritbin proposed openstack/ironic: fix bug virtualbox baremetal machine cannot boot from local disk  https://review.openstack.org/29095102:33
*** Sukhdev has quit IRC02:35
*** Sukhdev has joined #openstack-ironic02:35
*** jaybeale has quit IRC02:38
*** Sukhdev has quit IRC02:38
*** Sukhdev has joined #openstack-ironic02:39
*** baoli has quit IRC02:40
openstackgerritMerged openstack/ironic: Remove 'grub2' option in creating whole-disk-images  https://review.openstack.org/28979102:40
*** Sukhdev has quit IRC02:42
*** Sukhdev has joined #openstack-ironic02:42
*** Sukhdev has quit IRC02:45
*** Sukhdev has joined #openstack-ironic02:46
*** Sukhdev has quit IRC02:48
*** Sukhdev has joined #openstack-ironic02:49
*** Sukhdev has quit IRC02:52
*** Sukhdev has joined #openstack-ironic02:52
*** Marga_ has quit IRC02:54
*** Sukhdev has quit IRC02:55
*** Sukhdev_ has joined #openstack-ironic02:55
*** Marga_ has joined #openstack-ironic02:56
*** Sukhdev_ has quit IRC02:58
*** Sukhdev has joined #openstack-ironic02:59
*** dansmith has quit IRC03:01
*** Marga_ has quit IRC03:01
*** dansmith has joined #openstack-ironic03:01
*** Sukhdev has quit IRC03:01
*** Sukhdev has joined #openstack-ironic03:02
*** Sukhdev has quit IRC03:05
*** Sukhdev_ has joined #openstack-ironic03:05
*** Sukhdev_ has quit IRC03:08
*** Sukhdev has joined #openstack-ironic03:08
*** Sukhdev_ has joined #openstack-ironic03:11
*** Sukhdev has quit IRC03:11
*** Sukhdev_ has quit IRC03:14
*** dims has quit IRC03:14
*** Sukhdev has joined #openstack-ironic03:14
*** baoli has joined #openstack-ironic03:14
*** Sukhdev has quit IRC03:15
*** bapalm has quit IRC03:24
*** baoli has quit IRC03:24
*** Nisha has joined #openstack-ironic03:42
*** bapalm has joined #openstack-ironic03:43
*** Marga_ has joined #openstack-ironic03:50
*** links has joined #openstack-ironic03:50
*** Marga_ has quit IRC03:51
*** Marga_ has joined #openstack-ironic03:51
*** frontrunner2 has quit IRC03:58
*** dims has joined #openstack-ironic04:01
*** Nisha_away has joined #openstack-ironic04:12
*** Nisha has quit IRC04:12
*** dims has quit IRC04:13
*** Nisha_brb has joined #openstack-ironic04:16
*** Nisha_away has quit IRC04:16
*** Nisha_away has joined #openstack-ironic04:17
*** Nisha_brb has quit IRC04:17
*** spandhe has joined #openstack-ironic04:29
*** amotoki has joined #openstack-ironic04:38
*** kalpase has joined #openstack-ironic04:38
*** harlowja_at_home has quit IRC04:39
*** harlowja_at_home has joined #openstack-ironic04:39
*** kragniz has quit IRC04:52
*** kragniz has joined #openstack-ironic04:54
*** heyongli has joined #openstack-ironic04:57
*** heyongli has quit IRC05:00
*** heyongli has joined #openstack-ironic05:00
*** vinm213 has joined #openstack-ironic05:01
*** kragniz has quit IRC05:05
*** kragniz has joined #openstack-ironic05:05
*** heyongli has quit IRC05:10
*** heyongli has joined #openstack-ironic05:10
*** heyongli has quit IRC05:20
*** heyongli has joined #openstack-ironic05:20
*** heyongli has quit IRC05:30
*** baoli has joined #openstack-ironic05:36
*** harlowja_at_home has quit IRC05:36
*** kalpase has quit IRC05:40
*** dansmith has quit IRC05:40
*** yonglihe has quit IRC05:40
*** evgenyl has quit IRC05:40
*** cfarquhar has quit IRC05:40
*** baoli has quit IRC05:40
*** kalpase has joined #openstack-ironic05:46
*** dansmith has joined #openstack-ironic05:46
*** yonglihe has joined #openstack-ironic05:46
*** evgenyl has joined #openstack-ironic05:46
*** cfarquhar has joined #openstack-ironic05:46
*** suro-patz has joined #openstack-ironic05:47
*** Nisha_away has quit IRC05:57
*** appu103_ has joined #openstack-ironic06:00
*** appu103_ is now known as aparnav06:00
*** suro-patz has quit IRC06:02
*** kalpase has quit IRC06:02
*** dansmith has quit IRC06:02
*** yonglihe has quit IRC06:02
*** evgenyl has quit IRC06:02
*** cfarquhar has quit IRC06:02
*** suro-patz has joined #openstack-ironic06:08
*** kalpase has joined #openstack-ironic06:08
*** dansmith has joined #openstack-ironic06:08
*** yonglihe has joined #openstack-ironic06:08
*** evgenyl has joined #openstack-ironic06:08
*** cfarquhar has joined #openstack-ironic06:08
*** bapalm has quit IRC06:23
*** praneshp_ has joined #openstack-ironic06:27
*** praneshp has quit IRC06:29
*** praneshp_ is now known as praneshp06:29
*** shu-mutou has quit IRC06:33
*** bapalm has joined #openstack-ironic06:42
*** suro-patz has quit IRC06:45
*** praneshp has quit IRC06:53
*** praneshp has joined #openstack-ironic06:54
*** shu-mutou has joined #openstack-ironic06:56
*** spandhe has quit IRC06:58
*** shu-mutou is now known as shu-mutou-AFK07:00
*** praneshp has quit IRC07:07
*** ohamada has joined #openstack-ironic07:13
*** moshele has joined #openstack-ironic07:14
*** vishwanathj is now known as vishwanathj_zzz07:15
openstackgerrityolanda.robla proposed openstack/bifrost: Only pause in VMs when performing multinode testing  https://review.openstack.org/29053907:20
*** ohamada has quit IRC07:23
*** hoangcx has quit IRC07:23
*** hoangcx has joined #openstack-ironic07:33
*** rcernin has joined #openstack-ironic07:33
*** pece has joined #openstack-ironic07:42
*** baoli has joined #openstack-ironic07:48
*** baoli has quit IRC07:53
*** e0ne has joined #openstack-ironic07:53
*** Nisha has joined #openstack-ironic07:56
*** ifarkas has joined #openstack-ironic08:02
*** daemontool_ has joined #openstack-ironic08:11
*** hoangcx has quit IRC08:12
*** daemontool has quit IRC08:14
*** hoangcx has joined #openstack-ironic08:17
*** pas-ha has joined #openstack-ironic08:20
*** e0ne has quit IRC08:21
*** vaishnavi has joined #openstack-ironic08:24
*** e0ne has joined #openstack-ironic08:25
*** stendulker has joined #openstack-ironic08:28
*** mbound has joined #openstack-ironic08:31
*** davideagnello has quit IRC08:32
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Add discover nodes exercise  https://review.openstack.org/27610708:34
*** pcaruana has joined #openstack-ironic08:34
*** vishwanathj_zzz has quit IRC08:34
*** vishwanathj_zzz has joined #openstack-ironic08:35
*** davideagnello has joined #openstack-ironic08:35
*** permalac has joined #openstack-ironic08:35
*** permalac has quit IRC08:35
*** betherly has quit IRC08:36
*** romcheg has quit IRC08:36
*** sylwesterB has quit IRC08:36
*** cppforlife_ has quit IRC08:36
*** zhenguo_ has quit IRC08:36
*** agordeev has quit IRC08:36
*** zzzeek has quit IRC08:36
*** zzzeek has joined #openstack-ironic08:38
*** betherly has joined #openstack-ironic08:40
*** permalac has joined #openstack-ironic08:41
*** permalac has quit IRC08:41
*** romcheg has joined #openstack-ironic08:41
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Don't power off node in enroll state  https://review.openstack.org/29105308:41
*** e0ne has quit IRC08:42
*** cppforlife_ has joined #openstack-ironic08:44
*** zhenguo_ has joined #openstack-ironic08:44
*** sylwesterB has joined #openstack-ironic08:44
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Fix arg for "Port already exists" error  https://review.openstack.org/29105908:47
*** agordeev has joined #openstack-ironic08:48
*** wuhao has quit IRC08:49
*** wuhao has joined #openstack-ironic08:49
*** ohamada has joined #openstack-ironic08:55
*** ishant has joined #openstack-ironic08:58
*** permalac_ has joined #openstack-ironic09:00
*** permalac_ has quit IRC09:00
*** ndipanov has joined #openstack-ironic09:03
yuriyzmorning Ironic09:03
aarefievmorning all, yuriyz09:04
openstackgerritjxiaobin proposed openstack/ironic: Pass region_name to SwiftAPI  https://review.openstack.org/29106309:05
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Fix arg for "Port already exists" error  https://review.openstack.org/29105909:05
*** mgoddard has joined #openstack-ironic09:06
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Tests for testing node-create command  https://review.openstack.org/26205509:07
*** jistr has joined #openstack-ironic09:09
*** dtantsur|afk is now known as dtantsur09:11
*** jcoufal has joined #openstack-ironic09:12
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/29064409:14
*** lucas-dinner is now known as lucasagomes09:14
lucasagomesmorning all09:14
dtantsurmorning Ironic09:14
* dtantsur is on the RDO test day, so is only partially available09:14
*** amotoki has quit IRC09:15
*** sturivnyi has joined #openstack-ironic09:15
openstackgerritDebayan Ray proposed openstack/ironic: Follow-up of firmware update(iLO) as manual cleaning step  https://review.openstack.org/29106909:23
*** openstackgerrit has quit IRC09:30
*** openstackgerrit_ has joined #openstack-ironic09:31
*** openstackgerrit_ is now known as openstackgerrit09:31
*** openstackgerrit has quit IRC09:31
*** openstackgerrit_ has joined #openstack-ironic09:32
*** openstackgerrit_ is now known as openstackgerrit09:32
*** openstackgerrit has quit IRC09:32
*** openstackgerrit_ has joined #openstack-ironic09:33
*** openstackgerrit_ is now known as openstackgerrit09:33
*** openstackgerrit has quit IRC09:33
*** e0ne has joined #openstack-ironic09:33
*** openstackgerrit_ has joined #openstack-ironic09:34
*** openstackgerrit_ is now known as openstackgerrit09:34
openstackgerritMerged openstack/ironic-inspector: Refactor service init, shutdown and run into a separate class  https://review.openstack.org/28933809:40
openstackgerritMerged openstack/ironic-inspector: Remove redundant data copying in tests  https://review.openstack.org/28994109:40
dtantsurmerge party \o/09:40
*** electrofelix has joined #openstack-ironic09:41
*** _degorenko|afk is now known as degorenko09:42
openstackgerritMerged openstack/ironic-inspector: Officially deprecate rollback for introspection rules  https://review.openstack.org/28990909:45
*** daemontool__ has joined #openstack-ironic09:46
openstackgerritMerged openstack/ironic-inspector: Generate API documentation from Python modules  https://review.openstack.org/28993509:48
openstackgerritMerged openstack/ironic-inspector: Fixed warnings during the docs build  https://review.openstack.org/28993009:48
openstackgerritMerged openstack/ironic-inspector: Update the troubleshooting guide  https://review.openstack.org/28992809:48
openstackgerritMerged openstack/ironic-inspector: Clarify that we don't maintain compatibility for stored data  https://review.openstack.org/28655609:49
*** daemontool_ has quit IRC09:49
*** amotoki has joined #openstack-ironic09:50
aarefievdtantsur: hey, please trigger this one  https://review.openstack.org/#/c/288355/09:51
aarefievand party will go on :)09:51
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support  https://review.openstack.org/26425709:51
dtantsuraarefiev, it can't merge cause it depends on a patch which is not approved.. if you rebase it against master, we probably can merge it :)09:53
*** mgould has joined #openstack-ironic09:53
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add documentation for the IPMITool driver  https://review.openstack.org/29071809:53
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add documentation for the IPMITool driver  https://review.openstack.org/29071809:56
* lucasagomes fix a typo09:56
aarefievdtantsur: ohh my bad, I forget I've rebased it on top of test script to see changes09:57
dtantsur:)09:58
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Allow specify log level for Error exception  https://review.openstack.org/28835509:59
aarefievlong weekend :)10:00
dtantsurgood for you :)10:02
*** Marga_ has quit IRC10:03
lucasagomesNisha, re RAID documentaiton. There's a broken link caused by a typo in the ilo.rst10:04
lucasagomesNisha, other than that it looks very good, thank you!10:04
Nishalucasagomes, let me see10:04
Nishalucasagomes, can u review IPA patch also10:05
NishaJayF  has reviewed it10:05
Nishaironic patch i will update in some time10:05
lucasagomesNisha, it's on my list10:05
Nishalucasagomes, thanks :) i am just hoping it gets in this cycle10:06
*** Nisha has quit IRC10:06
*** Nisha has joined #openstack-ironic10:07
Nishalucasagomes, thanks10:09
openstackgerritNisha Agarwal proposed openstack/ironic: Add documentation for RAID  https://review.openstack.org/22633010:09
Nishaupdated the raid patch10:09
Nisha^^^10:09
*** aparnav has quit IRC10:10
*** sambetts|afk is now known as sambetts10:11
sambettsMorning all10:11
Nishasambetts, o/10:11
sambettshi Nisha10:12
Nisha:)10:13
*** derekh has joined #openstack-ironic10:13
dtantsurmorning sambetts10:13
dtantsurifarkas, sambetts, so I'm thinking we could have an inspector release today afternoon, and start a short feature freeze period10:14
dtantsurwdyt?10:14
sambettsdtantsur: yeah, makes sense, do we want to the the discovery devstack exercise merged first?10:14
dtantsursambetts, we can merge it during feature freeze, it's not a feature10:14
dtantsurbut yeah, I would love to merge it before :)10:15
ifarkasdtantsur, sounds good to me10:15
sambettsdtantsur: then I'm good with it :)10:15
aarefievsambetts: morning, I'm on discovery exercise now :)10:16
sambettsawesome :D10:16
dtantsursambetts, ifarkas, also we now have a non-voting iPXE gate which I'm fixing/implementing :)10:16
sambettssweet!10:17
dtantsurWIP patch: https://review.openstack.org/#/c/290013/10:17
dtantsurifarkas, sambetts, the last feature I'm actually waiting for is https://review.openstack.org/#/c/289825/10:18
dtantsurjtomasek promised me to finish it today :)10:18
sambettsdtantsur: I did wonder about if we wanted to get that in but as it had negative feedback i didn't bring it up :-P10:19
dtantsurUI folks are pretty keen on it, and it does not look risky.. so if we get it finished this afternoon, it's in10:20
sambettsdtantsur: yeah I'd like to get it in its a big deal for the UI guys10:20
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Tests for testing node-create command  https://review.openstack.org/26205510:21
*** hoangcx has quit IRC10:32
*** athomas has joined #openstack-ironic10:34
*** saibarspeis has joined #openstack-ironic10:36
mgouldmorning Ironic10:38
sambettshey mgould10:39
mgouldmorning sambetts10:39
vdrokgood morning ironic and morning aarefiev lucasagomes dtantsur Nisha sambetts and mgould !10:40
*** appu103_ has joined #openstack-ironic10:40
mgouldmorning vdrok aarefiev lucasagomes dtantsur Nisha!10:40
Nishavdrok, o/10:41
Nishamorning10:41
aarefievmgould, vdrok: morning10:41
openstackgerritVaishnavi proposed openstack/ironic: Add support for building ISO for deploy ramdisk  https://review.openstack.org/26459010:43
openstackgerritVaishnavi proposed openstack/ironic: Add support for dib based agent ramdisk in lib/ironic  https://review.openstack.org/26457910:43
Nishamgould, o/10:43
lucasagomesvdrok, mgould o/10:43
*** stendulker has quit IRC10:45
*** dims has joined #openstack-ironic10:47
*** Marga_ has joined #openstack-ironic10:47
*** bapalm has quit IRC10:51
*** dims has quit IRC10:52
openstackgerritNisha Agarwal proposed openstack/ironic-python-agent: Fixes the _install_grub()  https://review.openstack.org/29109710:55
*** dims has joined #openstack-ironic10:56
openstackgerritJane.zhang proposed openstack/ironic-python-agent: Stop tgtd before creating iscsi target via lio.  https://review.openstack.org/29033511:05
*** bapalm has joined #openstack-ironic11:08
sturivnyidtantsur, Haomeng: Hello. Would be grateful if you could review CLI tests patches https://review.openstack.org/#/c/240944/ https://review.openstack.org/#/c/251458/ https://review.openstack.org/#/c/259354/ https://review.openstack.org/#/c/262055/11:12
dtantsurI'll try to, but I'm on the RDO test day today-tomorrow11:18
dtantsurw00t, the iPXE gate is passing: http://logs.openstack.org/13/290013/2/check/gate-ironic-inspector-dsvm-ipa-ipxe-nv/d716acd/ :)11:19
*** dims has quit IRC11:19
sambettsw00t!!11:19
dtantsurno I need to find some time to finish this patch, update docs, etc :)11:20
*** dims has joined #openstack-ironic11:21
*** bapalm has quit IRC11:24
*** ishant has quit IRC11:25
openstackgerritLucas Alvares Gomes proposed openstack/ironic: SSH driver: Remove pipes from virsh's list_{all, running}  https://review.openstack.org/29111211:26
*** trown|outtypewww is now known as trown11:27
aarefievdtantsur: fyi about sudo in exercise http://logs.openstack.org/07/276107/23/check/gate-ironic-inspector-dsvm-discovery-nv/68e3515/console.html#_2016-03-10_08_56_04_49611:30
dtantsuryeah, we probably need to remove a notty flag.. I wonder how we do it e.g. for DIB11:31
aarefievsudo  may  prompt for a password which makes it a poor command for scripting11:39
dtantsurwell, the whole devstack relies on sudo without a password11:40
dtantsurand e.g. DIB as well11:40
aarefievdtantsur: any ideas on this case?11:42
dtantsurmkovacik, hey around? mind helping aarefiev with his script?11:42
dtantsurI'm a bit too busy with the rdo test day right now..11:42
mkovacikdtantsur, yeah11:43
sambettsyou can use sudo -v to extend an existing sudo session if there is one11:43
mkovacikaarefiev, would you mind me proposing a fix?11:43
aarefievmkovacik: sure, feel free11:44
*** bapalm has joined #openstack-ironic11:44
mkovacikaarefiev, w/r the sudo thing, how is the gate executing this? through ssh?11:45
*** mbound has quit IRC11:47
aarefievmkovacik: I dunno, I guess so, ansible ?11:47
aarefievsambetts: may work if the NOPASSWD tag is present for current cmd11:48
mkovacikyeah, ssh -T fixes that I think but let me see11:49
mkovacikaarefiev, we should run iptables in the exercise (wait for blacklist gone of a mac), too so not sure how to do without sudo; if this is ran within jenkins then  we need: jenkins ALL=(ALL) NOPASSWD: ALL  in /etc/sudoers11:52
mkovacikdtantsur, where to check this? ^11:52
dtantsurmaybe the problem is that we already run this script in one sudo level in gate... dunno11:53
mkovacikI mean where's the job conf repo (to make a pull request)11:53
aarefievhttps://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml11:56
*** dims has quit IRC11:56
*** trown is now known as trown|outtypewww11:56
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Add discover nodes exercise  https://review.openstack.org/27610711:57
*** moshele has quit IRC11:58
mkovacikaarefiev, thx, /me looks12:01
mkovacikaarefiev, I know the exercise.sh might be broken after my patch proposal, it's a wip12:02
mkovacikhowever, the discovery works OK12:03
mkovaciklocally12:03
aarefievok, let's wait for jenkins12:04
*** dims has joined #openstack-ironic12:04
openstackgerritMerged openstack/ironic-inspector: Allow specify log level for Error exception  https://review.openstack.org/28835512:05
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: [devstack] add support for using iPXE instead of plain PXE  https://review.openstack.org/29001312:12
dtantsurshould be fine now ^^12:12
lucasagomesis gate broken!?12:13
lucasagomesthe*12:14
*** vaishnavi has quit IRC12:16
aarefievmkovacik: fail https://jenkins01.openstack.org/job/gate-ironic-inspector-dsvm-discovery-nv/11/console12:17
*** moshele has joined #openstack-ironic12:18
mkovacikaarefiev, yeah :-/ however, gitgrep the config repo gives a ton of sudo commands12:19
lucasagomesyeah apparently some tempest stuff broke us :-( http://logs.openstack.org/12/291112/1/check/gate-tempest-dsvm-ironic-pxe_ssh/c7ad0ed/console.html#_2016-03-10_11_45_27_99712:19
sambettslucasagomes: well...12:20
sambettssurly our gate shouldn't even be loading that module?12:20
sambettswe don't test fwaas in our gate right?12:21
lucasagomessambetts, not very familiar with it :-( trying to figure that out12:21
lucasagomessambetts, afaik, we do not12:21
*** Nisha has quit IRC12:21
vdrokand it's strange that stuff like tempest.scenario.test_volume_boot_pattern.TestVolumeBootPattern.test_volume_boot_pattern is run12:21
lucasagomesnow why it's being triggered I've no clue12:21
lucasagomesreported to -infra12:21
sambettslooking at the test list it looks like the regex isn't working12:21
*** Nisha has joined #openstack-ironic12:22
*** nathan-h has joined #openstack-ironic12:25
lucasagomesfound the patch12:25
lucasagomeshttps://review.openstack.org/#/c/290108/12:25
lucasagomeslemme try to fix it12:25
lucasagomesurgh... f**k the tests shows ironic being broken but as it's -nv nobody cares12:25
sambettsgod damn it...12:26
sambettsalthough that doesn't explain why the fwaas senario is being loaded in our gate...12:28
dtantsuriirc tempest loads all known plugins12:28
openstackgerritLucas Alvares Gomes proposed openstack/ironic: DO NOT MERGE: Test tempest fix for gate breakage  https://review.openstack.org/29114412:30
lucasagomesthere's a patch fixing the problem, let's see if that works for us ^12:30
sambettsdtantsur: It appears to be trying to run the volume and snapshot senarios too :/12:30
lucasagomesapparently thursday is the day to break the gate12:32
sambettsbetter than a Friday I guess :-P12:32
*** saibarauei has joined #openstack-ironic12:33
lucasagomesheh yeah12:34
* lucasagomes brb will grab some bite12:34
*** lucasagomes is now known as lucas-hungry12:34
*** saibarspeis has quit IRC12:37
*** saibarauei is now known as saibarspeis12:37
TheJuliagood morning everyone12:39
*** alexpilo_ has joined #openstack-ironic12:39
mgouldTheJulia, morning!12:40
*** alexpilotti has quit IRC12:42
vdrokmorning TheJulia12:43
*** afaranha has quit IRC12:44
*** moshele has quit IRC12:45
*** pcaruana has quit IRC12:45
*** Goneri has quit IRC12:46
sambettsMorning TheJulia12:50
*** trown|outtypewww is now known as trown12:52
sambettslucas-hungry: https://review.openstack.org/#/c/290983/12:52
*** e0ne_ has joined #openstack-ironic12:53
*** e0ne has quit IRC12:54
*** baoli has joined #openstack-ironic12:55
*** afaranha has joined #openstack-ironic12:56
lucas-hungrysambetts, thanks, yeah added a patch that depends on that one to see if it's going to fix the problem for us12:59
*** pcaruana has joined #openstack-ironic13:00
*** dprince has joined #openstack-ironic13:01
*** moshele has joined #openstack-ironic13:04
*** appu103_ has quit IRC13:05
sambettslucas-hungry: cool :D13:06
*** thrash|g0ne is now known as thrash13:10
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Tests for CLI help messages  https://review.openstack.org/26227313:10
*** frontrunner has joined #openstack-ironic13:10
*** moshele has quit IRC13:12
openstackgerritNisha Agarwal proposed openstack/ironic: Add support for partition images in agent drivers  https://review.openstack.org/16200813:15
*** lucas-hungry is now known as lucasagomes13:17
*** jtomasek has joined #openstack-ironic13:17
*** nathan-h has quit IRC13:18
dtantsurjtomasek, o/13:19
lucasagomessambetts, patch works for us :-) https://review.openstack.org/#/c/291144/13:21
jtomasekdtantsur: Hi, I'd like to send an updated patch for this https://review.openstack.org/#/c/289825/ this evening (in about 5 hours) do you think it is going to be possible to get it in before the code freeze?13:23
dtantsursambetts, ifarkas, we should make a release before ^^. I'm fine to make a FF exception for CORS, wdyt?13:23
dtantsurso I make a release around right now, then the next one in 1.5 weeks around RC13:24
ifarkasdtantsur, +113:25
ifarkasdtantsur, or why not make only one release in 1.5 weeks?13:25
dtantsurifarkas, we kind of skipped M3.. and we already got a lot of features in, so would be good to get wider testing before we call it mitaka final13:26
ifarkasdtantsur, right, makes sense13:26
*** kalpase has left #openstack-ironic13:26
*** mtanino has joined #openstack-ironic13:27
sambettsmakes sense yup13:27
sambettslucasagomes: Great! we just need it merged then :D13:27
lucasagomessambetts, yeah, well out of my hands :-D13:28
* lucasagomes +1 that patch13:28
* sambetts same13:28
dtantsurjtomasek, ok, FFE granted. nonetheless the patch must merge by Monday evening (EU time)13:30
*** max_lobur has joined #openstack-ironic13:30
dtantsurifarkas, sambetts, anything else in flight you want me to get to the release?13:30
ifarkasdtantsur, nope13:31
jtomasekdtantsur: thanks!13:31
*** dims has quit IRC13:38
*** dims has joined #openstack-ironic13:39
*** moshele has joined #openstack-ironic13:42
sambettsdtantsur: nothing that I can see on the inbox13:43
openstackgerritDebayan Ray proposed openstack/ironic: Add documentation for firmware update(iLO) as manual clean step  https://review.openstack.org/27676513:43
*** max_lobur has left #openstack-ironic13:44
dtantsursambetts, ifarkas, thanks. release request: https://review.openstack.org/29118013:44
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Add discover nodes exercise  https://review.openstack.org/27610713:44
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Add discover nodes exercise  https://review.openstack.org/27610713:47
*** links has quit IRC13:48
*** vinm213 has quit IRC13:49
*** degorenko is now known as _degorenko|afk13:49
*** max_lobur has joined #openstack-ironic13:51
*** max_lobur has left #openstack-ironic13:53
*** mkovacik has quit IRC13:54
*** mkovacik has joined #openstack-ironic13:54
*** Goneri has joined #openstack-ironic13:57
*** mbound has joined #openstack-ironic14:00
*** keedya has joined #openstack-ironic14:03
*** openstackgerrit has quit IRC14:03
*** openstackgerrit_ has joined #openstack-ironic14:04
*** ifarkas has quit IRC14:04
*** openstackgerrit_ is now known as openstackgerrit14:04
*** openstackgerrit has quit IRC14:04
*** openstackgerrit_ has joined #openstack-ironic14:05
*** openstackgerrit_ is now known as openstackgerrit14:05
*** openstackgerrit has quit IRC14:05
*** openstackgerrit_ has joined #openstack-ironic14:06
*** openstackgerrit_ is now known as openstackgerrit14:06
*** openstackgerrit has quit IRC14:07
*** openstackgerrit_ has joined #openstack-ironic14:07
*** openstackgerrit_ is now known as openstackgerrit14:08
*** openstackgerrit has quit IRC14:08
*** openstackgerrit_ has joined #openstack-ironic14:08
gridinvvsaienko: online?14:08
*** openstackgerrit_ is now known as openstackgerrit14:09
*** openstackgerrit has quit IRC14:09
*** openstackgerrit_ has joined #openstack-ironic14:10
*** amotoki has quit IRC14:10
*** openstackgerrit_ is now known as openstackgerrit14:10
*** Guest41345 has joined #openstack-ironic14:11
*** cdearborn has joined #openstack-ironic14:15
openstackgerritNisha Agarwal proposed openstack/ironic: Add support for partition images in agent drivers  https://review.openstack.org/16200814:17
jrollmorning y'all14:19
Nishajroll morning14:20
Nishajroll, ^^^^14:20
jrollhi Nisha14:21
Nishajroll, hi14:22
NishaIPA patch has one +214:22
*** mtanino has quit IRC14:22
Nishaironic one needs review14:22
Nishai addressed lucasagomes  comments now14:23
lucasagomesjroll, morning14:23
jrollNisha: I have a ton of things to catch up on but I will get to it eventually14:23
jroll\o lucasagomes :)14:23
*** mmnelemane has joined #openstack-ironic14:23
Nishajroll, sure14:24
Nisha:)14:24
dtantsurmorning jroll14:27
jrollhey dtantsur :)14:28
dtantsurjroll, one complex upgrade concerning question: what to do about https://bugs.launchpad.net/ironic-python-agent/+bug/1554492 and https://review.openstack.org/28989714:28
openstackLaunchpad bug 1554492 in ironic-python-agent "IPA does not respect the "disk" kernel parameter" [Medium,In progress] - Assigned to Dmitry Tantsur (divius)14:28
dtantsur(details are in the bug report)14:28
*** dims has quit IRC14:29
jrolldtantsur: ew14:29
dtantsurall options are bad, I dunno...14:31
jrollyeah, thinking14:31
vdrokmorning jroll !14:31
*** bharath has joined #openstack-ironic14:31
jroll\o vdrok14:31
mmnelemaneHi all, This is Madhu Mohan. I only started ramping up on ironic since a few weeks. I just started analyzing https://bugs.launchpad.net/ironic/+bug/1535908. I was wondering if I can work on this to get better understanding of the project ?14:32
openstackLaunchpad bug 1535908 in Ironic "ironic-conductor no longer stops if exception raised during init_host()" [High,In progress] - Assigned to Madhu Mohan Nelemane (mmohan-9)14:32
jrollmmnelemane: yes, you can always work on bugs :)14:33
*** dims has joined #openstack-ironic14:35
gridinvvsaienko: hi, was playing with generic-switch,  plugin.sh will add duplicate mech driver name to ml2 config, if one is present in local.conf.14:36
gridinvvsaienko: and looks like neutron will load it twice, so postcommit on eg net-create will be called twice.14:36
*** cdearborn has quit IRC14:37
jrolldtantsur: anyway, seems like we do need to support it... it's always been a shoot-self-in-foot config option14:40
*** cdearborn has joined #openstack-ironic14:40
dtantsurjroll, yeah.. I'm telling anyone that they should be using root device hints, but they still don't......14:40
mmnelemanejroll: thanks.14:40
dtantsurjroll, could you please weigh in on the review then?14:41
jrolldtantsur: I'd be +1 to deprecating it, but we should support it for now14:41
jrollyep14:41
*** pcaruana has quit IRC14:41
dtantsurthanks14:41
jrolldtantsur: btw, did we decide not to do stable branches for IPA?14:42
jrollI can look back at mails if you don't remember14:42
dtantsurjroll, no final decision, but I guess we should14:43
dtantsurjroll, cause RDO attempt to use Mitaka IPA in Liberty has failed due to dependencies14:43
dtantsurtrown, right ^^14:43
TheJuliaI would absolutely argue that we need to maintaine stable branches for ipa14:43
jrolldtantsur: what dependencies?14:43
jrollTheJulia: yeah, I ask remembering your compliance argument14:43
dtantsurjroll, we introduced a patch in mitaka requiring newer oslo.service14:43
* TheJulia smiles14:44
trownreading up14:44
*** mgoddard_ has joined #openstack-ironic14:44
* TheJulia kind of thought we settled the whole thing in the argument before it, but *shrugs*14:44
jrolldtantsur: I don't get why that would break it?14:44
jrollTheJulia: we may have, my memory capacity is low right now14:44
TheJuliajroll: eh, the argument before was like the same thing but for different reasons and a complete reverse14:45
dtantsurjroll, because IPA from mitaka is still built with other dependencies from liberty. e.g. with old oslo.service14:45
trowndtantsur: RDO is using mitaka IPA in liberty14:45
TheJuliabut yeah, ERRINSUFFICENTCOFFEE14:45
trownoh... I see14:45
dtantsurtrown, oh, rly? so the oslo.service problem, how did you deal with it?14:45
jrollI'm fine with maintaining them, it isn't hard14:46
jrolland thinking about deprecating this disk fallback thing, would make it ewasier14:47
jrolleasier14:47
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add documentation about the disk_label capability  https://review.openstack.org/28852014:47
dtantsurit all depends on whether anyone is going to consume it :)14:47
trowndtantsur: ya we cant build mitaka IPA with liberty deps, but so far just using the mitaka package to build liberty IPA ramdisk is working fine for us :p14:47
dtantsurjroll, oh yeah, deprecating anything is so much easier14:47
*** mgoddard has quit IRC14:47
lucasagomesdtantsur, jroll that alone sounds like a good reason to keep doing stable branches14:47
*** nathan-h has joined #openstack-ironic14:47
jrollyeah, I say we just keep doing it14:47
lucasagomes+ the low maintenance14:47
trownjroll: I think if we can say IPA should work with the releases Ironic supports we dont need a stable branch14:47
trownoh14:48
trowneither way :P14:48
dtantsurI don't have a strong opinion either14:48
trownRDO just needs LIO support on liberty :)14:48
jrolldtantsur: commented on that thing14:48
dtantsurif we do keep stable branches, we should start consuming stable IPA in our stable gates14:48
jrollyeah14:48
*** rbudden has joined #openstack-ironic14:49
lucasagomesagreed14:49
dtantsurjroll, I'll post a patch deprecating this option and making sure we can work with it being empty (probably not today, debugging $stuff right now)14:49
dtantsurjroll, do you think we should ask people to set it to empty value explicitly?14:49
dtantsur(for now)14:49
* lucasagomes prior to all this conversation I thought we did14:49
jrolldtantsur: wait, what's the default?14:51
dtantsurjroll, currently? https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L201214:52
dtantsurjroll, and that's the problem: it applies to the bash ramdisk by default, but is ignored by IPA14:52
jrollthanks, couldn't find it.14:52
jrollalso sigh.14:52
jrollthat's terrible14:52
dtantsurit is :(14:53
jrollnow I'm more reluctant to support it14:53
jrollalso what is cciss/c0d014:53
*** pcaruana has joined #openstack-ironic14:53
lucasagomesjroll, some HP smart array thing14:54
lucasagomesthis code is very old, came from nova baremetal14:54
lucasagomesso...14:54
jrollyeah14:54
*** _degorenko|afk is now known as degorenko14:56
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Add discover nodes exercise  https://review.openstack.org/27610714:56
*** thiagop has joined #openstack-ironic14:57
thiagopGood morning folks14:57
jrolldtantsur: posted more comments14:57
jrollhi thiagop14:57
*** kan_ has quit IRC14:57
*** bharath has quit IRC14:58
thiagophey jroll, how was vegas?14:59
thiagopTheJulia: around?14:59
jrollthiagop: fantastic, needed a break14:59
TheJuliaI'm around14:59
TheJuliaI have a meeting in a little bit though14:59
lennybHi Ironic. Is it possible to run ironic_tempest_plugin API tests with a specific openstack-ironic-api-version ?15:01
*** mtanino has joined #openstack-ironic15:01
*** mgoddard_ has quit IRC15:03
*** mgoddard has joined #openstack-ironic15:03
thiagopjroll: I was meaning to ask you how's the work on network isolation and if you need some help with that15:03
thiagop(you and/or devananda)15:04
jrollthiagop: I'm still catching up, I can let you know a bit later15:04
thiagopjroll: sure :)15:04
*** thrash has quit IRC15:05
jrollthiagop: at a glance it looks like deva didn't get to it, so let me see what I can do this week15:05
*** moshele has quit IRC15:07
*** thrash has joined #openstack-ironic15:08
*** thrash has quit IRC15:08
*** thrash has joined #openstack-ironic15:08
sambettslucasagomes: I've poked some people to try and get that tempest plugin fix merged asap15:17
thiagopjroll: let me know15:17
lucasagomessambetts, thanks15:17
lucasagomesjroll, can't we make the ironic job voting in the tempest gate?15:18
lucasagomesit was clearly broken then but just got ignored15:18
jrolllucasagomes: we'd need to talk to mtreinish, we made it non-voting because it was performing so badly15:18
lucasagomeshttps://review.openstack.org/#/c/290108/15:18
lucasagomesjroll, right, it's more stable now. Maybe we should re-review that15:19
lucasagomesfunny thing is, the problem is in one of the neutron modules15:19
lucasagomesbut the neutron gate didn't break15:19
lucasagomes(y)15:19
mtreinishlucasagomes, jroll: part of having those tests as a plugin now is so the job doesn't need to run on tempest anymore15:20
lucasagomesfair enough15:21
lucasagomesI wonder why we do run these neutron_fwaas/tests/tempest_plugin tests as part of the ironic tests tho15:22
*** AndChat185600 has joined #openstack-ironic15:24
sambettslucasagomes: not sure, that log run you linked seemed to be running the nova volume/snapshot related senarios too15:25
lucasagomessambetts, strange15:26
lucasagomesmaybe we should looking at disabling those as well15:26
sambettslucasagomes: the regex should stop them all right?15:26
lucasagomessambetts, I'm not very familiar with tempest, but yeah, I imagine the regex looking for "baremetal" would filter those out15:27
lucasagomes2016-03-10 11:45:28.900 | tempest.scenario.test_snapshot_pattern.TestSnapshotPattern.test_snapshot_pattern[compute,id-608e604b-1d63-4a82-8e3e-91bc665c90b4,image,network]                         62.70215:27
lucasagomesyeah I see what you mean15:27
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add CLI tests to check JSON response body  https://review.openstack.org/29123915:28
sambettslucasagomes: I think the lastest runs should be running with the regex 'ironic' because they should be using the tempest plugin15:28
dtantsurinspector 3.1.0 released!15:31
sambetts\o/15:31
lucasagomesw00t15:31
Nishawoo15:31
dtantsurnow to finish polishing. please no big features until we get stable/mitaka in the coming 2 weeks :)15:32
* sambetts implements all the big features 15:33
jrollanyone have anything they want to block ironic 5.0.0 on?15:33
jrolldtantsur: ^^15:33
* dtantsur checks15:33
dtantsurnothing from me15:34
*** mbound has quit IRC15:34
jrollokay, you're the bug wizard so maybe I'll cut that today15:34
*** Goneri has quit IRC15:40
*** Goneri has joined #openstack-ironic15:40
thiagoplucasagomes: hey, ty!15:41
lucasagomesthiagop, hi there15:42
lucasagomesnp15:42
lucasagomesI'm sorry for the delay to re-review that :-(15:42
lucasagomesshame on me15:42
thiagoplucasagomes: np15:42
*** pas-ha has quit IRC15:42
*** pas-ha has joined #openstack-ironic15:43
*** pece has quit IRC15:45
thiagoplucasagomes: the window you mention on L206 can be hit, yes. But since it's a oneview-to-oneview users problem, shouldn't be an Ironic concern.15:48
lucasagomesthiagop, exactly15:50
*** baoli has quit IRC15:50
lucasagomesthiagop, yeah, I just commented to document that15:50
lucasagomesthiagop, that said, the shared pool of nodes approach not something very common (-:15:50
*** baoli has joined #openstack-ironic15:51
Nishajroll, wrt "anyone have anything they want to block ironic 5.0.0 on?" and "okay, you're the bug wizard so maybe I'll cut that today" ....So does it mean that stable ironic for mitaka will be released today?15:51
dtantsurNisha, not yet15:52
Nishadtantsur, ok...15:52
*** pas-ha_ has joined #openstack-ironic15:52
vdrokjroll, do you mind if i push an update to the network interface refactor? it resolves some comments and it passes more unittests :)15:52
*** pas-ha has quit IRC15:52
*** pas-ha_ is now known as pas-ha15:52
*** pas-ha has quit IRC15:52
sambettslucasagomes: the tempest fix got +A15:52
lucasagomesw00t!15:53
*** pas-ha has joined #openstack-ironic15:53
thiagoplucasagomes: innovation :P15:55
lucasagomesthiagop, hah15:56
lucasagomesmany innovation comes from difficult problems such as that so... why not15:56
*** jistr has quit IRC15:59
*** vishwanathj_zzz is now known as vishwanathj16:00
sambettsthiagop: is it possible for a OneView user to steal a node?16:00
*** jtomasek has quit IRC16:00
*** izaakk has joined #openstack-ironic16:00
*** max_lobur has joined #openstack-ironic16:00
sambettsi.e. its deployed by ironic and a OneView user replaces it profile and takes it over?16:00
*** max_lobur has left #openstack-ironic16:01
*** absubram has joined #openstack-ironic16:02
thiagopsambetts: yes, it is. But only purposely since it requires 2 or 3 confirmations to do so.16:02
sambettsthiagop: I was just wondering what Ironic would do if that happened?16:02
dtantsurfail miserably? :)16:03
dtantsurI think it will result in maintenance mode and "cannot sync power state"16:03
thiagopsambetts: error state + maintenance mode16:03
lucasagomesthat would be fun :-) specially with cleaning16:03
*** absubram_ has joined #openstack-ironic16:03
lucasagomesI mean, ironic will erase the user's data :D16:03
dtantsurto be fair, it's not the only way to screw everything16:03
lucasagomesyeah16:03
dtantsurit's just the same as changing IPMI credentials without Ironic knowing16:04
lucasagomesdtantsur, not sure if you've seem matthew garret's talk on IPMI16:04
dtantsurdefinitely16:04
thiagopI think Ironic requires some ammount of not-chaos to work16:04
lucasagomesthat's quite scary own it's own16:04
sambettsdtantsur: Not sure because Ironic still has control in that respect, it doesn't need a service profile to get power state afaik16:05
dtantsuraha16:05
dtantsurok, then I dunno16:05
*** mbound has joined #openstack-ironic16:05
sambettsdtantsur: would validate() get run on the power interface during the power periodic task?16:05
*** absubram has quit IRC16:06
*** absubram_ is now known as absubram16:06
sambettsdtantsur: if it did then at that point it would be possible the ownership logic could catch it and put it into error+ maintenance16:06
dtantsursambetts, should be, I guess16:06
sambettsthiagop: ^16:07
lucasagomessambetts, I don't think so... only if power_state is None16:07
lucasagomesso it doesn't attempt to sync the power state for an unconfigured node16:08
thiagopsambetts: we have some extra validations, checking code to see what happens... one sec16:08
lucasagomesfwiw https://github.com/openstack/ironic/blob/80fd454ef7e9f6b340af74a84c2a82e86bb5bf0e/ironic/conductor/manager.py#L2412-L241716:08
sambettslucasagomes: ah I see, I wonder if it would hurt to validate everytime, because that would catch if someone goofed up the ipmi creds via a node-update or something too16:10
lucasagomessambetts, if they do that the node goes into maintenance16:10
lucasagomessambetts, cause validate only checks if the configuration is set16:10
lucasagomesit doesn't actually validate if the credential are correct16:11
sambettslucasagomes: oh yeah :/16:11
sambettsbut if they deleted one of them it would catch it16:11
lucasagomesyeah16:11
thiagopsambetts dtantsur lucasagomes yeah, we have some validations in place to check if the server profile applied is the one that was applied by Ironic (or oneviewd previously without dynamic allocation)16:11
thiagopif the user removes that, the node goes into error state16:12
sambettsis it oneviewd that handles that?16:12
thiagopthe validation is done on the power and management interface to throw it on error, the maintenance is done by oneviewd, yes16:13
* lucasagomes shouts w00t. Patch in devstack fixing the agent_ipmitool job also got merged16:14
sambettsso it checks the profile every time it tries to do a get power state?16:14
* JayF notes Rackspcae launched a thing --> https://blog.rackspace.com/better-hybrid-connectivity-global-reach-and-improved-storage-onmetal-v2-now-available/16:14
JayFThanks for everyone who works with us upstream and helped make this possible. We don't have a product to launch without the foundation and collaboration provided by the community.16:14
*** sinval-afk is now known as sinval16:15
sinvalmorning guys16:15
lucasagomesJayF, nice one... and that's a pretty cool bear there16:15
lucasagomesJayF, I heard that google also joined the open compute project? Maybe you have more news about it from the conference than I do16:15
JayFI didn't hear that, that's nice!16:16
TheJuliait was on slashdot last night16:16
thiagopsambetts: yep16:16
JayFThe "meat" of the conference is more today; yesterday the big announcements were Microsoft releasing Linux software for switches and Bell Labs basically presenting that we're darn close to the theoretical max for fiber16:16
thiagopsambetts: that's how it is currently. With dynamic allocation, it'll not check since no server profile is applied.16:17
lucasagomesTheJulia, yeah, now amazon seems to be the only one left (from the big companies)16:17
*** max_lobur has joined #openstack-ironic16:17
*** max_lobur has left #openstack-ironic16:17
TheJulialucasagomes: I thought they were already involved16:17
lucasagomesJayF, yeah, that's a debian-based OS for the network switches16:17
lucasagomestho I wouldn't trust microsoft anyway. Doesn't matter how much the claim to love linux now16:18
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Port  https://review.openstack.org/24094416:18
lucasagomesTheJulia, maybe, I'm not 100% sure16:18
JayFI mean, Azure is a non-trivial part of their world, they're pretty invested in linux now. I'm less skeptical, it's not halloween anymore :)16:18
lucasagomesI think not... but gotta check16:18
sambettsthiagop: right, but once a node is in Active state, i.e deployed by Ironic how to you put a node into error if its server profile changes without having some code to check it in the ironic periodic task space16:18
lucasagomesJayF, agreed, we do have a partnership with them and all16:19
TheJuliaJayF: Are you sure, they might have a time machine16:19
thiagopsambetts: the check is in the validate() of power interface16:19
lucasagomesit's just historical conflicts... I mean, you don't try to kill something in the past and years later come back saying you love it16:19
sambettsthiagop: as me and lucas just discuses validate is only run for that interface if power_state = None16:19
sinvalsambetts, oh, I see16:20
thiagopsambetts: looking the code now16:20
thiagopsambetts:  I see16:21
lucasagomesthiagop, https://github.com/openstack/ironic/blob/80fd454ef7e9f6b340af74a84c2a82e86bb5bf0e/ironic/conductor/manager.py#L2412-L241716:21
Nisha lucasagomes i needed one help, how do we debug IPA when deploy ramdisk is built using DIB.....earlier i used to put pdb in the IPA source code on the server and which used to break there when we used to run /usr/bin/ironic-python-agent16:22
Nishabut now since the code is in venv, it doesnt break16:22
Nishado you know how to actually trace the IPA now when i have the iso built using DIB16:23
*** baoli has quit IRC16:23
sinvallucasagomes, sambetts that's right, so, during powe==none the driver will raise exception, but for other cases when power==on or power==off, the currently implementation does not raise anything, because validate is not called...16:24
lucasagomesNisha, hi there. I'm thinking here :-/ do you need to dynamically debug it with break points? Or would adding a bunch of debug messages do the job?16:24
sinvaldo you guys have any suggestion?16:25
sinvalis there a heartbeat for the node?16:25
sambettssinval: not once deployed16:25
lucasagomessinval, not off the top of my head16:25
lucasagomessinval, there's a heartbeat only while the ramdisk is running for cleaning or deploy16:26
lucasagomessinval, maybe this is something that a driver periodic task for oneview can do ?16:26
lucasagomesI know we discussed periodic task to replace the driver, but maybe we can use it now just to validate()16:26
lucasagomesreplace the daemon*16:27
sambettssinval: you have two options, put ownership checking logic in the actual get_power_state function which will be called perodically, or add a driver periodic task to check the ownership for all nodes16:27
sinvalwell, but, it will be the same case of a deamon, the task can be slower than the error16:27
sinvalor not?16:27
dtantsursee you tomorrow16:27
thiagopsee ya dtantsur16:27
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Add discover nodes exercise  https://review.openstack.org/27610716:27
sinvalbye dtantsur16:28
sambettso/ dtantsur16:28
thiagopsambetts lucasagomes good options, we'll consider that16:28
sinvalsambetts: makes sense16:28
lucasagomesdtantsur, night16:28
*** bharath has joined #openstack-ironic16:28
*** mmnelemane has quit IRC16:29
sambettssinval: any periodic task is going to have some delay in detecting the error compared to listening on a bus, but your periodic task could listen on the bus for events, and listen for a server profile change event or something and then trigger an error state event16:29
*** baoli has joined #openstack-ironic16:29
thiagopthiagop: probably we should file a bug about that since it already affects the merged driver16:29
sinvalsambetts, the bus will solve everything :)16:29
sinvalsambetts, I mean, I'm just trying to see if we could came up with a quick and effective solution, before going for a bus16:30
sambettsyup, my concern is that, that is definatly an error state, but an external deamon can't put a node into an error state16:31
Nishalucasagomes, dynamically debug16:31
JayFjroll: something to look into: If Ironic is ever going to provision network switches, you should look into ONIE16:32
*** bharathk has joined #openstack-ironic16:32
JayFjroll: basically Ironic could tie into ONIE and support all opencompute switches16:32
*** bharath has quit IRC16:32
JayF( http://onie.org/ ) -- basically an open source install environment for switches16:33
*** harlowja_at_home has joined #openstack-ironic16:33
jrollvdrok: go ahead man, thank you for the help16:35
sinvalsambetts, you're right16:36
jrollJayF: ++ for ONIE16:37
jrollI was looking at that a bit earlier16:37
sinvalsambetts, just 1 minute, I'll be back soon16:37
* lucasagomes looked at ONIE some time ago16:37
sambettssinval: Sure :)16:37
lucasagomesthat's cool16:37
openstackgerritMerged openstack/ironic: Add documentation for the IPMITool driver  https://review.openstack.org/29071816:37
lucasagomesNisha, hmm... the only way I think about that is kinda hacky. But, if you modify the ironic-agent element to not start the ironic-python-agent service at boot up time16:39
lucasagomesNisha, and you enable a way to log into the ramdisk once it's booted (dynamic-login element)16:39
lucasagomesyou could then log in, and start the service manually16:39
lucasagomesand debug it there16:39
Nishayes i have login enabled16:39
lucasagomesnot super straight forward but I don't think we have plenty of options here16:40
lucasagomesNisha, right... you can also set the deploy_timeout_callback config option to 016:40
Nishalucasagomes, i am facing issues there only in debugging16:40
sinvalsambetts: cool, so, should we register a bug for that issue16:41
lucasagomesthat will disable the call-back timeout and give you time to debug the service16:41
sinvaland we could add the validate call during get_power_state()16:41
lucasagomesNisha, which issue?16:41
sambettssinval: if it affects your current driver, if not I would add something in your spec to cover if16:42
*** ohamada has quit IRC16:42
sambettsit*16:42
*** ohamada has joined #openstack-ironic16:42
* lucasagomes thanks NobodyCam for approving the ipmitool doc patch (-:16:42
sinvalsambetts: yeah, it affects our current driver, so by registering the bug we could track this16:43
sinvalsambetts: because, the same case can happen for pre-allocation (current version) and dynamic allocation (next versions)16:43
sinvallucasagomes: do you have an opinions about that? ^16:46
*** e0ne_ has quit IRC16:46
* lucasagomes reads16:47
*** pcaruana has quit IRC16:47
lucasagomessinval, well not really... the event based approach (bus) will always beat a pooling based approach (check every X time)16:48
Nishalucasagomes, i want to debug _get_partition in IPA when it installs bootloader into the server during localboot....16:48
lucasagomessinval, beat in detection time I mean16:49
lucasagomesNisha, right, so if you modify the code to add a break point there16:49
lucasagomesNisha, then you modify the deploy-agent element to not start the ipa service at boot time16:49
lucasagomesNisha, then enable a way to log in in the ramdisk16:50
lucasagomesyou can do that no?16:50
Nishai modified the code code, added the pdb also16:50
lucasagomescause you: 1. tell ironic to deploy the node the noe will boot up16:50
lucasagomes2. access the node and start IPA manually /usr/bin/ironic-python-agent16:50
lucasagomes3. debug it16:50
lucasagomespdb /usr/bin ...16:50
lucasagomesNisha, right16:51
Nishaoh u mean i shud add pdb in front of binary ...16:51
Nishathanks i will try this way16:51
*** ohamada_ has joined #openstack-ironic16:52
*** ohamada has quit IRC16:52
lucasagomesNisha, maybe it was my mistake... I don't think you may need it16:52
*** ohamada_ has quit IRC16:52
lucasagomesif you just run the ironic-python-agent manually on ur current user section I believe it will stop at the break point16:53
*** ohamada_ has joined #openstack-ironic16:53
lucasagomesif not yeah you can run it as a script16:53
sinvallucasagomes, that's right, but the first idea is fixing the miserable failure by putting the "validate_if_node_is_mine" inside the get_power_state, this should work until we came up with the periodic task + bus16:53
lucasagomesNisha, python -m pdb /usr/bin...16:54
sinvallucasagomes: periodic + bus is the target16:55
* lucasagomes reads the scrollback 16:56
sinvalhahaha16:56
lucasagomessinval, right and the second suggestion is to add a driver periodic task that will call validate() (so we don't mess up with the sync_power_state one)16:58
lucasagomesbut both approachs are not perfect they leave a window of time where things can go bad16:58
lucasagomeswhich can be solved by the event-approach in the future16:59
lucasagomessinval, maybe I'm missing what is the question here16:59
* lucasagomes long day, I'm sorry, brain is a bit slow at the moment 16:59
sinvallucasagomes: hahaha, you answered my question, we should go for periodic + bus from now on17:00
sinvallucasagomes, sambetts: I'm registering the bug about that issue, thank you very much17:01
lucasagomessinval, ack :-D17:02
lucasagomessinval, well thank YOU17:02
sinvalI'll have to step back a little bit, see you guys tomorrow o/17:03
lucasagomessee ya17:03
lucasagomesI will also call it a day, gotta get the train back home yet17:03
lucasagomeshave a great evening all, talk soon17:03
*** lucasagomes is now known as lucas-afk17:04
sinvalnight lucas-afk17:04
*** dims has quit IRC17:04
*** ndipanov has quit IRC17:04
*** sinval is now known as sinval-afk17:04
*** derekh has quit IRC17:05
*** dims has joined #openstack-ironic17:06
mgouldlucas-afk, good night!17:06
*** suro-patz has joined #openstack-ironic17:06
*** praneshp has joined #openstack-ironic17:09
*** keedya has quit IRC17:14
openstackgerritlokesh s proposed openstack/ironic: Add support for the audit  middleware  https://review.openstack.org/27265817:16
*** MattMan has quit IRC17:17
*** MattMan has joined #openstack-ironic17:17
*** Sukhdev has joined #openstack-ironic17:21
*** praneshp_ has joined #openstack-ironic17:24
*** praneshp_ has joined #openstack-ironic17:24
*** praneshp_ has quit IRC17:26
*** praneshp has quit IRC17:27
*** praneshp has joined #openstack-ironic17:29
*** thrash is now known as thrash|biab17:32
*** mgarza has joined #openstack-ironic17:40
sambettso/ lucas-afk sinval-afk17:41
thiagopnight lucas-afk, good class sinval-afk17:41
*** athomas has quit IRC17:45
openstackgerritVladyslav Drok proposed openstack/ironic: WIP: add network drivers  https://review.openstack.org/28585217:45
thiagopvdrok: unittests is the only thing missing here^?17:48
vdrokthiagop, nope, it still real testing too :)17:49
vdrok*still needs17:49
*** praneshp has quit IRC17:50
*** Sukhdev has quit IRC17:52
*** e0ne has joined #openstack-ironic17:53
* vdrok goes home too17:53
vdrokgood night everyone!17:53
*** dims has quit IRC17:54
jroll\o night vdrok17:55
thiagopnight vdrok17:56
*** jaybeale has joined #openstack-ironic17:58
*** baoli has quit IRC18:00
devanandamorning, all18:00
*** baoli has joined #openstack-ironic18:01
openstackgerritDoug Hellmann proposed openstack/python-ironic-inspector-client: Update reno for stable/mitaka  https://review.openstack.org/29133618:01
*** dims has joined #openstack-ironic18:03
* sambetts is heading off18:04
sambettsGood night all18:04
*** sambetts is now known as sambetts|afk18:04
thiagopmorning devananda18:05
thiagopnight sambetts|afk18:05
openstackgerritGonéri Le Bouder proposed openstack/ironic-lib: use wipefs to erase FS meta information  https://review.openstack.org/28628318:09
*** moshele has joined #openstack-ironic18:09
*** Marga_ has quit IRC18:11
*** Marga_ has joined #openstack-ironic18:12
*** trown is now known as trown|lunch18:14
*** nathan-h has quit IRC18:14
*** thrash|biab is now known as thrash18:16
*** spandhe has joined #openstack-ironic18:16
*** electrofelix has quit IRC18:17
*** mgould has quit IRC18:18
*** moshele has quit IRC18:19
*** spandhe_ has joined #openstack-ironic18:19
*** spandhe has quit IRC18:21
*** spandhe_ is now known as spandhe18:21
*** dims has quit IRC18:21
mjturek1hey jlvillal, I have a question about one of your patches  https://review.openstack.org/#/c/279270 So I notice that you only grep for the 'private' network when checking for the cleaning network. There's no reason that a cleaning network couldn't be a public or private network right? Our target nodes are on a provider_network and we're wondering if we should patch to allow for that.18:21
*** ohamada_ has quit IRC18:21
mjturek1jlvillal: sorry your patch isn't doing the grepping18:22
*** dims has joined #openstack-ironic18:22
mjturek1but wondering if you had any insight18:22
*** pece has joined #openstack-ironic18:25
*** bharath has joined #openstack-ironic18:26
*** bharathk has quit IRC18:26
*** openstackgerrit_ has joined #openstack-ironic18:30
*** degorenko is now known as _degorenko|afk18:31
*** rloo has joined #openstack-ironic18:40
jlvillalmjturek1: Let me check...  In a meeting at the moment.18:40
jlvillalmjturek1: I don't really know.  I wouldn't say that "I grep for the 'private' network"18:41
jlvillalmjturek1: My patch isn't really changing the functionality of what is being done. So I'm not sure about private vs public. And I don't have time to look at it at the moment :(18:42
jlvillalI have meetings for the next 3-4 hours :(18:42
jrolldtantsur: still around? you good with openstack/python-ironic-inspector-client 1.5.0 == stable/mitaka?18:44
jrollnever mind, I see you replied18:45
*** keedya has joined #openstack-ironic18:50
mjturek1jlvillal: no worries at all, yeah you're patch isn't the culprit anyway. Thanks!18:53
jlvillalmjturek1: :)18:53
*** aginwala has joined #openstack-ironic18:56
*** rbudden has quit IRC18:57
*** mjturek1 has left #openstack-ironic19:02
*** trown|lunch is now known as trown19:12
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Add discover nodes exercise  https://review.openstack.org/27610719:14
*** jistr has joined #openstack-ironic19:15
NobodyCamGah ... wow calls atarted at 6 this mornig19:16
NobodyCamhappy late morning all :)19:16
*** harshs has joined #openstack-ironic19:18
*** absubram has quit IRC19:19
*** moshele has joined #openstack-ironic19:20
*** rbudden has joined #openstack-ironic19:21
openstackgerritjxiaobin proposed openstack/ironic: Pass region_name to SwiftAPI  https://review.openstack.org/29106319:27
*** dims_ has joined #openstack-ironic19:28
*** moshele has quit IRC19:28
*** jcoufal has quit IRC19:29
*** dims has quit IRC19:29
*** Sukhdev has joined #openstack-ironic19:29
*** bharath has quit IRC19:31
*** rcernin has quit IRC19:40
openstackgerritJiri Tomasek proposed openstack/ironic-inspector: Added CORS support to Ironic Inspector  https://review.openstack.org/28982519:41
NobodyCamJayF: BTW awesome set of tweets today!!!!!19:41
*** jaybeale has quit IRC19:43
*** e0ne has quit IRC19:43
*** Sukhdev has quit IRC19:43
*** baoli has quit IRC19:50
openstackgerritGonéri Le Bouder proposed openstack/ironic-python-agent: image: add the ability to clean disk metadata  https://review.openstack.org/28434719:53
*** trown has quit IRC19:56
*** zsmithnyc has quit IRC19:57
*** zsmithnyc has joined #openstack-ironic19:57
*** zsmithnyc has quit IRC19:57
*** zsmithnyc has joined #openstack-ironic19:58
*** twm2016 has joined #openstack-ironic19:59
*** trown has joined #openstack-ironic19:59
twm2016Hey everyone, I am new and am looking at this bug: https://bugs.launchpad.net/ironic/+bug/1531149 Can someone show me a review of what adding a "DSVM job" might look like?20:00
openstackLaunchpad bug 1531149 in ironic-python-agent "Add a DSVM job covering instance local boot" [Wishlist,Triaged]20:00
*** praneshp has joined #openstack-ironic20:03
*** aginwala has quit IRC20:03
*** aginwala has joined #openstack-ironic20:08
*** e0ne has joined #openstack-ironic20:12
jlvillalNobodyCam, Hello20:17
*** Nisha has quit IRC20:17
*** e0ne has quit IRC20:23
*** twm2016 has quit IRC20:26
*** thrash is now known as thrash|bbl20:28
*** mbound has quit IRC20:30
*** Sukhdev has joined #openstack-ironic20:30
mrdaMorning Ironic!20:36
*** absubram has joined #openstack-ironic20:44
openstackgerritMichael Turek proposed openstack/ironic: Allow user to specify cleaning network  https://review.openstack.org/29142721:06
*** krtaylor has quit IRC21:13
*** jaybeale has joined #openstack-ironic21:15
openstackgerritDoug Hellmann proposed openstack/python-ironicclient: Update reno for stable/mitaka  https://review.openstack.org/29143221:17
*** aginwala has quit IRC21:18
*** Marga_ has quit IRC21:20
*** penick has joined #openstack-ironic21:21
*** Sukhdev has quit IRC21:25
*** boris-42 has joined #openstack-ironic21:29
*** mbound has joined #openstack-ironic21:30
*** dprince has quit IRC21:32
*** mbound has quit IRC21:32
*** mbound has joined #openstack-ironic21:33
*** aginwala has joined #openstack-ironic21:34
NobodyCamjlvillal: :) morning :)21:34
NobodyCammorning mrda21:34
*** aginwala has quit IRC21:34
mrdaNobodyCam: o/21:39
NobodyCam:)21:39
*** Sukhdev has joined #openstack-ironic21:40
*** Sukhdev has quit IRC21:40
*** jistr has quit IRC21:40
*** Sukhdev has joined #openstack-ironic21:41
*** openstackstatus has quit IRC21:42
*** openstackstatus has joined #openstack-ironic21:44
*** ChanServ sets mode: +v openstackstatus21:44
*** keedya has quit IRC21:46
*** dims has joined #openstack-ironic21:52
*** dims_ has quit IRC21:52
*** pece has quit IRC21:53
*** Marga_ has joined #openstack-ironic21:54
*** saibarspeis has quit IRC22:05
*** rcernin has joined #openstack-ironic22:11
openstackgerritjxiaobin proposed openstack/ironic: Pass region_name to SwiftAPI  https://review.openstack.org/29106322:19
*** trown is now known as trown|outtypewww22:21
*** rcernin has quit IRC22:22
*** Goneri has quit IRC22:25
*** cdearborn has quit IRC22:25
*** cdearborn has joined #openstack-ironic22:26
*** chlong has quit IRC22:39
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Add discover nodes exercise  https://review.openstack.org/27610722:41
*** rloo has quit IRC22:47
*** aginwala has joined #openstack-ironic22:47
*** linuxgeek has quit IRC22:52
*** cdearborn has quit IRC22:54
*** mgarza has quit IRC22:54
*** trown|outtypewww has quit IRC22:55
*** AndChat|185600 has joined #openstack-ironic22:56
*** AndChat185600 has quit IRC22:56
*** trown has joined #openstack-ironic22:58
*** spandhe has quit IRC23:02
*** krtaylor has joined #openstack-ironic23:05
*** harshs has quit IRC23:06
*** rama_y has joined #openstack-ironic23:06
*** derekh has joined #openstack-ironic23:09
openstackgerritjxiaobin proposed openstack/ironic: Pass region_name to SwiftAPI  https://review.openstack.org/29106323:10
*** krtaylor has quit IRC23:11
*** krtaylor has joined #openstack-ironic23:25
*** absubram has quit IRC23:31
*** rbudden has quit IRC23:33
*** Goneri has joined #openstack-ironic23:36
*** derekh has quit IRC23:39
*** Sukhdev has quit IRC23:44
*** Sukhdev has joined #openstack-ironic23:45
*** mkovacik has quit IRC23:47
*** penick has quit IRC23:51
*** penick has joined #openstack-ironic23:54
*** penick has quit IRC23:57
*** Sukhdev_ has joined #openstack-ironic23:57
*** Sukhdev has quit IRC23:57

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