Thursday, 2016-03-17

*** lucas-afk has quit IRC00:02
*** achanda has joined #openstack-ironic00:03
*** mrda has quit IRC00:06
openstackgerritMerged openstack/ironic-specs: Move ssh-console-support spec to 4.3  https://review.openstack.org/28643200:07
*** mrda has joined #openstack-ironic00:08
*** lucasagomes has joined #openstack-ironic00:08
*** dims has joined #openstack-ironic00:08
*** dims_ has quit IRC00:09
*** dims has quit IRC00:10
*** mtanino has quit IRC00:13
*** achanda has quit IRC00:15
*** Sukhdev has joined #openstack-ironic00:19
*** garthb has quit IRC00:23
*** Sukhdev has quit IRC00:26
*** baoli has joined #openstack-ironic00:28
*** spandhe has quit IRC00:29
*** baoli has quit IRC00:30
*** vishwanathj has joined #openstack-ironic00:34
*** thrash is now known as thrash|g0ne00:44
*** tiendc has joined #openstack-ironic00:45
*** hoangcx has joined #openstack-ironic00:49
*** dims has joined #openstack-ironic00:52
*** jaybeale has quit IRC00:54
*** chlong has joined #openstack-ironic00:55
*** baoli has joined #openstack-ironic00:56
*** shu-mutou-AFK is now known as shu-mutou00:58
*** baoli has quit IRC00:59
*** baoli has joined #openstack-ironic01:05
*** baoli has quit IRC01:06
*** ramishra has quit IRC01:07
*** ramishra has joined #openstack-ironic01:08
*** ChrisAusten has joined #openstack-ironic01:13
*** chlong has quit IRC01:18
*** saripurigopi1 has quit IRC01:20
*** piet has quit IRC01:22
*** baoli has joined #openstack-ironic01:23
*** achanda has joined #openstack-ironic01:26
*** piet has joined #openstack-ironic01:28
*** chlong has joined #openstack-ironic01:31
*** alex_xu has quit IRC01:42
*** baoli has quit IRC01:43
*** alex_xu has joined #openstack-ironic01:44
*** piet has quit IRC01:49
*** alex_xu has quit IRC01:51
*** alex_xu has joined #openstack-ironic01:53
*** mannidi has joined #openstack-ironic01:53
*** mannidi has quit IRC02:01
*** mannidi has joined #openstack-ironic02:01
*** ramishra has quit IRC02:09
openstackgerritHaomeng,Wang proposed openstack/ironic: Devstack: add check of chassis creating  https://review.openstack.org/29378102:13
*** Haomeng has joined #openstack-ironic02:13
*** saripurigopi has joined #openstack-ironic02:13
*** Haomeng has quit IRC02:19
*** ChrisAusten has quit IRC02:19
*** lucasagomes has quit IRC02:19
*** mrda has quit IRC02:19
*** praneshp__ has quit IRC02:19
*** chenke has quit IRC02:19
*** kbyrne has quit IRC02:19
*** yolanda has quit IRC02:19
*** mtreinish has quit IRC02:19
*** mariojv has quit IRC02:19
*** zsmithnyc has quit IRC02:19
*** dansmith has quit IRC02:19
*** yonglihe has quit IRC02:19
*** evgenyl has quit IRC02:19
*** cfarquhar has quit IRC02:19
*** dims has quit IRC02:20
*** lucasagomes has joined #openstack-ironic02:24
*** Haomeng has joined #openstack-ironic02:25
*** ChrisAusten has joined #openstack-ironic02:25
*** mrda has joined #openstack-ironic02:25
*** 21WAAGMTC has joined #openstack-ironic02:25
*** chenke has joined #openstack-ironic02:25
*** kbyrne has joined #openstack-ironic02:25
*** yolanda has joined #openstack-ironic02:25
*** mtreinish has joined #openstack-ironic02:25
*** mariojv has joined #openstack-ironic02:25
*** zsmithnyc has joined #openstack-ironic02:25
*** dansmith has joined #openstack-ironic02:25
*** yonglihe has joined #openstack-ironic02:25
*** evgenyl has joined #openstack-ironic02:25
*** cfarquhar has joined #openstack-ironic02:25
*** kbyrne has quit IRC02:25
*** kbyrne has joined #openstack-ironic02:28
*** marios has quit IRC02:31
*** harshs has quit IRC02:36
*** piet has joined #openstack-ironic02:40
*** achanda has quit IRC02:48
*** achanda has joined #openstack-ironic02:50
openstackgerritHaomeng,Wang proposed openstack/ironic-specs: [WIP] Maintenance nodes automatic recovery  https://review.openstack.org/29219002:50
*** achanda has quit IRC02:52
*** achanda has joined #openstack-ironic02:55
*** piet has quit IRC03:10
*** Haomeng has quit IRC03:11
*** Haomeng has joined #openstack-ironic03:11
*** suro-patz has joined #openstack-ironic03:23
*** hoangcx has quit IRC03:31
*** hoangcx has joined #openstack-ironic03:31
openstackgerritMerged openstack/python-ironicclient: Trivial: Fix incorrect comments in test_baremetal.py  https://review.openstack.org/29153303:36
*** links has joined #openstack-ironic03:45
*** piet has joined #openstack-ironic03:52
*** baoli has joined #openstack-ironic03:55
*** baoli has quit IRC03:59
*** suro-patz has quit IRC04:05
*** Nisha has joined #openstack-ironic04:06
*** vaishnavi has joined #openstack-ironic04:07
*** stendulker has joined #openstack-ironic04:10
*** piet has quit IRC04:20
*** Sukhdev has joined #openstack-ironic04:43
*** 21WAAGMTC has quit IRC04:50
*** praneshp_ has joined #openstack-ironic04:50
*** vaishnavi has quit IRC04:50
*** vaishnavi has joined #openstack-ironic04:50
*** mannidi has quit IRC05:05
*** Marga_ has quit IRC05:05
*** suro-patz has joined #openstack-ironic05:11
*** baoli has joined #openstack-ironic05:15
*** vaishnavi has quit IRC05:19
*** baoli has quit IRC05:20
*** harlowja_at_home has quit IRC05:24
*** harlowja_at_home has joined #openstack-ironic05:25
*** getvasanth has joined #openstack-ironic05:25
*** vaishnavi has joined #openstack-ironic05:27
*** achanda has quit IRC05:37
*** praneshp_ has quit IRC05:38
*** kirankv has joined #openstack-ironic05:40
*** kalpase has joined #openstack-ironic05:45
*** jaybeale has joined #openstack-ironic05:49
openstackgerritAkira YOSHIYAMA proposed openstack/ironic-specs: Add nova-compatible-serial-console.rst to approved  https://review.openstack.org/29382705:50
*** sivaramakrishna has joined #openstack-ironic05:57
*** moshele has joined #openstack-ironic05:57
*** jaybeale has quit IRC05:59
openstackgerritNaohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft power off and inject nmi  https://review.openstack.org/21673806:03
*** deray has joined #openstack-ironic06:07
*** chlong has quit IRC06:08
*** Marga_ has joined #openstack-ironic06:12
*** vishwanathj is now known as vishwanathj_zzz06:12
derayg'morning Ironicers!06:16
*** Marga_ has quit IRC06:17
*** ChubYann has quit IRC06:17
*** moshele has quit IRC06:17
*** Marga_ has joined #openstack-ironic06:19
*** chlong has joined #openstack-ironic06:21
*** Marga_ has quit IRC06:22
*** Marga_ has joined #openstack-ironic06:22
*** lazy_prince has joined #openstack-ironic06:23
lazy_princeHi All, can you guys review https://review.openstack.org/#/c/287784/15 ..?06:26
*** chenke_ has joined #openstack-ironic06:30
*** chenke has quit IRC06:30
*** mannidi has joined #openstack-ironic06:34
getvasanthGood morning All :)06:34
getvasanthvdrok: Thanks a lot for the suggestion on the "No VIFS found error" got it fixed06:36
*** marios has joined #openstack-ironic06:55
*** hoangcx has quit IRC06:56
*** hoangcx_ has joined #openstack-ironic06:56
derayHaomeng, hi07:00
Haomengderay: morning:)07:00
derayHaomeng, o/ :)07:00
*** Marga__ has joined #openstack-ironic07:00
derayHaomeng, had a follow-up patch under review for ilo f/w update: https://review.openstack.org/#/c/291069/07:01
Haomengderay: ok, will review, thanks:)07:01
derayHaomeng, thx again :)07:02
Haomengalmost they are the comments changes07:02
*** Sukhdev has quit IRC07:02
*** Marga_ has quit IRC07:02
derayHaomeng, yeah .. sort of seom UT additions. alreday got 1 +207:02
*** getvasanth_ has joined #openstack-ironic07:02
Haomengyes:)07:03
*** suro-patz has quit IRC07:03
Haomengderay: Thank you Debayan, LGTM. Not approving, for other reviewers can have time to take a look.:)07:05
derayHaomeng, sure07:05
Haomengderay: :)07:05
derayHaomeng, thank you :)07:06
*** getvasanth has quit IRC07:06
*** moshele has joined #openstack-ironic07:06
*** Haomeng has quit IRC07:07
*** alex_xu has quit IRC07:10
*** alex_xu has joined #openstack-ironic07:12
openstackgerritNisha Agarwal proposed openstack/ironic-python-agent: Add support for partition images in agent driver  https://review.openstack.org/16022407:13
Nishavdrok, ^^^^07:14
*** afaranha has quit IRC07:14
*** tesseract has joined #openstack-ironic07:15
*** tesseract is now known as Guest5718207:15
*** shakamunyi has quit IRC07:18
*** saripurigopi has quit IRC07:18
*** saripurigopi has joined #openstack-ironic07:20
lazy_princeHi All, can you guys review https://review.openstack.org/#/c/287784/15 ..?07:22
*** shakamunyi has joined #openstack-ironic07:24
openstackgerritTan Lin proposed openstack/ironic: Add 'X-Openstack-Request-ID' header in the response  https://review.openstack.org/23800807:26
*** saripurigopi has quit IRC07:30
*** Haomeng has joined #openstack-ironic07:34
*** ohamada has joined #openstack-ironic07:34
*** zer0c00l has quit IRC07:37
*** rcernin has joined #openstack-ironic07:39
*** hoangcx_ has quit IRC07:49
*** Nisha has quit IRC07:49
*** hoangcx has joined #openstack-ironic07:56
*** chlong has quit IRC07:57
*** ChrisAusten has quit IRC08:00
*** moshele has quit IRC08:03
*** moshele has joined #openstack-ironic08:03
*** vaishnavi_r has joined #openstack-ironic08:04
*** vaishnavi has quit IRC08:08
openstackgerritAkira YOSHIYAMA proposed openstack/ironic-specs: Add nova-compatible-serial-console.rst to not-implemented  https://review.openstack.org/29382708:11
*** moshele has quit IRC08:19
*** sandhya has joined #openstack-ironic08:19
*** mbound has joined #openstack-ironic08:19
*** pcaruana has joined #openstack-ironic08:24
*** Nisha has joined #openstack-ironic08:25
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: console utils: add methods for serial console with socat  https://review.openstack.org/29387108:25
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: Add node serial console installation howto  https://review.openstack.org/29387208:25
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: IPMItool: add IPMISocatConsole class  https://review.openstack.org/29387308:25
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: IPMINative: add NativeIPMISocatConsole class  https://review.openstack.org/29387408:25
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: SSH: add SocatConsole class  https://review.openstack.org/29387508:25
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to set neutron port timeout from config  https://review.openstack.org/29387608:27
*** moshele has joined #openstack-ironic08:31
openstackgerritAkira YOSHIYAMA proposed openstack/ironic-specs: Add nova-compatible-serial-console.rst to not-implemented  https://review.openstack.org/29382708:35
*** ifarkas has joined #openstack-ironic08:36
*** moshele has quit IRC08:40
*** Marga__ has quit IRC08:42
*** Marga_ has joined #openstack-ironic08:42
*** moshele has joined #openstack-ironic08:42
*** barra204 has joined #openstack-ironic08:43
*** shakamunyi has quit IRC08:44
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: console utils: add methods for serial console with socat  https://review.openstack.org/29387108:45
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: SSH: add SocatConsole class  https://review.openstack.org/29387508:45
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: IPMINative: add NativeIPMISocatConsole class  https://review.openstack.org/29387408:45
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: IPMItool: add IPMISocatConsole class  https://review.openstack.org/29387308:45
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: Add node serial console installation howto  https://review.openstack.org/29387208:45
*** mkovacik has quit IRC08:47
*** jcoufal has joined #openstack-ironic08:48
openstackgerritMerged openstack/ironic: Follow-up of firmware update(iLO) as manual cleaning step  https://review.openstack.org/29106908:49
*** links has quit IRC08:53
*** mbound has quit IRC08:57
*** trusted has joined #openstack-ironic09:04
*** mbound has joined #openstack-ironic09:04
*** hoangcx has quit IRC09:06
*** sivaramakrishna has quit IRC09:07
*** hoangcx has joined #openstack-ironic09:07
*** athomas has joined #openstack-ironic09:11
*** ohamada has quit IRC09:11
*** d0ugal has quit IRC09:13
*** d0ugal has joined #openstack-ironic09:14
sturivnyiHaomeng, Morning=) Would be grateful if you could review my tests fot CLI https://review.openstack.org/#/c/240944/    Thanks!09:16
*** ohamada has joined #openstack-ironic09:18
*** jistr has joined #openstack-ironic09:26
openstackgerritYuiko Takada proposed openstack/ironic: Delete bridge "brbm" in devstack/unstack.sh  https://review.openstack.org/29390009:26
*** appu103_ has joined #openstack-ironic09:37
*** appu103_ is now known as aparnav09:39
*** Haomeng has quit IRC09:42
*** Haomeng has joined #openstack-ironic09:42
*** trusted has quit IRC09:43
*** achanda has joined #openstack-ironic09:44
*** mgould has joined #openstack-ironic09:44
*** achanda has quit IRC09:45
*** achanda has joined #openstack-ironic09:46
mgouldmorning Ironic!09:46
*** achanda has quit IRC09:46
*** _degorenko|afk is now known as degorenko09:47
ifarkasmorning all!09:52
deraymgould, ifarkas g'morning o/09:52
mgouldderay, ifarkas morning!09:54
*** sivaramakrishna has joined #openstack-ironic09:54
*** dims has joined #openstack-ironic09:57
*** tiendc has quit IRC09:59
dtantsurMorning Ironic10:05
deraydtantsur, g'morning :)10:05
aarefievmorning all!10:12
*** mkovacik has joined #openstack-ironic10:14
openstackgerritVasyl Saienko proposed openstack/ironic: Move _normalize_mac to driver utils  https://review.openstack.org/29392210:14
*** daemontool has joined #openstack-ironic10:14
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Tempest plugin initial commit  https://review.openstack.org/29392510:19
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: [WIP] Tempest: add basic test  https://review.openstack.org/29392610:19
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support  https://review.openstack.org/26425710:20
aarefievdtantsur: morning,  how are you feeling?10:21
dtantsurmorning! somewhat better, not perfect still10:21
aarefievdtantsur: I have started tempest tests, hope you will take a look when feel better10:22
*** hoangcx has quit IRC10:23
*** ohamada has quit IRC10:24
dtantsuraarefiev, many thanks! left the first comment about package name10:26
*** ohamada has joined #openstack-ironic10:26
openstackgerrityolanda.robla proposed openstack/bifrost: Remove generated default.leases file for libvirt  https://review.openstack.org/29340210:26
aarefievdtantsur: yeah I see, thanks, what the problem with package there ?10:27
*** bradjones has quit IRC10:27
dtantsuraarefiev, I don't think we need one more top-level package really... we have ironic_inspector.test, lets hide everything there10:28
*** links has joined #openstack-ironic10:29
*** mannidi has quit IRC10:29
aarefievok, I don't see any reason to have it, I'll move it10:29
*** daemontool has quit IRC10:31
lazy_princeHi All, can you guys review https://review.openstack.org/#/c/287784/15 ..? This change is for adding UEFI image build support in DIB10:31
openstackgerritDebayan Ray proposed openstack/proliantutils: Adds test coverage in tox.ini  https://review.openstack.org/29393610:32
sambettsMorning all o/10:33
dtantsurmorning sambetts10:34
*** absubram has quit IRC10:34
pas-hamorning dtantsur sambetts and all :)10:34
dtantsurpas-ha, o/10:34
sambettsHey pas-ha, dtantsur10:34
dtantsursambetts, ifarkas, release day today! please check https://review.openstack.org/293406 and https://review.openstack.org/29001310:35
dtantsur(and anything you think must go in mitaka)10:35
*** bradjones has joined #openstack-ironic10:36
*** bradjones has quit IRC10:36
*** bradjones has joined #openstack-ironic10:36
*** shu-mutou is now known as shu-mutou-AFK10:36
openstackgerritNisha Agarwal proposed openstack/ironic-python-agent: Add support for partition images in agent driver  https://review.openstack.org/16022410:37
sambettsdtantsur: the iPXE one looks good to me and the gate job is passing but I've not got a way to locally test it10:38
openstackgerrityolanda.robla proposed openstack/bifrost: Remove generated default.leases file for libvirt  https://review.openstack.org/29340210:42
openstackgerritMerged openstack/ironic-inspector: Update links to existing documentation  https://review.openstack.org/29340610:44
dtantsursambetts, lets treat it as experimental for now. I just want the iPXE gate to pass at all in stable/mitaka :)10:44
sambettsdtantsur: Happy for me to +2 without locally testing then?10:45
dtantsursambetts, yep. that's not a product feature after all10:47
*** e0ne has joined #openstack-ironic10:50
*** awiddersheim has quit IRC10:50
ifarkasdtantsur, do you want to reply to mgould's comment before I +A? https://review.openstack.org/#/c/290013/3/devstack/plugin.sh10:52
dtantsurifarkas, responded. not sure it's worth a separate comment tbh10:53
dtantsurall devstack plugins have configuration files diverged from what e.g. puppet does10:53
*** Nisha_away has joined #openstack-ironic10:55
*** Nisha has quit IRC10:55
*** sivaramakrishna has quit IRC10:58
ifarkasdtantsur, yeah, I also think it's not necessary10:58
*** sandhya has quit IRC11:00
*** cfarquhar has quit IRC11:03
mgouldOK, fair enough11:05
*** mannidi has joined #openstack-ironic11:05
* mgould adds a comment withdrawing his objection11:06
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: Updated eslint plugins  https://review.openstack.org/29395311:07
*** stendulker_ has joined #openstack-ironic11:13
*** boris-42 has joined #openstack-ironic11:14
*** stendulker has quit IRC11:16
openstackgerritNisha Agarwal proposed openstack/ironic: Add support for partition images in agent drivers  https://review.openstack.org/16200811:17
*** stendulker_ has quit IRC11:18
Nisha_awaylucasagomes, dtantsur sambetts vdrok ^^^^11:18
*** shu-mutou-AFK has quit IRC11:20
*** Nisha_away has quit IRC11:21
*** Nisha_brb has joined #openstack-ironic11:21
openstackgerritMerged openstack/ironic-inspector: [devstack] add support for using iPXE instead of plain PXE  https://review.openstack.org/29001311:25
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: Added phantomjs to package  https://review.openstack.org/29396211:26
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: Updated application dependencies  https://review.openstack.org/29396311:26
dtantsurifarkas, sambetts, anything else you guys want to see in mitaka before I request the final release?11:27
Nisha_brbdtantsur, lucasagomes i was looking into documenting the partition image support for agent drivers. I tried but could not find if we have the documentation saying that agent drivers support only partition images11:27
Nisha_brbdtantsur, lucasagomes it would be great if you could give me a pointer if such documetation is there11:27
dtantsursorry, I'm in the pre-release burden now :(11:28
Nisha_brbdtantsur, lucasagomes s/agent drivers support only partition images/agent drivers support only whole disk images11:28
Nisha_brbdtantsur, ok11:28
openstackgerritDebayan Ray proposed openstack/proliantutils: Adds test coverage in tox.ini  https://review.openstack.org/29393611:30
openstackgerritMoshe Levi proposed openstack/ironic: Adding InfiniBand Support  https://review.openstack.org/26426311:31
openstackgerritAparna proposed openstack/ironic: Doc update to enable HTTPS in Glance and Ironic comm  https://review.openstack.org/28053311:32
*** thrash|g0ne is now known as thrash11:32
ifarkasdtantsur, nothing from my side11:33
dtantsurok, no objections recorded, I'm proposing a release :)11:37
vdrokmorning ironic! getvasanth_ you're welcome :) Nisha_brb will take a look today11:37
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: Updated ui-bootstrap  https://review.openstack.org/29396911:37
dtantsurRelease request: https://review.openstack.org/#/c/293968/11:38
Nisha_brbvdrok, thanks11:40
*** Nisha_brb is now known as Nisha11:40
openstackgerritVaishnavi proposed openstack/ironic: Add support for building ISO for deploy ramdisk  https://review.openstack.org/26459011:42
deraylucasagomes, could you have a look at the doc patch: https://review.openstack.org/#/c/276765/ ? Already been +1'ed by TheJulia and stendulker11:46
*** chlong has joined #openstack-ironic11:48
*** openstackgerrit has quit IRC11:48
*** openstackgerrit has joined #openstack-ironic11:49
*** Goneri has quit IRC11:52
sambettsmgould: thanks for the review, I've fixed the commit message11:52
*** mtanino has joined #openstack-ironic11:52
mgouldsambetts, np11:53
mgouldnice, that's much clearer11:55
openstackgerritNisha Agarwal proposed openstack/ironic: Document partition image support for agent drivers  https://review.openstack.org/29397511:55
openstackgerritNisha Agarwal proposed openstack/ironic: Add support for partition images in agent drivers  https://review.openstack.org/16200811:56
openstackgerritDebayan Ray proposed openstack/proliantutils: Switch tox unit test command to use ostestr  https://review.openstack.org/29397711:59
*** ohamada has quit IRC12:01
*** awiddersheim has joined #openstack-ironic12:02
*** ohamada has joined #openstack-ironic12:05
lennybGood afternoon, Ironic. Do we have some kind of tests ( tempest/plugin ) for ironic-inspector?12:05
sambettslennyb: no tempest plugin, but we have devstack exercises that test it out12:06
lennybsambetts: 10x, where can I get more info/how to about it?12:07
dtantsurlennyb, aarefiev is working on a tempest plugin right now12:07
sambettslennyb: https://github.com/openstack/ironic-inspector/tree/master/devstack12:07
aarefievyep, I am12:08
dtantsurlennyb, right now we have a big bash script running in gate ^^. we also have functional tests running in place as a tox environment12:08
lennybaarefiev, dtantsur : thanks. I will take a look. Does it report to gerrit? ex: https://review.openstack.org/#/c/286556/  how can I see it's results?12:11
dtantsurlennyb, on this patch it didn't run, because it only touches docs12:12
deraydtantsur, https://review.openstack.org/#/c/276765/7/doc/source/drivers/ilo.rst@1382: is there any way I can fix that inline (and submit) w/o raising another patchset?12:12
dtantsurlennyb, check e.g. https://review.openstack.org/#/c/290013/12:12
dtantsurderay, these are nit, ignore them unless you have to create another patchset for other reasons12:13
*** awiddersheim has quit IRC12:13
deraydtantsur, there's a "fix" option and the tooltip says to it can fix it inline12:14
*** raildo-afk is now known as raildo12:14
lennybdtantsur : thanks. I assume gate-ironic-inspector-dsvm-ipa result is the most important one12:15
dtantsurderay, that's still a new patchset, just created via web UI12:15
dtantsurlennyb, right12:15
deraydtantsur, oo.. okat then. will leave it AS_IS12:15
*** baoli has joined #openstack-ironic12:18
deraydtantsur, posted my comments. thanks :)12:20
*** vaishnavi_r has quit IRC12:21
*** baoli_ has joined #openstack-ironic12:21
*** baoli has quit IRC12:24
*** trown|outtypewww is now known as trown12:25
*** dims has quit IRC12:25
openstackgerritNisha Agarwal proposed openstack/ironic-python-agent: Add support for partition images in agent driver  https://review.openstack.org/16022412:28
*** Nisha_away has joined #openstack-ironic12:29
*** Nisha has quit IRC12:29
*** odyssey4me has joined #openstack-ironic12:34
openstackgerrityolanda.robla proposed openstack/bifrost: Remove generated default.leases file for libvirt  https://review.openstack.org/29340212:37
*** Keedya has joined #openstack-ironic12:37
*** afaranha has joined #openstack-ironic12:39
KeedyaGood morning all! 0/12:41
openstackgerritNisha Agarwal proposed openstack/ironic: Add support for partition images in agent drivers  https://review.openstack.org/16200812:42
*** piet has joined #openstack-ironic12:42
*** Nisha_away is now known as Nisha12:42
*** Keedya has quit IRC12:45
openstackgerrityolanda.robla proposed openstack/bifrost: Remove generated default.leases file for libvirt  https://review.openstack.org/29340212:52
*** amotoki has joined #openstack-ironic12:57
*** mannidi has quit IRC12:59
*** Goneri has joined #openstack-ironic12:59
*** mannidi has joined #openstack-ironic12:59
derayNobodyCam, may be a low hanging one (doc patch): https://review.openstack.org/#/c/276765/13:01
*** kalpase has left #openstack-ironic13:03
*** aparnav has quit IRC13:03
*** mtanino has quit IRC13:07
sambettsmgould: seems like there may be resistance against the devstack patch :/13:09
TheJuliaanyone remember offhand what the absolute memory footprint minimum is for IPA?13:09
mgouldsambetts, I am shocked, shocked13:09
TheJuliain coreos that is13:10
sambettsTheJulia: 1gb13:10
dtantsur1 GiB for coreos, yeah13:10
sambettsTheJulia: tinyipa its 386mb13:10
sambetts;)13:10
dtantsur1.5 GiB bare minimum for DIB-based (at least used to be the last time I've checked)13:10
TheJuliaok13:10
TheJuliasambetts: nice13:10
TheJuliaI'm still waiting for this coreos build at 1GB to call the API :\13:10
mgouldsambetts, if only we had some system for checking which, if any, projects your change broke...13:11
sambettsmgould: haha yes :-p I'm hoping the CIs are going to prove him wrong13:11
TheJuliabingo, it worked finally13:11
TheJuliayolanda: yeah, it all has to do with ram :(13:11
mgouldsambetts, on the upside, he's suggested a possible workaround13:12
sambettsmgould: I'd really hate to do that though because it would mean that any project that creates its own flavor in a devstack plugin would have to implement the same workaround13:13
*** lazy_prince has quit IRC13:14
yolandaworked?13:14
TheJuliayolanda: on one of the vms I manually kicked to 1gb13:15
TheJuliagoing to kick the CI job to 1GB and retry13:15
sambettsmgould: the extra.d directory has files with leading numbers to ensure that they are run in order, tempest is 80_tempest so its obvs meant to run as near to last as possible13:15
yolandaTheJulia, dnsmasq doesn't seem to be related. I disabled the inventory options and failed anyway13:16
openstackgerritJulia Kreger proposed openstack/bifrost: DHCP test improvements  https://review.openstack.org/29374313:16
mgouldsambetts, yeah, I still think your patch is the best solution13:16
* mgould wonders if anyone relies on the patch running after extra.d13:17
mgouldyay, mutable global state...13:17
*** sinval-afk is now known as sinval13:18
sinvalmorning Ironic13:18
TheJuliayolanda: yeah, I put up another change last night to collect interface counters for CI  so we could see if data was transferred13:18
*** thiagop_ has joined #openstack-ironic13:19
sambettsmgould: reminds me of this https://xkcd.com/1172/13:20
*** ohamada has quit IRC13:20
sinvalhey TheJulia, I saw your comment about that bug involving OneView drivers, you're right, my description of the problem is wrong, I'll update it, thanks13:20
mgouldsambetts, ahahahaha *weeps*13:21
TheJuliayolanda: re-running locally as well where I've been able to reproduce the failure... although with tcpdump running in the background :)13:21
TheJuliasinval: oh cool, more context upfront is always better imho13:21
sambettsTheJulia: https://review.openstack.org/#/c/293648/13:22
thiagop_Good morning Ironic13:22
sambettso/ thiagop_13:22
*** thiagop has quit IRC13:23
*** ohamada has joined #openstack-ironic13:23
*** cdearborn has joined #openstack-ironic13:23
*** thiagop_ is now known as thiagop13:24
openstackgerrityolanda.robla proposed openstack/bifrost: Remove generated default.leases file for libvirt  https://review.openstack.org/29340213:24
*** Sukhdev has joined #openstack-ironic13:25
*** links has quit IRC13:28
*** Sukhdev has quit IRC13:28
*** Nisha has quit IRC13:31
*** mjturek1 has left #openstack-ironic13:32
*** krtaylor has quit IRC13:36
*** kalpase has joined #openstack-ironic13:37
*** mjturek1 has joined #openstack-ironic13:39
mjturek1hey, I'm seeing this error when deploying with the agent_ipmitool driver http://paste.openstack.org/show/490897/ has anyone seen it before? I'm wondering why 'swift_temp_url_key' would be needed13:40
*** piet has quit IRC13:45
NobodyCamgood morning Ironicers13:45
sambettsmjturek1: agent deploy uses swift temp urls to download the image, so you need to configure that option13:46
mjturek1ahhhh thanks sambetts didn't realize that13:47
dtantsurmorning NobodyCam13:47
sambettso/ NobodyCam13:48
yolandaTheJulia, test worked with vms of 1GB13:48
*** krtaylor has joined #openstack-ironic13:48
*** getvasanth_ has quit IRC13:48
yolandaso i can amend the test, and spin up 3 vms of 1gb insteaf of 5 of 500mb ...13:48
dtantsurNobodyCam, so, now I also got experience of being on a video :) music one though13:48
NobodyCammorning (and happy st patricks day) sambetts dtantsur lucasagomes jroll deray devananda jlvillal TheJulia thiagop sinval gabriel-bezerra yolanda mgould and anyone else not listed here13:49
NobodyCamdtantsur: awesome :)13:49
sinvalNobodyCam, o/13:49
NobodyCamo/13:49
dtantsurNobodyCam, https://youtu.be/A6Uxr_Zj7CA (not so much of me, mostly in the end)13:50
jlvillalGood morning NobodyCam dtantsur yolanda sinval sambetts mjturek1 TheJulia mgould lucasagomes and everyone else.13:50
dtantsurmorning jlvillal13:50
sinvaljlvillal, o/13:50
mjturek1good morning jlvillal13:50
* jlvillal trying to wake up...13:50
dtantsurand sinval and everyone else :)13:50
sinvaldtantsur, o/13:51
*** ifarkas has quit IRC13:52
*** dims has joined #openstack-ironic13:53
*** piet has joined #openstack-ironic13:53
NobodyCamdtantsur: great video. :)13:56
dtantsurthanks13:56
dtantsur:)13:56
sinvaldtantsur, you have a cool two hands on that bass13:56
Gonerihi all, can someone review https://review.openstack.org/#/c/286283/ (the use of wipefs instead of different dd call to wipe the disk)13:57
*** mtanino has joined #openstack-ironic13:57
thiagopNobodyCam: today is St. Patrick's? I need to find a green beer..13:58
dtantsurheh, we have some in the center13:58
NobodyCamI thnk it is13:58
NobodyCamwould have to ask lucasagomes to be sure13:58
NobodyCamheheheh13:58
openstackgerritHao Li proposed openstack/ironic-specs: Fix spelling error in local-boot-support-with-partition-images.rst  https://review.openstack.org/29403813:59
*** kalpase has quit IRC13:59
*** e0ne has quit IRC13:59
deraydtantsur, you rock! Is it just DIY? it's a full fledged professinal video. This Ironic is simply _not_ your place ;)14:00
*** e0ne has joined #openstack-ironic14:00
dtantsurLOL14:00
dtantsurthanks deray, sinval :)14:00
thiagopmorning NobodyCam jlvillal14:00
NobodyCammorning thiagop14:00
dtantsurderay, yep, it's fully DIY (and a pretty budget one)14:00
openstackgerritHao Li proposed openstack/ironic-specs: Fix a spelling error  https://review.openstack.org/29403814:01
derayNobodyCam, sinval thiagop jlvillal g'morning o/14:01
deraydtantsur, ok14:01
thiagopderay: o/14:01
*** mbound has quit IRC14:02
*** Nisha has joined #openstack-ironic14:03
*** ifarkas has joined #openstack-ironic14:03
derayNobodyCam, hi, possible to have a quick look at doc patch: https://review.openstack.org/#/c/276765/ ?14:06
NobodyCamderay: looking now14:08
TheJuliayolanda: \o/14:08
derayNobodyCam, okay14:08
*** piet has quit IRC14:10
TheJuliayolanda: https://review.openstack.org/#/c/293743/ <-- if you want to review it since I made some minor workflow changes, and overall I think I actually improved the test verification since the other vms halt when they can't get an address14:11
yolandaTheJulia, sure14:11
TheJuliahttps://review.openstack.org/#/c/293662/ <-- super simple easy review since the package was missing in the base debian config but present in one of the ubuntu variants.  Like one-liner14:14
*** jaypipes has quit IRC14:15
*** piet has joined #openstack-ironic14:15
NobodyCamderay: line 812 of https://review.openstack.org/#/c/276765/7/doc/source/drivers/ilo.rst ... "and OA." what is14:17
NobodyCamOA here?14:18
derayOA = Onboard Admininstrator14:18
TheJuliaOnboard Administrator14:18
NobodyCamack :) thank you14:18
NobodyCam(both)14:18
TheJulia:)14:18
*** mdnadeem has joined #openstack-ironic14:19
derayNobodyCam, :-)14:19
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Tests for testing chassis-create command  https://review.openstack.org/29363414:19
*** dims has quit IRC14:19
*** Goneri has quit IRC14:21
*** jaypipes has joined #openstack-ironic14:22
*** dims has joined #openstack-ironic14:22
mosheledtantsur: hi14:25
*** dims has quit IRC14:25
dtantsurmoshele, o/14:25
mosheledtantsur: in what log you see that the vendor name is empty https://review.openstack.org/#/c/279940/ in the gate?14:26
mosheledtantsur: I test it with  Ubuntu using this change in the DIB https://review.openstack.org/#/c/289233/14:27
dtantsurmoshele, e.g. http://logs.openstack.org/40/279940/11/check/gate-ironic-inspector-dsvm-ipa-src-nv/23aa583/logs/screen-ironic-inspector.txt.gz#_2016-03-08_16_44_56_87114:27
*** pcaruana has quit IRC14:28
dtantsurI'm not against keeping it there, I'm just pointing that it probably does not work on coreos..14:28
dtantsurmy main -1 reason is the formatting problem14:28
*** achanda has joined #openstack-ironic14:29
mosheledtantsur:  ok I will try it on coreos as well14:30
*** mkovacik_ has joined #openstack-ironic14:30
dtantsurthanks14:30
dtantsurmoshele, also note that the tinyipa build also has "lshw not found" http://logs.openstack.org/40/279940/11/check/gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-nv/b6a88de/logs/ironic-bm-logs/baremetalbrbm_0_console_08-03-2016-16:51:02.txt.gz14:31
mosheledtantsur: I didn't use the coreos and tinyipa because I wanted to test it with mellanox drivers so I used DIB, but I will debug this on coros and tinyipa  thanks14:33
*** mkovacik has quit IRC14:33
yolandaTheJulia, looks good14:34
*** garthb has joined #openstack-ironic14:35
*** garthb has quit IRC14:36
*** piet has quit IRC14:37
openstackgerritMerged openstack/ironic-specs: Fix a spelling error  https://review.openstack.org/29403814:37
*** Goneri has joined #openstack-ironic14:38
*** achanda has quit IRC14:40
*** piet has joined #openstack-ironic14:42
*** mkovacik__ has joined #openstack-ironic14:45
NobodyCamderay: when reading the type os firmware url schemes supported I see swift.. that is swift temp url?14:45
mosheledtantsur: where do you add packages to the tinyipa? in here https://github.com/openstack/ironic-python-agent/blob/master/imagebuild/tinyipa/build_files/buildreqs.lst14:46
derayNobodyCam, No. It's just a way of depicting a swift object under a container14:46
dtantsurmoshele, redirecting your question to sambetts14:47
moshelesambetts:  where do you add packages to the tinyipa? in here https://github.com/openstack/ironic-python-agent/blob/master/imagebuild/tinyipa/build_files/buildreqs.lst?14:47
*** mkovacik_ has quit IRC14:48
derayNobodyCam, it just facilitates the operator to provide a swift object which is eventually the firmware file to be updated14:48
sambettsmoshele: Are they required to install the ironic-python-agent package or do they just need to be installed on the final image?14:48
moshelesambetts: required for the ironic-python-agent package14:49
NobodyCamderay: no token or user name / password is required?14:49
*** Sukhdev has joined #openstack-ironic14:49
derayNobodyCam, No. not required as the supposition is that the fw file object has been created with the same credentials as that of conductor uses for the swift container14:52
NobodyCamany spec cores have a free minute to take a look at: https://review.openstack.org/#/c/275726 OneView Dynamic allocation of nodes ???14:52
derayso that object resides on the same account14:52
NobodyCamderay: That may be a good thing to NOTE on the doc14:53
sambettsmoshele: then you need to add it in both buildreqs.lst and finalreqs.lst14:53
derayand the conductor can also access the object gievn its own name and conatiner name14:54
moshelesambetts: ok thanks14:54
derayNobodyCam, okay14:54
dtantsursambetts, moshele, I'd say it should be only in the final image.. why do we need it to build IPA?14:55
*** Nisha has quit IRC14:55
derayNobodyCam, will be leaving now. Can you post ur queries/comments/suggestions on the patch? will address them t'morrow14:55
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: Updated UX for Default Cloud dropdown  https://review.openstack.org/29408014:56
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: Removed border from header  https://review.openstack.org/29408114:56
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: Introduced btn-gradient styles.  https://review.openstack.org/29408214:56
*** piet has quit IRC14:56
NobodyCamderay: doing now!14:56
*** piet has joined #openstack-ironic14:56
derayNobodyCam, sure :) thx again14:56
sambettsdtantsur: I'm not sure the context sorry :/14:56
mosheledtantsur: you are right I miss understood the answer14:56
sambettsmoshele, dtantsur: oh then you need only add it to finalreqs.lst then :)14:57
NobodyCamderay: awesome thank you .. and have a good night :)14:57
moshelesambetts, dtantsur: thanks :)14:58
*** ifarkas_ has joined #openstack-ironic15:00
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: console utils: add methods for serial console with socat  https://review.openstack.org/29387115:00
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: SSH: add SocatConsole class  https://review.openstack.org/29387515:00
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: IPMINative: add NativeIPMISocatConsole class  https://review.openstack.org/29387415:00
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: IPMItool: add IPMISocatConsole class  https://review.openstack.org/29387315:00
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: Add node serial console installation howto  https://review.openstack.org/29387215:00
*** deray has quit IRC15:01
*** ifarkas has quit IRC15:03
*** achanda has joined #openstack-ironic15:03
*** achanda has quit IRC15:08
*** rloo has joined #openstack-ironic15:08
openstackgerritbin proposed openstack/ironic: Fix VirtualBox cannot set boot device when powered on  https://review.openstack.org/29095115:11
*** dprince has joined #openstack-ironic15:11
*** mannidi has quit IRC15:14
*** dprince has quit IRC15:17
*** vishwanathj_zzz is now known as vishwanathj15:17
moshelesambetts: I am not able to build the tinyipa I get locale.Error: unsupported locale setting I tried this http://askubuntu.com/questions/205378/unsupported-locale-setting-fault-by-command-not-found but it didn't help any idea?15:17
*** [1]cdearborn has joined #openstack-ironic15:19
sambettsmoshele: are you building it by running make in the imagebuild/tinyipa directory?15:19
moshelesambetts: yes15:19
sambettscan you paste.openstack.org the build logs please?15:21
TheJuliaNobodyCam: cinerama: https://review.openstack.org/#/c/293743/ this change fixes the dhcp integration test sequence that yolonda added15:21
*** dprince has joined #openstack-ironic15:22
moshelesambetts: see http://paste.openstack.org/show/490922/15:22
sambettsmoshele: are you able to use pip on your machine outside of the tinyipa build?15:23
sambettsmoshele: what OS are you building it on/15:24
sambetts?15:24
*** rolfb_ has joined #openstack-ironic15:25
moshelesambetts: Ubuntu 14.04.1, and regarding the pip yes I installed there openstack using devstack15:25
jlvillalmgould: I noticed in my /var/log/syslog/ that I have a dnsmasq entry showing it refusing to hand out a DHCP address to the failing bare-metal node.15:25
jlvillalmgould: As an FYI :)15:25
*** rbrud has quit IRC15:26
*** rbrud has joined #openstack-ironic15:26
mgouldjlvillal, excellent!15:26
sambettsmoshele: I believe your seeing this error http://askubuntu.com/questions/446793/unsupported-locale-setting15:26
moshelesambetts: and my IPA repo is in nfs not local disk maybe it related15:26
mgouldwhat's the error message?15:26
jlvillalmgould: Something like "no available addresses".  I need to dig more into the neutron logs.15:27
jlvillalI'm trying to backport lucasagomes' patch https://review.openstack.org/293748   To get all the bare-metal console logs. Not sure if it is working yet. First pass last night didn't seem to work...15:28
*** Sukhdev has quit IRC15:28
mgouldjlvillal, cool, I'll see if I'm getting the same thing15:28
moshelesambetts: trying to see if their solution  works15:29
*** Nisha has joined #openstack-ironic15:30
dtantsursambetts, ifarkas_, wdyt about backporting https://review.openstack.org/#/c/293618/ and https://review.openstack.org/#/c/293475/ to liberty?15:31
ifarkas_dtantsur, right, I think you already mentioned that's the plan15:32
openstackgerritYuriy Zveryanskyy proposed openstack/ironic-specs: Driver configuration warnings  https://review.openstack.org/27907015:32
dtantsurifarkas_, yeah, just confirming with you guys15:33
*** cdearborn has quit IRC15:33
sambettsdtantsur: +2 from me15:33
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: Add vendor id and vendor name to interface information  https://review.openstack.org/27994015:34
sambettsdtantsur: When did I lose my workflow rights from stable inspector, maybe I never had them I can't remember15:34
dtantsursambetts, that's for stable branch, it was initially like that15:34
sambettsdtantsur: oh :/ I guess I never noticed15:35
dtantsurmaybe it's a bit too paranoid, but I'd like to be able to sync stable changes with policies, releases, etc15:35
*** rbrud has quit IRC15:35
dtantsursambetts, if you feel like carrying about stable branch policies, I can add you to the release group15:35
*** rolfb_ is now known as rbrud15:35
* jlvillal hopes the line at the coffee machine is not too long...15:35
*** rolfb_ has joined #openstack-ironic15:36
dtantsurjlvillal, line to coffee is always too long, even if it's 1 person15:36
sambettsdtantsur: I'm happy as it is, its proably best to only have one person to hit the button on stable15:36
jlvillaldtantsur: We have "fancy" machines which grind the coffee and make each cup one at a time...15:36
jlvillalsambetts: Coward! :P15:36
* jlvillal doesn't blame sambetts at all :)15:37
sambettshahaha ;)15:37
mgoulddtantsur, you know that the first webcam was set up to monitor the Cambridge CS department's coffee machine?15:37
dtantsurlol, why?15:37
sambettsmoshele: thats brilliant!15:38
sambettsmgould: *15:38
mgouldso the person running it could check if there was a queue, I suppose...15:38
*** kirankv has quit IRC15:38
sambettsmoshele: sorry for the noise15:38
* dtantsur though about someone stealing all the coffee at first :D15:38
mgouldah, no, it was to check if the coffee pot had finished brewing: https://en.wikipedia.org/wiki/Trojan_Room_coffee_pot15:38
mgouldor, indeed, to see if someone had drunk it all :-)15:39
dtantsurso I was not too far from the truth!15:40
jrollNobodyCam: I think that docker issue you found in IPA gate yesterday is a real thing and busted our gate :/15:41
jrolldtantsur: fyi ^15:41
dtantsuroh wut?15:41
dtantsurwe've just fixed the previous breakage...15:41
*** sivaramakrishna has joined #openstack-ironic15:41
NobodyCamTheJulia: is there a bug for 293743?15:42
NobodyCamjroll: ugh :(15:42
TheJuliaNobodyCam: afaik no, didn't really think it was a bug since yolanda has been setting up a new test job that just didn't work out of the gate (pun not intended)15:45
openstackgerritYuriy Zveryanskyy proposed openstack/ironic-specs: Driver configuration warnings  https://review.openstack.org/27907015:46
NobodyCamTheJulia: ack ... I was just asking15:46
TheJulia:)15:46
jrolldtantsur: http://logs.openstack.org/24/160224/20/check/gate-tempest-dsvm-ironic-pxe_ipa-src/6be8f96/logs/devstacklog.txt.gz#_2016-03-17_12_54_16_96115:48
*** vaishnavi_r has joined #openstack-ironic15:50
dtantsurdamn15:51
*** piet has quit IRC15:51
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: Introduced btn-gradient styles.  https://review.openstack.org/29408215:51
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: Adjusted flow for header elements  https://review.openstack.org/29411415:51
*** piet has joined #openstack-ironic15:51
dtantsurjroll, we didn't use ironic-lib in liberty, did we? trown is asking for confirmation15:51
jrolldtantsur: no, don't believe so15:52
trownthanks jroll15:53
jrollnp15:53
*** krtaylor has quit IRC15:55
dtantsurjroll, JayF, do I get it right: IPA advertises the first IP address it could find for a NIC, right?15:55
JayFUh, I think so?15:56
JayFI know right where that code is though, checking15:56
jrollI have no clue without reading code15:56
dtantsurnow I have people with many NIC's, and the first NIC is on a external network15:57
dtantsurseems like IPA can connect to Ironic, but Ironic can't connect back15:57
dtantsurI think the bug is public: https://bugzilla.redhat.com/show_bug.cgi?id=130898115:57
openstackbugzilla.redhat.com bug 1308981 in openstack-ironic "ironic provisioning fails after several retries" [Urgent,New] - Assigned to lmartins15:57
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add CLI tests to check JSON response body  https://review.openstack.org/29123915:57
JayFhttps://github.com/openstack/ironic-python-agent/blob/d25d94b3168deb9b3183a3c59bebd4b9c9ec2463/ironic_python_agent/agent.py#L18615:58
JayFdtantsur: ^15:58
jrollhm, maybe we should change the code to find the route to ironic15:58
dtantsurwe have https://github.com/openstack/ironic-python-agent/blob/d25d94b3168deb9b3183a3c59bebd4b9c9ec2463/ironic_python_agent/cmd/agent.py#L70-L74 but I don't feel safe using interface names15:59
JayFI remember originally that was an option you passed in15:59
dtantsurwe had enough problems with block device names15:59
JayFyou had to tell IPA what its own IP is15:59
dtantsurI'd rather have something like ipa-provisioning-subnet=192.0.2.1/2416:00
dtantsurand then IPA would pick an IP matching this subnet16:00
jrolloh I think it still is JayF16:00
jrollhttps://github.com/openstack/ironic-python-agent/blob/d25d94b3168deb9b3183a3c59bebd4b9c9ec2463/ironic_python_agent/cmd/agent.py#L4616:01
jrolldtantsur: ^^16:01
JayFI remember ellen wrote that ip detect code16:01
JayFbam that's pretty solid actually16:01
JayFyou could have Ironic pass that into IPA16:01
dtantsurjroll, does not look too usable to me, tbh.. how do I set it? I can't set it in conductor, right?16:01
JayFif you're using neutron provider16:01
JayFdtantsur: the idea would be to add something to ironic which will pass it in when setting up dhcp :)16:01
jrolloooo, good question :)16:01
dtantsurhmm, with neutron yes. though the same problem might appear with inspection16:01
JayFI'm not saying we should not fix the IPA issue :) I just think there are multiple interesting paths16:02
jrollyou could add something to the unit file too16:02
jrolluntil ironic has a thing16:02
JayFwell it's the kernel command line16:02
jrollOR a cli option16:02
JayFbut you have to have a separate option for each agent ip16:02
jrollall of these can be specified on the command line16:02
JayFyou'd have to do something like write a bash script to detect the correct ip and toss it in a file16:03
dtantsurjroll, JayF, what if we make https://github.com/openstack/ironic-python-agent/blob/d25d94b3168deb9b3183a3c59bebd4b9c9ec2463/ironic_python_agent/cmd/agent.py#L70 also accept net/subnet?16:03
JayFand read that into the cli option in the unit16:03
jrollright, so /usr/bin/ironic-python-agent --advertise-host $(find my ip)16:03
*** mgoddard has quit IRC16:03
jrollit's terrible16:03
jrollbut until ironic has a thing16:03
*** mgoddard has joined #openstack-ironic16:03
jrolldtantsur: maybe16:03
JayFdtantsur: I'd prefer it be a different option, and the actual interface name option be deprecated16:03
JayFdtantsur: for the reasons you pointed out re: block device naming16:03
jrolldtantsur: I think the best thing to do, is in the detection code, ip route get $ironic_ip16:03
jrolland use that interface16:03
JayFthese are all leftover bad decisions from teeth-agent, I think16:04
JayFlol16:04
dtantsurhehe16:04
dtantsurok, lemme experiment with the 'ip route' thing16:04
JayFip route get $IP16:04
JayFwill return the nexthop route16:04
JayFcan you can see what interface that would go out of16:04
* dtantsur looks for something like "machine-readable" argument16:04
JayFit's still a two-step process I think16:04
JayF[vagrant@localhost ~]$ ip route get 4.2.2.116:05
JayF4.2.2.1 via 10.0.2.2 dev eth0  src 10.0.2.1516:05
JayF    cache16:05
JayFdtantsur: ^ just gotta pull the address after "src"16:05
JayFdtantsur: the output of iproute2 is infinitely parsable, but there's no --csv option or the like16:05
dtantsur:)16:05
dtantsurok, lemme try to put together a patch16:05
JayFif you get stuck or need a review ping me16:06
JayFI've written more than my share of bs like that :P16:06
dtantsurthanks!16:06
JayFdtantsur: one request/piece of advice: short circuit all your logic if only one ip16:06
JayFdtantsur: that way you can reduce the surface area significantly of folks who could be impacted by the change16:06
dtantsurJayF, sorry, I didn't quite get what exactly you suggest to short-circuit16:07
JayFonly run crazy-ip-route-get logic in cases of >1 IP on the server the agent is running on16:08
JayFyou can get all network interfaces via the hwm method, then iterate over, if >1, use logic to differentiate, if exactly one, just use that IP and skip the route get logic16:08
dtantsurJayF, what's the point to use this one IP, if 'ip route get' fails?16:09
*** krtaylor has joined #openstack-ironic16:09
JayFThat's a reasonable question, and my only answer would be those awful switches from a decade ago that proxy arped the entire internet16:10
JayFlol16:10
dtantsurheh16:10
JayFwas more out of a thought to only change behavior in the possibly broken case; i.e. >1 IP16:11
JayFbut what you say makes sense too so .... *shrug*16:11
jroll"proxy arped the entire internet" <- I don't even want to know16:11
JayFtl;dr, you had a /0 subnet, and the switch would arp for any internet ip you wanted and just pass the traffic that way16:12
JayFI don't understand why they existed, but I've heard of them from people who had to run them16:12
JayFand it sounds like one of the worst ideas possible16:12
jrollI said I *don't* want to know :P16:12
dtantsurwow16:12
*** harshs has joined #openstack-ironic16:13
mgouldJayF, I think it was to handle hosts with misconfigured subnet masks16:13
* JayF may have, as a rookie sysadmin, misset a subnet mask on a router once and broke the whole offices' access to yahoo16:14
mgouldhost tries to ARP for an IP that isn't on the local subnet => router proxy-ARPs it => no tech-support call16:15
*** mbound has joined #openstack-ironic16:19
openstackgerritVasyl Saienko proposed openstack/ironic: Move _normalize_mac to driver utils  https://review.openstack.org/29392216:21
*** pcaruana has joined #openstack-ironic16:22
*** praneshp__ has joined #openstack-ironic16:24
*** ifarkas_ has quit IRC16:25
openstackgerritJulia Kreger proposed openstack/ironic-python-agent: Fall back from ATA erase to shredding  https://review.openstack.org/27090216:25
openstackgerritJohn L. Villalovos proposed openstack/ironic: Don't depend on existing file perm for qemu hook  https://review.openstack.org/29413916:27
openstackgerritNisha Agarwal proposed openstack/ironic-python-agent: Add support for partition images in agent driver. It also adds the ironic-lib in the requirements list of the IPA package.  https://review.openstack.org/16022416:27
JayFTheJulia: re: 270902, I'd be a little careful with that16:28
JayFTheJulia: most ATA erasable drives *can* get into a state where they're locked and secure erase won't work unless manually unlocked (I bet we could make IPA attempt this on its own)16:28
JayFTheJulia: so I could easily see someone flipping that option on, and grinding their disks into a pulp shredding them over and over even though the server has support for ISE16:29
TheJuliaJayF: yeah, I don't exactly like it myself, but a few people i've talked to have encountered that situation and I didn't see any way to try and unlock the disks16:29
TheJuliabeyond what IPA already tries to do16:29
JayFI mean at the least, if we fail to lock the drive in step 1, we should still try to unlock it in step 216:30
JayFsorta building the assumption that whatever locked the drive last was IPA when it failed cleaning16:30
JayFso if IPA got the drive "stuck in a lock" it could get it out w/o shredding16:30
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to set neutron port timeout from config  https://review.openstack.org/29387616:30
JayFif the user locked the drive, optionally just shred it (in my case I'd never enable that, because my users aren't supposed to be able to lock drives :P)16:30
TheJuliaso do you think a conductor level config entry of some sort?16:31
JayFI'm saying if you're going to add that 'feature'16:32
JayFwe should probably add logic in the Generichwm to also make damn sure ipa can "fix itself" if cleaning failed after IPA locked it16:32
JayFbecause otherwise we setup a situation where *an IPA bug* or cleaning failure could lead to hardware destruction16:33
TheJuliawell, if you read the docs, users already make the asumption that the behavior it puts in place is the default16:33
TheJuliathe exception/failure to unlock becomes a surprise16:33
TheJuliaand some people just don't care... they should though, but yeah16:33
JayFdo you understand my point though? That IPA can initiate this failed state?16:33
TheJuliayes16:33
JayFSo I'm not saying -1 to your change16:34
JayFI'm saying it needs to *also* fix that case to prevent IPA causing hardware failure16:34
JayFi.e. if the step 1 (locking the drive) fails, then still attempt step 2 (secure erasing a locked drive)16:34
TheJuliamy perception, and I'll go back and check this, is that the IPA code already attempts that16:34
JayFI'm 99.99% sure that's not true, checking16:34
TheJulia:16:34
TheJuliae:(16:34
TheJuliaerr16:34
* TheJulia just gives up on typing for the day16:34
JayFhttps://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L63616:35
TheJuliaJayF: so... IF I was looking at code from the future, I guess I should have saved it :)16:35
JayFwe actually bail explicitly if the drive is locked at the start of ata_erase16:35
JayFmaybe if we see it's already locked, we should try to ata erase it with IPA's password (NULL)16:35
TheJuliayeah16:35
JayFand then pass HardwareNotSupported (or whatever it's called) if your option is set16:35
TheJuliaI'll revise it based on that16:35
JayFif your option isn't set, have it bail there16:35
JayFs/pass/raise/16:36
dtantsursambetts, also this cherry-pick: https://review.openstack.org/#/c/294104/ (as it's devstack-only, I think it's valuable that we run iPXE gate on liberty as well)16:37
*** thrash is now known as thrash|biab16:37
sambettsdtantsur: I've got no problems with it16:37
dtantsurplease +2 then :)16:38
sambettsdone :)16:38
mgouldJayF, TheJulia: this is probably a stupid idea, but could we use whole-disk encryption and then just throw away the key?16:38
JayFmgould: that's literally what ISE enabled drives do16:39
mgould\o/16:39
jrollcan we be sure to review https://review.openstack.org/#/c/160224/21 and https://review.openstack.org/#/c/162008/18 ? those are the last two things I'm waiting on before IPA/ironic releases for mitaka16:39
* mgould feels vindicated, but will Google next time :-)16:39
* jroll digs in to IPA gate now16:39
JayFheh it's okay, I knew none of this stuff until I had to run a bare metal cloud16:39
jrollJayF: before I start flailing around, do you have any thoughts what might be up with http://logs.openstack.org/24/160224/20/check/gate-tempest-dsvm-ironic-pxe_ipa-src/6be8f96/logs/devstacklog.txt.gz#_2016-03-17_12_54_16_96116:40
jrollkeep in mind there's been lots of devstack image changes lately, could be related (still trusty though)16:40
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Add support for partition images in agent driver.  https://review.openstack.org/16022416:41
jrolland yes, the daemon should be running: http://logs.openstack.org/24/160224/20/check/gate-tempest-dsvm-ironic-pxe_ipa-src/6be8f96/logs/devstacklog.txt.gz#_2016-03-17_12_54_14_91016:41
jrollheh thanks for fixing that :P16:41
JayFjroll: ^ fixed your commit message nit16:41
JayFlol yeah16:41
dtantsurJayF, jroll, do we expect people to be able to provide ironic host name instead of IP address to IPA?16:42
jrollbut yeah IPA gate is hard down with this docker thing :/16:42
dtantsurI think no, cause nameservers are unlikely to be available16:42
dtantsurbut dunno16:42
JayFjroll: ...ugh16:42
jrolldtantsur: yeah, I think that's the intent16:42
JayFdtantsur: We certainly pass an IP, not a DNS name, but I could see both working16:42
jrollI don't see any reason we should block domain names16:42
dtantsurok, then I have to try resolve it... mmm, interesting16:43
JayFjroll: that looks almost like a version mismatch. Is docker_build.bash running on the same host ad the dockerd?16:43
dtantsurunfortunately 'ip route' does not seem to work with host names16:43
JayFare you sure?16:43
jrollJayF: yeah, should be16:43
jroll$ ip r g google.com16:43
jrollError: an inet prefix is expected rather than "google.com".16:43
JayFdammit that's stupid ip route216:43
jrollheh16:43
dtantsuryep :(16:43
dtantsurnow I have to 1. figure out if something is an IP address, 2. if not - try resolving it16:44
JayFI bet there are py libraries16:44
dtantsura-lot-of-fun.jpg16:44
JayFthat do a noop resolution if it's an ip16:44
JayFyou could always something like16:44
TheJuliadtantsur: I'm fairly sure there are canned calls for that16:44
JayFtry: execute(ip route get google.com); except processexecutionerror: execute(ip route get `dig +short google.com`)16:44
dtantsurheh, that's possible :)16:45
JayFjroll: that could also be a perms issue16:45
JayFjroll: like docker.sock isn't readable by the user that build is running as16:45
JayFjroll: that's an AWFUL error, but it's just basically "I can't use docker.sock for some reason"16:45
JayFjroll: without a version mismatch, perms are the most likely culprit.16:46
JayFjroll: either that or docker started only listening on tcp by default or something16:46
jrollJayF: yeah, dropping some debugging stuff in16:46
jrollJayF: well... 2016-03-17 12:54:16.891 | + imagebuild/coreos/build_coreos_image.sh:main:L24:   sudo -E make16:46
jrollI hope root has access16:46
JayFwhat distro are they using now? Ubu trusty?16:47
jrollyep16:47
*** harshs has quit IRC16:47
JayFAnd are they using the stock docker or the one from docker directly16:47
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: [WIP] Updated test libraries  https://review.openstack.org/29416016:47
jroll2016-03-17 12:39:31.966 | Setting up docker.io (1.6.2~dfsg1-1ubuntu4~14.04.1) ...16:48
JayFk going to look at hte default configs that puts in16:48
*** yarkot_ has joined #openstack-ironic16:48
* krotscheck grumps.....16:49
*** aarefiev22 has joined #openstack-ironic16:50
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: [WIP] Bind to interface routable to the ironic host, not a random one  https://review.openstack.org/29416216:51
dtantsurjroll, JayF, rough draft ^^ will finish tomorrow16:51
openstackgerritChris Krelle proposed openstack/bifrost: Allow mysql_username and mysql_password to be passed via env var.  https://review.openstack.org/29373316:51
dtantsurjroll, JayF and there is the second part of the problem: iSCSI also serves on a wrong interface...... will figure out tomorrow16:52
openstackgerritJim Rollenhagen proposed openstack/ironic-python-agent: debug gate  https://review.openstack.org/29416416:53
jrolldtantsur: fun!16:53
*** garthb has joined #openstack-ironic16:53
jrollJayF: ^ hoping to get some debug stuff from there16:53
JayFjroll: almost certainly a race16:53
jrollJayF: you think docker just isn't up yet?16:53
JayFI'm looking but thinking that is true16:53
JayFbecause the socket creation is done AFTER the service reports up16:53
JayFbecause crappy initscript is crappy16:54
JayFhttps://github.com/docker/docker/issues/664716:55
JayFsays it's fixed, but I'm not sure I believe it16:55
JayFbecause I also don't think we're waiting on the upstart event16:55
JayFI think we're waiting for the script to return, which happens before the post_script finishes16:55
JayFjroll: ^ there's confirmation of your race16:55
openstackgerritJim Rollenhagen proposed openstack/ironic-python-agent: debug gate  https://review.openstack.org/29416416:55
JayFjroll: whatever code starts docker needs to spin until /var/run/docker.sock exists16:55
jrollJayF: oh, lovely16:56
* JayF has 95% confidence in that16:56
JayFI'd need a real VM to confirm, as docker doesn't run in docker :P16:56
JayF(and containers are my playground to see inside stuff like this)16:57
JayFjroll: can we see /var/log/upstart/docker.log on a failed host?16:58
JayFit doesn't appear to be uploaded :(16:59
jrollJayF: add it to 294164?16:59
*** saripurigopi has joined #openstack-ironic16:59
JayFyou're going to heisenbug yourself there, fwiw16:59
jrollheh I know17:00
openstackgerritJim Rollenhagen proposed openstack/ironic: Wait for docker to be up before building IPA  https://review.openstack.org/29417217:00
jrollthere's the real fix17:00
* jroll files a bug to add to that17:00
moshelesambetts: I want to had lshw to tinyipa but I get this error http://logs.openstack.org/40/279940/12/check/gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-nv/03db744/logs/devstacklog.txt.gz#_2016-03-17_15_58_54_97517:01
openstackgerritJim Rollenhagen proposed openstack/ironic: Wait for docker to be up before building IPA  https://review.openstack.org/29417217:01
moshelesambetts: and it is in http://repo.tinycorelinux.net/7.x/x86/tcz/17:02
dtantsurjroll, ouch, so it's my fault then :(17:02
sambettsmoshele: we build x86_6417:02
jrolldtantsur: what'd you do :D17:03
JayFand why didn't tests catch it and prevent you from doing it17:03
dtantsurjroll, this service restart bit17:03
*** suro-patz has joined #openstack-ironic17:03
jrolldtantsur: oh, interesting17:03
jrollJayF: we don't build from source in ironic17:03
dtantsurthis17:03
JayFooooh17:03
sambettsmoshele: http://distro.ibiblio.org/tinycorelinux/6.x/x86_64/tcz/17:03
jrollaha, https://github.com/openstack/ironic/commit/f3167750d96d3065ce5e9b5d7994e1545a9b01ad17:04
jrolldtantsur: it happens :)17:04
*** Sukhdev has joined #openstack-ironic17:04
sambettsmoshele: it does look like lshw is built for tinycore x86_64, we may have to manually build it like we do for qemu_utils17:04
JayF...should we run a -src gate against any change to lib/ironic in ironic?17:05
jrollJayF: we could, but I think the risk of breakage is pretty low17:05
jrollBUT17:05
moshelesambetts: I see17:05
jrollwe should totally use depends-on with a dummy commit in IPA to verify things if we think it could break the IPA build17:05
JayFjroll: cost of breakage is high though, is the counter17:05
jrollJayF: yeah17:06
dtantsuryeah, lesson learned17:06
*** afaranha has quit IRC17:07
JayFwell fwiw not really your bug dtantsur17:07
JayFthat initscript shouldn't return to shell until the service starts17:07
JayFand they fixed it for the upstart event, so things like dependant upstart services wait17:08
JayFbut not in the primary script so it just returns early17:08
*** Sukhdev has quit IRC17:10
*** praneshp__ has quit IRC17:13
*** saripurigopi has quit IRC17:15
*** jaybeale has joined #openstack-ironic17:16
*** Sukhdev has joined #openstack-ironic17:16
*** rbrud has quit IRC17:16
*** Sukhdev has quit IRC17:17
*** MattMan has quit IRC17:18
*** MattMan has joined #openstack-ironic17:19
*** yarkot_ has quit IRC17:19
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: [WIP] Updated test libraries  https://review.openstack.org/29416017:26
*** degorenko is now known as _degorenko|afk17:27
*** jistr has quit IRC17:27
*** e0ne has quit IRC17:27
*** Marga_ has quit IRC17:28
*** vaishnavi_r has quit IRC17:28
krotscheckBest response to a bug report ever:17:30
krotscheckMe: "Hey, you're over quota"17:30
*** penick has joined #openstack-ironic17:30
krotscheckThem: "Some random person set up a mirror in china, go get it from there"17:30
* krotscheck facepalms.17:30
*** krtaylor has quit IRC17:31
*** saripurigopi has joined #openstack-ironic17:33
*** Guest57182 has quit IRC17:36
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: Removed phantomjs from test suite  https://review.openstack.org/29416017:38
*** Sukhdev has joined #openstack-ironic17:38
*** dprince has quit IRC17:38
*** penick has quit IRC17:39
NobodyCamjroll: looking at https://review.openstack.org/#/c/294172/2/devstack/lib/ironic ... what would you think about a for loop so we could time out: maybe something like http://paste.openstack.org/show/x1k6HW0Adipb1QKhXLZW17:43
*** krtaylor has joined #openstack-ironic17:43
JayFNobodyCam: I already put that as a comment on there17:44
jrollNobodyCam: oh, good point, JayF said the same17:44
NobodyCamnote ^^^ written in paste screen17:44
NobodyCamdoh17:44
JayFGMTA, Chris!17:44
jrollNobodyCam: you're welcome to update the patch ;D17:44
jrollor I can do it in a bit17:44
*** Sukhdev has quit IRC17:45
NobodyCamack... I can in a few :)17:45
jrollthank you!17:46
*** mkovacik__ has quit IRC17:47
*** spandhe has joined #openstack-ironic17:47
*** praneshp_ has joined #openstack-ironic17:49
*** dims has joined #openstack-ironic17:50
*** vishwana_ has joined #openstack-ironic17:51
*** mgould has quit IRC17:52
*** vishwanathj has quit IRC17:54
*** rcernin has quit IRC17:55
*** rodrigods has joined #openstack-ironic17:59
*** baoli_ has quit IRC18:01
*** sivaramakrishna has quit IRC18:05
*** thrash|biab is now known as thrash18:07
openstackgerritChris Krelle proposed openstack/ironic: Wait for docker to be up before building IPA  https://review.openstack.org/29417218:11
openstackgerritChris Krelle proposed openstack/ironic: Wait for docker to be up before building IPA  https://review.openstack.org/29417218:12
NobodyCamjroll: JayF: ^^^^18:12
*** piet has quit IRC18:13
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: [wip] Removed phantomjs from test suite  https://review.openstack.org/29416018:13
*** ohamada has quit IRC18:15
NobodyCamdoh forgot to add a break in htere18:16
openstackgerritChris Krelle proposed openstack/ironic: Wait for docker to be up before building IPA  https://review.openstack.org/29417218:16
*** pcaruana has quit IRC18:18
*** e0ne has joined #openstack-ironic18:18
openstackgerritChris Krelle proposed openstack/ironic: Wait for docker to be up before building IPA  https://review.openstack.org/29417218:21
* NobodyCam stops editing now18:21
*** mkovacik has joined #openstack-ironic18:26
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: Add vendor id and vendor name to interface information  https://review.openstack.org/27994018:30
*** pcaruana has joined #openstack-ironic18:30
*** piet has joined #openstack-ironic18:34
*** dtantsur is now known as dtantsur|afk18:36
NobodyCamnight dtantsur|afk18:36
dtantsur|afksee you18:36
thiagopnight dtantsur|afk , have a green beer for us18:37
NobodyCammmmm (green) beer18:40
thiagopNobodyCam: I'll have to be good with the red ale I have in my refrigerator today18:40
* TheJulia thinks redbreast18:41
*** penick has joined #openstack-ironic18:42
NobodyCamthiagop: Killian's Irish Red???18:43
thiagopNobodyCam: no, it's national. It's called Baden Baden.18:44
*** sambetts is now known as sambetts|afk18:45
*** baoli has joined #openstack-ironic18:46
*** sivaramakrishna has joined #openstack-ironic18:50
*** Marga_ has joined #openstack-ironic18:51
*** Nisha has quit IRC18:52
*** ChubYann has joined #openstack-ironic18:58
*** mgoddard has quit IRC18:58
*** baoli has quit IRC19:00
*** baoli has joined #openstack-ironic19:02
*** jcoufal has quit IRC19:03
*** krtaylor has quit IRC19:08
*** athomas has quit IRC19:10
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: Add vendor id and vendor name to interface information  https://review.openstack.org/27994019:19
*** krtaylor has joined #openstack-ironic19:20
*** baoli has quit IRC19:20
*** baoli has joined #openstack-ironic19:21
*** moshele has quit IRC19:21
openstackgerritJim Rollenhagen proposed openstack/ironic: Wait for docker to be up before building IPA  https://review.openstack.org/29417219:24
*** e0ne has quit IRC19:24
*** jaybeale has quit IRC19:24
jrollNobodyCam: ^ fixed bashate there19:24
JayFNobodyCam: commented19:26
JayFoh19:26
JayFwell commented on it after jroll's change, so still valid, lol19:26
jrollJayF: yeah, was wondering the same19:27
jrollJayF: or even just die inside that if19:27
*** baoli has quit IRC19:28
*** baoli has joined #openstack-ironic19:28
JayFwell that's why I'd use a while loop with an increment19:29
JayFbreak on success, die on i=20 or whatever19:29
Gonerihi, can someone take a look on this review (to make use of wipefs): https://review.openstack.org/#/c/286283/19:31
*** davidlenwell has quit IRC19:36
*** davidlenwell has joined #openstack-ironic19:42
*** baoli has quit IRC19:43
*** moshele has joined #openstack-ironic19:43
*** baoli has joined #openstack-ironic19:43
*** piet has quit IRC19:48
*** baoli has quit IRC19:51
*** Sukhdev has joined #openstack-ironic20:01
NobodyCamhave a alienware tech here will take a look at the commets as soon as he is done fixing my system20:01
jrollNobodyCam: I'm on it20:06
*** cdearborn has joined #openstack-ironic20:08
openstackgerritJim Rollenhagen proposed openstack/ironic: Wait for docker to be up before building IPA  https://review.openstack.org/29417220:09
openstackgerritJim Rollenhagen proposed openstack/ironic-python-agent: Kicking CI for dependent change  https://review.openstack.org/29427020:11
jroll^ the IPA patch is to make sure it works20:11
JayFjroll: missed a comment, commented20:12
jrollwat20:13
jrolloops20:13
JayFthat was a nice review, who is that in IRC?20:13
openstackgerritJim Rollenhagen proposed openstack/ironic: Wait for docker to be up before building IPA  https://review.openstack.org/29417220:14
jroll¯\_(ツ)_/¯20:14
jrollphillipe, you here?20:14
*** pcaruana has quit IRC20:17
JayFI want to give him a o/ for that good review20:17
jrollikr20:18
openstackgerritJim Rollenhagen proposed openstack/ironic-python-agent: Add support for partition images in agent driver.  https://review.openstack.org/16022420:21
*** e0ne has joined #openstack-ironic20:21
jrollNobodyCam: ^ changed the depends-on there if you want to re-+220:21
*** [1]cdearborn has quit IRC20:22
*** vishwanathj has joined #openstack-ironic20:23
*** jaybeale has joined #openstack-ironic20:25
*** piet has joined #openstack-ironic20:25
*** vishwana_ has quit IRC20:25
*** pcaruana has joined #openstack-ironic20:30
*** mbound has quit IRC20:34
*** sivaramakrishna has quit IRC20:36
*** ametts has joined #openstack-ironic20:42
*** trown is now known as trown|PTO20:54
*** spandhe has quit IRC21:04
*** e0ne has quit IRC21:04
*** Sukhdev has quit IRC21:12
*** spandhe has joined #openstack-ironic21:15
*** baoli has joined #openstack-ironic21:23
*** Goneri has quit IRC21:26
*** amotoki has quit IRC21:29
*** Nisha has joined #openstack-ironic21:32
*** maratoid has joined #openstack-ironic21:38
*** Nisha has quit IRC21:39
NobodyCamjroll: looks like 160224 still failed21:40
NobodyCamlooking at logs now21:40
NobodyCamlooks like the same error though the new code did run: http://logs.openstack.org/24/160224/23/check/gate-tempest-dsvm-ironic-agent_ssh-src/84e4363/logs/devstacklog.txt.gz#_2016-03-17_21_02_39_615 (new code) and http://logs.openstack.org/24/160224/23/check/gate-tempest-dsvm-ironic-agent_ssh-src/84e4363/logs/devstacklog.txt.gz#_2016-03-17_21_02_41_833 (error)21:44
JayFlooking21:45
JayFNobodyCam: while I'm waiting for that to load, wanna replace your +2 here? https://review.openstack.org/#/c/160224/2321:45
JayFwell.. shit21:46
NobodyCameven with it failing tests still?21:46
JayFCan we get infra to hold on to one of these nodes post-failure21:46
JayFIf I could just ssh to one I could figure it out in minutes :(21:46
JayFI thought I had anyway, lol21:46
*** afaranha has joined #openstack-ironic21:48
*** baoli has quit IRC21:49
*** raildo is now known as raildo-afk21:49
*** baoli has joined #openstack-ironic21:50
*** aNupoisc has joined #openstack-ironic21:50
openstackgerritChris Krelle proposed openstack/bifrost: Allow mysql_username and mysql_password to be passed via env var.  https://review.openstack.org/29373321:50
*** harshs has joined #openstack-ironic21:59
jrollugh, seriously?21:59
NobodyCamyep JayF is working with infra atm on the issue21:59
jrollJayF: yes, you can request a node be held. kick tests, get to jenkins console via status.o.o/zuul, get hostname21:59
jrollah ok22:00
*** amotoki has joined #openstack-ironic22:00
JayFjroll: look in -infra, I have a devstack science box up trying to repro22:00
JayFjroll: if I don't get to the bottom of the rabbithole today I can put your key on it22:00
jrollyeah, ok22:00
*** dims has quit IRC22:01
jlvillalJayF: You might be interested in my project:  https://github.com/JohnVillalovos/devstack-gate-test22:01
*** dims_ has joined #openstack-ironic22:01
JayFapparently http://logs.openstack.org/24/160224/23/check/gate-tempest-dsvm-ironic-agent_ssh-src/84e4363/logs/reproduce.sh is a thing now22:01
JayFthat's what I'm using22:02
jlvillalIf you do a 'vagrant up' with that, then it creates a devstack gate environment.22:02
* NobodyCam didn't know about that until today22:02
*** rolfb_ has quit IRC22:03
*** harshs has quit IRC22:05
cineramajlvillal: cool22:08
jlvillal:)22:08
JayFNeither did I, NobodyCam22:08
JayFand now I'm already making it better22:08
JayFinfra is good at nerd sniping :P22:08
NobodyCamhehehe22:09
*** moshele has quit IRC22:09
*** baoli has quit IRC22:10
*** harshs has joined #openstack-ironic22:10
*** ametts has quit IRC22:14
*** piet has quit IRC22:16
*** piet has joined #openstack-ironic22:16
*** amotoki has quit IRC22:17
mrdaMorning Ironic22:18
NobodyCammorning mrda :)22:18
mrdaMorning NobodyCam22:20
thiagopmorning mrda22:20
thiagopnight mrda, night Ironic!22:20
mrdanight thiagop!22:21
*** dims has joined #openstack-ironic22:23
*** baoli has joined #openstack-ironic22:23
*** thiagop has quit IRC22:25
*** dims_ has quit IRC22:26
*** mbound has joined #openstack-ironic22:26
*** baoli has quit IRC22:26
*** baoli has joined #openstack-ironic22:27
*** moshele has joined #openstack-ironic22:28
NobodyCamnight thiagop22:32
JayFNobodyCam: jroll: science fail22:42
JayFNobodyCam: jroll: the provided script doesn't even appear to have cloned ipa22:42
jrollJayF: :/22:42
NobodyCam:(22:42
JayFAnd trying to repro it manually didn't work either22:46
JayFbut I think that means it's almost certainly a race22:46
JayFor something really funky that only happens when Ironic's ds stuff is run22:46
JayFso you all might have to ask infra to leave one around when it fails22:46
JayFalthough I susspect when you get on it, you might find that it works22:47
JayFraces are no fun :(22:47
jrollurgh22:47
jrollJayF: I'll have someone hold one22:48
JayFthat being said, I'm curious what removing dtantsur|afk's restart would do22:48
JayFgiven that seems to be the suspected catalyst22:48
JayFperhaps there's a better fix to whatever problem he had and we can trade a hard problem for an easy one, lol22:48
jrollwell, his bug was "fedora doesn't auto-start docker"22:50
jrollso I guess we could check if it's running first22:50
*** maratoid has quit IRC22:53
*** Sukhdev has joined #openstack-ironic22:54
*** Keedya has joined #openstack-ironic22:55
*** Keedya has quit IRC22:57
*** baoli has quit IRC22:59
*** baoli has joined #openstack-ironic23:00
*** piet has quit IRC23:00
*** harshs has quit IRC23:01
*** piet has joined #openstack-ironic23:01
*** mbound has quit IRC23:02
*** baoli has quit IRC23:05
*** cfarquhar has joined #openstack-ironic23:05
*** cfarquhar has quit IRC23:05
*** cfarquhar has joined #openstack-ironic23:05
*** baoli has joined #openstack-ironic23:05
*** baoli has quit IRC23:08
*** dims has quit IRC23:08
*** baoli has joined #openstack-ironic23:08
openstackgerritMerged openstack/bifrost: Fix and improve test job data collection  https://review.openstack.org/29375223:09
*** harshs has joined #openstack-ironic23:12
*** piet has quit IRC23:14
jrollthat thing where you kick tests but forget you were waiting for them to start23:17
jrolland now they're done and you can't hold back the node again23:17
*** lucasagomes has quit IRC23:18
*** Sukhdev has quit IRC23:24
*** harshs has quit IRC23:25
*** lucasagomes has joined #openstack-ironic23:25
*** dims has joined #openstack-ironic23:28
*** aNupoisc has quit IRC23:30
*** dims has quit IRC23:32
*** Sukhdev has joined #openstack-ironic23:34
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: SSH: add SocatConsole class  https://review.openstack.org/29387523:35
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: IPMINative: add NativeIPMISocatConsole class  https://review.openstack.org/29387423:35
openstackgerritAkira YOSHIYAMA proposed openstack/ironic: IPMItool: add IPMISocatConsole class  https://review.openstack.org/29387323:35
*** Sukhdev has quit IRC23:36
*** jaybeale has quit IRC23:36
*** baoli has quit IRC23:37
*** jaybeale has joined #openstack-ironic23:37
*** jaybeale has quit IRC23:37
*** jaybeale has joined #openstack-ironic23:39
*** alex_xu has quit IRC23:41
*** penick has quit IRC23:44
*** baoli has joined #openstack-ironic23:44
*** smoriya_ has joined #openstack-ironic23:48
*** baoli has quit IRC23:49
*** alex_xu has joined #openstack-ironic23:50
*** baoli has joined #openstack-ironic23:52
*** davideagnello has quit IRC23:53
jrollJayF: you want in on this docker debugging adventure?23:53
jrollthe node is held back, though if you want to wait for devstack to finish it's gonna be a few23:53
*** davideagnello has joined #openstack-ironic23:54
JayFjroll: I mean, realistically what I was going to do is try to run the docker build23:54
*** baoli has quit IRC23:54
JayFif it failed, troubelshoot23:54
JayFif it doesn't, weep23:54
jrollyeah23:55
JayFand the failure will be easy to t-shoot, basically perms on the socket or something funky like double-docker-installs23:55
*** baoli has joined #openstack-ironic23:55
JayFbut I'm 99.99999% sure it's the timing of that restart23:55
jrollwell, it's run as root23:55
JayFand the sock probably exists even if the daemon isn't backing it23:55
jrollso shouldn't be permissions23:55
jrollah yeah23:55
JayFwhich means the check we're doing may not be effective23:55
JayFI think the better way to go would be remove that restart23:55
jrollmaybe just run some docker command until it succeeds23:56
JayFand go back to where docker package is installed and start it there, if fedora23:56
JayFI mean, that's possible too, but kinda gross :P23:56
jrollya23:56
jlvillalJayF: Any chance IPA can send a lot more messages to the console?23:56
JayFjlvillal: pass a --debug on the command line23:56
jlvillalJayF: I'm trying to debug a different issue and don't see much detail.23:56
JayFjlvillal: if you wanna get it logging to console, there's an option in systemd to make journald log to console, I don't know it right off but it's not too hard23:56
JayFso do oyu want debug mode, or do you want useful logs to /dev/console?23:57
jlvillalJayF: Thanks. I am doing this in my grenade test work.23:57
jrollsystemd.journald.forward_to_console=yes23:57
jrollon kernel cmd line23:57
JayF...how?23:57
JayFdid you just google it or do you have a hotkey for irc log searching now23:57
JayFlol23:57
jlvillalThanks :)23:57
jrollI started when he asked, took about 60s :P23:58
JayFlol23:58
* jlvillal makes a note...23:58
JayFI think we run ipa in the gate with that option23:58
jrollpretty slow imo23:58
jlvillalJayF: We do? I don't think so...23:58
jlvillalFrom looking at the ironic-bm-logs/* files...23:58
JayFI'm 99% sure we do, but it's rarely effective because only the last boot is logged in those files23:58
jlvillalJayF: Not anymore!!!23:58
jrollJayF: so newer znc moved log locations, I now have a split between old and new :/23:59
jlvillal:)23:59
JayFso you almost always get the log of the successful boot of the OS that's booting23:59
jrollyeah not anymore23:59
JayFwait, what?23:59
jrollwe keep them all now23:59
JayFif that's fixed we need to run with that option in the gate23:59
jroll+123:59
jlvillalYeah lucasagomes (aka superstar) made it keep each one.23:59
*** davideagnello has quit IRC23:59
JayFWhat's the right place to make that change? I can throw up a fix real quick23:59
* jlvillal is unsure...23:59

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