Tuesday, 2015-09-29

*** naohirot has joined #openstack-ironic00:02
*** harshs has joined #openstack-ironic00:06
*** mbound has quit IRC00:06
openstackgerritJohn L. Villalovos proposed openstack/bifrost: Update deprecated options for Mitaka release  https://review.openstack.org/22826700:09
jlvillalmrda: Good morning00:14
jlvillalmrda: And good night :)00:14
mrdahey jlvillal00:15
jlvillalHey!00:15
*** shadower has quit IRC00:23
*** shadower has joined #openstack-ironic00:23
*** dims has joined #openstack-ironic00:24
*** Sukhdev has joined #openstack-ironic00:31
*** tsekiyama has quit IRC00:33
*** harshs_ has joined #openstack-ironic00:44
*** harshs has quit IRC00:45
*** amit213 has quit IRC00:45
*** harshs_ is now known as harshs00:45
*** yuriyz has quit IRC00:47
*** shakamunyi has joined #openstack-ironic00:55
*** marzif has joined #openstack-ironic00:56
*** penick has quit IRC00:57
*** baoli has joined #openstack-ironic00:58
*** mbound has joined #openstack-ironic01:00
openstackgerritKATO Tomoyuki proposed openstack/ironic: Fix typo: add a missing white space  https://review.openstack.org/22868101:01
*** harshs has quit IRC01:01
*** r-daneel has quit IRC01:04
*** mbound has quit IRC01:06
*** chenke has joined #openstack-ironic01:06
*** mtanino has quit IRC01:09
mrdaJust a small poke, I have this patch review out for a bug fix, and so far it's seen no love.  If anyone would like to review, it'd be appreciated: https://review.openstack.org/#/c/225556/01:11
*** lazy_prince has joined #openstack-ironic01:13
*** Marga_ has quit IRC01:18
jrollmrda: +1'd01:18
*** Marga_ has joined #openstack-ironic01:18
* jroll needs to learn more about partitioning and such01:18
mrdata01:20
*** Sukhdev has quit IRC01:26
*** harshs has joined #openstack-ironic01:29
jrollI have no clue how to run devstack in japanese :(01:31
jrollaha, got it01:33
*** praneshp has quit IRC01:49
*** harshs has quit IRC01:54
*** baoli has quit IRC01:55
openstackgerritKATO Tomoyuki proposed openstack/ironic: Fix typo: add a missing white space  https://review.openstack.org/22868101:56
*** baoli has joined #openstack-ironic02:06
*** dims has quit IRC02:09
*** achanda has joined #openstack-ironic02:25
*** baoli has quit IRC02:32
*** zhenguo has joined #openstack-ironic02:35
*** Marga__ has joined #openstack-ironic02:36
*** Marga_ has quit IRC02:39
*** Marga__ has quit IRC02:40
*** harshs has joined #openstack-ironic02:41
*** garthb has joined #openstack-ironic02:44
*** praneshp has joined #openstack-ironic02:49
*** baoli has joined #openstack-ironic02:49
*** lazy_prince has quit IRC02:50
*** ijw has quit IRC02:51
*** baoli has quit IRC02:56
*** baoli has joined #openstack-ironic02:57
*** baoli has quit IRC03:03
*** harshs has quit IRC03:05
*** saripurigopi has joined #openstack-ironic03:05
*** achanda has quit IRC03:09
*** dims has joined #openstack-ironic03:09
*** dims has quit IRC03:16
*** bnemec has quit IRC03:17
*** boris-42 has quit IRC03:19
*** hemna has joined #openstack-ironic03:20
*** boris-42 has joined #openstack-ironic03:29
*** baoli has joined #openstack-ironic03:31
*** Nisha has joined #openstack-ironic03:34
*** links has joined #openstack-ironic03:36
*** baoli has quit IRC03:37
*** chenke has quit IRC03:37
*** chenke has joined #openstack-ironic03:38
*** harshs has joined #openstack-ironic03:42
saripurigopiMorning Ironic03:51
*** hemna has quit IRC04:06
*** VikasC has joined #openstack-ironic04:12
*** Marga_ has joined #openstack-ironic04:12
openstackgerritMerged openstack/ironic: Fix typo: add a missing white space  https://review.openstack.org/22868104:17
*** benhur_marcel has quit IRC04:22
*** benhur_marcel has joined #openstack-ironic04:24
*** sdake_ has joined #openstack-ironic04:28
*** itsuugo has joined #openstack-ironic04:28
*** achanda has joined #openstack-ironic04:29
*** lazy_prince has joined #openstack-ironic04:31
*** sdake has quit IRC04:31
*** Nisha has quit IRC04:31
*** Marga_ has quit IRC04:34
*** Nisha has joined #openstack-ironic04:34
*** Marga_ has joined #openstack-ironic04:35
*** Marga_ has quit IRC04:39
*** itsuugo has quit IRC04:42
*** harshs_ has joined #openstack-ironic04:45
*** harshs has quit IRC04:47
*** harshs_ is now known as harshs04:47
*** Marga_ has joined #openstack-ironic04:50
*** itsuugo has joined #openstack-ironic04:56
*** hemna has joined #openstack-ironic05:03
*** hemna has quit IRC05:03
*** vinbs has joined #openstack-ironic05:12
*** dims has joined #openstack-ironic05:13
*** Marga_ has quit IRC05:15
*** Marga_ has joined #openstack-ironic05:15
*** dims has quit IRC05:18
*** praneshp_ has joined #openstack-ironic05:22
*** krtaylor has quit IRC05:23
*** praneshp has quit IRC05:24
*** praneshp_ is now known as praneshp05:24
*** Sukhdev has joined #openstack-ironic05:25
*** killer_prince has joined #openstack-ironic05:25
*** rameshg87 has joined #openstack-ironic05:26
*** lazy_prince has quit IRC05:29
*** krtaylor has joined #openstack-ironic05:32
*** garthb has quit IRC05:32
*** yog_ has joined #openstack-ironic05:33
*** achanda has quit IRC05:36
*** Haomeng has joined #openstack-ironic05:42
*** harshs has quit IRC05:44
*** Marga_ has quit IRC05:49
*** sdake_ is now known as sdake05:56
*** amotoki has joined #openstack-ironic05:56
*** yog_ has quit IRC05:57
*** lsmola has joined #openstack-ironic05:58
*** yog_ has joined #openstack-ironic06:07
*** itsuugo has quit IRC06:09
openstackgerritJuPing proposed openstack/python-ironicclient: Fix the bug of incorrect spelling  https://review.openstack.org/22873706:10
*** jamielennox is now known as jamielennox|away06:12
*** benhur_marcel has quit IRC06:13
*** itsuugo has joined #openstack-ironic06:14
*** itsuugo has quit IRC06:16
*** openstackgerrit has quit IRC06:16
*** openstackgerrit has joined #openstack-ironic06:17
*** Sukhdev has quit IRC06:20
*** vinbs has quit IRC06:23
*** e0ne has joined #openstack-ironic06:27
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Zanata  https://review.openstack.org/22873906:28
*** sdake_ has joined #openstack-ironic06:33
*** dtantsur|afk is now known as dtantsur06:33
dtantsurMorning Ironic06:33
*** e0ne has quit IRC06:34
*** sdake has quit IRC06:36
*** Marga_ has joined #openstack-ironic06:39
*** e0ne has joined #openstack-ironic07:07
*** lazy_prince has joined #openstack-ironic07:09
*** e0ne has quit IRC07:09
*** 32NAAGKSY has joined #openstack-ironic07:09
*** sdake_ is now known as sdake07:10
*** killer_prince has quit IRC07:11
*** benhur_marcel has joined #openstack-ironic07:18
*** e0ne has joined #openstack-ironic07:21
*** benhur_marcel has quit IRC07:22
*** 32NAAGKSY has quit IRC07:30
*** alexpilotti has joined #openstack-ironic07:31
*** lazy_prince has quit IRC07:32
*** lazy_prince has joined #openstack-ironic07:32
*** praneshp_ has joined #openstack-ironic07:32
*** praneshp has quit IRC07:32
*** praneshp_ is now known as praneshp07:32
*** ifarkas has joined #openstack-ironic07:36
*** lazy_prince has quit IRC07:37
*** lazy_prince has joined #openstack-ironic07:38
*** itsuugo has joined #openstack-ironic07:38
*** itsuugo has quit IRC07:40
*** lazy_prince has quit IRC07:42
*** lazy_prince has joined #openstack-ironic07:42
*** romainh has joined #openstack-ironic07:43
*** itsuugo has joined #openstack-ironic07:49
openstackgerritMerged openstack/ironic: Prepare for functional testing  https://review.openstack.org/22861207:49
*** praneshp has quit IRC07:50
*** alexpilotti has quit IRC07:52
*** praneshp has joined #openstack-ironic07:52
*** karimb has joined #openstack-ironic07:56
*** stendulker has joined #openstack-ironic08:04
sambettsMorning Ironic o/08:04
sambettsYou feeling a little less stressed today dtantsur?08:04
*** lazy_prince has quit IRC08:05
dtantsursambetts, morning, thanks for asking. I'm feeling not too well, as I had a bad night after all this, but anyway much better :)08:05
mrdahi sambetts and dtantsur o/08:05
dtantsurmorning mrda08:05
*** lazy_prince has joined #openstack-ironic08:05
sambettso/ mrda08:06
*** derekh has joined #openstack-ironic08:07
sambettsdtantsur: Ah man, thats not great, take it easy today and recover a bit08:07
dtantsurI'll try to. anyway, we've released the thing, time to relax and think about future achievements :)08:08
sambetts:) exactly08:09
dtantsurooops, I forgot about my coffee >_<08:10
rcarrillocruzdtantsur: heya, i adressed your comment on https://review.openstack.org/#/c/222914/, thanks, it was a copy-pasta :-)08:11
dtantsurthanks, cool08:12
*** MattMan has joined #openstack-ironic08:15
*** jistr has joined #openstack-ironic08:16
*** yog_ has quit IRC08:17
*** marzif has quit IRC08:18
*** yuriyz has joined #openstack-ironic08:18
*** yog_ has joined #openstack-ironic08:19
*** ukalifon1 has joined #openstack-ironic08:21
*** athomas has joined #openstack-ironic08:21
*** lazy_prince has quit IRC08:28
*** killer_prince has joined #openstack-ironic08:28
*** ukalifon1 has quit IRC08:30
*** ifarkas has quit IRC08:44
*** ifarkas has joined #openstack-ironic08:45
*** ifarkas has quit IRC08:45
*** ifarkas has joined #openstack-ironic08:45
*** chenke has left #openstack-ironic08:47
rameshg87<ALL PATHES HITTING MERGE FAILURES> :D08:49
dtantsurhehe :D08:50
*** killer_prince has quit IRC08:51
*** dtantsur is now known as dtantsur|brb08:51
*** killer_prince has joined #openstack-ironic08:51
*** sambetts has quit IRC09:07
*** itsuugo has quit IRC09:07
*** ndipanov has quit IRC09:07
*** itsuugo has joined #openstack-ironic09:08
*** sambetts has joined #openstack-ironic09:08
*** davideagnello has quit IRC09:12
*** romcheg has joined #openstack-ironic09:13
*** davideagnello has joined #openstack-ironic09:13
*** dims has joined #openstack-ironic09:16
*** killer_prince has quit IRC09:16
*** killer_prince has joined #openstack-ironic09:16
*** ndipanov has joined #openstack-ironic09:18
*** dims has quit IRC09:22
*** itsuugo has quit IRC09:28
*** itsuugo has joined #openstack-ironic09:29
*** MattMan has left #openstack-ironic09:32
*** digimonde has joined #openstack-ironic09:33
*** MattMan has joined #openstack-ironic09:36
*** zhenguo has quit IRC09:43
*** priteau has joined #openstack-ironic09:43
*** shuquan has joined #openstack-ironic09:46
*** dims has joined #openstack-ironic09:47
*** dtantsur|brb is now known as dtantsur09:49
*** MattMan has left #openstack-ironic09:50
*** MattMan has joined #openstack-ironic09:50
*** MattMan has quit IRC09:50
*** pelix has joined #openstack-ironic09:51
sambettsargh! the grub2 element is still broken in disk-image-builder!09:56
*** naohirot has quit IRC09:56
*** MattMan has joined #openstack-ironic09:56
*** MattMan has quit IRC09:57
dtantsurnot wow09:57
*** MattMan has joined #openstack-ironic09:57
*** kbyrne has quit IRC09:58
*** shuquan has quit IRC09:58
*** MattMan has quit IRC09:59
sambettshmm, trying again after making sure everything is up to date09:59
*** MattMan has joined #openstack-ironic10:00
sambettsnope.. still fails... the way to build the tenant image in the ironic install guide is broken then10:03
*** kbyrne has joined #openstack-ironic10:03
sambettshttps://bugs.launchpad.net/diskimage-builder/+bug/147040010:04
openstackLaunchpad bug 1470400 in diskimage-builder "element grub2 worked with centos7, but not ubuntu." [Medium,Triaged]10:04
sambettsdtantsur: ^10:04
sambetts:'(10:04
dtantsuroh damn...10:04
dtantsursambetts, the problem is that there's not too much people working on tripleo tooling, and the majority is from RH10:05
sambettsI'm just surprised that no one else from ironic has picked it up as its part of our install guide :(10:07
*** MattMan has quit IRC10:08
rameshg87sambetts: I have seen people reporting grub2 element failing, but didn't happen for me10:08
rameshg87sambetts: previously10:09
*** MattMan has joined #openstack-ironic10:09
rameshg87sambetts: I am wondering for some reason, it has to do something with the current environment10:09
sambettsI've ended up using the local-boot element from the tripleo elements and that works but its a pain to have to pull another repo10:10
*** MattMan has quit IRC10:11
*** MattMan has joined #openstack-ironic10:11
sambettsrameshg87: I think it might have been you that pointed me to that solution originally10:12
*** MattMan has quit IRC10:12
rameshg87sambetts: not me :)10:12
sambetts:/ hmmm, my env I'm trying to build it in is Ubuntu server 14.04 with devstack running, what env do you have it working in?10:14
*** MattMan has joined #openstack-ironic10:14
*** romainh has quit IRC10:15
sambettsrameshg87: ^10:16
rameshg87sambetts: I too have the same10:16
* rameshg87 wondered the same thing last time too10:16
sambettsrameshg87: what version of diskimage-builder do you have?10:17
rameshg87sambetts: I use it from github, I don't pip install it10:17
rameshg87sambetts: I mean master10:17
sambettsah10:17
sambetts, I'm using the pip installed version10:18
sambetts1.1.210:18
rameshg87sambetts: probably give a try from master ?10:18
rameshg87sambetts: just to see if issue is still there ..10:18
*** MattMan has quit IRC10:20
*** MattMan has joined #openstack-ironic10:21
sambettsrameshg87: Ok, I'm giving it a go now10:22
sambettsrameshg87: Nope failed, again ... does disk-image-builder do any caching that I can delete to make sure previous runs aren't affecting it?10:24
rameshg87sambetts: it caches base image10:25
rameshg87sambetts: in ~/.cache/disk* or something10:26
rameshg87sambetts: but apart from that I dont think it matters, rest of things it regenerates including pip cache afaik10:26
rameshg87*apt cache10:26
rameshg87sambetts: /opt/stack/diskimage-builder/bin/disk-image-create ubuntu grub2 baremetal is all you are trying ?10:28
rameshg87sambetts: let me give it a go10:28
sambettsrameshg87: I've deleted .cache/image-create and I'm re-running it too10:28
sambettsrameshg87: disk-image-create ubuntu baremetal dhcp-all-interfaces grub2 -o my-image10:28
rameshg87sambetts: I am giving it a go too, but it might take some time, network is slow10:29
rameshg87oh it's fast for a change :)10:29
sambettsrameshg87: ^ thats the command I'm running, its straight from the ironic install guide10:29
rameshg87sambetts: okay10:29
rameshg87I don't have dhcp-all-interface, but probably it doesn't matter10:29
*** Nisha has quit IRC10:29
sambettsrameshg87: I'd hope not :/10:29
openstackgerritYuriy Zveryanskyy proposed openstack/ironic-specs: Ceph Object Gateway Temp URL support for ironic  https://review.openstack.org/22882910:31
sambettsrameshg87: nope :( still failed even after deleting the cache10:32
openstackgerritYuriy Zveryanskyy proposed openstack/ironic-specs: Ceph Object Gateway Temp URL support for Ironic  https://review.openstack.org/22882910:33
rameshg87sambetts: hold on, mine still in progress10:33
rameshg87sambetts: meanwhile can you paste in paste.openstack.org the complete log if possible ?10:33
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Add "logs" and "extra-hardware" inspection collectors  https://review.openstack.org/22509210:33
*** liliars has quit IRC10:33
sambettsrameshg87: I'll give it a go :)10:34
rameshg87sambetts: my speed has slowed down considerably :(10:34
sambetts:(10:35
sambettsrameshg87: http://paste.openstack.org/show/474624/10:38
rameshg87ack, looking10:38
sambettsrameshg87: seems like something weird happens when it trys to install the grub package10:44
rameshg87sambetts: to me it looks like grub configure scripts try to check for a block device during post-install or something10:45
rameshg87sambetts: and it's not finding anything in /dev10:45
rameshg87sambetts: I had hoped dib would have done a bind mount of /dev/ on the chrooted filesystem10:45
rameshg87sambetts: but not sure what's happening10:45
rameshg87sambetts: same errror, when others used to get, it used to work for me well.10:45
rameshg87sambetts: let us check if that's the case again10:46
rameshg87sambetts: meanwhile I would suggest using the break debugging mechanism of dib and give a try breaking before the install phase in dib10:46
sambettsrameshg87: how do I do that?10:46
rameshg87sambetts: http://docs.openstack.org/developer/diskimage-builder/developer/developing_elements.html#debugging-elements10:47
rameshg87sambetts: something like export break=before-install10:47
rameshg87sambetts: dib will drop to chrooted environment just before the install phase (in this context, just before installing grub)10:47
rameshg87sambetts: that would give some window to do debugging10:48
rameshg87sambetts: first thing I would check is what is in /dev10:48
rameshg87sambetts: meanwhile we can see how my run goes10:48
sambettsyup :)10:48
*** jamielennox|away is now known as jamielennox10:49
*** sdake_ has joined #openstack-ironic10:50
*** sdake has quit IRC10:54
*** dtantsur is now known as dtantsur|brb10:54
sambettsrameshg87: Ok, I'm in the debug console11:03
rameshg87sambetts: okay, mine is installing grub2 package now11:03
rameshg87sambetts: and it seems to have gone through and installed properly11:03
sambetts:/11:04
rameshg87sambetts: I am on this - http://paste.openstack.org/show/474628/11:04
sambettsrameshg87: Mines a little different http://paste.openstack.org/show/474629/11:07
rameshg87so you have a newer one11:07
sambettsseems like it11:08
*** ifarkas has quit IRC11:09
*** ifarkas has joined #openstack-ironic11:09
rameshg87sambetts: it worked for me11:10
rameshg87sambetts: retrying with dhcp-all-interfaces to rule out11:10
sambettsrameshg87: ok :)11:10
sambettsrameshg87: anything that you think i should check in my debug console?11:11
rameshg87sambetts: /dev ?11:11
rameshg87sambetts: is there something in it ?11:11
rameshg87may be I should drop to shell too11:11
*** itsuugo has quit IRC11:11
sambettsrameshg87: Yeah, theres a bunch of stuff in /dev11:11
rameshg87sambetts: may be try to install grub2 manually ?11:12
*** ramineni has joined #openstack-ironic11:12
*** romcheg1 has left #openstack-ironic11:12
rameshg87apt-get install grub-efi-amd64-signed shim-signed grub-pc-bin11:12
rameshg87??11:12
*** ramineni_ has joined #openstack-ironic11:13
*** ramineni1 has joined #openstack-ironic11:13
sambettsrameshg87: errors11:14
rameshg87sambetts: okay, goes through for me when I tried to run ^^^ above command manually11:15
rameshg87sambetts: so that's where error is11:15
sambettscannot find a device for / (is /dev mounted?)11:15
*** ramineni2 has joined #openstack-ironic11:15
sambetts /bin/df cannot read table of mouted file systems11:15
sambettsNo such file or directory11:16
*** ramineni3 has joined #openstack-ironic11:16
*** ramineni has quit IRC11:16
rameshg87sambetts: apt-cache madison grub-common grub-efi-amd64 grub-efi-amd64-bin grub-efi-amd64-signed grub-pc-bin grub2-common11:17
rameshg87sambetts: what does that give11:17
*** ramineni1 has quit IRC11:17
rameshg87sambetts: mine gives this - http://paste.openstack.org/show/474630/11:17
*** ramineni has joined #openstack-ironic11:18
sambettsif you run df what output does it give you?11:19
*** ramineni2 has quit IRC11:19
*** ramineni3 has quit IRC11:20
rameshg87sambetts: no mounts11:20
sambettsrameshg87: http://paste.openstack.org/show/474641/11:20
rameshg87sambetts: so it's exactly same packages11:21
sambettsrameshg87: mine says 'df: cannot read table of mounted file systems: No such file or directory'11:21
*** itsuugo has joined #openstack-ironic11:22
rameshg87sambetts: yeah, mine is same11:22
rameshg87sambetts: mount output is empty11:22
*** ramineni has quit IRC11:22
sambettsrunning "mount" on mine lists 7 things11:22
rameshg87sambetts: oh what are they ?11:23
* rameshg87 takes grub source code11:23
sambettsrameshg87: http://paste.openstack.org/show/474643/11:24
rameshg87sambetts: this is where your error comes from11:25
rameshg87http://git.savannah.gnu.org/cgit/grub.git/tree/util/grub-probe.c#n27311:26
rameshg87sambetts: strange, my mount is empty within chrooted environment11:26
*** baoli has joined #openstack-ironic11:27
*** marzif has joined #openstack-ironic11:28
*** baoli_ has joined #openstack-ironic11:28
sambetts:/11:29
*** trown|outttypeww has quit IRC11:30
*** trown has joined #openstack-ironic11:31
*** baoli has quit IRC11:31
*** ramineni_ has quit IRC11:32
rameshg87sambetts: http://paste.openstack.org/show/474644/11:32
rameshg87sambetts: this is the output of grub-probe in my chrooted environment11:33
sambettsrameshg87: http://paste.openstack.org/show/474645/11:34
rameshg87sambetts: that was my question too, why is it not able to find anything in /dev11:35
*** stendulker has quit IRC11:35
rameshg87sambetts: ls /dev/sd* shows your disks ?11:36
sambetts /dev/sda /dev/sda1 /dev/sda2 /dev/sda511:36
rameshg87sambetts: I will need to leave now, but I think there is something to debug11:38
rameshg87sambetts: grub-probe -vv /11:38
rameshg87it shows even more information11:38
sambettsok :)11:39
rameshg87sambetts: this is mine with -vv - http://paste.openstack.org/show/474647/11:41
rameshg87sambetts: may be that might give a clue as to why grub-probe on your system is not detecting device files in /dev11:41
rameshg87sambetts: will try to join you if possible later in the evening11:41
rameshg87sambetts: comparing with -vv output of yours might give a clue11:42
rameshg87sambetts: see you later .. have a good day11:42
* rameshg87 goes home11:42
*** rameshg87 has quit IRC11:42
sambettsrameshg87: Thanks for your help! Cya later!11:42
*** itsuugo has quit IRC11:50
*** dtantsur|brb is now known as dtantsur11:52
*** thrash|g0ne is now known as thrash11:55
*** VikasC has quit IRC11:56
openstackgerritShinn'ya Hoshino proposed openstack/ironic: Fix a bug error by passwords includes just numbers  https://review.openstack.org/22038512:00
*** itsuugo has joined #openstack-ironic12:01
ifarkasdtantsur, hi, could you please review https://review.openstack.org/#/c/222623/ ?12:01
openstackgerritOm Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - RPC  https://review.openstack.org/20624312:05
openstackgerritOm Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/20624412:05
openstackgerritOm Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - net  https://review.openstack.org/20624512:05
openstackgerritOm Kumar proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326212:05
openstackgerritOm Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - DB  https://review.openstack.org/20623212:05
openstackgerritOm Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - objs  https://review.openstack.org/20623812:05
openstackgerritOm Kumar proposed openstack/ironic: Add network provider interface and implementations  https://review.openstack.org/13968712:05
*** nicodemos has joined #openstack-ironic12:07
TheJuliaGood morning12:07
sambettso/ TheJulia12:08
*** aarefiev has quit IRC12:09
*** romainh has joined #openstack-ironic12:10
dtantsurifarkas, sure, it's on my list12:11
ifarkasdtantsur, thanks!12:11
*** aarefiev has joined #openstack-ironic12:12
openstackgerritJulia Kreger proposed openstack/bifrost: Ansible 2.0 compatibility  https://review.openstack.org/19683212:12
*** itsuugo has quit IRC12:12
*** itsuugo has joined #openstack-ironic12:13
*** itsuugo has quit IRC12:30
dtantsurjroll, I though we have/had a patch up to support DEBUG level logging in IPA, do you remember something like that?12:31
TheJuliadtantsur: I thought there kind of was, but you had to get the output to the console12:32
*** trown is now known as trown|afk12:33
dtantsurTheJulia, morning. No, my idea is just to enable DEBUG level logging in oslo.config, output is a bit unrelated12:33
TheJuliaFor some reason I thought it was already that way, but I've not looked deeply at IPA in a few months12:34
dtantsurI also though it's there already, but I can't find any traces...12:34
dtantsur* thought12:34
TheJuliaTime for more coffee?12:35
dtantsur:)12:35
dtantsurit's midday here, I'm supposed to be awake already12:35
dtantsur(I'm definitely not though)12:35
TheJuliaI'm still on my frist cup of coffee12:36
dtantsurfix it asap12:36
TheJulialol12:36
*** dims_ has joined #openstack-ironic12:36
TheJuliarcarrillocruz: By chance did you have a chance to look at 228178?12:37
*** dims has quit IRC12:40
*** amotoki has quit IRC12:42
*** dims_ has quit IRC12:53
dtantsursambetts, wdyt about https://blueprints.launchpad.net/ironic-inspector/+spec/node-states12:58
*** priteau has quit IRC13:01
*** dims has joined #openstack-ironic13:01
*** rloo has joined #openstack-ironic13:02
sambettsdtantsur: Sounds like a good idea to prevent processing running multiple times for the same node and it'll also provide more in depth feedback of the introspection process to people querying status13:02
dtantsurcool13:03
sambetts:)13:03
dtantsursambetts, and one more question: what would you say about a feature to rerun the whole processing chain for a node (given, that we start storing unprocessed data)?13:04
dtantsur(sorry if I already asked you)13:04
dtantsursambetts, i.e. like https://blueprints.launchpad.net/ironic-inspector/+spec/reapplying-rules but rerun everything?13:04
*** trown|afk is now known as trown13:05
dtantsurI'm not decided if it's worth effort, or we'd better only rerun rules13:05
dtantsurtrown, ifarkas, question for you too ^^13:05
dtantsurand morning trown :)13:05
trownmorning dtantsur sambetts TheJulia ifarkas13:06
*** priteau has joined #openstack-ironic13:06
ifarkasmorning all13:07
sambettsdtantsur: I think its worth re-runing both the processing plugins and rules, because the plugins may modify parts of the data, e.g. like extra hardware does, and so if you don't rerun all the plugins too then it won't be the same as it was the first time13:07
ifarkasdtantsur, +1, the blueprint provides a good argument to support that13:07
dtantsurhmm, year fair. also pretty useful for debugging plugins...13:08
sambettsthats another good poit13:09
sambettspoint*13:09
dtantsurok, I'll update this blueprint. thanks all!13:09
*** thiagop has joined #openstack-ironic13:09
openstackgerritJulia Kreger proposed openstack/bifrost: Ansible 2.0 compatibility  https://review.openstack.org/19683213:09
thiagopGood morning Ironicers13:09
sambettso/ thiagop13:10
sambettso/ trown13:10
*** daemontool_ has joined #openstack-ironic13:10
openstackgerritJulia Kreger proposed openstack/bifrost: Initial support to collect node facts like UUID  https://review.openstack.org/22817913:10
*** links has quit IRC13:22
dtantsursambetts, trown, ifarkas, so now it looks like this: https://blueprints.launchpad.net/ironic-inspector/+spec/rerun13:23
dtantsur(API endpoint is subject to bikeshedding)13:23
thiagopjroll: Hey you, congratz!13:26
*** MattMan has left #openstack-ironic13:26
sambettsdtantsur: Does it need to be syncronous? If we implement the spec you mentioned previously with the different states, then it could run safely async with and be checkable via status13:27
*** MattMan has joined #openstack-ironic13:28
dtantsursambetts, it does not, I was just thinking that it could be more convenient. on the other hand, being sync is a hard commitment, maybe we don't want to maintain it...13:28
sambettsdtantsur: just thinking in case any custom rules take a long time, and if you wanted to rerun several nodes in parrell it would be easier if it was async13:29
dtantsurright13:29
jrollmorning everyone :)13:30
jrolldtantsur: not sure about that patch :/13:30
trowndtantsur: I like the idea of node-states13:30
jrollthiagop: thanks :D13:30
dtantsurmorning jroll!13:31
jroll\o13:31
thiagopjroll: morning too13:31
jrollthanks for breaking everything in flight13:31
jroll:D13:31
dtantsuryou're always welcome ;)13:31
dtantsurI got around 15 notifications from Gerrit about it13:33
jrollI /dev/null gerrit emails, too many things :P13:34
dtantsurI would really want it to send only notifications from my patches and answers to my reviews13:34
openstackgerritRuby Loo proposed openstack/ironic: Use standard locale when executing 'dd' command  https://review.openstack.org/21647813:35
jrollrloo: thanks \o/13:35
jrolltrying to test that.13:35
dtantsursambetts, updated the endpoint to be async13:36
rloohey, morning jroll. I was going to trust yuiko on that :)13:36
dtantsurmorning rloo!13:36
jrollrloo: I guess we have to, I can't reproduce13:36
rloohi everyone, dtantsur, sambetts, thiagop, ifarkas13:37
jrollrloo: it's awful odd that the error happens far below the dd command though :(13:37
thiagopgood morning rloo13:37
rloojroll: well vdrok made a comment, he thinks stderr wasn't log'd cuz of the exception13:37
jrollah.13:37
rloojroll: I can only +1 so I won't be blamed ;)13:38
jrollheh13:39
jlvillaljroll, dtantsur I did say in the email that I thought most things would break in-flight ;)13:39
* jlvillal woke up way too early today. 13:40
NobodyCammorning Ironicers :)13:40
jlvillalNobodyCam, Good morning!13:40
rloojlvillal: no worries. the rebasing itself (for one patch I did) was painless.13:40
sambettsdtantsur: Is it worth adding a dependecy on the states blueprint? and it should perhaps error if the node's not in finsihed already?13:40
NobodyCamwow jlvillal you are up early :)13:40
rloomorning NobodyCam13:40
thiagopgood morning NobodyCam13:41
jlvillalNobodyCam, Yep. Unfortunately ;)  Woke up at 4:30 and couldn't fall back asleep even though I tried.13:41
NobodyCammorning jlvillal, rloo, dtantsur, sambetts, jroll, thiagop and all others not listed here13:41
sambettso/ NobodyCam13:41
jlvillalrloo, Good to know. Thanks.13:41
NobodyCamjlvillal: uggh13:42
NobodyCambut been there done that many times13:42
jrollheya NobodyCam :)13:42
jrolljlvillal: more time for activities \o/13:42
NobodyCammorning jroll :)13:42
NobodyCamnow coffee time :) brb13:42
openstackgerritRuby Loo proposed openstack/ironic: Refactor deploy_utils methods  https://review.openstack.org/22226413:42
krtaylorgood morning UGT13:44
*** jaypipes has quit IRC13:44
thiagopgood morning krtaylor13:45
openstackgerritJim Rollenhagen proposed openstack/ironic: Switch tox unit test command to use ostestr  https://review.openstack.org/22810513:46
*** mdbooth_ has joined #openstack-ironic13:46
*** cdearborn has joined #openstack-ironic13:46
jrolljlvillal: ^ can you make sure my rebase makes sense?13:47
jrollalso wondering if we can drop the whitelist_externals thing13:47
jlvillaljroll, I will take a look13:47
jrollthanks, no rush13:47
NobodyCammorning krtaylor13:48
NobodyCamUGT ofc13:48
jlvillaljroll, makes sense to me. And I bet we could drop the whitelisting of 'bash'13:48
jlvillaljroll, Did you try a tox run? If so, did you see what I mean by much much much more output?13:49
*** marzif has quit IRC13:49
jlvillalPersonally I kind of like it. I think it will make debugging things easier.13:49
jrolljlvillal: I saw it in the logs, I like it personally13:49
*** jaypipes has joined #openstack-ironic13:49
*** mdbooth has quit IRC13:49
*** mdbooth_ is now known as mdbooth13:49
jrollI also run production in DEBUG, so...13:49
* jroll tests dropping the bash thing13:50
*** romcheg has quit IRC13:53
openstackgerritJim Rollenhagen proposed openstack/ironic: Switch tox unit test command to use ostestr  https://review.openstack.org/22810513:54
jrolltotes works13:54
jrollhowever apparently the bot doesn't?13:54
jrolljlvillal: updated13:54
jrolloh, so slow openstackgerrit13:55
jlvillallooking....13:55
*** saripurigopi has quit IRC13:56
jlvillaljroll, LGTM13:56
jrollthanks :)13:56
* jlvillal for some reason when he first started thought LGTM meant Let's Get That Merged!13:56
jrollhah13:56
jrollI like it13:56
rloojroll, jlvillal: whoa, that's A LOT of output13:58
dtantsursambetts, we can figure out if node is finished already13:58
jrollhehe13:58
jlvillalrloo, I told you ;)13:58
dtantsursambetts, we have something like finished_at13:58
dtantsurmorning NobodyCam13:58
dtantsurmorning jlvillal, krtaylor13:59
jlvillaldtantsur, Good morning. Privyet13:59
dtantsur:)13:59
sambettsdtantsur: Yh, tbh they were kinda separate thoughts, because we already have finsihed = True in the statys13:59
dtantsurjlvillal, привет13:59
sambettsstatus*13:59
rloojlvillal: why can't it just spew that stuff into .testrepository/*13:59
jlvillalSorry, I'm clueless on typing in Cyrillic13:59
dtantsurhehe14:00
*** lazy_prince has joined #openstack-ironic14:00
jlvillalrloo, That would probably be a great question for mtreinish...14:00
jrollif you don't want the output, run tox -e py27 > test.log14:01
jrollor whatever14:01
sambettsdtantsur: I just thought it would make sense to mention that it'll only work for already finished nodes, so would error if attempted on a node that wasn't already finished14:01
jrollI don't believe "too much output" exists for running tests14:01
dtantsursambetts, good point14:01
dtantsursambetts, updated14:02
* jroll looks at what failing tests look like14:02
sambettsdtantsur: awesome :D14:02
jlvillaljroll, I like your attitude. But sometimes reading those jenkins logs. I'm not so sure ;)14:02
dtantsurheh, I like spec review via IRC :D14:02
jrolljlvillal: as long as failures are at the bottom, I'm good with it :)14:02
*** killer_prince has quit IRC14:02
rloojroll, jlvillal: failures are at the bottom14:03
jlvillaljroll, I wish they were. Usually it seems I'm scrolling up and up and up and up...14:03
jrollaaaaand they are.14:03
jrollFailed 1 tests - output below:14:03
jroll==============================14:03
jroll<insert test fail spam>14:03
jlvillalExcellent :)14:03
dtantsuryeah, sounds good14:04
rloojroll, jlvillal: it would be nice if the commit mentioned a lot of output, with the failures at the bottom.14:04
jrollalso, our tests are getting slow :( Ran: 2703 tests in 38.0000 sec.14:04
dtantsurjroll, btw IPA currently is troublesome: it doesn't even show print's during unit tests14:04
jrollthat's with 2 cores14:04
jrolldtantsur: :(14:04
dtantsurIIRC it was somehow fixed in ironic, but not in IPA14:04
jlvillal2 cores??????14:04
jlvillalOh, right. Probably a VM14:05
jlvillalWhich is the smart way to do things.14:05
*** itsuugo has joined #openstack-ironic14:05
jlvillalrloo, Sounds reasonable to me.14:05
openstackgerritJim Rollenhagen proposed openstack/ironic: Switch tox unit test command to use ostestr  https://review.openstack.org/22810514:05
jroll^ updated commit message rloo14:06
rloodtantsur: maybe try switching to ostestr: https://review.openstack.org/#/c/228105/14:06
jrolldtantsur: I wonder what happens if you use ostestr there :)14:06
rloothx jroll14:06
jrollnp14:06
dtantsuryeah, that's what I mean :)14:06
* jlvillal goes to make some breakfast...14:07
dtantsursambetts, have you also seen https://blueprints.launchpad.net/ironic-inspector/+spec/abort-introspection ? it's around for a while, but we never had time to implement it...14:07
dtantsurnow that we have 'abort' word in ironic, aborting in-band inspection seems a good feature to have14:08
jrolljlvillal: heh, I don't trust code that is designed to erase and partition disks without a VM, no matter how many mocks14:08
*** romcheg has joined #openstack-ironic14:08
dtantsurare you running unit tests as root? ;)14:08
jrollheh14:09
jrollstill don't trust it!14:09
sambettsdtantsur: I think I agree about adding an abort, as you say if ironic can abort deploy then we should be able to abort introspection without having to wait for it to timeout14:10
dtantsurcool14:13
*** achanda has joined #openstack-ironic14:15
openstackgerritVladyslav Drok proposed openstack/ironic: Add retries to ssh._get_hosts_name_for_node  https://review.openstack.org/22482814:15
*** achanda has quit IRC14:22
*** mgoddard1 has joined #openstack-ironic14:31
*** mgoddard has quit IRC14:31
openstackgerritRuby Loo proposed openstack/ironic: Update help strings for DRAC configs  https://review.openstack.org/22897614:35
*** mtanino has joined #openstack-ironic14:47
*** digimonde has quit IRC14:48
*** achanda has joined #openstack-ironic14:51
*** [1]cdearborn has joined #openstack-ironic14:55
*** cdearborn has quit IRC14:59
*** achanda has quit IRC14:59
*** lazy_prince has quit IRC14:59
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Add .eggs/ to gitignore  https://review.openstack.org/22898515:00
*** mgoddard1 has quit IRC15:04
rloois Lucas on vacation this week? Seems like he mentioned something about that.15:04
*** dims has quit IRC15:04
dtantsurrloo, yes15:04
jvmariomorning ironic \o15:04
rloodtantsur: thx. (maybe I should have taken vacation too!)15:04
TheJuliagood morning jvmario15:04
*** mgoddard has joined #openstack-ironic15:04
dtantsuroh, me too, for a loooong time15:04
dtantsurmorning jvmario15:04
rloohey, morning jvmario, TheJulia15:04
* TheJulia needs a vacation15:05
rloodtantsur, TheJulia, we all deserve it!!! :)15:05
*** amotoki has joined #openstack-ironic15:10
* jlvillal wants a vacation...again ;)15:10
*** dims__ has joined #openstack-ironic15:16
*** nicodemos has quit IRC15:16
dtantsurgoing a bit earlier today, see you tomorrow15:17
rloogood night dtantsur15:18
sambettsNight dtantsur o/15:18
thiagopnight dtantsur15:20
*** MattMan has left #openstack-ironic15:24
*** dtantsur is now known as dtantsur|afk15:26
*** lekha is now known as lekha||away15:33
rcarrillocruzTheJulia: i haven't i'm afraid, i've had an issue with the baseline config of my blades and have spent the whole day redeploying15:36
rcarrillocruzis that for the name missing thing?15:36
rcarrillocruzi saw on the weekend you sent several fixes15:36
*** saripurigopi has joined #openstack-ironic15:37
TheJuliarcarrillocruz: :( it is for the name field missing, apparently I completely forgot to add it to be passed through15:41
*** thrash is now known as thrash|bbl15:41
TheJuliaat least for enrollment, referencing by strictly name instead of uuid is in the following revisions15:41
BadCubMorning Ironicville15:43
thiagopmorning BadCub15:43
BadCubhiya thiagop15:43
*** marzif has joined #openstack-ironic15:54
*** mgoddard has quit IRC15:56
*** mgoddard has joined #openstack-ironic15:57
*** Nisha has joined #openstack-ironic15:57
openstackgerritMario Villaplana proposed openstack/ironic: Allow vendor drivers to acquire shared tasks  https://review.openstack.org/22605816:00
jvmariojlvillal: thanks for the review - i updated the patch based on your comments16:01
jlvillaljvmario: Thanks for updating. Sorry if too nit-picky...16:02
jvmariojlvillal: not at all16:02
*** itsuugo has quit IRC16:03
openstackgerritMario Villaplana proposed openstack/ironic-python-agent: Adds more functional tests for commands  https://review.openstack.org/22424716:05
jvmariomrda: thanks for the review - i added the comment you recommended16:05
*** marzif has quit IRC16:05
*** vishwanathj has joined #openstack-ironic16:06
*** ifarkas has quit IRC16:08
*** Nisha_away has joined #openstack-ironic16:10
*** alexpilotti has joined #openstack-ironic16:12
*** Nisha has quit IRC16:13
*** praneshp has quit IRC16:14
jlvillaldtantsur|afk or anyone else have suggestion on a different name for: https://review.openstack.org/#/c/228612/3/ironic/tests/unit/api/base.py16:17
jlvillalPecanUnitTest?16:17
* jlvillal working on follow-up patch16:17
*** yog_ has quit IRC16:19
*** digimonde has joined #openstack-ironic16:23
*** romcheg has quit IRC16:23
sambettsjlvillal: maybe something like BaseAPIFuncTestCase?16:25
jlvillalsambetts: Maybe. I think we wanted to get rid of Functional. Since we have created a new 'functional' directory for functional testing.16:26
*** jistr has quit IRC16:26
*** ijw has joined #openstack-ironic16:26
openstackgerritRuby Loo proposed openstack/ironic: Fix documentation for installing mariaDB  https://review.openstack.org/22903216:26
openstackgerritJohn L. Villalovos proposed openstack/ironic: Follow up for: Prepare for functional testing patch  https://review.openstack.org/22903316:29
sambettsjlvillal: ah ok maybe BaseAPITestCase? then16:31
jlvillalsambetts: That sounds pretty good to me. But maybe drop 'Case'16:31
jlvillalBaseAPITest or should it be BaseApiTest ?16:31
sambettsjlvillal: :/ not sure I tend to keep the classes called TestCases because to me the functions are the tests, but thats just what I would do, I'd be happy with BaseApiTest too16:33
jlvillalsambetts: Thanks for your help :)16:34
sambetts:)16:34
*** romainh has left #openstack-ironic16:37
*** dims__ has quit IRC16:38
*** dims__ has joined #openstack-ironic16:39
*** karimb has quit IRC16:40
openstackgerritJohn L. Villalovos proposed openstack/ironic: Follow up for: Prepare for functional testing patch  https://review.openstack.org/22903316:41
*** alexpilotti has quit IRC16:44
*** digimonde has quit IRC16:51
*** trown is now known as trown|lunch16:54
*** derekh has quit IRC16:56
*** garthb has joined #openstack-ironic17:00
openstackgerritMerged openstack/ironic: Imported Translations from Zanata  https://review.openstack.org/22873917:04
openstackgerritMerged openstack/ironic: Replace six.iteritems() with .items()  https://review.openstack.org/22856817:06
*** ukalifon has joined #openstack-ironic17:11
openstackgerritJohn L. Villalovos proposed openstack/ironic: Follow up for: Prepare for functional testing patch  https://review.openstack.org/22903317:13
*** lsmola has quit IRC17:14
sambettsNight everyone17:14
sambettso/17:14
thiagopnight sambetts17:16
*** harshs has joined #openstack-ironic17:16
*** ijw has quit IRC17:18
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: [WIP] Configuration Resource  https://review.openstack.org/22905517:18
*** marzif has joined #openstack-ironic17:19
*** itsuugo has joined #openstack-ironic17:26
rloobye sambetts17:27
*** ijw has joined #openstack-ironic17:27
*** athomas has quit IRC17:27
*** itsuugo has quit IRC17:27
*** garthb has quit IRC17:27
*** garthb has joined #openstack-ironic17:28
*** achanda has joined #openstack-ironic17:29
*** coolsvap has quit IRC17:33
*** coolsvap has joined #openstack-ironic17:37
*** itsuugo has joined #openstack-ironic17:39
*** krtaylor has quit IRC17:42
*** ijw has quit IRC17:44
*** trown|lunch is now known as trown17:45
*** baoli_ has quit IRC17:46
jrolljlvillal: looks like global-reqs is open for mitaka17:47
*** mgoddard1 has joined #openstack-ironic17:49
*** mgoddard has quit IRC17:49
*** praneshp has joined #openstack-ironic17:50
jlvillaljroll: Thanks17:51
jrollnp17:51
*** penick has joined #openstack-ironic17:51
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/22906717:52
*** krtaylor has joined #openstack-ironic17:55
*** Marga_ has quit IRC17:56
*** penick has quit IRC17:58
*** Nisha_away has quit IRC17:58
*** Nisha_away has joined #openstack-ironic17:58
*** dims has joined #openstack-ironic18:00
*** dims__ has quit IRC18:02
*** penick has joined #openstack-ironic18:04
BadCubhey thiagop you still around?18:05
thiagopBadCub: Yes, I am18:05
*** penick has quit IRC18:06
*** marzif has quit IRC18:09
*** Nisha_away has quit IRC18:09
*** Nisha_away has joined #openstack-ironic18:09
*** penick has joined #openstack-ironic18:13
*** ijw has joined #openstack-ironic18:13
*** ijw has quit IRC18:14
*** Sukhdev has joined #openstack-ironic18:14
*** Sukhdev has quit IRC18:16
*** Sukhdev has joined #openstack-ironic18:17
openstackgerritJulia Kreger proposed openstack/bifrost: Make use of host fact collection for name based inventories  https://review.openstack.org/22818018:19
*** Marga_ has joined #openstack-ironic18:19
*** ijw has joined #openstack-ironic18:23
openstackgerritJulia Kreger proposed openstack/bifrost: Ansible 2.0 compatibility  https://review.openstack.org/19683218:24
*** itsuugo has quit IRC18:26
*** boris-42 has quit IRC18:29
*** Nisha_away has quit IRC18:31
*** Nisha_away has joined #openstack-ironic18:31
*** Sukhdev has quit IRC18:44
*** Sukhdev has joined #openstack-ironic18:45
*** jasondotstar is now known as jasondotstar_afk18:47
*** thrash|bbl is now known as thrash18:54
*** e0ne has quit IRC18:54
*** saripurigopi has quit IRC18:55
*** afaranha has joined #openstack-ironic18:57
*** Sukhdev has quit IRC18:58
*** pelix has quit IRC19:01
*** ukalifon has quit IRC19:06
*** priteau has quit IRC19:09
*** davideagnello has quit IRC19:09
*** Sukhdev has joined #openstack-ironic19:09
*** itsuugo has joined #openstack-ironic19:10
*** priteau has joined #openstack-ironic19:15
*** priteau has quit IRC19:15
*** priteau has joined #openstack-ironic19:16
*** mgoddard1 has quit IRC19:18
*** davideagnello has joined #openstack-ironic19:20
*** baoli has joined #openstack-ironic19:21
*** tsekiyama has joined #openstack-ironic19:22
*** ijw has quit IRC19:24
*** priteau has quit IRC19:25
*** itsuugo has quit IRC19:27
*** digimonde has joined #openstack-ironic19:28
*** itsuugo has joined #openstack-ironic19:28
*** baoli has quit IRC19:31
*** baoli has joined #openstack-ironic19:32
*** krtaylor has quit IRC19:40
*** krtaylor has joined #openstack-ironic19:43
rcarrillocruzhey folks19:43
rcarrillocruzi was wondering19:43
rcarrillocruzis it possible to add metadata to an ironic node?19:43
rcarrillocruzor arbitrary properties, however the naming19:44
thiagoprcarrillocruz: I think the driver_info field could be used to do that19:45
*** baoli has quit IRC19:45
rcarrillocruzhmm, ok19:45
rcarrillocruzTheJulia: you did anything like that on baremetal.json entries?19:45
*** baoli has joined #openstack-ironic19:46
*** Sukhdev has quit IRC19:48
rcarrillocruzhmm19:48
rcarrillocruzhttp://docs.openstack.org/developer/ironic/webapi/v1.html19:48
rcarrillocruzit may seem the Node object has an 'extra' field just for that19:49
rcarrillocruzweeee19:49
thiagopso many fields... :P19:53
*** baoli has quit IRC19:57
TheJuliarcarrillocruz: in the baremetal.json file they get passed as vars, we don't have the extra field mapped through though.... :(19:58
rcarrillocruzdoh19:58
rcarrillocruzok, then i'll take a stab at the code19:59
openstackgerritJohn L. Villalovos proposed openstack/ironic: Rename 'message' attribute to '_msg_fmt' in IronicException  https://review.openstack.org/21536219:59
rcarrillocruzlooks a valid use case to add to bifrost, no?19:59
TheJuliafor the ansible mods, it would be valid imho19:59
rcarrillocruzk thx19:59
*** [1]cdearborn has quit IRC20:00
*** penick has quit IRC20:01
*** cdearborn has joined #openstack-ironic20:01
TheJuliarcarrillocruz: if you want to sub it in via the bifrost repo/playbooks, I will fully support that :)20:01
jlvillalgreghaynes: Any chance you could look at: https://review.openstack.org/224928  I realize it is a minor thing for the diskimage-builder.20:02
*** penick has joined #openstack-ironic20:05
*** cdearborn has quit IRC20:05
*** cdearborn has joined #openstack-ironic20:05
rcarrillocruzTheJulia: i really hope i can test your fixes tomorrow, appreciate your code20:06
*** [1]Rick has joined #openstack-ironic20:06
rcarrillocruztoday has been a madness20:06
rcarrillocruzhad to deploy 45 blades20:06
rloojroll: were you going to approve this? https://review.openstack.org/#/c/228105/20:07
*** ukalifon1 has joined #openstack-ironic20:07
*** Nisha_away has quit IRC20:09
*** Marga_ has quit IRC20:12
*** Marga_ has joined #openstack-ironic20:13
*** e0ne has joined #openstack-ironic20:14
*** harshs has quit IRC20:14
*** mrda has quit IRC20:18
greghaynesjlvillal: Ive been on pto for a bit, back tomorrow and im planning to review all the dib things20:19
jlvillalgreghaynes: Thank you much :)20:19
jlvillaljvmario: Reviewed the functional testing patch. Put a pointer about requests.request()20:20
jvmariothanks jlvillal20:20
openstackgerritMerged openstack/ironic-specs: Update spec file locations for 4.2 / Liberty  https://review.openstack.org/22660620:23
jlvillaljvmario: I did have question on the tests though. Is the step_* thing to make sure they run one at a time?20:28
*** dims has quit IRC20:28
jvmariojlvillal: right, and that they execute with a given order20:28
jlvillaljvmario: Never mind. I have read the comment....20:28
jlvillaljvmario: Thanks!20:28
jvmarionp20:28
*** dims has joined #openstack-ironic20:29
jrollrloo: oh, I was thinking I made a change to it and shouldn't, was fairly minor though, I'll go ahead now20:29
rloojroll: yeah, I think it is OK20:29
*** olaph has quit IRC20:32
*** Marga_ has quit IRC20:32
*** Marga_ has joined #openstack-ironic20:33
*** Sukhdev has joined #openstack-ironic20:35
*** baoli has joined #openstack-ironic20:38
*** karimb has joined #openstack-ironic20:42
*** ukalifon1 has quit IRC20:44
*** [1]cdearborn has joined #openstack-ironic20:45
jvmariojlvillal: good point about sorting the steps in the ipa functional tests20:47
jvmariojlvillal: what would you think about using this lib for sorting the steps? https://pypi.python.org/pypi/natsort20:47
jvmarioit looks useful, but i'm not sure whether we want to add another test dependency20:47
jlvillaljvmario: Maybe: foo, index, rest = step_name.split('_', 2)20:48
jlvillalreturn int(foo)20:48
jvmariothat works, too20:48
jlvillalSomething like that as a 'key' function20:48
jlvillaljvmario: A little fragile, but it is a unit test, so I think it is fine :)20:48
*** ijw has joined #openstack-ironic20:58
*** cdearborn has quit IRC20:59
jlvillaljroll: As a note, the experimental grenade jobs failed. I haven't yet explored why.21:00
jrolljlvillal: I figured, usually if something is working it would be at least non-voting :)21:01
*** olaph has joined #openstack-ironic21:03
jlvillalYeah21:03
*** ijw has quit IRC21:03
*** mrda has joined #openstack-ironic21:04
mrdaMorning21:04
*** trown is now known as trown|outttypeww21:04
jlvillalmrda: Good morning21:05
mrdahey jlvillal21:05
jvmario\o21:05
*** harshs has joined #openstack-ironic21:05
jrollmorning mrda21:05
mrdahey jroll and jvmario21:07
thiagopmorning mrda21:08
mrdathiagop: o/21:08
*** hemna has joined #openstack-ironic21:09
rloomorning mrda21:10
mrdarloo: o/21:10
mgagneregarding https://review.openstack.org/#/c/228004/5 can someone provide an example of this manifest.xml?21:12
mgagneI'm having a hard time finding example of a valid manifest.xml21:13
*** itsuugo has quit IRC21:13
*** alexpilotti has joined #openstack-ironic21:14
rloomgagne: first I've heard of that. jlvillal may know.21:14
jlvillalmgagne: git clone https://android.googlesource.com/platform/manifest21:15
*** olaph has quit IRC21:15
mgagnejlvillal: yea, I got that. I'm wondering about the fetch attribute in remote element21:16
mgagneas I'm getting fatal: no fetch in <remote> within21:16
mgagnewhen I put fetch, I can't access the repo, tells me no such repo21:16
jlvillalmgagne: I'm not exactly sure what 'git-repo' does with that21:17
jlvillalmgagne: I'll peek at the source code. I forget what it does.21:17
mgagneok, I'll restore it and see if I can debug the problem from there. maybe removing it is a bad idea21:17
jlvillalmgagne: It can also be like: fetch="ssh://..."  From some internal ones we use here21:19
jlvillalmgagne: git clone https://gerrit.googlesource.com/git-repo21:20
jlvillalAnd there is a doc file which explains the manifest layout21:20
mgagneI'll read there21:20
mgagneI just wish to try the tool and see what it can do. so far, I'm more confused than anything else21:20
jlvillalmgagne: Attribute `name`: A unique name for this project.  The project's21:20
jlvillalname is appended onto its remote's fetch URL to generate the actual21:20
jlvillalURL to configure the Git remote with.21:20
jlvillalmgagne: ${remote_fetch}/${project_name}.git21:21
jlvillalmgagne: Feel free to ask me questions. I'll do by best to answer them!21:21
* jroll sticks to for project in ...; do git clone21:22
jroll:P21:22
mgagnesure. My ambition is bigger than ironic, put all openstack projects I care about in a manifest21:22
jlvillalmgagne: Me too. I just used ironic as an example :)21:22
*** baoli has quit IRC21:22
*** baoli has joined #openstack-ironic21:22
*** thrash is now known as thrash|g0ne21:27
openstackgerritMario Villaplana proposed openstack/ironic-python-agent: Adds more functional tests for commands  https://review.openstack.org/22424721:29
*** baoli has quit IRC21:30
*** olaph has joined #openstack-ironic21:30
*** baoli has joined #openstack-ironic21:32
*** baoli has joined #openstack-ironic21:32
*** harshs has quit IRC21:32
openstackgerritMerged openstack/ironic: Switch tox unit test command to use ostestr  https://review.openstack.org/22810521:36
jlvillaljvmario: Almost there! :)21:37
jvmarioty21:38
jlvillalWell at least from me.21:38
*** e0ne has quit IRC21:38
*** harlowja has quit IRC21:40
*** harlowja has joined #openstack-ironic21:40
jvmariojlvillal: also updated the shared lock ironic patch with comments21:43
jlvillaljvmario: Will look at it21:43
openstackgerritMario Villaplana proposed openstack/ironic-python-agent: Adds more functional tests for commands  https://review.openstack.org/22424721:44
*** Sukhdev has quit IRC21:45
jlvillaljvmario: On unit tests for decorators21:45
*** garthb has quit IRC21:45
jlvillaljvmario: Why not just call it directly? Not as a decorator?21:45
jlvillalor am I missing something????21:46
*** garthb has joined #openstack-ironic21:46
*** amotoki has quit IRC21:46
jvmariojlvillal: that could work, i was trying to use them as decorators before21:46
jlvillalres = vendor_passthru(blah, blah, blah)21:46
jlvillalself.assertBlah(expected, res.blah)21:46
jvmariojlvillal: one point though is that whenever a vendor uses it as a decorator, they will error out, but that way could work too21:47
jlvillaljvmario: They are just functions.21:47
* jvmario tries it21:47
jvmarioi know21:47
jlvillal:)21:47
*** Sukhdev has joined #openstack-ironic21:47
jlvillaljvmario:  As far as unit testing them. Some using them as a decorator as expected. But for tough cases, call them as functions.21:48
jlvillalJust a thought.21:48
NobodyCamjust  puttin it out I really dislike CWT, though I now have travel booked21:50
NobodyCamto tokyo21:51
*** devlaps has joined #openstack-ironic21:52
*** ijw has joined #openstack-ironic21:52
*** digimonde has quit IRC21:53
* jlvillal does not know what CWT is. Maybe he is lucky... ;)21:53
NobodyCamjlvillal: it is the hp travel system21:54
jvmariohttp://www.carlsonwagonlit.com/en/21:54
jlvillalNobodyCam: Ah. We use AmericanExpress.21:54
jlvillalNobodyCam: But in other news. Woo hoo!!!  You are going to the summit! :)21:55
jlvillalI'll see you there.21:55
NobodyCamjvmario: yep thats them21:55
NobodyCamjlvillal: awesome!!!!21:55
*** hemna has quit IRC21:56
*** ijw has quit IRC21:57
*** ijw has joined #openstack-ironic21:57
*** hemna has joined #openstack-ironic22:01
jrollNobodyCam++ for hating CWT22:04
*** hemna has quit IRC22:04
openstackgerritMario Villaplana proposed openstack/ironic: Allow vendor drivers to acquire shared tasks  https://review.openstack.org/22605822:05
NobodyCamtook like over to get the flights... just nuts22:05
jvmarioNobodyCam: did you get a flight into haneda? i was pretty much only able to get narita through that tool unfortunately (it's an hour and a half away)22:07
jvmariostill, very happy to be going :)22:08
NobodyCamjvmario: yep same here. nrt22:08
NobodyCamlooks like a long cab ride22:08
jvmarioNobodyCam: I would highly advise against the cab, it's $200+ to downtown tokyo. i was planning on using the subway system22:08
cineramayou can do that or take the "airport limousine" which is not really a limousine at all22:09
NobodyCamyou are more brave then I...22:09
jvmariobut, if your expense policy is lenient... :)22:09
* jlvillal has not yet figured out how to get to the hotel22:09
jlvillalmgagne: repo init -u https://github.com/JohnVillalovos/git-repo-manifests -m ironic22:10
jlvillalmgagne: repo sync22:10
NobodyCamnice and a travel warning to not go to Fukushima Dai-ichi22:10
NobodyCamlol22:10
jlvillalmgagne: I threw that together pretty quickly22:10
mgagnefetch="https://git.openstack.org/"22:10
mgagneI think that's what I was missing22:10
mgagnecould be the example expect you to have ALL repositories in the same folder, including the repo one22:11
jlvillalmgagne: I'm hoping the 'review' entry will send it to the correct server when uploading a patch :)22:11
mgagnehaha, the android project: sorry, we don't use python here. -222:11
*** boris-42 has joined #openstack-ironic22:12
*** baoli has quit IRC22:12
mgagneI see 404 error at the console. that's confusing22:12
jlvillalmgagne: I don't understand?  The whole git-repo program is written in Python22:12
*** baoli has joined #openstack-ironic22:13
mgagnejlvillal: was jk =)22:13
jlvillalmgagne: Yeah, that seems to be something with clone bundle22:13
jlvillalmgagne: Not sure if I changed fetch to 'review.openstack.org' if it would stop doing that.22:13
mgagnejlvillal: nan, I got the same22:13
mgagnejlvillal: and thought changing it to git.o.o like you suggested would fix it22:13
mgagnejlvillal: it did not22:14
mgagnejlvillal: will try with git.o.o so review.o.o isn't overwhelm22:14
jlvillalmgagne: Good plan :)22:15
*** [1]cdearborn has quit IRC22:22
*** harshs has joined #openstack-ironic22:26
*** jobewan has joined #openstack-ironic22:40
mgagnejlvillal: https://gist.github.com/mgagne/58e06be2fba077b6a3df22:42
jobewanHas anyone used ironic w/ a DHCP helper?22:42
jlvillalmgagne: Cool :)22:42
mgagnejlvillal: some repositories aren't clonable :-/22:43
jlvillalmgagne: Really?22:43
jlvillalStrange22:43
mgagnefatal: https://git.openstack.org/openstack/networking-hpe/info/refs?service=git-upload-pack not found: did you run git update-server-info on the server?22:43
*** baoli has quit IRC22:44
*** baoli has joined #openstack-ironic22:44
jlvillalmgagne: Does a plain 'git clone' work?22:44
*** [1]Rick has quit IRC22:44
jlvillalmgagne: You can also do:  repo --trace CMD22:45
jlvillalTo see the git commands that get executed22:45
mgagnejlvillal: the repository does not exist22:45
mgagnejlvillal: https://github.com/openstack/networking-hpe22:46
jlvillalmgagne: Well that could be a problem ;)  Maybe ask about that project in #openstack-infra22:46
jlvillalIt might need to be removed from projects.yaml22:46
mgagnejlvillal: there is a lot of repo in projects that doesn't exist22:46
*** Sukhdev has quit IRC22:46
jlvillalmgagne: :(22:46
*** amotoki has joined #openstack-ironic22:46
mgagnelike https://github.com/openstack/deb-openstack-pkg-tools22:46
*** amotoki has quit IRC22:51
jrolljobewan: what do you mean by a DHCP helper?23:02
*** Sukhdev has joined #openstack-ironic23:04
jobewanjroll:  I mean as opposed to a dhcp relay, a helper simply sits on the layer3 and forwards broadcast bootp via unicast to a dhcp server on a remote layer 3.23:04
jrollah23:05
jobewanit doesn't "hold" any mac or state data like a relay23:05
jrollI don't see why that wouldn't work, as long as it's pointed at the neutron dhcp thing23:05
jobewanit seems that I could get the discovery image to boot, but it didn't have any network connectivity after boot up23:06
jrolldid you include the dhcp-all-interfaces element in that image? (assuming disk-image-builder)23:06
openstackgerritJohn L. Villalovos proposed openstack/ironic: testing  https://review.openstack.org/22917923:06
jrollit might be dhcp'ing on the wrong interface or something23:06
mgagnealright, got it to work by querying gerrit for list of projects. The result: https://gist.github.com/mgagne/29f97326bf837e984be923:06
jlvillalmgagne: ^^^ My patch uploaded with 'repo upload' :)23:06
jrolljlvillal: +2+A23:07
jobewanwell, it gets an IP - but lays down the gateway incorrectly23:07
jlvillalLOL23:07
jrolljobewan: hmm. no clue. might be a question for neutron folks? or someone that knows more than me, at least :)23:07
jobewanif I hop into the node's console and manually apply the gateway, it begins to work23:07
jlvillalmgagne: All I can say is you got a lot of programs you work on ;)23:08
jrollyeah, that's interesting23:08
mgagnejlvillal: more often, I read the code =)23:08
jlvillalmgagne: As a note, I am a bad person and forced push to my repo. In case you were using it.23:09
mgagnejlvillal: not using it =)23:10
mgagnejlvillal: hacking my script23:10
jlvillalwhew ;)23:10
*** harshs has quit IRC23:12
*** jobewan has quit IRC23:13
jlvillalmgagne: One note on projects that don't exist. They may actually be in 'stackforge/*' instead of 'openstack/*'23:15
jlvillalI've had a few projects like that in the past.23:15
mgagnejlvillal: right but still, some don't exist at all23:16
jlvillalOkay23:16
*** baoli has quit IRC23:16
*** ijw has quit IRC23:18
*** dims_ has joined #openstack-ironic23:19
*** Sukhdev has quit IRC23:21
*** dims has quit IRC23:22
*** sdake_ has quit IRC23:30
*** sdake has joined #openstack-ironic23:30
*** mdbooth_ has joined #openstack-ironic23:31
*** dims_ has quit IRC23:31
*** dims_ has joined #openstack-ironic23:32
*** sambetts_ has joined #openstack-ironic23:32
*** mdbooth has quit IRC23:32
*** mrda has quit IRC23:32
*** sambetts has quit IRC23:32
*** johnthetubaguy has quit IRC23:32
*** tristanC has quit IRC23:32
*** mdbooth_ is now known as mdbooth23:32
*** trown|outttypeww has quit IRC23:33
*** sinval has quit IRC23:33
*** tristanC has joined #openstack-ironic23:33
mgagnejlvillal: updated version with repo validating against gerrit: https://gist.github.com/mgagne/58e06be2fba077b6a3df23:33
*** vdrok has quit IRC23:33
*** keekz has quit IRC23:33
*** sinval has joined #openstack-ironic23:33
*** harshs has joined #openstack-ironic23:33
*** vdrok has joined #openstack-ironic23:34
jlvillalmgagne: I like it. If the project gets approved maybe we can add things like that?23:34
*** johnthetubaguy has joined #openstack-ironic23:34
*** keekz has joined #openstack-ironic23:34
mgagnejlvillal: sure, feel free to use the script. I'm just playing with repo to see if it can make my life easier23:34
*** trown has joined #openstack-ironic23:35
mgagnejlvillal: would be great to be able to switch to stable branches too. I'm not sure how projects without stable branches would react23:35
mgagnejlvillal: would be something to add to the script I guess so it checks if the project has the release:has-stable-branches tag23:36
jlvillalmgagne: Sounds reasonable23:36
mgagneanyway, it's just a poc =)23:37
jlvillalmgagne: Looks like you have made quick progress23:39
mgagneI've got all the projects organized to my taste so I'm satisfied23:40
*** garthb has quit IRC23:44
*** sdake has quit IRC23:44
*** achanda_ has joined #openstack-ironic23:44
*** sdake_ has joined #openstack-ironic23:44
*** achanda_ has quit IRC23:45
*** sdake_ is now known as sdake23:45
*** achanda has quit IRC23:48
*** saripurigopi has joined #openstack-ironic23:52
openstackgerritMerged openstack/ironic: Fix some iBoot strings  https://review.openstack.org/22862923:54
*** Sukhdev has joined #openstack-ironic23:54

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