Thursday, 2016-02-25

*** Sukhdev has quit IRC00:03
*** harshs has joined #openstack-ironic00:04
*** yonglihe has quit IRC00:12
*** klindgren_ has joined #openstack-ironic00:13
*** klindgren has quit IRC00:15
*** keedya has quit IRC00:16
*** blakec has joined #openstack-ironic00:18
*** jaybeale has joined #openstack-ironic00:19
*** Sukhdev has joined #openstack-ironic00:24
*** Sukhdev has quit IRC00:27
*** Sukhdev has joined #openstack-ironic00:35
*** Sukhdev has quit IRC00:44
*** blakec has quit IRC00:46
openstackgerritNaohiro Tamura proposed openstack/ironic: Follow up nits of irmc oob inspection  https://review.openstack.org/28006000:49
*** ppiela has quit IRC00:52
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/28398400:56
*** Sukhdev has joined #openstack-ironic01:01
*** yolanda_ has joined #openstack-ironic01:05
*** harshs has quit IRC01:08
*** harshs has joined #openstack-ironic01:09
*** Sukhdev has quit IRC01:10
*** harshs has quit IRC01:10
*** harshs has joined #openstack-ironic01:11
*** garthb has quit IRC01:14
*** garthb_ has quit IRC01:14
*** harshs has quit IRC01:15
*** yolanda_ is now known as yolanda01:18
*** blakec has joined #openstack-ironic01:20
*** jaypipes has quit IRC01:24
*** spandhe has quit IRC01:26
*** mtanino has quit IRC01:32
*** blakec has quit IRC01:34
*** lhcheng has quit IRC01:35
*** suro-patz has quit IRC01:39
*** zzzeek has quit IRC01:45
*** zzzeek has joined #openstack-ironic01:45
*** zzzeek has quit IRC01:47
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/28398301:47
*** zzzeek has joined #openstack-ironic01:50
*** kan_ has joined #openstack-ironic01:51
*** dims has quit IRC01:52
*** zzzeek has quit IRC01:52
*** zzzeek has joined #openstack-ironic01:54
*** praneshp_ has joined #openstack-ironic01:56
*** praneshp has quit IRC01:58
*** praneshp_ is now known as praneshp01:58
*** ijw has quit IRC02:02
*** Sukhdev has joined #openstack-ironic02:05
*** Sukhdev has quit IRC02:09
*** penick has quit IRC02:19
*** hoangcx has joined #openstack-ironic02:25
*** amotoki has joined #openstack-ironic02:29
*** ChrisAusten has joined #openstack-ironic02:30
*** sivaramakrishna has joined #openstack-ironic02:39
*** smoriya_ has quit IRC02:41
*** links has joined #openstack-ironic02:47
*** jaybeale has quit IRC02:48
openstackgerritSivaramakrishna Garimella proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326202:57
*** boris-42 has quit IRC03:04
*** yuanying has quit IRC03:16
*** amotoki has quit IRC03:19
*** awiddersheim has quit IRC03:22
*** amotoki has joined #openstack-ironic03:30
*** baoli_ has quit IRC03:31
*** links has quit IRC03:34
*** amotoki has quit IRC03:35
*** appu103_ has joined #openstack-ironic03:38
*** baoli has joined #openstack-ironic03:40
*** amotoki has joined #openstack-ironic03:40
*** stendulker has joined #openstack-ironic03:41
*** baoli has quit IRC03:42
*** baoli has joined #openstack-ironic03:43
openstackgerritTan Lin proposed openstack/ironic: Add ensure_thread_contain_context() to task_manager  https://review.openstack.org/28452003:44
*** chlong_ has quit IRC03:47
*** baoli has quit IRC03:48
*** chlong_ has joined #openstack-ironic03:49
*** harshs has joined #openstack-ironic04:00
*** suro-patz has joined #openstack-ironic04:01
*** amotoki has quit IRC04:01
*** hoangcx has quit IRC04:02
*** hoangcx has joined #openstack-ironic04:03
*** yuanying has joined #openstack-ironic04:07
*** chenke__ has quit IRC04:08
*** chenke__ has joined #openstack-ironic04:09
*** amotoki has joined #openstack-ironic04:13
openstackgerritMerged openstack/proliantutils: Support for iLO based firmware update  https://review.openstack.org/20354304:23
*** yolanda has quit IRC04:28
*** links has joined #openstack-ironic04:29
*** achanda has quit IRC04:35
*** Marga__ has joined #openstack-ironic04:40
*** vaishnavi has joined #openstack-ironic04:43
*** Marga__ has quit IRC04:44
*** Marga_ has quit IRC04:44
openstackgerritZhenguo Niu proposed openstack/ironic: Restart consoles on conductor startup  https://review.openstack.org/25725204:45
*** achanda has joined #openstack-ironic04:55
*** Marga_ has joined #openstack-ironic04:56
*** Marga_ has quit IRC05:00
*** Marga_ has joined #openstack-ironic05:01
*** Marga_ has quit IRC05:01
*** Marga_ has joined #openstack-ironic05:01
*** Marga_ has quit IRC05:02
openstackgerritDebayan Ray proposed openstack/proliantutils: Added changelog and authors  https://review.openstack.org/28407505:02
*** Marga_ has joined #openstack-ironic05:02
*** yonglihe has joined #openstack-ironic05:05
*** Fdaisuke has quit IRC05:15
*** Fdaisuke has joined #openstack-ironic05:17
*** suro-patz has quit IRC05:18
openstackgerritTan Lin proposed openstack/ironic: Add possible values for config options  https://review.openstack.org/25013205:43
*** yuanying has quit IRC05:47
*** dims has joined #openstack-ironic05:52
*** suro-patz has joined #openstack-ironic05:52
*** jaybeale has joined #openstack-ironic05:56
*** jaybeale has quit IRC06:03
harlowja_at_homedtantsur|afk, https://review.openstack.org/#/c/284390/06:07
harlowja_at_homefixes some glitches, ha06:07
*** praneshp_ has joined #openstack-ironic06:09
*** harlowja_at_home has quit IRC06:11
*** kalpase has joined #openstack-ironic06:12
*** praneshp has quit IRC06:12
*** praneshp_ is now known as praneshp06:12
*** harlowja_at_home has joined #openstack-ironic06:13
*** harshs has quit IRC06:14
*** suro-patz has quit IRC06:24
*** praneshp_ has joined #openstack-ironic06:26
*** praneshp has quit IRC06:28
*** praneshp_ is now known as praneshp06:28
*** ChubYann has quit IRC06:29
*** mbound has joined #openstack-ironic06:30
*** harlowja_at_home has quit IRC06:33
*** harlowja_at_home has joined #openstack-ironic06:33
*** mbound has quit IRC06:34
*** dims has quit IRC06:43
*** achanda_ has joined #openstack-ironic06:45
*** achanda has quit IRC06:46
*** vaishnavi has quit IRC06:46
*** suro-patz has joined #openstack-ironic06:49
*** praneshp has quit IRC06:49
*** hoangcx_ has joined #openstack-ironic06:51
*** hoangcx has quit IRC06:52
*** harlowja_at_home has quit IRC06:54
*** yuanying has joined #openstack-ironic06:54
*** zzzeek has quit IRC06:56
*** vaishnavi has joined #openstack-ironic06:57
openstackgerritZhenguo Niu proposed openstack/ironic: Add nodes tagging suport - objs  https://review.openstack.org/23335706:59
openstackgerritZhenguo Niu proposed openstack/ironic: Joined 'tags' column while getting node  https://review.openstack.org/25306506:59
openstackgerritZhenguo Niu proposed openstack/ironic: Add db api layer for CRUD operations on node tags  https://review.openstack.org/23259806:59
*** zzzeek has joined #openstack-ironic06:59
openstackgerritZhenguo Niu proposed openstack/ironic: Add DB layer support for node tags filter  https://review.openstack.org/27262407:00
openstackgerritTan Lin proposed openstack/ironic: Add 'X-Openstack-Request-ID' header in the response  https://review.openstack.org/23800807:02
*** zzzeek has quit IRC07:10
*** ifarkas has joined #openstack-ironic07:33
*** mbound has joined #openstack-ironic07:43
openstackgerritvinay kumar muddu proposed openstack/proliantutils: Changes to NIC info retrieving logic  https://review.openstack.org/27311807:45
Haomengdtantsur|afk: can you help to review my spec, which addressed your comments already - https://review.openstack.org/#/c/266688/, thank you.07:52
*** suro-patz has quit IRC07:54
*** smoriya_ has joined #openstack-ironic07:59
*** yuanying has quit IRC08:01
*** kalpase1 has joined #openstack-ironic08:01
*** kalpase has quit IRC08:01
Haomengdtantsur|afk: and this ipa patch - https://review.openstack.org/#/c/272420/, thanks:)08:04
*** mbound has quit IRC08:04
*** mkovacik has quit IRC08:09
*** yuanying has joined #openstack-ironic08:11
*** athomas has joined #openstack-ironic08:15
*** yuanying has quit IRC08:16
*** yuanying has joined #openstack-ironic08:18
*** Oku_OS has quit IRC08:18
*** yuanying has quit IRC08:26
*** vinm213 has joined #openstack-ironic08:26
vinm213morning Ironicers08:26
vinm213haomeng o/08:27
Haomengvinm213: morning:)08:27
vinm213deray /o08:27
*** deray has joined #openstack-ironic08:29
openstackgerritVaishnavi proposed openstack/ironic: Activate iLO license as part of manual cleaning process  https://review.openstack.org/25992108:29
*** yuanying has joined #openstack-ironic08:32
*** yuanying has quit IRC08:38
*** jcoufal has joined #openstack-ironic08:41
*** yuanying has joined #openstack-ironic08:41
*** yuanying has quit IRC08:47
*** boris-42 has joined #openstack-ironic08:51
*** amotoki has quit IRC08:53
*** hoangcx_ has quit IRC08:59
*** athomas has quit IRC08:59
*** hoangcx has joined #openstack-ironic09:00
*** achanda_ has quit IRC09:00
*** jistr has joined #openstack-ironic09:01
*** achanda has joined #openstack-ironic09:04
aarefiev morning!09:05
*** athomas has joined #openstack-ironic09:06
*** NobodyCa1 has joined #openstack-ironic09:07
*** trusted has joined #openstack-ironic09:08
*** trusted has quit IRC09:10
*** amotoki has joined #openstack-ironic09:10
*** sivaramakrishna has quit IRC09:10
*** Oku_OS has joined #openstack-ironic09:10
*** samueldmq1 has joined #openstack-ironic09:11
*** sivaramakrishna has joined #openstack-ironic09:11
*** sivaramakrishna is now known as Guest9228709:11
lennyb_Hi Ironic, are there any tempest scenario tests for this project? Or something 'more aggressive" than API testing ?09:13
*** samueldmq1 has quit IRC09:15
*** NobodyCa1 has quit IRC09:15
aarefievlennyb_: hi, yep there is basic one https://github.com/openstack/ironic/blob/master/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py09:15
*** yolanda has joined #openstack-ironic09:19
*** ohamada has joined #openstack-ironic09:19
*** achanda has quit IRC09:19
*** kalpase1 has quit IRC09:20
lennyb_aarefiev: thanks I will look into it.09:21
openstackgerritVaishnavi proposed openstack/ironic: Add support for dib based agent ramdisk in lib/ironic  https://review.openstack.org/26457909:21
*** ndipanov has joined #openstack-ironic09:21
*** achanda has joined #openstack-ironic09:24
*** lucas-hungry is now known as lucasagomes09:26
lucasagomesmorning all :-)09:26
Haomenglucasagomes: morning lucas:)09:27
Haomenglucasagomes: can you help to review my spec - https://review.openstack.org/#/c/266688/, and this ipa patch  https://review.openstack.org/#/c/272420 , if you have time:) thank you, nice day:)09:28
lucasagomesHaomeng, sure, will do09:28
aarefievlucasagomes, Haomeng: morning09:28
Haomenglucasagomes: thanks a lot, nice day:)09:28
Haomengaarefiev: morning:)09:29
lucasagomesyou too09:29
lucasagomesaarefiev, morning09:29
*** achanda has quit IRC09:29
*** kalpase has joined #openstack-ironic09:32
*** yolanda has quit IRC09:35
openstackgerritVaishnavi proposed openstack/ironic: Activate iLO license as part of manual cleaning process  https://review.openstack.org/25992109:39
vaishnavigood morning everyone.. :)09:39
vinm213morning lucassgomes09:41
vinm213morning aarefiev, lennyb_09:41
yuriyzmorning all Ironicers09:43
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Set node last_error in TaskManager  https://review.openstack.org/27399209:44
aarefievyuriyz, vinm213, vaishnavi: morning09:46
vaishnavimornign aarefiev09:52
*** mgould has joined #openstack-ironic09:52
*** mkovacik has joined #openstack-ironic09:54
*** moshele has joined #openstack-ironic09:55
*** alineb has joined #openstack-ironic09:58
mgouldmorning Ironic!09:58
mgouldand happy birthday, lucasgomes!09:58
*** derekh has joined #openstack-ironic10:00
*** alexpilotti has joined #openstack-ironic10:01
mkovacikmorning mgould!10:03
mkovaciklucasagomes, what, a party? :D10:03
mgouldmkovacik, morning!10:03
*** hoangcx has quit IRC10:06
*** max_lobur has quit IRC10:06
*** daemontool has joined #openstack-ironic10:07
lucasagomesmkovacik, hah sure thing! Everyone is invited10:10
mkovaciklucasagomes, cheers! :) happy birthday!10:10
lucasagomesmkovacik, thanks a lot :-)10:11
mkovacik;)10:11
*** yhvh has quit IRC10:11
*** joemonk has quit IRC10:11
*** yhvh has joined #openstack-ironic10:13
vinm213Happy birthday lucasagomes10:14
openstackgerritSivaramakrishna Garimella proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326210:15
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Update ironic.conf.sample  https://review.openstack.org/28421810:15
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add deprecated_for_removal config info in ironic.conf.sample  https://review.openstack.org/28421910:15
lucasagomesvinm213, :D thanks you very much10:15
lucasagomesthank you*10:16
*** joemonk has joined #openstack-ironic10:16
Guest92287good morning all :-)10:18
*** Guest92287 has quit IRC10:19
*** sivaramakrishna has joined #openstack-ironic10:20
sivaramakrishnagood morning!10:20
*** max_lobur has joined #openstack-ironic10:20
*** daemontool has quit IRC10:21
*** chenke__ has quit IRC10:21
*** chenke__ has joined #openstack-ironic10:22
*** daemontool has joined #openstack-ironic10:24
sivaramakrishna:sambetts, can you review https://review.openstack.org/#/c/21326210:25
sivaramakrishnaI've made changes to factor in more logic, so user can override functions in isci deploy10:26
*** shu-mutou is now known as shu-mutou-AWAY10:29
*** achanda has joined #openstack-ironic10:29
*** sivaramakrishna has quit IRC10:30
*** _degorenko|afk is now known as degorenko10:30
sambetts|afksivaramakrishna: Thanks! I'll check it out!10:32
*** electrofelix has joined #openstack-ironic10:32
*** sambetts|afk is now known as sambetts10:32
sambettsMorning everyone!10:33
sambettsHappy Birthday lucasagomes!!!10:33
lucasagomessambetts, hi there! Thanks much man10:33
*** achanda has quit IRC10:35
*** pas-ha has joined #openstack-ironic10:35
aarefievlucasagomes: Happy Birthday!!!10:37
lucasagomesaarefiev, hi there, thank you very much!10:38
aarefievmorning sambetts10:38
sambettso/ aarefiev10:39
*** kan_ has quit IRC10:42
*** pcaruana has joined #openstack-ironic10:42
lucasagomesgate is broken?!10:47
sambetts?!10:47
lucasagomessambetts, apparently it is10:47
* lucasagomes checks10:47
sambetts2016-02-25 10:37:51.869 | + /tmp/ansible/bin/ansible subnodes -f 5 -i /home/jenkins/workspace/gate-tempest-dsvm-ironic-pxe_ssh/inventory -m synchronize -a 'mode=pull src='\''/opt/stack/logs/'\'' dest='\''/opt/stack/logs/subnode-{{ host_counter }}'\'''10:48
sambetts2016-02-25 10:37:52.073 | ERROR! Specified hosts options do not match any hosts10:48
sambetts??10:48
sambettsansible probem ?10:48
lucasagomesapparently yes10:50
lucasagomeshttp://logs.openstack.org/18/284218/3/check/gate-tempest-dsvm-ironic-pxe_ssh/766e9fc/logs/screen-n-cpu.txt.gz#_2016-02-25_10_36_33_96910:51
sambetts:/ well ... what XD10:52
vdrokmorning Haomeng vinm213 mgould sambetts and happy birthday lucasagomes :)10:52
sambettso/ vdrok10:52
lucasagomesvdrok, hey buddy! Thank you10:52
vinm213morning vdrok10:52
vdrokhm, so devstack install is successful, but it returns 1 in result10:58
*** stendulker has quit IRC11:06
*** dtantsur|afk is now known as dtantsur11:09
dtantsurMorning Ironic11:09
dtantsurlucasagomes, happy birthday man, I hope you (will) enjoy it :)11:10
lucasagomesdtantsur, morning! Thank you very much!11:10
lucasagomesso far so good11:10
dtantsurgood to know :)11:14
dtantsurjroll, an easy stable backport to +A: https://review.openstack.org/#/c/276692/11:15
mgouldvdrok, morning!11:15
vdrokmorning dtantsur !11:16
sambettso/ dtantsur11:16
*** derekh has quit IRC11:16
mgouldmorning dtantsur11:17
dtantsurlucasagomes, sambetts, wait WUT?? devstack finishes successfully but the gate think it failed??11:18
lucasagomesdtantsur, yeah, trying to run locally here11:18
lucasagomes[stack@localhost devstack]$ echo $?11:19
lucasagomes011:19
lucasagomesmine worked fine tho :-/11:19
dtantsurlucasagomes, sdague says that it affects everyone11:19
sambetts:/11:19
lucasagomes:-/11:20
lucasagomesthe good part of this is that we def have more people looking at it now11:20
dtantsurlol true11:21
*** mgould has quit IRC11:23
vdrokyep, it seems that something goes wrong after ./stack.sh run in devstack-vm-gate.sh11:30
vdrokand it is not logged anywhere11:31
lucasagomesvdrok, :-(11:35
lucasagomesthe last patch merged in devstack-gate is from feb 1911:35
lucasagomesbut yeah, it seems to be something there cause I can't reproduce locally11:35
*** mgould has joined #openstack-ironic11:39
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib: Add support for choosing the disk label  https://review.openstack.org/28353211:42
openstackgerritDebayan Ray proposed openstack/proliantutils: Added changelog and authors  https://review.openstack.org/28407511:58
*** afaranha has joined #openstack-ironic12:00
*** vaishnavi has quit IRC12:03
*** sinval-afk is now known as sinval12:05
sinvalmorning folks12:06
vdrokgood morning sinval12:06
sinvalo/ vdrok12:07
lucasagomesapparently the problem is setuptools12:07
lucasagomeshttps://bitbucket.org/pypa/setuptools/issues/502/packaging-164-does-not-allow-whitepace12:07
lucasagomesa new version was released today12:07
sambettsnice...12:08
openstackgerritAparna proposed openstack/ironic: Documentation update to enable HTTPS in Glance and Ironic communication  https://review.openstack.org/28053312:08
vdrokheh all the logs point to that :D12:09
*** kalpase has quit IRC12:15
*** NobodyCa1 has joined #openstack-ironic12:15
*** NobodyCam has quit IRC12:17
*** kalpase has joined #openstack-ironic12:28
*** achanda has joined #openstack-ironic12:33
*** appu103_ has quit IRC12:36
*** raildo-afk is now known as raildo12:37
TheJuliagood morning everyone12:37
sambettso/ TheJulia12:37
lucasagomesTheJulia, morning12:38
TheJuliaohh,  driver composition reform spec updated, just the spec I need to digest before my first meeting of the day12:38
*** lucasagomes is now known as lucas-hungry12:39
*** achanda has quit IRC12:40
vdrokgood morning TheJulia12:42
sinvalmorning TheJulia12:43
*** daemontool_ has joined #openstack-ironic12:43
*** yolanda has joined #openstack-ironic12:45
*** daemontool has quit IRC12:46
*** daemontool_ has quit IRC12:47
*** dnikishov has quit IRC12:48
*** thrash|g0ne is now known as thrash12:49
*** ifarkas has quit IRC12:52
vinm213vdrok, a quick question about the https://review.openstack.org/#/c/24603612:54
*** pcaruana has quit IRC12:55
*** baoli has joined #openstack-ironic12:55
vinm213i see your coments on using get_admin_context() instead of creating the RequestContext object in service.py12:55
openstackgerritMoshe Levi proposed openstack/ironic: Adding InfiniBand Support  https://review.openstack.org/26426312:55
*** daemontool has joined #openstack-ironic13:01
*** smoriya_ has quit IRC13:02
*** trown|outtypewww is now known as trown13:06
*** pcaruana has joined #openstack-ironic13:09
jrollmorning y'all13:09
jroll:( for gate13:09
sambettshey jroll13:09
jroll:D for lucas-hungry birthday \o/13:09
*** dprince has joined #openstack-ironic13:09
dtantsurmorning jroll, TheJulia, sinval and everyone else13:09
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: [WIP] Allow enabling debug level via kernel cmdline  https://review.openstack.org/28470513:11
dtantsurI realized I can't debug without ^^13:12
*** derekh has joined #openstack-ironic13:13
jroll++ I like that13:13
jrollput a it there13:13
jrollnit13:13
dtantsurthnx13:13
dtantsurdamn, I always mistype no, not and now >_<13:14
sinvalmorning dtantsur13:15
sambettsdtantsur: your hard coding it to 1 right now?13:15
sinvalmorning jroll13:15
jroll\o13:15
dtantsursambetts, that's for testing :) I'll remove this bit once I see gate passing with it13:16
jrollsambetts: I assumed that's for testing13:16
jrolland hence the wip13:16
dtantsurexactly13:16
*** ifarkas has joined #openstack-ironic13:17
sambettsah ok :-P I wondered if you'd done it like that until we'd added it as an option into the pxe config template in Ironic or something13:17
sambettswas going to suggest we make it a params.get('ipa-debug', '1') XD13:18
dtantsursambetts, well, it will default to off, I guess13:18
sambettsYeah once testing is done :)13:18
*** lucas-hungry is now known as lucasagomes13:19
lucasagomesjroll, o/ thanks a lot man!13:19
jroll:D13:19
sinvalhappy birthday lucasagomes o/13:20
sambettswhat might be nice on the Ironic side is to make sure when ipa-debug is true that the right kernel params are there to make it output to the screen too, because at least on my hardware even if ipa was on debug it still woldn't output anything because of the tty settings13:21
openstackgerritDebayan Ray proposed openstack/proliantutils: Added changelog and authors  https://review.openstack.org/28407513:21
lucasagomessinval, valeu man!13:21
jrollsambetts: honestly I feel like we should forward to console by default13:22
sambettsjroll: yeah, that would be nice13:24
sambettswhen I first started testing IPA I'm sure it used to, then at some point it stopped doing ti13:24
sambettson my hardware even all the kernel booting output is hidden, it just sits on the pxe booting from kernel screen until it pops up with a login13:26
*** baoli_ has joined #openstack-ironic13:26
*** links has quit IRC13:26
*** baoli has quit IRC13:28
lucasagomesgate should be good now13:30
* lucasagomes recheck a patch to test13:30
*** ChrisAusten has quit IRC13:31
lucasagomesbtw stable branch maintainers: http://logs.openstack.org/92/276692/1/gate/gate-ironic-python27/d9dc0fe/console.html#_2016-02-25_13_08_55_66913:32
jrolllucasagomes: war13:34
jrollwat*13:34
lucasagomesdunno, I wonder if it's related to the same 403 we were seeing yesterday13:35
* lucasagomes didn't investigate it yet13:35
jrollidk, that's definitely new though13:35
jrollin the last 24h or so13:35
* jroll diffs package versions13:36
*** achanda has joined #openstack-ironic13:37
lucasagomesyeah def last 24h... too many breakages today13:37
openstackgerritMerged openstack/proliantutils: Added changelog and authors  https://review.openstack.org/28407513:38
sambetts:( \13:38
* lucasagomes runs unittests locally against stable/liberty13:38
jrollnothing in diff13:38
jrollwtf13:38
jrollgotta step away for a few13:38
lucasagomesjroll, np I will take a look at it13:41
lucasagomesI can reproduce it locally13:41
*** achanda has quit IRC13:41
lucasagomesjroll, https://review.openstack.org/#/c/284436/ this seems to fix it (at least when I apply it locally)13:42
*** mgould has quit IRC13:43
openstackgerritZhenguo Niu proposed openstack/ironic: Add db api layer for CRUD operations on node tags  https://review.openstack.org/23259813:44
*** kalpase has left #openstack-ironic13:45
openstackgerritZhenguo Niu proposed openstack/ironic: Joined 'tags' column while getting node  https://review.openstack.org/25306513:45
openstackgerritZhenguo Niu proposed openstack/ironic: Add nodes tagging suport - objs  https://review.openstack.org/23335713:46
openstackgerritZhenguo Niu proposed openstack/ironic: Add DB layer support for node tags filter  https://review.openstack.org/27262413:47
*** dims has joined #openstack-ironic13:47
openstackgerritvinay kumar muddu proposed openstack/proliantutils: Changes to NIC info retrieving logic  https://review.openstack.org/27311813:49
*** jaypipes has joined #openstack-ironic13:51
*** cdearborn has joined #openstack-ironic13:54
jrolllucasagomes: oh, we were warned about that, didn't realize he'd release it without landing that first :/13:55
jrollalso turns out I'm an idiot13:55
jrollcat broke | uniq | sort > broke13:55
jrollgives an empty file13:55
jroll:/13:55
jrollanyway, approving the backport in just a sec13:57
*** zzzeek has joined #openstack-ironic13:57
lucasagomesjroll, :-/ thanks man13:57
*** zzzeek has quit IRC13:57
jrolland done13:58
*** zzzeek has joined #openstack-ironic13:58
*** daemontool_ has joined #openstack-ironic13:59
lucasagomeso/14:00
* TheJulia already has a headache this morning14:01
*** daemontool has quit IRC14:02
sambettsTheJulia: :(14:03
TheJuliaI might be completely off my rocker and misinterpretting the logic path that gets taken but yeah... https://review.openstack.org/#/c/284347/14:05
*** links has joined #openstack-ironic14:07
*** sinval is now known as sinval-brb14:07
jrollTheJulia: you sound correct to me there14:08
dtantsuraren't we cleaning the partition table in ironic already?14:08
dtantsurTheJulia, ^^14:08
dtantsurah, that's not a question to you, sorry14:09
TheJuliadtantsur: I beleive we do14:09
TheJuliawhen appropriate14:09
TheJuliajroll: good to know, so I'm just rocker crazy and not strait jacket in padded room crazy :)14:09
jrollheh14:10
*** Goneri has joined #openstack-ironic14:10
* dtantsur jumps on the bugzilla14:11
TheJuliasinval-brb: w/r/t https://review.openstack.org/#/c/264266/, i think it is part of a larger quesiton of an alternative workflow for driver users, besides I think we need to go-ahead and land that doc :)14:11
jrolldtantsur: yay, ty14:12
jrollI was going to say that bug seems odd to me14:12
TheJuliadtantsur: ty14:12
dtantsurjroll, TheJulia, I've seen the fun failure when the machine had several hard drives and root device hints were not used. then on different deployments ironic could use different hard drives, and it ended up with 2 config drives14:13
dtantsurbut that does not seem to be the case14:13
jrolldtantsur: the irc logs link in the bugzilla seems completely irrelevant too14:14
TheJuliadtantsur: yeah, I had a fun discussion with someone that was offering what was intended as the root volume with multipath io14:14
TheJuliavia fiberchannel14:14
TheJuliafun deployment errors!14:14
dtantsuroh yeah, something like that was mentioned to me once as well14:15
sambettslucasagomes: Did you say the gate was fixed?14:16
lucasagomessambetts, hopefully, I recheck a patch to see14:16
sambettslucasagomes: ah ok :) I'll wait to hear from you then before I recheck anything14:17
TheJuliadtantsur: I'm going to keep it in the back of my mind, basically told them to kill the multipath io down to a single path for deployment14:17
*** KennethWilke has quit IRC14:17
*** chaustin has joined #openstack-ironic14:17
*** sinval-brb is now known as sinval14:18
sinvalTheJulia: cool, np14:19
EmilienMlucasagomes: happy birthday man \o/14:20
lucasagomesEmilienM, hey man! Thank you very much, merci!14:20
*** keedya has joined #openstack-ironic14:23
*** mgoddard_ has joined #openstack-ironic14:27
*** vinm213 has quit IRC14:28
*** blakec has joined #openstack-ironic14:29
*** mgoddard has quit IRC14:30
Gonerilucasagomes, https://review.openstack.org/#/c/284347/2 I pushed some log samples (and Happy birthday!)14:32
lucasagomesGoneri, thanks a lot! I will take a look soonish14:33
*** blakec has quit IRC14:33
jrolldtantsur: soooo I started working on this network interface refactor... do you have any POC code for how you were thinking to load drivers?14:34
dtantsurjroll, no.. to be honest, I was just throwing ideas around.. and then we agreed we won't merge it in liberty, so I postponed it14:35
jrollhehe14:35
jrollyeah, I think I have a path forward, was just curious to see if you had anything14:35
*** amotoki has quit IRC14:37
vdrokmorning jroll !14:37
jrollhai vdrok :)14:37
vdrokvinm213, sorry, was afk, what was your question about?14:38
lucasagomesGoneri, why you need to zero 10M after doing the --zap-all there?14:39
vdrokbrb14:40
Gonerilucasagomes, sgdisk -Z does not wipe all trace of of partition tables14:40
lucasagomesGoneri, you mean different types != gpt and mbr?14:41
Gonerilucasagomes, and this my example, grub will still find some trace of the previous ISO9660 system. I had the same problem with a FAT system.14:41
sambettslucasagomes: looks like the gate is alive14:41
lucasagomessambetts, w00t14:41
sambettslucasagomes: at least I just got a jenkin pass result email14:41
sambetts:-P14:41
Gonerilucasagomes, yes, I guess it clean just the minimal bits14:41
Gonerilucasagomes, but grub sanity check go beyond.14:42
Gonerigoes14:42
Goneriso at this end, grub just give up (take a look on the log samples)14:43
Gonerilucasagomes, this is bad, because the node will reboot and stay in the power stats unless an human take a look (force_power_state_during_sync=False).14:44
lucasagomesGoneri, gotcha, we have a similar function that runs after the iscsi is exposed to the disk14:45
lucasagomesGoneri, https://github.com/openstack/ironic-lib/blob/5e642ee4cb632f2f8035fa7c15c091a85df48bba/ironic_lib/disk_utils.py#L28414:45
jrolldtantsur: devananda: blah, going to need to do *most* of this driver comp work to handle this refactor14:45
dtantsurouch14:46
lucasagomesI wonder if we should just use that function instead of having different ways of getting rid of disk metadatas in Ironic and IPA14:46
sambetts:(14:46
dtantsurjroll, I'll help with that, but the timing is nearly impossible for mitaka14:46
jrolldtantsur: the main thing being that we'll need to load an instance of the driver per node14:46
jrollunless I'm missing something14:46
jrollyeah, that's my concern :(14:46
sambettsdtantsur: I thought we were just going to load singletons of the interfaces?14:47
sambettsjroll: ^14:47
jrollsingletons of the interfaces14:47
jrollbut the main self.driver thing will need to be per node14:47
jrollelse how do you determine which singleton of the network interface to access14:47
sambettswe can add a enabled_network_interfaces = Flat,Neutron14:48
sambetts?14:48
jrollsure14:48
jrolland if both are enabled?14:48
sambettsnode.network_interface?14:48
* dtantsur had a feeling we'd need this reform done back at the summit..14:48
jrollwell, if the main driver object is a singleton14:48
lucasagomesGoneri, thanks, commented on the patch14:48
jrollyou can't decide which node14:48
*** mgould has joined #openstack-ironic14:48
jrollso it needs to be an instance per node14:48
lucasagomesthe ironic-lib was created to share code between ironic and IPA (and other ironic related projects) so, it would be aligned14:49
lucasagomesand good if we could share the same function between both14:49
jrolland then driver.enabled_interfaces.network is a singleton14:49
Gonerilucasagomes, so far IPA does not depend on this lib, right?14:49
sambettsI thought we were going to have, driver.allowed_network_interfaces and then a global enabled_network_interfaces, then we can just load all the interfaces at conductor start and then have a check in the node.validate or something that node.network_interface is in node.driver.allowed_network_interfaces?14:52
jrollsambetts: yes... but driver is currently a singleton14:53
jrollso how do you resolve driver.network?14:53
*** sinval is now known as sinval-afk14:53
sambettsyou don't? the interface is set on the node14:53
jrollso node.network.configure_cleaning_network()14:54
lucasagomesGoneri, not at the moment14:54
jrollhmmm14:54
lucasagomesbut we can add, because the goal is to use ironic-lib in IPA14:54
sambettsnet_provier = common.get_network_interface(node.network_interface)14:54
sambettsbasicly what exists now, but its loaded as a singleton14:55
jrollyeah, I think the eventual goal is to make self.driver.network a thing14:56
jrollbut maybe that will work in the meantime14:56
sambettswhen we talked about the driver comp that all the drivers were going to be for reference only, and the actual interfaces would be part of the node object14:57
jrollhm, okay, maybe that's in the etherpad, don't see it on the spec14:58
dtantsurjroll, I didn't update the spec after the summit, my bad15:00
* dtantsur got side-tracked to $things15:00
jrolldtantsur: lol, ok15:00
* jroll redoes everything15:00
*** deray has quit IRC15:00
dtantsurjroll, you may want to check with devananda cause he was also planning to work on this spec15:01
* dtantsur feels sorry that people finish what he didn't finish :)15:01
NobodyCa1good morning Ironicers15:02
*** chaustin has quit IRC15:03
jrolldtantsur: oh, I won't be finishing it, just using it for inspiration so I don't make it hard to finish later15:03
jroll:)15:03
dtantsurok, good to know :)15:04
dtantsurmorning NobodyCa115:04
*** KennethWilke has joined #openstack-ironic15:05
vdrokgood morning NobodyCa115:07
NobodyCa1morning dtantsur lucasagomes jroll JayF devananda vdrok sambetts jlvillal mgould rloo and all others not directly listed here15:08
sambettso/ NobodyCa115:08
jrollheya NobodyCa1 :)15:08
mgouldNobodyCa1, morning!15:08
*** NobodyCa1 is now known as Nobodycam15:09
jlvillalGood morning NobodyCa1 (where's NobodyCam?) dtantsur lucasagomes jroll JayF vdrok sambetts mgould and everyone else :)15:09
dtantsurmorning jlvillal :)15:09
vdrokmorning jlvillal :)15:09
jlvillal:)15:09
jroll\o jlvillal15:09
Nobodycamhehehehe15:09
Nobodycam:)15:10
sambettshey jlvillal15:10
dtantsurNobodycam, still not correct ;)15:10
* dtantsur has an IRC client with nick coloring, so he always knows15:10
sambettsjlvillal: how far have you got with the multinode Ironic tempest?15:10
*** Nobodycam is now known as NobodyCam15:10
*** daemontool__ has joined #openstack-ironic15:10
NobodyCam:)15:11
dtantsurbetter now :)15:11
lucasagomesjlvillal, NobodyCam morning15:11
jlvillalsambetts, Not far. As in back-tracked to single-node to keep moving forward on grenade.15:11
jlvillalsambetts, Trying to run the grenade job as closely to how our current gate runs.15:12
jlvillalsambetts, Ran into a stable/liberty issue: https://bugs.launchpad.net/bugs/154909515:12
openstackLaunchpad bug 1549095 in Ironic "devstack fails while running Ironic grenade job: init__() got an unexpected keyword argument 'token'" [Undecided,In progress] - Assigned to John L. Villalovos (happycamp)15:12
*** jmccrory has quit IRC15:12
*** links has quit IRC15:12
*** jmccrory has joined #openstack-ironic15:13
jlvillalI think some library strangeness. Trying to figure out why it is happening. Working with devstack/openstackclient person on it.15:13
sambettsjlvillal: yeah, makes sense to get grenade going, I'm looking forward to multinode to see what tinyipa is really capable of :-P15:13
jlvillal:)15:13
jrollAdd `driver_interfaces` table: { `id` INT PK, `interface` VARCHAR, UX(interface) }15:13
* jroll tries to figure out what UX() is15:13
*** e0ne has joined #openstack-ironic15:14
*** daemontool_ has quit IRC15:14
jlvillalUser eXperience :)15:14
*** praneshp has joined #openstack-ironic15:15
*** chaustin has joined #openstack-ironic15:15
sambettsI think that table is so that we don't have the interfaces stored on the node object row directly, like how capablities is going to be15:15
jlvillalUnix?  Air Europa?15:15
sambettsso the columns should be id, interface_type, node_id, interface_name15:16
sambettsI think15:16
jrollsambetts: nah, it's just a list of interfaces, with node.*_interface FK to it15:17
jrollI'm assuming UX meaning unique constraint15:17
*** garthb_ has joined #openstack-ironic15:18
*** garthb has joined #openstack-ironic15:18
sambettsah ok, that makes sense, now you say that I rememeber I think we added that table for doing rolling upgrades, so we could track which interfaces had be loaded before, and then we could also just have an int ref in the node table instead of a string15:19
sambettsunique contraint makes sense15:19
sambettsI think if we want it extensible, so we can add new interfaces types on the fly without adding new columns to the node row, then having a joining table, with columnds node_id interface_id might be cool15:21
*** penick has joined #openstack-ironic15:23
*** praneshp_ has joined #openstack-ironic15:24
*** Haomeng|2 has joined #openstack-ironic15:24
*** max_lobur1 has joined #openstack-ironic15:24
sambettsdtantsur, jroll: we should have a virtual sprint for the driver comp spec15:24
dtantsur+115:24
*** ifarkas_ has joined #openstack-ironic15:26
*** Haomeng has quit IRC15:26
*** max_lobur has quit IRC15:26
*** praneshp has quit IRC15:26
*** praneshp_ is now known as praneshp15:26
*** ifarkas has quit IRC15:26
jrollyeah probably should15:27
*** chaustin has quit IRC15:30
*** yolanda has quit IRC15:32
*** raddaoui has joined #openstack-ironic15:32
jristhappy birthday lucasagomes !15:32
lucasagomesjrist, hey man! Thank you very much15:33
*** ppiela has joined #openstack-ironic15:33
NobodyCamoh happy birthday lucasagomes !!!15:33
jristaw yeah15:33
jristall aboard the happy birthday train!15:33
lucasagomesjrist, your birthday is on saturday right? Happy early birthday! (my turn now :-)15:33
NobodyCam+++15:33
jristhaha15:33
jristyes it is!15:33
jristthanks15:33
jrist:)15:33
lucasagomesNobodyCam, thanks a lot!15:33
NobodyCamhappy almost birthday jrist :)15:33
*** mgoddard_ has quit IRC15:33
jristthanks NobodyCam !15:34
jristwhen lucasagomes was born, I was in high school15:34
lucasagomeshah well I'm not that young anymore :-(15:35
* lucasagomes turned 2715:35
jristoh!15:35
jristI thought you turned 21 ;)15:35
lucasagomeshah15:35
* jrist gets 6 years back15:35
lucasagomesand my hair is falling out :-P (no kidding)15:36
*** penick has quit IRC15:36
* mgould remembers turning 27 and thinking he was SO OLD15:36
*** mgoddard has joined #openstack-ironic15:37
NobodyCamhahahaha15:38
jristlol15:38
* NobodyCam is almost dbl that15:38
jrollI'm 28, can confirm I'm super old and lucas is a baby15:39
jristLOL15:39
lucasagomeswhat hah15:39
NobodyCamlol15:39
*** achanda has joined #openstack-ironic15:39
jristmy hair is turning grey :(15:39
jristespecially my beard hair15:39
jrollmine too, don't feel bad :P15:39
mgouldjroll: you're officially in your late twenties rather than your mid-twenties now15:39
jristsoon I'll be a curmudgeon15:39
jrollmgould: I know :(15:39
* mgould doesn't understand you millennials with your emojis and your dubstep15:40
jlvillalI will also say Happy Birthday Lucas Alvares Gomes~ (lucasagomes)15:40
mgould++15:40
*** e0ne has quit IRC15:40
openstackgerritMerged openstack/ironic: Update ironic.conf.sample  https://review.openstack.org/28421815:41
*** e0ne_ has joined #openstack-ironic15:41
NobodyCamanyone here remember this: http://3.bp.blogspot.com/-hDt5gtMe3Go/T2EQmE5aJrI/AAAAAAAAAc0/xjrOOyaFt1c/s1600/logans-run-carousel.jpg15:41
mgouldnot from when it came out, but I have seen the movie :-)15:41
lucasagomesjlvillal, thanks much!15:41
NobodyCam:)15:41
* jlvillal prepares to go to work...15:41
lucasagomesjrist, jroll at least you guys have gair15:42
lucasagomeshair*15:42
jristoverrated15:42
lucasagomeswhen I look at my dad and all my uncles I can project my future15:42
jristgets in my face15:42
jristhaha15:42
lucasagomesand there's no hair there15:42
jristhair is too much work15:42
jrolllol15:42
lucasagomesindeed15:42
lucasagomesI won't need to buy shampoo anymore, see that's the bright side of it15:43
*** mtanino has joined #openstack-ironic15:43
jristhaircuts15:43
NobodyCamlol15:43
jristsave $30 a month at least15:43
* jrist turns on some Journey15:43
lucasagomesjrist, oddly I feel much better about it now15:43
jristyou should man15:43
jrist:)15:43
jristno need for swim caps15:44
jristI can go on and on15:44
lucasagomeslol15:44
jristhttps://www.youtube.com/watch?v=jifOSQAI0G815:44
* dtantsur has a home-made haircut15:45
*** achanda has quit IRC15:45
mgoulddtantsur, me too :-)15:46
mgouldI don't save quite as much as lucasgomes will, because clippers occasionally burn out, but it does save a lot of money and time and hassle15:47
openstackgerritPavlo Shchelokovskyy proposed openstack/python-ironicclient: Stop ignoring _ as builtin in pep8  https://review.openstack.org/28477615:51
*** e0ne_ has quit IRC15:52
*** mtanino has quit IRC15:52
dtantsurjroll, IPA with debug logging :) http://logs.openstack.org/05/284705/1/check/gate-tempest-dsvm-ironic-pxe_ipa-src/fe865fd/logs/ironic-bm-logs/baremetalbrbm_0_console_25-02-2016-15:43:09.txt.gz15:54
dtantsurnow I'm removing the temporary thing15:54
devanandamorning, all15:54
mgoulddevananda, morning15:54
NobodyCamgood morning devananda15:54
dtantsurmorning devananda15:54
*** dnikishov has joined #openstack-ironic15:55
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Allow enabling debug level via kernel cmdline  https://review.openstack.org/28470515:55
dtantsurjroll, sambetts ^^15:55
sambettsdtantsur: sweet!15:55
dtantsurI deadly need it, so please review :)15:55
dtantsurlucasagomes, ^^15:55
lucasagomesdtantsur, 1 sec (looking at that odd problem where the disks are not detected)15:56
vdrokmorning devananda15:56
dtantsuroh yeah15:56
sambettso/ devananda15:56
*** mgoddard_ has joined #openstack-ironic15:59
*** mgoddard has quit IRC16:03
*** mtanino has joined #openstack-ironic16:03
mgoulddtantsur, LGTM16:04
openstackgerritMoshe Levi proposed openstack/ironic: Adding InfiniBand Support  https://review.openstack.org/26426316:05
*** chaustin has joined #openstack-ironic16:05
*** garthb has quit IRC16:06
*** garthb_ has quit IRC16:06
*** harlowja_at_home has joined #openstack-ironic16:06
*** dims has quit IRC16:06
*** moshele has quit IRC16:07
*** mkovacik has quit IRC16:08
*** praneshp has quit IRC16:14
openstackgerritJim Rollenhagen proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/20624416:14
openstackgerritJim Rollenhagen proposed openstack/ironic: Add network provider interface and implementations  https://review.openstack.org/13968716:14
openstackgerritJim Rollenhagen proposed openstack/ironic: WIP: refactor network provider stuff  https://review.openstack.org/28478716:14
*** praneshp has joined #openstack-ironic16:15
jrollurgh, didn't mean to edit that bottom one :/16:15
sambettsjroll: its so annoying when that happens16:15
jrolldevananda: dtantsur: sambetts: curious what you think of the direction in https://review.openstack.org/28478716:15
jrollmostly around how I'm loading drivers and such16:15
*** daemontool__ has quit IRC16:16
dtantsurjroll, I'd take a look tomorrow, cause today I'm pretty bad at thinking :(16:16
jrolldtantsur: heh, no worries16:16
jrollI'll probably keep hacking on it16:16
harlowja_at_homedtantsur, u want that futurist change soon right? (for the work tuple)16:17
harlowja_at_homelet's see if we can merge and release before tommorow if so, ha16:17
harlowja_at_home*last week for more oslo releases for a while16:18
dtantsurharlowja_at_home, would be cool, but not required16:19
harlowja_at_homek16:19
*** chaustin has left #openstack-ironic16:19
harlowja_at_homelet's see how the moons align16:19
harlowja_at_homelol16:19
harlowja_at_home*the moon16:19
harlowja_at_homewhatever16:19
harlowja_at_homeha16:19
*** rloo has joined #openstack-ironic16:19
dtantsuranyone wants to check the manual cleaning docs https://review.openstack.org/#/c/264266/ before I press the button?16:22
sambettsdtantsur: I can do a quick english scan :)16:23
dtantsuryep16:23
dtantsur(though it's written by rloo, I don't think you can find any english problems there)16:23
sambettstrue :-P16:23
jlvillalsambetts you will probably just add extra u letters to words ;-)16:24
jlvillaland change z to s16:24
sambettshahaha thats just the flavour of our language ;)16:24
NobodyCam:-p16:24
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Add db mingration test  https://review.openstack.org/28479216:25
dtantsuroh, I see "Cisco CI" commenting on our patches, nice :)16:26
sambettsyup :D16:26
NobodyCamnice :)16:27
sambettsI'm still building the actual test bed underneith but the zuul and jenkins bit is alive :D16:27
dtantsurgreat16:27
rloosambetts: ++ go for the extra u and s :)16:31
lucasagomesdtantsur, left a comment in the patch (+2'd anyway)16:31
sambettsrloo: haha, when I used to work on Horizon I got shot down so much for my Britishisms16:31
rloosambetts: my rule is if I write a new doc, I can add u and s. If I edit/modify an existing doc, then I will go with whatever convention is there. Cleaning doc already existed so I left it americanized :)16:33
sambetts:-P good rule16:33
dtantsurlucasagomes, hmm, good idea. I'll probably follow-up with that, if you don't mind. Without this patch I have no chances of debugging what I debug right now :)16:33
lucasagomesdtantsur, it would be good to have it prior to merge this first version right? You want me to update that patch?16:34
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Rescan SCSI devices prior to list them  https://review.openstack.org/28479616:36
vdrokmorning rloo :)16:36
dtantsurlucasagomes, sorry, I didn't get the question. is it still about ipa-debug? I agree with your comment, I just don't feel like retesting it right now, will take more time16:36
dtantsurso I want to change to oslo_utils in the follow-up, leaving '1' and 'true' now16:36
lucasagomesdtantsur, fair enough16:36
sambettslucasagomes, dtantsur I wondered if there was a function somewhere to do that!!!16:37
lucasagomessambetts, yeah there's16:37
rloohi vdrok !16:38
sambettsdtantsur, rloo: Cleaning Docs read well to me16:39
*** cdearborn has quit IRC16:43
*** absubram has joined #openstack-ironic16:43
*** jaybeale has joined #openstack-ironic16:44
*** ifarkas_ has quit IRC16:44
openstackgerritMerged openstack/ironic: Add deprecated_for_removal config info in ironic.conf.sample  https://review.openstack.org/28421916:46
openstackgerritJim Rollenhagen proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/20624416:48
openstackgerritJim Rollenhagen proposed openstack/ironic: WIP: refactor network provider stuff  https://review.openstack.org/28478716:48
openstackgerritJim Rollenhagen proposed openstack/ironic: Add network provider interface and implementations  https://review.openstack.org/13968716:48
jrollrabsed that on master because jlvillal moved stuff from under me :P16:48
sambetts:)16:49
sambettsdamn I'm getting 503s from Gerrit again ...16:49
rloothx sambetts!16:49
rloojroll etc. i have an hour or so today to review -- anything urgent that needs looking at?16:50
sambettshas gerrit blow up for anyone else?16:50
* jlvillal looks around and says, "Who me??"16:50
jrollrloo: manual cleaning, raid docs/client16:50
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Rescan SCSI devices prior to list them  https://review.openstack.org/28479616:51
jrollsambetts: super slow for me right now16:51
*** dtantsur is now known as dtantsur|afk16:51
dtantsur|afksee you tomorrow16:51
rloojroll: ok. isn't manual cleaning almost done? i still need to do that GET cleansteps API thing but I think that can be deferred to later.16:51
jlvillaldtantsur|afk: Ciao16:51
rloojroll: will look at RAID today.16:51
jlvillaldtantsur|afk: Paka paka  (spelling?)16:51
dtantsur|afklol16:51
dtantsur|afkпока16:51
jlvillal:)16:51
jrollrloo: raid stuff is https://review.openstack.org/226330, https://review.openstack.org/22623416:52
jlvillaljroll: I'm guessing it was the _from_db_object function....16:52
jrollrloo: and yes, cleaning is almost done, I haven't checked on it yesterday/today16:52
jrolljlvillal: yeah, no worries man :)16:52
jlvillal:)16:52
rloojroll: thx will look at raid stuff16:53
*** dims has joined #openstack-ironic16:54
*** amotoki has joined #openstack-ironic16:54
*** garthb has joined #openstack-ironic16:58
*** garthb_ has joined #openstack-ironic16:59
sambettsjroll: I'm not sure you need the enabled_interfaces dictionary, I would make something like InterfaceFactory, inherited into a NetworkInterfaceFactory, then in base manager do something like for interface in network_int_factory.interfaces() and in task manager do _interface = network_int_factory.get_interface(self.node.network_interface) self.driver.network = _interface ??17:00
*** dnikishov has quit IRC17:01
openstackgerritGonéri Le Bouder proposed openstack/ironic: ipxe: add --timeout parameter to kernel and initrd  https://review.openstack.org/28389317:01
*** raddaoui has quit IRC17:03
*** raddaoui has joined #openstack-ironic17:04
openstackgerritMerged openstack/ironic: Add manual cleaning to documentation  https://review.openstack.org/26426617:05
*** mgoddard_ has quit IRC17:07
*** mgoddard has joined #openstack-ironic17:07
*** jistr has quit IRC17:08
*** electrofelix has quit IRC17:12
harlowja_at_homedtantsur|afk, https://review.openstack.org/28436717:19
*** penick has joined #openstack-ironic17:20
*** krtaylor has quit IRC17:21
*** max_lobur1 has quit IRC17:24
*** electrofelix has joined #openstack-ironic17:24
*** sivaramakrishna has joined #openstack-ironic17:27
*** mgould has quit IRC17:29
jrollsambetts: yeah, I could, I just have a major aversion to things with the name factory in them :D17:31
jrollalso, that was in the spec, can't remember if it was in etherpad17:31
jrollsambetts: can you leave that comment on the patch and I'll play with it later?17:31
sivaramakrishnasambetts,jroll: shall I go ahead and rename the method finish_deploy for iscsi_deploy?17:32
jrollsivaramakrishna: context?17:33
sivaramakrishnahttps://review.openstack.org/#/c/21326217:33
*** krtaylor has joined #openstack-ironic17:33
sivaramakrishnacheck the latest comment from sambetts17:33
*** achanda has joined #openstack-ironic17:36
*** sinval-afk is now known as sinval17:36
sambettsjroll: comment left :)17:36
*** trown is now known as trown|lunch17:37
sivaramakrishnajroll: not sure if my prev message went..  can you review https://review.openstack.org/#/c/21326217:37
openstackgerritSivaramakrishna Garimella proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326217:40
sambettssivaramakrishna: I'm happy either way, hence my +1 but others might have more defined opinions on it17:40
sivaramakrishnaokay.. for now I'll fix the tests17:40
sivaramakrishnalet me know what name I should use17:41
openstackgerritlokesh s proposed openstack/python-ironicclient: Log SHA1 hash of X-Auth-Token value  https://review.openstack.org/27895117:42
jrollsivaramakrishna: oh, I'm not opinionated17:43
sivaramakrishnadrok, jroll: let me know if we're final on naming it bash_finish_deploy17:43
sivaramakrishnajroll: thanks17:43
sivaramakrishnajroll: so I guess I'll rename it then17:44
jrollsivaramakrishna: sure, I don't mind17:44
*** mgould has joined #openstack-ironic17:45
vdroksivaramakrishna, works for me too17:49
*** electrofelix has quit IRC17:50
sivaramakrishnagreat! thanks.. will put in the change today.17:51
*** raddaoui has quit IRC17:51
*** electrofelix has joined #openstack-ironic17:51
mgouldharlowja_at_home, just left some mostly superficial comments17:52
jlvillalmgould: Thanks for the pull request. I'll take a closer look at it17:55
*** harshs has joined #openstack-ironic17:55
*** amotoki has quit IRC17:56
mgouldjlvillal, thanks17:57
mgouldI'm about to go home soon, but I've run ironic-grenade.sh to completion17:58
*** thiagop has joined #openstack-ironic17:58
*** stendulker has joined #openstack-ironic17:58
mgouldexited with status 1; no obvious error apart from "WARNING, provided hosts list is empty"17:59
thiagopGood morning, folks17:59
jrollmgould: oh, that's the ansible bug that hit the gate today18:00
* jroll finds fix18:00
mgouldjroll, I've been seeing that warning for ages, has it just been fixed?18:01
*** hemna has joined #openstack-ironic18:01
rlooJayF: btw, you had asked me last year, to look at the cleaning doc. Months later, Done!  :) https://review.openstack.org/#/c/264266/18:02
*** athomas has quit IRC18:02
jrollmgould: https://review.openstack.org/#/c/284652/18:02
jrollmgould: idk, it broke all gates this morning18:03
*** derekh has quit IRC18:03
jrollI *think*18:03
vdrokgood night everyone!18:03
* sivaramakrishna getting sleepy :-18:03
sivaramakrishnavdrok: good night!18:03
rloonight vdrok !18:03
jrollmgould: oh, what it was is in latest ansible that became an error, I believe18:03
mgouldjroll, gotcha18:04
mgouldvdrok, good night!18:04
*** sivaramakrishna has quit IRC18:04
*** degorenko is now known as _degorenko|afk18:04
mgouldrloo, high-five!18:04
rloomgould: :D18:05
*** harshs has quit IRC18:07
lucasagomesfolks I'm calling it a day18:07
*** ndipanov has quit IRC18:07
lucasagomesgoing grab some pints to celebrate with the folks in the office18:07
*** harshs has joined #openstack-ironic18:07
lucasagomeshave a great night all18:07
lucasagomessee you tomorrow18:07
thiagoplucasagomes: night!18:07
mgouldlucasagomes, good night!18:07
sambettsnight lucasagomes, have a great BD evening!!18:07
mgouldhave a fun celebration :-)18:07
lucasagomescheers :-)18:07
*** lucasagomes is now known as lucas-afk18:08
*** jcoufal has quit IRC18:08
mgouldjroll, jlvillal: digging into the logs a bit I find the error "dial tcp: lookup index.docker.io: no such host"18:08
jroll:|18:08
jlvillalmgould: Which logs?  I'm missing the context.18:10
mgouldI can curl index.docker.io from outside the VM but not from inside, so I think it's maybe that the virtualbox networking setup is wrong18:10
mgouldjlvillal, /opt/stack/logs/grenade.sh.txt.gz after an ironic_grenade.sh run18:10
*** jaybeale has quit IRC18:11
*** praneshp_ has joined #openstack-ironic18:11
jlvillalAh18:11
mgouldhttp://paste.openstack.org/show/488239/18:12
mgouldanyway, hometime for me now; I'll have a look into it tomorrow18:12
jlvillalmgould: I don't see that error in my run18:12
jrollmgould: looks like dns weirdness honestly18:13
jrollbut yeah, I like to blame vbox18:13
jlvillalmgould: My IPA build works fine. Building the docker image.18:13
mgouldweird18:14
mgouldI can't ping Google from inside the VM either18:14
*** praneshp has quit IRC18:14
mgouldmaybe my patch broke networking :-(18:14
*** praneshp_ is now known as praneshp18:14
mgouldanyway, home! or rather, pub :-)18:14
*** suro-patz has joined #openstack-ironic18:14
mgouldgood night, everybody!18:14
sambettso/ mgould18:14
*** mgould has quit IRC18:14
*** spandhe has joined #openstack-ironic18:15
*** jcoufal has joined #openstack-ironic18:16
*** jcoufal has quit IRC18:16
jlvillalmgould: http://paste.openstack.org/show/488241/  My run.18:17
*** penick has quit IRC18:20
*** max_lobur has joined #openstack-ironic18:21
*** cdearborn has joined #openstack-ironic18:22
*** penick has joined #openstack-ironic18:25
*** max_lobur has quit IRC18:26
*** max_lobur has joined #openstack-ironic18:28
*** stendulker has quit IRC18:28
*** stendulker has joined #openstack-ironic18:29
*** stendulker has quit IRC18:29
*** stendulker has joined #openstack-ironic18:29
*** ohamada has quit IRC18:30
*** hemna has quit IRC18:33
*** stendulker has quit IRC18:35
cfarquharhi all.  what components are responsible for keeping nova (e.g. nova hypervisor-show) updated with attributes of ironic nodes (e.g ironic node-show)?18:36
*** hemna has joined #openstack-ironic18:40
*** piet has joined #openstack-ironic18:41
cfarquharmy initial problem was that memory on the nova side was 0, despite being correct on the ironic side.  I deleted and recreated the ironic node in an attempt at a quick fix.  nova shows the old node still and is missing the new node.  if anyone has tips on where I should be looking for problems, they would be appreciated.18:41
*** chenke_ has joined #openstack-ironic18:42
praneshpcfarquhar: afaik, its the resource tracker on nova side18:43
praneshpnova.compute.resource_tracker makes a periodic call, to update_avaliable_resources I think18:43
praneshpit eventually goes down to https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L55118:44
praneshpcfarquhar: if deleting and recreating is not a probem, I suggest deleting the ironic node, then also nuking the nova side hypervisort before adding again18:45
*** chenke__ has quit IRC18:45
NobodyCamUsing username "nobodycam".18:46
*** piet has quit IRC18:46
* sambetts is heading off for today 18:46
sambettssee everyone tomorrow18:46
NobodyCam    night sambetts18:47
*** sambetts is now known as sambetts|afk18:47
cfarquharpraneshp: thanks, I'll try that!18:47
*** trown|lunch is now known as trown18:49
*** Haomeng has joined #openstack-ironic18:50
*** hemna has quit IRC18:52
*** Haomeng|2 has quit IRC18:52
*** ChubYann has joined #openstack-ironic18:55
thiagopnight sambetts|afk18:56
*** alineb has quit IRC18:56
*** piet has joined #openstack-ironic18:58
*** hemna has joined #openstack-ironic18:58
*** praneshp_ has joined #openstack-ironic18:59
*** Marga_ has quit IRC19:00
*** praneshp has quit IRC19:01
*** praneshp_ is now known as praneshp19:01
*** piet has quit IRC19:02
*** piet has joined #openstack-ironic19:03
*** aarefiev22 has joined #openstack-ironic19:05
openstackgerritShivanand Tendulker proposed openstack/ironic: Update proliantutils version required for Mitaka release  https://review.openstack.org/28488419:09
*** penick has quit IRC19:09
*** baoli_ has quit IRC19:11
*** raddaoui has joined #openstack-ironic19:13
*** dprince has quit IRC19:16
*** raddaoui has quit IRC19:24
*** raddaoui has joined #openstack-ironic19:25
*** mkovacik has joined #openstack-ironic19:28
*** Sukhdev has joined #openstack-ironic19:29
*** dprince has joined #openstack-ironic19:35
openstackgerritMerged openstack/proliantutils: Changes to NIC info retrieving logic  https://review.openstack.org/27311819:36
*** Marga_ has joined #openstack-ironic19:36
*** electrofelix has quit IRC19:40
*** chenke_ has quit IRC19:44
*** chenke_ has joined #openstack-ironic19:45
*** caiobo has joined #openstack-ironic19:46
*** baoli has joined #openstack-ironic19:49
*** krotscheck is now known as krotscheck_dcm19:50
*** pas-ha has quit IRC19:50
devanandaTheJulia: wdyt of bifrost including a playbook for assigning IP addresses to the IPMI ports of things?19:50
*** baoli has quit IRC19:51
*** praneshp has quit IRC19:53
*** jaybeale has joined #openstack-ironic19:56
*** hemna has quit IRC19:56
*** raddaoui has quit IRC20:05
openstackgerritNisha Agarwal proposed openstack/ironic-lib: Enables partition images for agent drvers.  https://review.openstack.org/28104520:11
*** piet has quit IRC20:11
*** piet has joined #openstack-ironic20:14
*** piet has quit IRC20:14
*** penick has joined #openstack-ironic20:15
*** Marga_ has quit IRC20:15
*** Marga_ has joined #openstack-ironic20:15
*** raddaoui has joined #openstack-ironic20:18
*** Marga_ has quit IRC20:25
*** Marga_ has joined #openstack-ironic20:26
*** Marga_ has quit IRC20:27
*** aarefiev22 has quit IRC20:29
*** piet has joined #openstack-ironic20:33
*** _degorenko|afk has quit IRC20:42
*** _degorenko|afk has joined #openstack-ironic20:43
sinvalsee you tomorrow guys20:49
*** sinval is now known as sinval-afk20:49
*** piet has quit IRC20:51
*** Marga_ has joined #openstack-ironic20:55
*** Goneri has quit IRC20:58
*** Marga_ has quit IRC20:59
*** ChrisAusten has joined #openstack-ironic21:01
mrdaMorning Ironic21:01
*** jaybeale has quit IRC21:03
NobodyCammorning mrda21:04
NobodyCam:)21:04
*** raddaoui has quit IRC21:04
*** vishwanathj has quit IRC21:04
*** raildo is now known as raildo-afk21:04
*** daemontool has joined #openstack-ironic21:05
*** jaybeale has joined #openstack-ironic21:06
*** thiagop has quit IRC21:06
mrdahey NobodyCam :)21:07
*** piet has joined #openstack-ironic21:08
*** rama_y has joined #openstack-ironic21:11
openstackgerritKurt Taylor proposed openstack/ironic-python-agent: Update DIB description for IPA docs  https://review.openstack.org/28493221:17
*** dprince has quit IRC21:19
*** Sukhdev has quit IRC21:21
*** Marga_ has joined #openstack-ironic21:22
*** achanda has quit IRC21:23
*** Marga_ has quit IRC21:27
*** achanda has joined #openstack-ironic21:31
*** piet has quit IRC21:32
*** ijw has joined #openstack-ironic21:33
*** achanda has quit IRC21:33
*** nathan-h_ has joined #openstack-ironic21:35
*** achanda has joined #openstack-ironic21:37
*** ijw has quit IRC21:41
*** ijw has joined #openstack-ironic21:42
*** MattMan has quit IRC21:45
*** MattMan has joined #openstack-ironic21:46
*** Nisha has joined #openstack-ironic21:46
*** daemontool has quit IRC21:48
*** causten_ has joined #openstack-ironic21:50
*** ChrisAusten has quit IRC21:50
*** piet has joined #openstack-ironic21:51
*** Marga_ has joined #openstack-ironic21:51
*** penick_ has joined #openstack-ironic21:59
*** penick has quit IRC21:59
*** penick_ is now known as penick21:59
*** causten_ has quit IRC22:00
*** Nisha has quit IRC22:00
*** Nisha has joined #openstack-ironic22:01
*** nathan-h_ has quit IRC22:08
rloojroll, dtantsur|afk : wrt the RAID CLI patch, there is a similar thing as with cleaning. A file with target_raid_config which is being handled at the library/API level, not the shell level.22:08
*** achanda has quit IRC22:09
rloojroll, dtantsur|afk : I'm not sure I grok why config-drive is at the API level, so not sure if target_raid_config should be handled at shell or API level.22:09
jrollrloo: honestly I think configdrive should be at the shell level22:09
rloojroll, dtantsur|afk : referring to https://review.openstack.org/#/c/22623422:09
jrollwe just didn't think about it at the time22:09
rloojroll: Oh. ok, please mention in the raid patch then :) thx.22:10
jrollI still think shell level is the right thing to do22:10
jrollrloo: done! thanks22:11
rloojroll: i don't know if i have much of an opinion. it seems like it is more flexible at the api level if someone wants to pass in a file name. but that can be done later too, if we decide it is needed.22:11
jrollyeah, I just don't get the use case for api level - why would a python application write a file before calling that?22:12
rloojroll: ok, i reviewed both raid patches. i'll look again next week. i think it is doable to get them done next week :)22:12
*** Goneri has joined #openstack-ironic22:13
rloojroll: no idea why anyone would but doesn't mean we should prevent. but again, we can add that in later when someone wants.22:13
jrollrloo: awesome, thank you so much :)22:13
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Allow rerunning introspection on stored data  https://review.openstack.org/26763722:13
rloojroll: np. ciao :)22:13
jrolllaters22:14
*** rloo has left #openstack-ironic22:14
*** daemontool has joined #openstack-ironic22:17
*** piet has quit IRC22:18
*** awiddersheim has joined #openstack-ironic22:19
*** rama_y has quit IRC22:24
*** Sukhdev has joined #openstack-ironic22:28
*** hemna has joined #openstack-ironic22:30
*** suro-patz has quit IRC22:40
*** Nisha has quit IRC22:43
*** Nisha has joined #openstack-ironic22:43
*** Nisha has quit IRC22:46
*** Nisha_away has joined #openstack-ironic22:46
*** absubram has quit IRC22:48
openstackgerritNisha Agarwal proposed openstack/ironic-python-agent: Add support for partition images in agent driver  https://review.openstack.org/16022422:48
*** suro-patz has joined #openstack-ironic22:53
*** hemna has quit IRC22:55
openstackgerritJim Rollenhagen proposed openstack/ironic: WIP: refactor network provider stuff  https://review.openstack.org/28478722:58
jrollsambetts|afk: ^ that did turn out much nicer, ty22:58
jrolldevananda: ^ would still love to have your eyes on that early22:59
*** achanda has joined #openstack-ironic23:04
Nisha_awayjroll, i was looking at the comments on raid cli patch23:06
Nisha_awayI would say 'target_raid_config' is more like configdrive23:07
Nisha_awayand less like clean_steps23:07
*** hemna has joined #openstack-ironic23:07
jrollNisha_away: I disagree, it's a single json file23:07
jrollright?23:07
Nisha_awayYes, but how does that matter23:08
jrollif I'm developing a python application, I'd expect to pass a dict or list into that method, not a string23:08
devanandajroll: TaskManager calling deepcopy(_driver) concerns me, but I'm not sure why23:09
jrolldevananda: yeah, I feel like it might get ram-heavy under load23:10
devanandaactually, yea, that part of taskmanager is really weird23:10
devananda*is now really weird23:10
jrollnot sure what else there is to do, though, task.driver will need to be per-node23:10
jrollor rather, need to be an instance of the driver per node23:10
devanandaooooh23:11
devanandaright now, task.driver is a pointer to that driver's singleton23:11
jrollthis feels like the best place to put it, but not totally sure23:11
*** cdearborn has quit IRC23:11
jrollyep23:11
devanandawe want task.driver to become an instance of a new "driver object" that has references to singletons for each interface23:12
devanandaor rather, I think that's the most logical modelling of this23:12
devanandabut that might be a much bigger change23:12
jrollyep, I agree, the spec/etherpad also agree23:12
devanandacool23:12
jrollI do think this proves we can do each interface one-by-one, which makes me happy23:13
devanandayah23:13
openstackgerritMerged openstack/python-ironicclient: Remove unused `anyjson`  https://review.openstack.org/28368423:13
*** thrash is now known as thrash|g0ne23:14
*** xek_ has joined #openstack-ironic23:18
jrolldevananda: so, good proceeding with this direction?23:19
jrollI can finish up the refactor tomorrow probably23:19
*** xek has quit IRC23:19
devanandajroll: ++23:20
devanandajroll: do you think that deepcopy will be needed as an intermediate step, pending a lot more work, or something you can refactor out quickly?23:21
jrolldevananda: I mean, I'm thinking the rest of the composition stuff will need to wait for Newton, so yeah it's longer term23:22
jrollI'm not sure if we'll even fully get rid of it, unless eventually self.driver is just a bare object instead of a singleton23:22
jrollthree options, as I see it:23:23
jroll1) use deepcopy forever23:23
jroll2) refactor such that self.driver is a bare object, with interfaces attached23:23
jroll3) instantiate the driver at the time we're currently doing deepcopy23:23
jroll(2) seems best to me23:23
jrollbut the composition spec plans for self.driver to continue to be the power driver, so dunno23:24
*** hemna has quit IRC23:28
*** Sukhdev has quit IRC23:29
*** yuanying has joined #openstack-ironic23:33
*** yuanying has quit IRC23:34
*** yuanying has joined #openstack-ironic23:34
*** achanda has quit IRC23:38
*** smoriya_ has joined #openstack-ironic23:45
devanandajroll: agreed on (2)23:45
*** Nisha_away has quit IRC23:48
devanandajroll: can we do the network stuff without a) completely refactoring all the things, and b) deepcopy'ing drivers, and c) waiting for newton23:49
*** achanda has joined #openstack-ironic23:49
jrolldevananda: assuming this code works, the necessary bits are done, just need to finish refactoring the actual networky bits23:50
jrollmoving stuff out of common/network, checking provisioning_network config at startup, etc23:51
*** Goneri has quit IRC23:51
jrolland then changing the calls in the following patch to self.driver.network.*()23:51
devanandaneat23:52
devanandawithout diving in deeper rght now, I'm sort of expecting the work to make interfaces into singletons to be greater than that23:53
jrollwell, there's backwards compat concerns, at a minimum23:53
devanandayah23:53
jrolland additinoal columns23:53
jrollother than that I don't actually think it will be much work23:53
devanandacontinuing to work with out of tree drivers / interfaces23:54
devanandawill be challenging23:54
devanandadeepcopy would be necessary for those23:54
jrollmaybe some refactoring of what node.driver does, too23:54
jrollyeah23:54
jrollso I'm almost thinking, if node.driver is set, use old behavior23:54
devanandaperhaps add a method to the base driver class ...23:54
devanandaright23:54
jrollya23:54
devanandathat detects if it's a new driver (or new interface), and falls back to deepcopy if not23:54
devananda*ifnot new23:54
jrollyeah, or that23:55
jrollI feel like I'd rather fully deprecate node.driver23:55
jrolland break everything out23:55
jroll(see above with task.driver being a bare object)23:55
*** trown is now known as trown|outtypewww23:58
*** krtaylor has quit IRC23:58
devanandainstead of having task.driver being a copy of task.node.driver, you'd drop the node.driver interface entirely?23:59

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