Tuesday, 2016-05-10

*** Sukhdev has quit IRC00:30
*** rloo has quit IRC00:31
*** Sukhdev has joined #openstack-ironic00:31
*** Goneri has joined #openstack-ironic00:32
*** Sukhdev has quit IRC00:37
*** Goneri has quit IRC00:39
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/31372900:44
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/31435300:44
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/31435400:44
*** mbound has joined #openstack-ironic00:47
*** mbound has quit IRC00:52
*** hoangcx has joined #openstack-ironic00:59
*** wajdi has joined #openstack-ironic01:00
*** baoli has quit IRC01:16
*** baoli has joined #openstack-ironic01:16
*** baoli has quit IRC01:22
*** baoli has joined #openstack-ironic01:34
*** sacharya has joined #openstack-ironic01:39
*** sacharya has quit IRC01:44
*** baoli has quit IRC01:48
*** baoli has joined #openstack-ironic01:48
*** sacharya has joined #openstack-ironic02:02
*** baoli has quit IRC02:03
*** spandhe has joined #openstack-ironic02:06
*** ijw_ has quit IRC02:12
*** spandhe_ has joined #openstack-ironic02:16
*** ayoung has quit IRC02:17
*** spandhe has quit IRC02:18
*** spandhe_ is now known as spandhe02:18
*** amotoki has quit IRC02:25
*** spandhe_ has joined #openstack-ironic02:35
*** spandhe has quit IRC02:35
*** spandhe_ is now known as spandhe02:35
*** rajinir has joined #openstack-ironic02:44
*** yuanying has quit IRC02:51
*** amotoki has joined #openstack-ironic02:52
*** amotoki has quit IRC03:00
*** stendulker has joined #openstack-ironic03:04
*** Sukhdev has joined #openstack-ironic03:05
*** amotoki has joined #openstack-ironic03:10
*** moshele has joined #openstack-ironic03:26
*** links has joined #openstack-ironic03:29
*** amotoki has quit IRC03:31
*** amotoki has joined #openstack-ironic03:39
*** ayoung has joined #openstack-ironic03:42
*** yuanying has joined #openstack-ironic03:48
*** irf has joined #openstack-ironic03:53
*** sirushti has quit IRC03:57
*** sirushti has joined #openstack-ironic04:06
*** appu103_ has joined #openstack-ironic04:08
*** stendulker_ has joined #openstack-ironic04:19
*** stendulker has quit IRC04:23
*** stendulker has joined #openstack-ironic04:27
*** stendulker_ has quit IRC04:31
*** jiangfei has joined #openstack-ironic04:42
*** rajinir has quit IRC04:46
*** Nisha_away has joined #openstack-ironic04:52
*** fragatina has quit IRC04:59
*** Sukhdev has quit IRC05:01
openstackgerritShivanand Tendulker proposed openstack/proliantutils: Add RIS support for power operations  https://review.openstack.org/28980005:04
*** vmud213 has joined #openstack-ironic05:06
*** fragatina has joined #openstack-ironic05:17
*** fragatina has quit IRC05:22
*** appu103_ is now known as aparnav05:29
*** itamarl has joined #openstack-ironic05:31
openstackgerritZhenguo Niu proposed openstack/ironic-inspector: Place db and cmd files to seperate directories  https://review.openstack.org/31286205:32
*** fragatina has joined #openstack-ironic05:33
*** fragatina has quit IRC05:33
*** fragatina has joined #openstack-ironic05:34
*** moshele has quit IRC05:39
*** ChubYann has quit IRC05:40
openstackgerritAparna proposed openstack/ironic: Deployment vmedia operations to run when cleaning  https://review.openstack.org/30722505:43
*** d0ugal has joined #openstack-ironic05:52
*** itamarl has quit IRC05:53
*** vsaienko has quit IRC05:58
*** spandhe has quit IRC06:04
*** itamarl has joined #openstack-ironic06:06
*** amotoki has quit IRC06:10
*** Nisha_away has quit IRC06:11
*** Nisha has joined #openstack-ironic06:25
*** itamarl has quit IRC06:26
*** rcernin has joined #openstack-ironic06:27
*** vsaienko has joined #openstack-ironic06:36
*** moshele has joined #openstack-ironic06:41
*** moshele has quit IRC06:42
*** ChrisAusten has quit IRC06:42
*** moshele has joined #openstack-ironic06:42
openstackgerritVasyl Saienko proposed openstack/ironic: TEST Checking Gates  https://review.openstack.org/31110906:43
*** itamarl has joined #openstack-ironic06:45
*** sacharya has quit IRC06:53
*** amotoki has joined #openstack-ironic06:59
*** tesseract has joined #openstack-ironic07:03
*** d0ugal has quit IRC07:23
pas-hamorning Ironicers07:25
*** lennyb has quit IRC07:25
*** d0ugal has joined #openstack-ironic07:26
*** d0ugal has quit IRC07:26
*** d0ugal has joined #openstack-ironic07:26
lazy_princeHi.. I am running tox -e py27 for ironic-python-agent but it keeps failing ironic_python_agent.tests.unit.test_hardware.TestGenericHardwareManager.test_get_os_install_device with error about pyudev.device._errors.DeviceNotFoundByFileError: [Errno 2] No such file or directory: '/dev/sdb'07:31
lazy_princeis there anything that I might be missing..?07:32
*** lennyb has joined #openstack-ironic07:34
*** irf has quit IRC07:40
*** irf has joined #openstack-ironic07:40
*** ifarkas has joined #openstack-ironic07:43
*** sacharya has joined #openstack-ironic07:54
*** sacharya has quit IRC07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** irf has quit IRC08:02
openstackgerritVasyl Saienko proposed openstack/ironic: TEST Checking Gates  https://review.openstack.org/31110908:11
*** ohamada has joined #openstack-ironic08:12
*** irf has joined #openstack-ironic08:14
*** athomas has joined #openstack-ironic08:14
*** aparnav has quit IRC08:14
*** appu103_ has joined #openstack-ironic08:14
*** dtantsur|afk is now known as dtantsur08:16
dtantsurMorning Ironic08:16
dtantsurlazy_prince, from git master?08:16
*** e0ne has joined #openstack-ironic08:18
irfmorning Ironic !08:22
*** lucas-dinner is now known as lucasagomes08:22
lucasagomesmorning all08:22
*** Nisha_away has joined #openstack-ironic08:28
*** Nisha has quit IRC08:28
*** daemontool__ is now known as daemontool08:30
irfas continue to the issue, "No valid host was found", i am still facing the issue ...08:31
irftheer are still issues nova is not communicating well with Ironic or vice versa ...08:31
irfdo we require nova.conf and nova-compute files both ?08:35
dtantsurlucasagomes, mind looking at https://review.openstack.org/#/c/313511/ again please? now it must be less controversial (even though I like it less)08:36
lucasagomesdtantsur, hi there, sure thing08:37
lucasagomes1 sec08:37
*** jistr has joined #openstack-ironic08:37
dtantsurno worries.. just want it landing reasonably soon, I have a guy blocked on it downstream08:37
lucasagomesirf, you want to have 1 compute per type right? Like one for baremetal and another for virtual?08:37
lucasagomesirf, so yes, you need another nova.conf (that's where the driver is specified, plus you need to tweak things like ram_ratio for the baremetal one)08:38
*** Nisha_away has quit IRC08:39
*** Nisha_away has joined #openstack-ironic08:39
irfyes i have one compute per type ...08:40
irfon the controller node , i have set the nova compute service for baremetal ...08:41
irflet me paste the contents of both the file , nova.conf and nova-compute.conf08:42
*** amotoki has quit IRC08:42
*** amotoki has joined #openstack-ironic08:43
*** Nisha_away has quit IRC08:47
irfhttp://pastebin.com/DhEN0rn508:47
irfplease review ...08:47
irfthis nova compute service i am using for baremetal08:47
*** derekh has joined #openstack-ironic08:47
stendulkerdtantsur, lucasagomes: Can you please have a look at config drive related review https://review.openstack.org/#/c/230924/08:50
dtantsurstendulker, it's pretty big, but I'll try to fit it into today's schedule08:52
dtantsurthanks08:52
lucasagomesstendulker, ++ on my list as well08:52
stendulkerdtantsur, lucasagomes: thank you08:52
odyssey4memrda thanks - it seems clear to me that the ironicclient needs some love then cc jroll08:53
dtantsur+100 to ironicclient needs some love, even though I don't know the context :)08:53
lucasagomessambetts, re your comment https://review.openstack.org/#/c/314149/2 that's the bug, pecan/WSME gets confused when routing the request08:57
lucasagomesi totally agree that v1/nodes/detail/detail should be fine, but it's not due the technology we use08:57
*** sacharya has joined #openstack-ironic09:00
*** ppiela has quit IRC09:03
*** Nisha has joined #openstack-ironic09:03
*** ppiela has joined #openstack-ironic09:03
sambettslucasagomes: woah?! really...09:04
lucasagomessambetts, yeah, check the bug out09:04
*** sacharya has quit IRC09:04
lucasagomesthat's why I linked it to the bug cause there's info there that would be hard to summarize in a comment09:05
lucasagomessambetts, and yes, pecan/wsme kinda sucks :-)09:05
*** itamarl_ has joined #openstack-ironic09:06
sambettslucasagomes: I didn't read the bug as such, I read the bug as the reason v1/nodes/detail/detail is invalid is because we've created the v1/nodes/detail endpoint and therefore underminded wsmes routing09:07
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: Add unittests for the utils.py module  https://review.openstack.org/31419709:07
lucasagomessambetts, that's it really... Maybe I missed something in your comment on the patch09:08
* lucasagomes re-reads09:08
*** itamarl has quit IRC09:08
*** itamarl_ is now known as itamarl09:09
lucasagomessambetts, right, I agree with the comment... but not sure what actions you want on that? To update the note I left inline about new sub-controllers?09:10
sambettslucasagomes: I think the comment needs to be that if we add any further actions to the nodes controller like detail then it needs to be done using query, I think new sub-controllers/resources should be fine to continue as normal unless they also expose a detail like extension then that should be a query WDYT?09:13
sambettse.g. If I add a new drivers resource then I should be fine to have /v1/nodes/<uuid or name>/drivers09:13
sambettsthen I need to do v1/nodes/<uuid or name>/drivers?detail=true if I want more details09:14
sambettsand then I can have v1/nodes/<uuid or name/drivers/<uuid or name>09:14
sambettswith no conflicts09:15
lucasagomesright, the problem will come if someone name a node "drivers" in this case09:15
sambettsthere isn't because we don't define a v1/nodes/drivers endpoint yet09:15
sambettsso there isn't an existing endpoint to confused wsme09:16
irfplease review09:16
lucasagomessambetts, right, lemme check that... cause validate is the same story right? We do have v1/nodes/<uuid>/validate09:16
lucasagomesbut not v1/nodes/validate09:16
sambettsyeah, validate shouldn't get confused if you have a noded called validate09:17
sambettsit'd just  be v1/nodes/validate/validate09:17
lucasagomesright on! yeah I will update that comment to reflect it09:20
lazy_prince@dtantsur: from stable/liberty09:23
sambettslucasagomes: with this in mind I wonder if the current reserve words cover to many things? Do we have v1/nodes/ports?09:23
lucasagomessambetts, nop only nodes/<ident>/ports09:24
lucasagomesyeah, it may be covering more than needed09:24
dtantsurlazy_prince, hmm, maybe we didn't backport the fix... there was a breaking change in pyudev09:25
dtantsurI'll check in a few09:25
lazy_princedtantsur: thanks for looking into it..09:25
dtantsurweird, https://review.openstack.org/#/c/254612/ looks in liberty...09:26
dtantsurlazy_prince, does you checkout have this patch applied ^^^?09:27
sambettslucasagomes: looking at the ref, only detail and validate are the ones we need to watch out for09:27
*** milan has quit IRC09:28
*** daemontool has quit IRC09:28
lucasagomessambetts, well, there's a bunch of comments on that patch09:29
lucasagomesI will have to verify each situations to see09:29
lucasagomesapparently nodes/ports will get routed to the ports controller anyway09:30
lucasagomeseven tho we only suport nodes/<uuid>/ports09:30
*** chlong has quit IRC09:30
sambetts:/ sounds like we added a fall back route somewhere :/09:30
lazy_princedtantsur: let me check09:31
lucasagomessambetts, http://paste.openstack.org/show/496542/09:31
sambettshmmm ...09:32
sambettsthat sounds like a real bug in our API to me09:32
lucasagomesit is09:33
lucasagomesthat's the thing, just the way things get routed is problematic09:33
sambettslucasagomes: I think if we don't support v1/nodes/ports we should fix that in the code itself09:34
*** mgould has joined #openstack-ironic09:34
lucasagomessambetts, so that's the pecan/wsme thingy that we need to fix09:34
lucasagomesbut, clearly we are avoiding it :-/09:35
irfguys , please help ... i am tsruggling lot but no luck09:35
sambettslucasagomes: I'm looking at our controllers now, and I think its our fault09:36
mrdaodyssey4me: that's still my problem then :)09:37
sambettsirf: are you passing both those files into your nova compute service?09:37
sambettsbecause one says one compute_driver and one says a different compute_driver09:37
sambettsand infact the ironic virt driver is commented out09:37
lucasagomessambetts, so idk, I need to check the api code and see if we can avoid that09:42
lucasagomesperhaps using custom_actions instead of subcontrollers or things like that09:42
mgouldmorning Ironic!09:43
sambettslucasagomes: ok so our problem is that the ports controller has been added as a sub-controller of Nodes, but the ports controller hasn't been written as a sub-controller and its a complete fudge...09:45
lucasagomessambetts, right, that's what we thought that was the wsme magic, making it easy, reuse the same controller09:46
lucasagomesperhaps we should not do that :-/09:47
sambettslucasagomes: so I think you can reuse the same controller, however I think there is magic that looks at the number of arguments a function can take09:47
*** Nisha has quit IRC09:47
sambettslucasagomes: e.g. ports get_one current takes self, port_id and fields, but to be a sub resource it needs to take self, node_id, port_id, field=None09:48
sambettslucasagomes: looking at these controllers makes me shiver ...09:50
*** Nisha has joined #openstack-ironic09:50
lucasagomessambetts, heh I know09:50
vdrokmorning ironic, lucasagomes dtantsur sambetts and mgould !09:51
lucasagomesvdrok, hi there, good morning09:51
sambettso/ mgould09:51
lucasagomessambetts, maybe we should put a patch up changing one bit of it, say ports09:51
lucasagomesand if viable we can re-structure the api09:51
sambettsyeah, I think that might be worth doing09:52
lucasagomessambetts, right, should we hold-on on the follow up aptch then you think?09:53
mgouldvdrok, lucasagomes, sambetts dtantsur morning!09:53
vdrokanother problem with controllers - ports are subcontroller for both nodes and portgroups, meaning that either node_id or port_id can get into first function argument09:53
vdroks/port_id/portgroup_id09:54
dtantsurmorning mgould, vdrok09:54
lucasagomesvdrok, :-/09:55
irfok.. so what is the correct value for compute_driver ?10:01
*** milan has joined #openstack-ironic10:01
irfcompute_driver=nova.virt.ironic.IronicDriver10:02
irfis this correct ?10:02
*** daemontool has joined #openstack-ironic10:08
sambettsyes10:08
irfok10:08
irfnow another doubt is , what is the difference between nova.conf and nova-compute.conf file ? are they both really required OR it is just simply i messed up the things ...10:09
openstackgerritNisha Agarwal proposed openstack/ironic: Extend IRONIC_RAMDISK_TYPE to support 'dib'  https://review.openstack.org/30240310:11
mgouldquestion about Nisha's patch above: if IRONIC_RAMDISK_TYPE is dib, is the ramdisk image build on demand?10:16
lucasagomessomeone wants to take a quick peek at https://review.openstack.org/#/c/314197/ ? (it's just unittests for vbmc)10:17
mgouldlucasagomes, looking now10:18
*** Nisha_away has joined #openstack-ironic10:18
lucasagomesmgould, thanks, it currently has no unittests what so ever, that's the first ones10:18
mgouldmorning Nisha10:18
Nisha_awaymgould, morning10:19
lucasagomesso there's some files being moved around etc10:19
vmud213morning all10:26
irfi made the same entries in both the files ...10:26
irfcompute_driver=nova.virt.ironic.IronicDriver10:26
mgouldvmud213, morning10:26
vmud213mgould o/10:27
Nishamgould, u can build on demand if u set IRONIC_BUILD_DEPLOY_RAMDISK as true10:27
Nishamgould, or if these variables are not set it will build by itself10:28
irfcompute_driver=nova.virt.ironic.IronicDriver10:28
mgouldNisha, thanks10:28
Nishafor me i have tested the second way for building thru devstack10:28
vmud213Hello all, Can you review https://review.openstack.org/#/c/303503/10:29
NishaI dont remember using IRONIC_BUILD_DEPLOY_RAMDISK option till now in CI10:30
openstackgerritMerged openstack/python-ironicclient: Fix typo in docstring of assertTableHeaders method  https://review.openstack.org/31001710:34
mgouldlucasagomes, done10:35
lucasagomesthanks a lot!10:36
*** hoangcx has quit IRC10:37
irfeven after uncommenting compute_driver ,, error still remains same10:37
Nishahas anyone built the ramdisk successfully using these options in DIB?10:37
Nishasudo -E disk-image-create ubuntu-minimal simple-init ironic-agent iso -p openssh-server -o ubuntu-minimal-ramdisk10:37
Nishai am trying this as part of comment on patch https://review.openstack.org/#/c/264579/ by devananda10:38
Nishafor me it fails as "+ sudo sh -c 'http_proxy=http://web-proxy.ind.hp.com:8080 debootstrap --verbose         --variant=minbase         --include=python,sudo,linux-image-generic         --components=main,restricted,universe         --arch=amd64                           trusty         /tmp/image.3JCwN7HU/mnt         http://archive.ubuntu.com/ubuntu         '10:38
Nishash: 1: debootstrap: not found10:38
NishaUnmount /tmp/image.3JCwN7HU/mnt/tmp/ccache10:38
Nisha"10:38
Nishalucasagomes, dtantsur mgould ^^^^ any idea on above10:39
dtantsurNisha, you're asking 3 Red Hat guys about how often we build images with ubuntu? ;)10:40
Nishadtantsur, :) i didnt know all are Redhat guys10:40
dtantsurnothing strikes me about your command line10:40
dtantsuryou may want to ask on #tripleo10:40
dtantsurFWIW I'd prefer we do the same as we recommend in the docs10:41
Nishadtantsur, i would add this on the patch and will let devananda come back as of now10:42
*** MattMan has quit IRC10:42
*** MattMan has joined #openstack-ironic10:42
Nishadtantsur, ++ to what u said10:42
openstackgerritMerged openstack/virtualbmc: Add unittests for the utils.py module  https://review.openstack.org/31419710:43
mgouldNisha, I've never tried those options, sorry10:44
Nishamgould, thanks.10:44
mgouldNisha: dtantsur lucasagomes ifarkas and I are all Red Hatters10:45
Nishamgould, i knew dtantsur lucasagomes are from redHat10:46
lucasagomesNisha, and milan :-)10:46
* mgould still has Ubuntu installed on most of his personal machines: I only started working at RH a few months ago :-)10:46
Nishai came to know about you now10:46
mgouldah, milan, I was wondering why mkovTABTABTAB wasn't working :-)10:46
lucasagomeslol10:46
dtantsurmkovTABTABTAB is a great nick fwiw10:46
mgould:-)10:46
ifarkas+100!10:47
mgouldNisha, it looks like you have to install debootstrap on your host machine10:48
mgouldit's a cross-compiler10:48
mgould[cross-builder? you know what I mean]10:48
Nishamgould, no ...10:48
mgouldNisha, from elements/debootstrap/README.rst "Use of this element will also require the tool 'debootstrap' to be available on your system."10:49
mgouldhttps://wiki.debian.org/Debootstrap10:49
Nishamgould, ok i will try10:50
mgouldanyway, it's a tool for installing a Debian base system onto *another* filesystem10:50
mgouldwhich means it needs to exist on the machine building the image10:50
mgould"cross-compiler" == compiler running on one architecture, building binaries for a different architecture, eg gcc running on x86 but outputting ARM binaries10:51
mgouldnot a great analogy, sorry10:51
openstackgerritAparna proposed openstack/ironic: Deployment vmedia operations to run when cleaning  https://review.openstack.org/30722510:51
*** Nisha_away has quit IRC10:54
*** dtantsur is now known as dtantsur|bbl10:55
*** Nisha has quit IRC10:55
milanmorning guys! :)10:58
milanmgould, yeah, I've just changed that ;)10:58
milanlucasagomes, hi :)10:58
lucasagomesmorning10:58
sambettslucasagomes: I think I've got a solution :)10:59
lucasagomessambetts, cool10:59
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Update devstack plugin with new auth options  https://review.openstack.org/29731210:59
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Migrate to using keystoneauth Sessions  https://review.openstack.org/23698210:59
mgouldmilan, https://www.youtube.com/watch?v=rERG3x6vsGY11:00
*** sacharya has joined #openstack-ironic11:01
milanmgould, lol :D11:01
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Update devstack plugin with new auth options  https://review.openstack.org/29731211:02
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Migrate to using keystoneauth Sessions  https://review.openstack.org/23698211:02
*** sacharya has quit IRC11:05
*** stendulker has quit IRC11:06
jrollnisha needs debootstrap installed is all :|11:08
jrollmorning11:09
mgouldmorning jroll11:09
* mgould does the "I was right" dance11:09
jrolloh, I didn't read far enough heh11:09
sambettsmorning jroll11:09
mgouldjroll, np, it's great to have my suspicions confirmed :-)11:09
* sambetts is solving all the API issues ;) 11:10
sambetts(rm -rf ironic/api) ;) haha11:10
mgouldheh11:10
jrollitym sed -i -e 's/pecan/flask/g' ironic/api/11:17
sambettsnahhh, /me likes pecan hehe, we just need to use it properly11:18
*** dims has quit IRC11:20
*** dims has joined #openstack-ironic11:25
*** mgould is now known as mgould|afk11:26
openstackgerritSam Betts proposed openstack/ironic: Prevent URL collisions with sub-controllers: nodes/ports  https://review.openstack.org/31451411:30
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: Add unittests for the cmd/vbmc.py module  https://review.openstack.org/31451511:32
*** appu103_ has quit IRC11:32
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: "vbmc list" to sort the result by domain name  https://review.openstack.org/31409511:34
TheJuliao/ just wanted to say hi folks, trying to rest and hea11:34
lucasagomesTheJulia, hello :-)11:34
TheJuliarest and heal, brain hasn't had coffee in days11:34
sambettsTheJulia: :(11:35
*** thrash|g0ne is now known as thrash11:35
TheJuliasambetts: smiley faces, I'm smiling, as much as I can at the moment.  Anyway, I think I'm going to give up on the nurse who is snoring super loudly and go hunt down some juice... although she promised to take me to real coffee today11:37
sambettsTheJulia: the nurse is snoring?!11:37
TheJuliaYeah, my nurse apparently has had a few long shifts recently,  I'm doing fairly good so I'm being nice to her :)11:38
sambetts:)11:39
mat128morning everyone o/11:50
*** baoli has joined #openstack-ironic11:50
*** lucasagomes is now known as lucas-hungry11:52
*** baoli_ has joined #openstack-ironic11:53
vdrokmorning jroll TheJulia and mat12811:53
vdrokhey milan, what happened to mkovacik?11:53
milanvdrok, morning :D11:54
milanyeah, I thought I'd change that11:54
vdrok:)11:54
milanmgould|afk things milan is too sexy though ;)11:54
vdrok"I'm too sexy for my nick" then :)11:55
milan:D11:55
*** baoli has quit IRC11:56
* milan was know as mkovTABTABTAB for a while too11:56
openstackgerritSam Betts proposed openstack/ironic: WIP: Prevent URL collisions with sub-controllers: nodes/ports  https://review.openstack.org/31451411:57
*** jtomasek_ has joined #openstack-ironic11:58
irfin the conductor logs i am getting the same error :12:02
irfhttp://pastebin.com/xuU1GZU312:02
*** jcoufal has joined #openstack-ironic12:04
irfplease suggest12:05
openstackgerritImre Farkas proposed openstack/ironic: [WIP] DRAC RAID configuration  https://review.openstack.org/17631912:07
*** jtomasek_ has quit IRC12:08
jrollirf: you might try the openstack users mailing list (http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack) or https://ask.openstack.org/en/questions/12:10
*** dprince has joined #openstack-ironic12:16
*** kromanenko_ has joined #openstack-ironic12:17
*** rbudden has joined #openstack-ironic12:18
*** jjohnson2 has joined #openstack-ironic12:19
milansambetts, hi, I've got a question about Inspector API mod patch, you've got a minute? https://review.openstack.org/#/c/308379/10..12/ironic_inspector/main.py12:20
sambettsmilan: sure, 1 minute12:20
*** jcoufal has quit IRC12:21
*** jcoufal has joined #openstack-ironic12:22
milansambetts, :)12:22
* milan wonders should we have a GET method when we allow POST with data here?12:23
*** xavierr has joined #openstack-ironic12:23
milanthe reapply part12:23
vdrokjroll: seems that we need to use ClusteredComputeManager in grenade job to be able to run tests in 3 vms12:23
milan* to GET the (raw) image data12:24
milansambetts, ^12:24
vdrokbecause otherwise hypervisor resources are not being updated in time12:24
vdrokafter instance deletion12:24
sambettsmilan: I think we discussed being able to fetch the raw data12:24
jrollvdrok: are you sure it isn't because they're busy cleaning?12:24
vdrokjroll: I changed that, it waits for cleaning to finish now12:25
milansambetts, that might be true but I've forgotten the outcome here, sry12:25
sambettsmilan: GET ..../data/unprocessed -> return raw ramdisk data12:25
vdrokand I still see 0s in hypervisor-show12:25
sambettsI think we said that it made sense12:25
sambettsmilan: ^12:25
jrollvdrok: maybe we sleep 30 seconds or something to wait for it12:25
milansambetts, yeah, I had the same impression12:26
jrollvdrok: or fix the actual computemanager? CCM is bad :/12:26
milansambetts, thx. I'll -1 it again12:26
vdrokjroll: well, adding time.sleep(30) in all the tests using >1 node is worse :)12:26
jrollvdrok: right, why don't we make the in-tree ComputeManager do the right thing?12:27
vdrokjroll: yep, I think we should, IIRC you wanter to get rid of CCM completely?12:27
vdrokin our tree12:28
jrollvdrok: yes, very much so. it's a hack to make two computes look like one12:28
vdrokjroll: I think for now I'll switch to CCM, to further do the smoke stuff, then will try to look into how to fix the compute manager12:30
jrollvdrok: sure, I'd just rather not have that in the gate12:30
vdrokjroll: yep, completely agree, that will be too much new stuff in one job - tinyipa, smoke and CCM :)12:31
jrollheh12:31
xavierrgood morning ironic :)12:36
*** irf has quit IRC12:36
*** dtantsur|bbl is now known as dtantsur12:37
dtantsurmorning jroll, TheJulia, mat12812:38
dtantsurand xavierr12:38
vdrokmorning xavierr !12:40
mat128morning dtantsur xavierr12:40
vsaienkomorning dtansur, jroll, vdrok and all!12:40
vdrokvsaienko: o/12:41
openstackgerritSam Betts proposed openstack/ironic: Prevent URL collisions with sub-controllers: nodes/ports  https://review.openstack.org/31451412:42
*** wajdi has quit IRC12:42
sambettslucas-hungry: try that out for size ^^12:42
sambettslucas-hungry: "ironic node-create --driver fake --name ports" then "ironic node-show ports" works12:43
xavierrmorning dtantsur vdrok mat12812:43
xavierr:)12:43
mat128sambetts: that review allows for reserved names as node names?12:43
mat128haha clever12:44
sambettsmat128: it allows "ports" as a node name, there are still a couple of others that need reworking and unfortunatly we can never allow "detail" as a node name :'(12:44
mat128I was even fine with the original limitation, so this is icing on the cake :)12:45
sambettsmat128: but IMO the ones that were breaking like v1/nodes/ports/ports was a bug that needed squashing12:45
sambettsif people are happy with that approach then I'll put together the rest of the patches12:46
*** lucas-hungry is now known as lucasagomes12:51
*** mjturek1 has joined #openstack-ironic12:52
lucasagomessambetts, that's really cool!12:52
sambettsas I said pecan can do it we just have to use it right :-P12:53
*** mjturek1 has quit IRC12:56
*** mbound has joined #openstack-ironic12:57
*** amotoki has quit IRC13:01
*** sacharya has joined #openstack-ironic13:02
*** Goneri has joined #openstack-ironic13:02
jrollvdrok: thinking about it more, it doesn't sound insane to wait for hypervisor-stats to show >0 before proceeding13:02
*** keedya has joined #openstack-ironic13:02
*** mbound has quit IRC13:02
*** mgould|afk is now known as mgould13:03
sambettsmat128: which call broke when you tried my patch? I'll try to work out if its in the code or your machine13:04
mgouldmorning TheJulia mat12813:05
mat128test_nodes.TestPost.test_post_ports_subresource, test_nodes.TestPatch.test_patch_ports_subresource, test_nodes.TestListNodes.test_ports_subresource_noid, test_nodes.TestDelete.test_delete_ports_subresource13:05
mat128Not saying it broke any of the real api calls, just unit tests (but that probably means something broke)13:05
sambettsoh right the tests, yeah I didn't fix any of those yet :-P13:06
mat128MismatchError: 403 != 40013:06
mat128Ahh13:06
*** vmud213 has quit IRC13:06
mat128ok good to hear, then your patch works for it's intended purpose, I changed "spam" for "ports" in a get test and it worked a1 :D13:06
sambettswanted consensus on the approach before I fell down the test rabbit hole13:06
*** sacharya has quit IRC13:06
sambettsawesome :D I've tried, ironic node-port-list ports, and it works, so it likely that it might have changed an abort code somewhere or something, because there were some conditionals that I was able to remove because of the changes I made13:08
*** mjturek1 has joined #openstack-ironic13:10
*** mjturek1 has quit IRC13:10
*** mjturek1 has joined #openstack-ironic13:11
openstackgerritMerged openstack/ironic: Cleanup unused conf variables  https://review.openstack.org/28049313:15
*** afaranha has joined #openstack-ironic13:15
openstackgerritMerged openstack/virtualbmc: Add unittests for the cmd/vbmc.py module  https://review.openstack.org/31451513:17
openstackgerritMerged openstack/virtualbmc: "vbmc list" to sort the result by domain name  https://review.openstack.org/31409513:17
*** daemontool_ has joined #openstack-ironic13:21
*** links has quit IRC13:23
openstackgerritAndre keedy proposed openstack/ironic-inspector: Replace node data via uuid/data/unprocessed API  https://review.openstack.org/30837913:23
*** cdearborn has joined #openstack-ironic13:23
*** daemontool has quit IRC13:24
*** mgould has quit IRC13:29
*** hshiina has joined #openstack-ironic13:31
*** mgould has joined #openstack-ironic13:31
*** ChrisAusten has joined #openstack-ironic13:32
*** ChrisAusten has quit IRC13:35
*** ChrisAusten has joined #openstack-ironic13:35
jrolldtantsur: hey, a todo popped up on my list reminding me to send an email about dropping the bash ramdisk - would you mind doing that?13:38
*** rloo has joined #openstack-ironic13:43
jrolldtantsur: also, just saw the last kilo release go out :D13:44
dtantsurjroll, \o/13:44
dtantsuryeah, will do, gimme a few minutes - I'm on a meeting13:45
jrolldtantsur: awesome, thank you. I'm around if you want help with words or dates or whatever13:45
sambettstrying to shrink tinyipa more :( 25mb of tinyipa is libicu... a unicode library requirement imported from gdisk ... and then like 49mb is python deps13:47
*** wajdi has joined #openstack-ironic13:50
wajdigood morning all13:51
*** thiagop has joined #openstack-ironic13:51
thiagopGood morning, Ironic13:51
dtantsurmorning wajdi, thiagop13:51
thiagopdtantsur: o/13:52
openstackgerritAndreas Jaeger proposed openstack/pyghmi: Remove unused pngmath Sphinx extension  https://review.openstack.org/28621713:56
*** ametts has joined #openstack-ironic13:58
*** hoangcx_ has joined #openstack-ironic13:59
NobodyCamMorning ironic folks, says the man with out internets :-P14:01
sambettsNobodyCam: Morning14:01
*** amotoki has joined #openstack-ironic14:01
NobodyCamMorning sambetts14:01
*** causten_ has joined #openstack-ironic14:01
dtantsurmorning NobodyCam; internets are so-so today, no worries :D14:01
*** ChrisAusten has quit IRC14:02
*** hoangcx_ is now known as hoangcx14:02
NobodyCamLol my provider (time Warner)  is down in the entire neighborhood14:02
*** rloo has quit IRC14:02
NobodyCamMorning dtantsur14:02
*** rloo has joined #openstack-ironic14:02
*** causten__ has joined #openstack-ironic14:03
NobodyCamKinda thinking it might be a pto day if things don't get fixed soon ish:-P14:03
jrollNobodyCam: but what would you do with a day off and no internet? :P14:04
jrollmorning :)14:04
*** mtanino has joined #openstack-ironic14:04
NobodyCamMorning jroll probably go pull tvs out of the old rv and move them into the new one... lol14:06
NobodyCam:-)14:06
*** causten_ has quit IRC14:06
jrollheh14:07
*** hoangcx has quit IRC14:08
openstackgerritSam Betts proposed openstack/ironic-python-agent: TinyIPA: Shave off some file size from tinyipa ramdisk  https://review.openstack.org/31367514:09
rloomorning everyone.14:20
dtantsurmorning rloo14:20
*** rajinir has joined #openstack-ironic14:20
dtantsurjroll, sorry, I constantly get distracted, I'll get to the bash ramdisk email as soon as I'm finally free14:20
rloojroll: are we still using launchpad for the client? https://blueprints.launchpad.net/python-ironicclient14:20
jrolldtantsur: not a huge rush, thanks14:21
jrollrloo: not for blueprints, just bugs...14:21
*** jjohnson2 has quit IRC14:21
rloojroll: guess those didn't get moved to the new rfe tags blah blah14:22
*** jjohnson2 has joined #openstack-ironic14:22
jrollrloo: maybe not14:22
rloojroll: I just updated our link to our feature tracker, from https://wiki.openstack.org/wiki/Ironic. that's when i noticed this.14:22
*** mjturek1 has quit IRC14:23
rloojroll: if i get a chance, i'll update it. although if anyone else wants to volunteer...14:23
jrollrloo: oh, fun, thank you14:23
jrollvdrok might be interested14:23
jrolldo we really need 4 RFEs for functional testing? :)14:23
jrollalso py34 is the future, and works in the client, we should probably kill the py33 thing14:23
*** Nisha_away has joined #openstack-ironic14:24
*** Nisha has joined #openstack-ironic14:24
rloodtantsur: i'm guessing the feature tracker link for ironic-inspector is also incorrect at https://wiki.openstack.org/wiki/Ironic#ironic-inspector14:24
rloojroll: i didn't actually look at the proposed bps.14:24
jrollrloo: I only looked at the names14:24
dtantsurrloo, true, I forgot about this page14:25
rloojroll: you looked at more than I did :)14:26
*** mjturek1 has joined #openstack-ironic14:27
*** keedya has quit IRC14:35
*** keedya has joined #openstack-ironic14:36
vdrokmorning rloo14:39
rloohi vdrok. Did you hear your name mentioned? :)14:39
mgouldmorning rloo14:39
vdrokyep :)14:40
rloohi mgould14:40
vdrokyeah, we shouldn't use it for blueprints14:40
rloovdrok: if you would like to move those bps to our bug rfe, that would be great.14:40
vdrokrloo: sure, will do14:40
rloovdrok: thx!14:40
*** spandhe has joined #openstack-ironic14:41
mgouldsambetts, wow, that's a lot of Unicode library14:41
mgouldis there another fdisk implementation we could use instead?14:42
vdrokjroll: btw, should we change the icon in https://launchpad.net/ironic to https://wiki.openstack.org/wiki/File:Bear_metal.svg ? :)14:43
jrollvdrok: hehe, maybe, remind me later14:43
* jroll jumps into meetings14:43
mgouldvdrok, definitely14:44
sambettsmgould: yeah, really sucks, I've tried looking into how to get rid of it, but I cant find a slimer alternative to gdisk14:44
mgouldoh wow, gdisk is hosted on sourceforge14:44
* mgould wonders if we could compile it from source with a --no-unicode option or something14:45
jlvillalmgould, Is that a problem?14:45
vdrokmorning jlvillal !14:45
jlvillalvdrok, Good morning. Privyet14:45
mgouldjlvillal, not a problem, I'm just surprised anyone still uses sourceforge14:46
jlvillalmgould, True. I think they are getting better since they were sold.14:46
* jlvillal kind of likes gitlab.com for free private repos, unlike github.com14:46
sambettsjroll, lucasagomes, mgould: I tried making a tripleOwl style sticker out of BearMetal wdyt?14:47
mgouldsambetts, excellent14:47
sambettshttp://imgur.com/wmIBNfJ14:47
jlvillalsambetts, photos or it didn't happen?14:47
mgouldjlvillal, I haven't tried gitlab, but it looks good14:47
mgouldsambetts, :-)14:48
sambettsstill a concept that needs refining, and I also have no idea how/where you get them printed/cut :-p14:48
jrollsambetts: ha, nice14:49
jrollsambetts: https://twitter.com/jayofdoom/status/72967678301702144014:49
jlvillalsambetts, http://bfy.tw/1ZoZ14:50
vdrokooooo14:50
sambettshaha awesome :D, I like the round shaped ones, because the cover the apple on my mac book14:50
sambettsjlvillal: ;) thanks14:50
jlvillal:D14:50
*** kcalman has joined #openstack-ironic14:50
mariojvhi - is there a bug # i should put for recheck for the gate failures that dtantsur fixed yesterday?14:52
JayFmariojv: that's not a thing anymore14:52
JayFmariojv: just do recheck14:52
mariojvthanks14:52
*** pcaruana has joined #openstack-ironic14:53
mgouldsambetts, I'll definitely order some of those if you get them printed14:54
sambettsmgould: :D14:56
*** stendulker_ has joined #openstack-ironic14:58
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: DNM Remove openvswitch-datapath-dkms from debs  https://review.openstack.org/31461615:00
*** piet has joined #openstack-ironic15:01
*** piet has quit IRC15:02
*** piet has joined #openstack-ironic15:02
*** milan has quit IRC15:04
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add tempest to the test-requirements.txt  https://review.openstack.org/31017915:04
sambettswoah?! when running devstack just now, enroll nodes just tried to do 2 port creates with the same mac addresses...15:09
*** sacharya has joined #openstack-ironic15:09
lucasagomessambetts, hah looks pretty good!15:10
lucasagomesjroll, JayF mannn I really want one sticker like that!15:11
JayFWhy you taggin' jroll15:11
JayFI am the keeper of the stickers!15:11
lucasagomes:-)15:12
lucasagomesthe stickers looks pretty awesome15:13
JayFFor anyone who wants pixie boots stickers: PM me an address and a number; and try to coordinate if you work in the same place as other folks on project (aka I don't wanna send 4 envelopes to 4 people if you all work in the same office). Stickers look like -> https://twitter.com/jayofdoom/status/729676783017021440/photo/115:13
JayFmariojv: ^ you mind being keepers of the stickers for Rackspace castle again? If so I'll ship you a couple dozen15:13
jlvillalmgould, lucasagomes Wondering if any updates/news on Grenade stuff during your time zone?15:13
lucasagomesjlvillal, not here :-( (was working on virtualbmc/pyghmi's)15:15
jlvillallucasagomes, Thanks15:15
sambettsjlvillal: just in the middle of trying to run it15:16
jlvillalsambetts, cool :)15:16
lucasagomesJayF, btw, not sure if you've seem my ping yesterday. But tweaking the ipmi session timeout in pyghmi made the vbmcs way more stable15:16
*** causten__ has quit IRC15:16
JayFI saw that in scrollbacka nd have the review open15:16
JayFbut I haven't looked at it at all15:17
JayFbut more than happy to have broken it and seen it fixed; I presume you did some testing with that in the devstack I stood up15:17
lucasagomesJayF, yup, I applied the fix locally because I still need a release from pyghmi to be able to ship it15:18
JayFfine by me, but very glad you saw it15:20
JayFjust help with stability or did it speed it up too?15:20
lucasagomesafaict stability, I'm still looking into it15:22
lucasagomesbut for now changing pyghmi's a bit painful since there's no unittests15:22
lucasagomeswe really need to do some work on that, cause things can break easily15:22
*** kbyrne has quit IRC15:22
*** keedya has quit IRC15:23
mgouldjlvillal, I got a bit further after the meeting (notes added to Etherpad), but no progress today15:24
mgouldjlvillal, how are you getting on?15:24
jlvillalmgould, I am getting an error when creating the server15:26
*** wshao has joined #openstack-ironic15:26
*** keedya has joined #openstack-ironic15:28
mgouldjlvillal, me too15:28
*** sacharya_ has joined #openstack-ironic15:29
rloojroll: wrt the poll on midcycle, is there a deadline?15:29
mariojvJayF: might want to ask nate or mark, i will be moving in a few months and out of office end of this month15:29
jrollrloo: thinking I'll just collect/announce monday mornign15:29
mgouldI noticed that the nova flavor specified cpu_arch as an extra_spec, but Ironic had it as a property rather than in properties/capabilities15:29
rloojroll: ok, i'll reply and indicate that.15:30
mgouldremoving that extra_spec from the flavor let me get a bit further, but then I hit another error15:30
jrollrloo: cool, thanks15:30
JayFmariojv: oh that's right15:31
*** sacharya has quit IRC15:31
mgouldsambetts, what version of gdisk are you using? The README says that libicu isn't required after v0.8.915:32
mgouldcurrent version is 1.0.115:32
sambettsmgould: just chrooted into by build dir and it says its version 1.0.015:33
mgouldsambetts, OK15:36
*** kbyrne has joined #openstack-ironic15:36
sambettsmgould: so maybe we can just nuke it but I don't know :/15:36
mgouldsambetts, you shouldn't need libicu15:36
mgouldapparently it was only needed for UTF-16 partition name support15:37
mgouldand UTF-16 should be killed with fire anyway15:37
sambettshmm, I wonder if it'll be sad if I just rm the data lib :/15:37
sambettslibicudata.so.54 ... is the file15:38
mgouldthe gdisk README has instructions for modifying the Makefile by hand to enable libicu, it's not even a ./configure option15:38
jlvillalmgould, This is what I see http://ironicstack.com/grenade-logs/2016-05-10/grenade.sh.txt15:38
jlvillallucasagomes, sambetts ^^^15:38
mgouldso unless your distro packager has gone out of their way to bloat the image, you shouldn't need it15:38
mgouldjlvillal, looking now15:38
* lucasagomes checks15:38
jlvillalI added --verbose --debug  to the openstack server create command15:39
lucasagomesjlvillal, that's what jroll's patch was fixing wasn't it?!15:39
jlvillalI still need to debug15:39
*** milan has joined #openstack-ironic15:39
lucasagomes https://review.openstack.org/31360015:39
mgouldjlvillal, yes, that's the same error as me15:39
jrolllucasagomes: so actually mine wasn't intended to fix that, and likely doesn't15:39
jlvillallucasagomes, Maybe I screwed up and not applying it15:40
jlvillalLooking15:40
*** kbyrne has quit IRC15:40
* lucasagomes checks... looks very similar15:40
jrolllucasagomes: I got some other error running by hand which that fixes, but it seems invalid15:40
lucasagomesoh15:40
*** itamarl has quit IRC15:40
lucasagomesso we should not apply that ?15:40
lucasagomesI see your comment on the patch now15:40
jrollyeah, probably not15:40
lucasagomescause after that I saw something like http://paste.openstack.org/show/496372/15:40
jlvillallucasagomes, jroll's patch is after the create.15:41
dtantsurjroll, mind taking one more look at https://review.openstack.org/313511 ? seems we've found a compromise...15:41
*** amotoki has quit IRC15:41
lucasagomesand worked around by creating a bunch of loopdevices15:41
lucasagomesfor ((i=0;i<200;i++)); do   [ -e /dev/loop$i ] || sudo mknod -m 0600 /dev/loop$i b 7 $i; done15:41
* dtantsur gets nagged about this bug all the time...15:41
jlvillallucasagomes, The create is on line 82. And the patch touches line 88 or so15:41
jlvillalAnd I am applying the patch.15:41
lucasagomesright, lemme run it again15:42
lucasagomesI probably need a new env cause mine's all snapshotted and things may have changed15:42
jlvillalmgould, Glad you got the same error as me :)15:43
* jlvillal needs to go to work after WFH meeting...15:43
mgouldjlvillal, I'll keep plugging away at it for another hour and note anything I find on the etherpad15:43
jlvillalmgould, Thank you!15:44
mgoulddtantsur, my suggestion of webcam + OpenCV was, on second thoughts, overcomplicated15:44
mgouldinstead, we should use laser interferometry15:44
*** moshele has quit IRC15:44
mgouldops people won't mind mounting lasers on their racks, right?15:45
*** rama_y has joined #openstack-ironic15:45
sambettsmgould: https://review.openstack.org/#/c/314086/ my attempt to prevent the cause of inconsitant file sytems15:45
mgouldsambetts, is this an alternative to dtantsur's patch, or will it work with it?15:46
jrolldtantsur: +2 but I'd prefer one more vote on that, I think15:47
sambettsmgould: not sure if its an alternative really because I can't reproduce the scenario where the file sytem carrier file is unreliable, but it should make dhcp-all-interfaces finish before IPA starts hopfully preventing races15:48
jrolldtantsur: one that hasn't been involved in this patch as much :P15:48
JayFI'm looking15:50
JayFjroll: dtantsur: Maybe dtantsur fix the nits real quick and then jroll and I can land it/15:51
JayFWe can also land as-is and do a followup, just with people here and watching might be easier to just fix the patch15:51
jrolleither way15:51
*** tesseract has quit IRC15:52
dtantsurlemme fix it, sure thing15:54
mgoulddtantsur, I left a few nitpicks in case you had to do another patch15:55
*** Sukhdev has joined #openstack-ironic15:55
dtantsurI will right now15:55
mgoulddtantsur, cool15:56
mgoulddtantsur, was it your machine that had the unreliable carrier files in sysfs?15:56
* mgould is wondering if it's buggy hardware or kernel drivers15:56
mgouldsambetts, cool, so it would make sense to land both?15:57
JayFA buggy NIC driver? Never! /s15:57
mgouldI know, right? million-to-one chance15:57
sambettsmgould: it needs testing it was just something I put together real quick, I'm not even sure it does what I hope it does xD my first time doing major systemd stuff15:58
dtantsurmgould, no, not mine15:58
*** dprince has quit IRC15:58
NobodyCamoh looks like we may be getting some internets15:59
mat128sambetts: looks like icu is pulled by this https://github.com/ooyala/tinycore_gdisk/blob/master/gdisk.tcz.dep16:02
sambettsmat128: yup16:03
JayFsambetts: unless we had a fix that worked in all the images, that's not really a fix is it?16:03
mat128not sure if this is source for official package (not a TCL expert), but if it does it's getting pulled all the time16:03
JayFsambetts: re: 31408616:03
*** wshao has quit IRC16:03
jrolloh man, I thought we were about to use the TCL language16:03
mat128jroll: lol, not proper case for TinyCoreLinux16:04
*** keedya has quit IRC16:04
dtantsursambetts, we should ignore imagebuild directory in the pep8 check, otherwise imagebuild/tinyipa/tinyipabuild gives hundreds of errors :D16:04
jrollah right, Tcl16:04
sambettsdtantsur: haha16:04
jrolldtantsur: well, we should ignore imagebuild/tinyipa16:04
jrollthe coreos builder has python :P16:04
*** penick has joined #openstack-ironic16:04
JayFsambetts: dtantsur: What file are you talking about? I don't see any python in that dir16:05
dtantsurimagebuild/tinyipa/tinyipafinal/usr/local/bin/...16:05
dtantsurditto imagebuild/tinyipa/tinyipabuild/usr/local/lib/16:06
JayFsambetts: dtantsur: OH, that's like, an artifact of the build process? If so only +1 to excluding the build dirs, not to excluding the code places16:06
dtantsuryeah, yeah, that's what I meant16:06
JayFcool perfectg16:06
jrolloh heh16:06
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: [inspection] wait for the PXE DHCP by default and remove the carrier check  https://review.openstack.org/31351116:06
dtantsurjroll, JayF ^^^16:06
JayFshould the tinyipa readme be updated to not say it's experimental16:06
dtantsurmgould, ^^16:06
*** e0ne has quit IRC16:06
JayFgiven we're about to make it the gate and the devstack default?16:06
jrollJayF: yep16:06
devanandamornin, all16:07
wajdimorning devananda16:07
*** appu103_ has joined #openstack-ironic16:07
*** appu103_ is now known as aparnav16:08
jrolldtantsur: dare I ask for unit tests for the mac normalization thing? :)16:08
dtantsursigh...16:08
dtantsurwell, you definitely might :)16:08
dtantsurmorning devananda16:08
JayFdtantsur: https://review.openstack.org/#/c/313035/ should not change inspector jobs due to https://review.openstack.org/#/c/313799/16:09
sambettsJayF: what questions about 314068 do you gave?16:09
sambettshave?16:09
JayFsambetts: I have more context now; the breakage only being in DIB image was something I was missing16:09
sambettsJayF: ah :-P16:09
JayFsambetts: just on patrol for having logic in only one imaeg when it's needed in three16:09
dtantsurJayF, inspector jobs don't use this template16:09
JayFoooh16:09
sambettsJayF: ah makes sense16:09
JayFdtantsur: do you want me to push project-config change to force it to ipxe off?16:10
JayFdtantsur: er, tinyipa off16:10
JayFdtantsur: or whatever, just prevent breakage, lol16:10
mgoulddtantsur, thanks16:11
* mgould raises his estimate of "buggy driver"16:12
dtantsurJayF, nevermind I guess. we should default to iPXE anyway, and looks like tinyipa works for us (judging by your patch)16:12
mgouldbuggy drivers for RH-issued developer hardware would be embarrassing :-)16:12
*** sacharya_ has quit IRC16:12
mgouldmat128, sambetts I see that .tcz file is three years old, it might well date from when gdisk did need libicu16:13
*** daemontool_ has quit IRC16:13
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: [inspection] wait for the PXE DHCP by default and remove the carrier check  https://review.openstack.org/31351116:13
mat128yeah, it also says it builds 0.8.7 which required libicu16:13
dtantsurjroll, JayF now with moar tests ^^^16:13
jrollsambetts: how was it we dealt with the tempest choosing the wrong flavor thing again?16:13
jrolldtantsur: <316:13
mat128mgould: do you know where to find the official sources for TinyCore builds?16:13
mgouldmat128, sambetts says he's getting v1.0.1 IIRC, so we're probably not using that version16:14
jrolldtantsur: +216:14
mgouldmat128, no idea16:14
*** dprince has joined #openstack-ironic16:14
sambettsmgould, mat128: http://distro.ibiblio.org/tinycorelinux/6.x/x86_64/tcz/16:15
jrollsambetts: never mind, found it16:15
mat128sambetts: how does one go and submits a change for these things?16:16
mgouldmat128, sambetts http://tinycorelinux.net/6.x/x86_64/tcz/src/gdisk/compile_gdisk16:16
mat128.info says it builds 1.0.0, which corresponds to what we found16:17
mgouldso they are deliberately pulling in ICU16:17
* mgould shakes his fist at TCL16:17
mat128http://distro.ibiblio.org/tinycorelinux/6.x/x86_64/tcz/gdisk.tcz.dep mentions icu16:17
mgouldsambetts, how hard would it be to build it from source?16:17
sambettsmgould: are they actually using it though or are they jut including it in their dep for lolz16:17
jrolldtantsur: https://review.openstack.org/#/c/314662/16:18
mat128sambetts: I bet just for fun, since 0.8.9 mentions getting rid of it16:18
mat128as a requirement16:18
mgouldsambetts, not sure: what does the compile_gdisk file I linked to mean?16:18
openstackgerritJim Rollenhagen proposed openstack/ironic: Test post don't upvote  https://review.openstack.org/31186516:19
mgouldyep, they're deliberately patching the Makefile to drag in ICU: http://tinycorelinux.net/6.x/x86_64/tcz/src/gdisk/gptfdisk-1.0.0-convenience-1.patch16:19
sambettsmgould: -_-16:19
sambettsbulding from source wouldn't be hard16:20
mgouldand they statically link it in16:20
sambettsman ...16:21
* sambetts is going to add it to the list of thing he builds from source16:23
mat128sambetts, mgould: asked on #tinycorelinux, not expecting a reply but maybe16:24
mgouldmat128, thanks!16:24
*** ifarkas has quit IRC16:28
*** Sukhdev has quit IRC16:29
*** Sukhdev has joined #openstack-ironic16:30
rlooby the way, did we decide on whether to merge or not the portgroups API? https://review.openstack.org/#/c/206244/10016:32
*** ijw has joined #openstack-ironic16:32
rlooit is up to 100 revisions. cool. warm. good. bad. sigh.16:32
dtantsurfolks want to wait with it...16:32
rloodtantsur: ok, so leave it as is. I won't review it again until we think we're close to merging the rest.16:33
dtantsur++ me neither16:33
rloowhat about network drivers. i heard some mention of splitting it up: https://review.openstack.org/#/c/285852/16:34
dtantsurI did one review round for it, I want it to come closer to the driver composition (or the opposite - folks driving the driver composition closer to it)16:34
krotscheckjroll, betherly, sambetts: Should ironic-ui be publishing its docs, or should it be a part of the ironic docs? I'm leaning former. https://review.openstack.org/#/c/305661/16:35
krotscheckppiela ^^16:35
sambettskrotscheck: I think it should be publishing its own docs16:35
*** electrofelix has joined #openstack-ironic16:36
jrollkrotscheck: what docs does ironic-ui have? I also lean former16:36
sambettskrotscheck: you've already got your own in tree, so its easy to add the publish job to project-config16:36
rloodtantsur: i just looked at your comment. seems reasonable. i thought someone said that they wanted that patch split up into smaller patches though.16:36
*** ohamada has quit IRC16:38
betherlykrotscheck: jroll sambetts i think it should be the former as well16:38
betherlyon my to do list for tomorrow to look into sorting that then16:38
rloojroll, krotscheck: I think it makes sense to be part of the ironic docs; although I don't know what 'part of the ironic docs' means. Do you mean the docs that ironic keeps intree and that end up at http://docs.openstack.org/developer/ironic/16:38
dtantsurironic-inspector has separate docs, so it might make sense for ironic-ui as well..16:39
jrollrloo: I guess I think it depends what the docs contain :)16:39
rlooso i don't think i want ironic-ui documentation source in ironic tree.16:39
jrollagree16:40
rloobut if there were a central place for users to find documentation about ironic + friends, I think that the ironic ui docs should be available there.16:40
jrollI think the question is should it be at /developer/ironic/ or /developer/ironic-ui/16:40
mat128sambetts mgould created this on their bug tracker16:40
mat128http://forum.tinycorelinux.net/index.php/topic,19951.0.html16:40
rloojroll: i would say /developer/ironic-ui then.16:40
betherlyrloo: jroll maybe if the ironic docs has a section added to provide links to subproject docs?16:40
sambettsjroll: I think ironic-ui, and we can add a link in the ironic docs to it16:40
mat128had to go through 8 impossible captchas simply to post a bug :)16:40
jrollrloo: yeah, I agree, we can link from main docs16:40
* jroll == sambetts == betherly == rloo16:41
betherlylol16:41
sambettsmat128: Nice!16:41
rloo++ betherly16:41
mat128that bug tracker was hard to find :P16:41
sambettshehe great minds and all that16:41
betherlyjroll: rloo ppiela krotscheck so tomorrow i will look into sorting ironic-ui docs and let you know here when thats done and ready for it to be linked from ironic docs16:42
mat128I have a feeling it's going to be either a) not answered, or b) be told 6.x is not supported anymore16:42
rloobetherly: is there ironic-ui information available from https://wiki.openstack.org/wiki/Ironic?16:42
sambettsmat128, mgould: tbh at some point we might want to move to tinycorelinux v7 instead of v616:42
betherlyjroll: do you want me to submit a patch to update ironic docs too with space for subproject linking?16:42
sambettsmat128, mgould: newer kernel etc16:42
betherlyrloo: hmm doesnt seem to be16:43
jrollbetherly: it's super easy, and sure go for it16:43
mat128sambetts: yeah, I figured. Let me check real quick but I'm pretty sure gdisk still pulls icu16:43
jlvillalmgould: I am bumping up the number of tinyipa VMs to 7.16:44
rloobetherly: might be good to update that page then. you should be able to do it yourself.16:44
sambettsmat128: yup it does16:44
jlvillalmgould: I had switched to tinyipa but forget to increase from the 1 VM.16:44
mat128why are we using gdisk again?16:44
jlvillalmgould: I also changed from 1024MB per VM to 384MB16:44
sambettsmat128: for gpt partitioning16:44
jlvillalmgould: Rebuilding now...16:45
betherlyjroll: rloo will do on both accoutns16:45
jrollbetherly: I think you want https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L6766 and https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/projects.yaml#L244916:46
jrollbut I may be wrong16:46
jrolljlvillal: you're gonna need a mitaka patch16:46
vdrokjlvillal: done the same for smoke, the only difference - I run with 3 vms, do we need more? all the tests do 3 vms max I think16:46
jrolljlvillal: https://review.openstack.org/#/c/314662/16:46
betherlyjroll: thanks :)16:47
jrollnp16:47
jlvillaljroll: Thanks. Will check it out.16:47
*** sacharya has joined #openstack-ironic16:47
jlvillalvdrok: Okay. So as long as we have at least 3 VMs we should be good?16:48
jrollvdrok: well, more VMs will help with the capacity problems you're seeing :)16:48
mat128sambetts: https://review.openstack.org/#/c/296466/3 makes it so that it doesnt seem to use gdisk anymore, and gdisk isn't used in the 2 Depends-On or in ironic-lib directly16:48
devanandajroll, betherly: fwiw, I think the deployer documentation (http://docs.openstack.org/developer/ironic/deploy/install-guide.html) should mention the UI and describe how to install it16:48
jrolldevananda: I'm not opposed to that16:49
*** e0ne has joined #openstack-ironic16:49
vdrokI'm seeing some really weird things now - using clustered compute manager from ironic tree, after termination, used fields, like local_gb_used are set to 0, all the other fields are OK. after some time, ALL the fields are set to 0, like memory_mb, local_gb. Then after some more time, things are back to normal. Has anyone seens this?16:49
* krotscheck just -2'd his first patch!16:49
* krotscheck feels POWERFUUUUL16:49
devanandakrotscheck: congradulations! :)16:49
jrollkrotscheck: and here you didn't want core :D16:50
mgouldkrotscheck, I hope it wasn't one of mine :-)16:50
mgouldmat128++16:50
betherlydevananda: jroll does it need to be a repeat of the ironic ui docs or can we just link to them (once the link works lol)16:50
*** sacharya has quit IRC16:50
krotscheckjroll: I still don't, because I'm keenly aware of how I'd abuse it :D16:50
mat128vdrok: maybe this? https://bugs.launchpad.net/nova/+bug/157921316:50
openstackLaunchpad bug 1579213 in OpenStack Compute (nova) "ComputeFilter fails because compute node has not been heard from in a while" [Undecided,New]16:50
jrollbetherly: links would be fine16:50
*** derekh has quit IRC16:50
devanandabetherly: links ++16:50
*** sacharya has joined #openstack-ironic16:50
vdrokmat128: in this case, memory_mb is 0, so ram filter fails :(16:51
krotscheckjroll, betherly, devananda: To be fair, it's the docs patch which tries to point ironic-ui's docs link to ironic's docs, rather than actually publishing ironic-ui's docs.16:51
betherlykrotscheck: hoping it wasn't one of mine :p16:51
devanandabetherly: also, what do ya'll plan to put in the ironic-ui docs?16:51
mgouldOK, I need to go - jlvillal, let me know how you get on (I'll leave this machine connected)16:51
betherlydevananda: installation notes and features for now16:51
mat128vdrok: thought 0 was the symptom of "not updated recently enough"16:51
betherlyany additions welcome :)16:52
jlvillalmgould: Thanks!  And good night16:52
jrollkrotscheck: <316:52
jrollbest -2 ever16:52
*** stendulker_ has quit IRC16:53
openstackgerritNisha Agarwal proposed openstack/ironic: Add support for dib based agent ramdisk in lib/ironic  https://review.openstack.org/26457916:54
*** rama_y has quit IRC16:54
*** rama_y has joined #openstack-ironic16:54
mat128mgould sambetts: my bad, using your browser's find function in gerrit doesnt seem to work very well, ironic-lib starts using gdisk once the changes are applied, thus the ramdisk still needs it16:55
*** mgould has quit IRC16:55
sambettsmat128: yeah :( shame I think I've managed to get somewhere with manually building it instead though16:56
JayFmat128: sambetts: switch to the unified diff16:56
JayFmat128: sambetts: if you want inline search to work16:56
sambetts++16:57
vdrokjroll: mat128: not sure, but I definitely need to look at this resource tracking. here is a paste of what happens in my case - http://paste.openstack.org/show/496597/16:57
vdrokwhich is the reason smoke tests fail sometimes16:57
jrollvdrok: oooooo16:58
jrollI feel like I know the answer here16:58
jrolldoes CCM just set the used fields on tear down?16:58
*** mgould has joined #openstack-ironic16:58
*** mgould is now known as mgould|afk16:59
vdrokjroll: https://github.com/openstack/ironic/blob/master/ironic/nova/compute/manager.py#L83-L9216:59
jrollvdrok: here's the thing https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L31216:59
*** penick has quit IRC16:59
vdrokoh16:59
jrollthat's why it drops to 0 during cleaning16:59
vdrokyeah, that explains things :)17:00
openstackgerritNisha Agarwal proposed openstack/ironic: Add support for building ISO for deploy ramdisk  https://review.openstack.org/26459017:00
vdrokthanks jroll17:00
*** ChrisAusten has joined #openstack-ironic17:00
jrollvdrok: also since the resource tracker will "correct" us, that's why you see 0 used, 384 available for a short time, I think17:00
*** daemontool_ has joined #openstack-ironic17:00
jrollit sees there's no instance and changes it to 0 used17:00
*** causten_ has joined #openstack-ironic17:01
vdrokso it seems we still need to wait for actual resources set correctly, not for cleaning finish17:01
jrollright17:01
*** aparnav has quit IRC17:02
*** causten_ has quit IRC17:03
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: Add unittests for the manager.py module  https://review.openstack.org/31468117:04
*** ChrisAusten has quit IRC17:04
*** causten_ has joined #openstack-ironic17:04
JayFlucasagomes: sorta been chewing on what you said earlier. I don't think we should use virtualbmc in the gate until pyghmi starts following more ironic-style review processes (rather than the instamerge that tends to happen now)17:05
JayFlucasagomes: and has better test coverage17:05
*** e0ne has quit IRC17:06
lucasagomesJayF, I agree with that, we need to check with jjohnson217:06
lucasagomesJayF, I'm trying to get vbmc enough test coverage as well17:06
jjohnson2lucasagomes, ok17:06
jlvillalsambetts: Do you know the status on moving the gate to use tinyipa?17:07
lucasagomesjroll, (if jjohnson2 agrees) do you think pyghmi's could be part of the ironic umbrealla?17:08
*** sacharya has quit IRC17:08
sambettsjlvillal: I think JayF might have put patches together to do it17:08
*** sacharya has joined #openstack-ironic17:08
JayFjlvillal: I have some patches up, incomplete, on my top 3 list to finsih17:09
jlvillalJayF: sambetts Thanks17:10
JayFhttps://review.openstack.org/#/c/313799/ is the frist of them, to make ipxe config explicit so we can make ipxe/tinyipa teh default in devstack without chanign gate17:10
lucasagomesJayF, one thing we can do as well, is to pin on specific version of pyghmi17:10
*** jaybeale has joined #openstack-ironic17:10
JayFhttps://review.openstack.org/#/c/313800/ is the second, but needs the projects.yaml updated17:10
lucasagomesor set a upper-bound at least17:10
lucasagomes(which is the same we do for other things such as PrettyTable etc)17:10
JayFlucasagomes: I mean, I don't think that really gets to the heart of my objection, unless we really treat pyghmi 100% as an external lib17:11
lucasagomesJayF, that's kinda how we treat it now, pyghmi's in the global-requirements (and has a upper-bound)17:12
lucasagomeswhat I mean is, that will give virtualbmc stability and we can work on pyghmi and improve it in parallel17:12
*** e0ne has joined #openstack-ironic17:13
jrolllucasagomes: yeah, totally re: pyghmi17:13
jjohnson2lucasagomes, JayF I am missing part of the conversation...  If I interpret 'ironic-style review process' right, there's two interesting points: 1) I haven't had a lot of volunteers to code review 2) there's also a lot of functionality that ironic doesn't yet touch that are probably even more boring for the general audience17:14
jrollJayF: putting a vbmc gate on pyghmi solves a decent amount of that issue, no?17:14
jjohnson2I certainly wouldn't mind that17:15
JayFjroll: if we're treating pyghmi 100% as an external lib, any concerns I have are mostly gone17:15
jrollJayF: sure17:16
devanandaJayF jroll: at the moment, I think we should be treating pyghmi like we treat other hardware drivers / libs, eg. proliantutils17:16
lucasagomesthere's a job already, it's non-voting for now until we get some stability17:16
*** athomas has quit IRC17:17
JayFdevananda: I agree; but I think that relationship would change if we used vbmc in the gate which is built on pyghmi, which is why I started the conversation initially17:17
devanandaJayF: ah. I agree.17:17
*** causten_ has quit IRC17:17
jjohnson2lucasagomes, you might want to do a testcommit that will actualy pass the gate, since I'm bogged down from fixing the python3 commit17:17
devanandahadn't read all the backscroll, sorry17:17
lucasagomesjjohnson2, yup17:17
lucasagomesthere are some already17:17
lucasagomesjjohnson2, e.g https://review.openstack.org/#/c/286217/17:17
vdrokjroll, jlvillal, then maybe we can do the following wrt smoke - set the vm count to 6, and during instance termination, wait for ironic node cleaning to finish. This way, we don't have to wait for resource tracker, and we won't use more memory/cpu as we will have 3 vms max running at the same time. WDYT? Or wait for actual hypervisor-show command's correct17:17
vdrokoutput?17:17
jrollvdrok: I'm fine with that, we can fit 3x 384MB VMs in the gate17:18
jroller.17:18
jroll7x17:18
jjohnson2lucasagomes, ah, ok17:18
vdrokaha, great :)17:19
lucasagomesjjohnson2, so the job is passing, since it does not block the gate yet (non-voting) we should keep an eye on it before merging any patches17:19
JayFjroll: should I change my tinyipa in gate patch to lower ram from 512 to 384?17:19
JayFjroll: it would mean it would fail gate if we ever made tinyipa too big, which I think could be a good thing17:19
jrollJayF: idk if it matters much, unless we need the space17:19
jrollcould be17:19
jrollI guess I'm not opinionated17:19
jjohnson2I still don't get notified on pyghmi reviews...17:20
devanandajjohnson2: you can change your subscription settings in gerrit17:21
*** jistr has quit IRC17:21
JayFjroll: the question boiled down is: do we want to fail a tinyipa gate if we can't run the agent in 384mb of ram17:21
jrollJayF: right, I'm not sure17:22
vdrokIf we're going to use 7 vms for grenade, I think we should :)17:24
*** kromanenko_ has quit IRC17:24
*** ChubYann has joined #openstack-ironic17:24
sambettsJayF: won't it fail anyway? because it won't even boot17:24
*** daemontool_ has quit IRC17:25
jrollvdrok: yeah, that's a good point17:25
jrollJayF: sure, do it, or at least the -src job17:25
jjohnson2hehdevah, thanks17:25
jjohnson2devananda, thanks17:25
vdrokjroll: otoh, if it will be only 3 at the same time, we can do 512?17:25
jrollvdrok: right, we need to decide which is best17:26
vdrok3 running at the same time I mean17:26
jjohnson2pyghmi is interesting in general, it's in openstack, but has some commitments beyond (e.g. I'm not allowed to drop python 2.6 testing)17:27
lucasagomesoh17:28
jjohnson2there's some non-cloud users of RHEL6 that won't even consider EL7 at this juncture17:29
lucasagomesI think you can keep then, I mean, not sure if OpenStack would enforce dropping 2.6 support to the libraries17:29
openstackgerritMerged openstack/pyghmi: Remove unused pngmath Sphinx extension  https://review.openstack.org/28621717:29
jjohnson2of course, those same users are served through rpms...17:29
sambettsOMG OMG OMG OMG OMG OMG First tinyIPA boot at 256MB!!! waiting for devstack to prepare so I can tempest test it but ZOMG!17:30
*** e0ne has quit IRC17:30
*** penick has joined #openstack-ironic17:30
dimssambetts : wow, someone is excited :)17:30
sambettsMuch excitment :D17:30
jjohnson2hmm, I've been spending some of my day working on our HPC bootstrap, it's about 17 MB filesystem, got downright bloated after adding python ;)17:31
JayFsambetts: only if I limited it to 384mb (or maybe 256 now? lol) in teh gate17:31
JayFsambetts: that's what I was asking: should the size be 512 or 38417:31
sambettsJayF: ah right, missed that bit17:31
* sambetts -> gym, then I'm going to try to push a new patch to IPA with the improvements after local testing 17:33
*** sambetts is now known as sambetts|afk17:33
jjohnson2oh well, back to writing my filesystem for a while17:33
jjohnson2this is a nice prelude for me manifesting hardware management api as a filesystem.... it's at least fun17:34
jjohnson2maybe not so much the 'profit'17:34
*** hshiina has quit IRC17:36
*** spandhe has quit IRC17:38
dtantsurhttps://review.openstack.org/#/c/313511/ passed gates, could use +W17:42
dtantsurJayF, ^^^17:43
lucasagomesjjohnson2, heh which filesystem are you writing?17:43
JayFdone17:44
dtantsurour of memory in tinyipa gate, wow: http://logs.openstack.org/62/314662/1/check/gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-nv/c607a1a/logs/ironic-bm-logs/baremetalbrbm_0_console_10-05-2016-17:24:24.txt.gz17:45
dtantsursambetts|afk, ^^^17:45
dtantsurjroll, do you want to recheck https://review.openstack.org/#/c/314662/ due to ^^^?17:46
*** keedya has joined #openstack-ironic17:46
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: Add unittests for the manager.py module  https://review.openstack.org/31468117:46
jrolldtantsur: O_o17:46
lucasagomesdtantsur, ouch, weird17:47
dtantsurI suspect that low memory is great, but we should not go too low in gate :)17:47
dtantsurstill, intriguing17:47
jrollthat's with 51217:47
jrollalso keep in mind that's stable17:47
jrollJayF: stable branches are probably a reason to keep it at 512 for most things :)17:47
jrollstable/mitaka anyway17:48
*** lucasagomes is now known as lucas-afk17:49
dtantsurg'night, see you tomorrow17:49
jrollnight!17:50
* dtantsur will be on irc for some time, but can be considered gone for a day :)17:50
jjohnson2lucas-afk, right now, something to provide a utility that only understands file I/O http access, long story17:50
JayFjroll: we force ramdisk to coreos for stable; we could force size for stable too17:50
jrolldtantsur: don't say that, I'll keep pinging you17:50
jrollJayF: yeah, it's just harder because project-config17:50
jrollbut it is possible17:50
JayFjroll: honestly I kinda have an itch to refactor how it works to make the ramdisk type dictate size17:50
jjohnson2lucas-afk, but I also have thoughts of having a confluent client fs.  Already the CLI presents things like a filesystem when at its prompt, might be fun to extend out17:50
*** absubram has joined #openstack-ironic17:50
dtantsurafter saying that I'm free to ignore any pings :-P17:50
jrollJayF: not a bad idea17:51
* vdrok leaves too, good night!17:52
*** thiagop has quit IRC17:52
*** lucas-afk is now known as lucasagomes17:55
lucasagomesjjohnson2, sounds kinda cool17:55
*** garthb has joined #openstack-ironic17:57
lucasagomeswill call it a day as well, have a great evening (ugt) all!17:57
*** lucasagomes is now known as lucas-dinner17:57
jjohnson2lucasagomes, yeah, I really need to make a new video covering the state of things17:57
jjohnson2lucas-dinner, have a good evening17:57
*** Sukhdev_ has joined #openstack-ironic17:57
NobodyCamhave a good night lucas-dinner17:58
*** Sukhdev has quit IRC18:00
*** Sukhdev_ has quit IRC18:02
*** ChrisAusten has joined #openstack-ironic18:02
*** e0ne has joined #openstack-ironic18:03
*** rajinir has quit IRC18:06
*** Nisha_away has quit IRC18:09
*** Nisha has quit IRC18:10
*** causten_ has joined #openstack-ironic18:10
*** jaybeale has quit IRC18:12
*** ChrisAusten has quit IRC18:12
*** rcernin has quit IRC18:13
openstackgerritMerged openstack/ironic-python-agent: [inspection] wait for the PXE DHCP by default and remove the carrier check  https://review.openstack.org/31351118:17
*** electrofelix has quit IRC18:19
*** sacharya has quit IRC18:31
*** baoli_ has quit IRC18:32
dtantsurjroll, hope you don't mind backporting ^^^ (patch https://review.openstack.org/#/c/314713/)18:35
* dtantsur disappears now18:36
NobodyCamnight dtantsur18:36
*** dtantsur is now known as dtantsur|afk18:36
dtantsur|afko/18:36
jrolldtantsur|afk: yeah, it's fine, remind me tomorrow if I don't do it now18:36
mat128dtantsur: you can always use a newer IPA with an older Ironic, that's what we do here18:36
jrollnow/today18:36
dtantsur|afkmat128, a bit trickier for distributions like RDO which strictly follow upstream branches...18:37
mat128oh, right18:37
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/31372918:37
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/31435418:37
*** penick has quit IRC18:41
*** penick has joined #openstack-ironic18:41
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/31375618:42
*** joprovost has joined #openstack-ironic18:43
*** rcernin has joined #openstack-ironic18:43
*** jcoufal has quit IRC18:47
*** thiagop has joined #openstack-ironic18:48
openstackgerritMathieu Mitchell proposed openstack/ironic: Allow configuring shred's final overwrite with zeros  https://review.openstack.org/30410118:53
*** rajinir has joined #openstack-ironic18:53
jjohnson2Well, now I know for certain pyghmi can juggle 2,000 real servers from a single process18:55
jjohnson2though the keepalive to keep sessions alive alone are enough to take up 25% of a core...18:56
*** piet has quit IRC18:58
*** fragatina has quit IRC19:02
*** spandhe has joined #openstack-ironic19:03
*** jaybeale has joined #openstack-ironic19:07
openstackgerritMerged openstack/ironic-lib: Fix coverage option and execution  https://review.openstack.org/30555719:08
*** jaybeale_ has joined #openstack-ironic19:09
*** fragatina has joined #openstack-ironic19:09
*** jaybeale has quit IRC19:11
cineramajlvillal: how far are you getting with the grenade run lately?19:12
*** pcaruana has quit IRC19:12
*** piet has joined #openstack-ironic19:13
jlvillalcinerama: I'm running some tests now. My latest changes seemed to have made me take a step back.19:13
jlvillalTrying to figure out which change it is.19:13
openstackgerritMathieu Mitchell proposed openstack/ironic: Allow configuring shred's final overwrite with zeros  https://review.openstack.org/30410119:13
jlvillalcinerama: It is either the 7 VM nodes are a patch I am merging.19:13
jlvillalI think19:13
cineramamm. i am getting failures in the base smoke often, and sometimes in the next bit after that19:14
jlvillalcinerama: My latest changes are causing failures in base smoke.19:17
jlvillalSo reverting some changes.19:17
mat128sambetts|afk, mgould|afk: there is hope! http://forum.tinycorelinux.net/index.php/topic,19951.0.html19:18
cineramajlvillal: cool, i'll try again when you land that19:18
*** sacharya has joined #openstack-ironic19:22
*** causten_ has quit IRC19:24
*** jaybeale_ has quit IRC19:29
*** jaybeale has joined #openstack-ironic19:29
openstackgerritRuby Loo proposed openstack/ironic-specs: Priorities that were done in Mitaka  https://review.openstack.org/31473919:35
*** jaybeale has quit IRC19:36
*** rcernin has quit IRC19:51
*** piet has quit IRC19:51
*** daemontool_ has joined #openstack-ironic19:51
*** e0ne has quit IRC19:53
*** rcernin has joined #openstack-ironic19:59
*** afaranha has quit IRC20:00
*** dprince has quit IRC20:05
*** daemontool_ has quit IRC20:06
*** absubram has quit IRC20:06
sambetts|afkmat128: thats cool!20:10
*** Goneri has quit IRC20:12
*** sacharya_ has joined #openstack-ironic20:13
openstackgerritMerged openstack/ironic-python-agent: Remove oslo.i18n  https://review.openstack.org/31358620:13
*** sacharya has quit IRC20:16
*** rama_y has quit IRC20:17
xavierrguys, have you faced some problem with agent deploy images these days?20:18
jrollxavierr: you'll need to be more specific20:19
sambetts|afkdtantsur|afk, jroll: woah?! that log with the out of memory error is booting the coreos image!!!!!20:20
*** rama_y has joined #openstack-ironic20:21
sambetts|afkdtantsur|afk, jroll: even though its the tinyipa job... WTF?!20:21
xavierrI'm trying to boot a baremetal using coreos_production_pxe.vmlinuz and coreos_production_pxe_image-oem.cpio.gz, but the machine stucks when trying to load kernel image20:22
devanandaxavierr: can you observe / record the machine's console during boot?20:23
jrollsambetts|afk: waaaat20:23
xavierrdevananda, yeap20:23
sambetts|afkjroll: http://logs.openstack.org/62/314662/1/check/gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-nv/c607a1a/logs/ironic-bm-logs/baremetalbrbm_0_console_10-05-2016-17:24:24.txt.gz you can tell because the coreos kernel outputs [<seconds since boot>] before each line and tinyipa doesn't20:24
xavierrdevananda, let me upload the print screen20:24
sambetts|afkjroll: thats why it out of memoryed, because coreos doesn20:24
jrollsambetts|afk: right, looking20:24
sambetts|afkdoesn't fit in 512mb20:25
openstackgerritSean Dague proposed openstack/ironic: DNM: testing paramiko bump  https://review.openstack.org/31475620:26
jrollsambetts|afk: oh, we don't support downloading tinyipa on stable/mitaka20:26
jrollfeel free to backport that20:26
jrollbut that would be why20:26
sambetts|afkjroll: oooo of course!!!20:26
*** fragatina has quit IRC20:27
*** keedya has quit IRC20:27
*** aNupoisc has joined #openstack-ironic20:27
xavierrdevananda, https://drive.google.com/open?id=0Bx3iHZG5D0neMk5kbDAxeHd4aWc20:27
sambetts|afkjroll: we'll have to trigger the post job on IPA stable/mitaka to provide one too20:28
xavierrdevananda, thats all20:28
*** ijw_ has joined #openstack-ironic20:28
*** ijw has quit IRC20:28
jrollsambetts|afk: right20:29
xavierrdevananda, after that the machine stops and reboot20:29
jrollsambetts|afk: something to hack on tomorrow, go beer20:29
*** joprovost has quit IRC20:33
*** absubram has joined #openstack-ironic20:33
*** baoli has joined #openstack-ironic20:36
*** baoli has quit IRC20:41
*** baoli has joined #openstack-ironic20:42
sambetts|afkjroll: heh :-P cya tomorrow20:44
jrollsambetts|afk: thanks for confirming we can land this, though :D20:45
JayFsambetts|afk: jroll: That means that my changes, which force !tinyipa is stable/mitaka or earlier, would fix this too20:45
openstackgerritKurt Taylor proposed openstack/ironic-specs: Revise third party continuous integration spec for new drivers  https://review.openstack.org/31476820:46
jrollJayF: well, how does that fix "tinyipa job doesn't actually run tinyipa on mitaka"?20:46
JayFoh. OH20:46
jroll:)20:46
JayFtinyipa job shouldn't run against mitaka ever, right?20:46
JayFbecause we didn't support it in mitake20:46
JayFseems like the most anti-stable thing possible to start gating a thing we didn't support on release post-release20:47
JayFI mean really we don't support it now because it doesn't gate anything (working on it)20:47
*** baoli has quit IRC20:48
jrollJayF: we did though20:49
sambetts|afkJayF: we support tinyipa build from source in stable/mitaka20:49
*** KennethWilke has quit IRC20:49
jrollwe just didn't support *downloading* tinyipa in mitaka20:49
sambetts|afkjust not prebuilt20:49
jroll(apparently)20:49
JayFI guess I don't understand then20:49
JayFbecause in OpenStack it's been hammered over and over20:49
JayFif you don't gate it, you don't support it20:49
*** baoli has joined #openstack-ironic20:49
jrollwe supported tinyipa built from source20:49
JayFand tinyipa has zero voting jobs; even the -src job on ipa is nonvoting20:49
jrollwell, it wasn't voting20:49
jrollhere's my actual issue20:50
*** baoli has quit IRC20:50
jrollgrenade starts with tests on $previous_stable and upgrades to current20:50
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/31435320:50
jrolltherefore, I need tinyipa working on $previous_stable, in this case mitaka20:50
jrolland tinyipa does exist in mitaka, and the job was previously okay, until we made it fetch from tarballs.o.o20:51
jrollso I want to backport the patch that allows that20:51
jrollmake sense?20:51
*** lucas-dinner has quit IRC20:51
JayFThat makes some sense; would it not work to just make a -src job for stable/mitaka since that was supported at that point?20:51
*** baoli has joined #openstack-ironic20:52
jrollwell, it would be more of adding the special things to the grenade job to make it a -src thing20:53
jrollwhich will just make it longer and more prone to failure because internets20:54
JayFgotcha20:54
JayFI'm like +1 in pragmatism20:54
JayFbut it does break the pattern20:54
sambetts|afka merge into IPA stable/mitaka will generate a tinyipa-mitaka.tar.gz on tarballs, so we just need to backport the devstack change to make it useable :)20:54
JayFsambetts|afk: wanna push a commit to both master and stable updating the readme to say it's not experimental anymore?20:55
sambetts|afksure :)20:55
*** KennethWilke has joined #openstack-ironic20:55
*** baoli has quit IRC20:57
*** lucasagomes has joined #openstack-ironic20:58
*** baoli has joined #openstack-ironic21:00
JayFDid anyone else want a pixie boots sticker? Just need an address to mail them to21:00
JayFI'm going downstairs for stamps and to mail off the first batch in about 20m, so lmk soon or you'll have to wait longer :)21:01
*** baoli_ has joined #openstack-ironic21:01
devanandaJayF: o/21:01
JayFdevananda: pm me your address + how many stickers21:01
JayFdevananda: only request is that you know the stickers will be used :)21:01
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/31372921:02
mat128JayF: I'm poking around to know who really wants one21:04
JayFmat128: wanna hit me up in pm with an address? I mean, I'm assuming if I gave you an extra one or two someone would use it given you all use the software21:04
*** baoli has quit IRC21:04
JayFmat128: just have to make sure to put all the onmetal stickers I'll send you on your laptop too /s21:04
mat128Haha21:05
mat128Will do :) Walking around the office with this21:05
mat128Let's trade stickers for pictures of stickers :)21:05
JayFhahaha21:05
*** rama_y has quit IRC21:05
openstackgerritMerged openstack/ironic: Adds RAID interface for 'iscsi_ilo'  https://review.openstack.org/31401621:06
*** rama_y has joined #openstack-ironic21:06
*** garthb has quit IRC21:09
*** wajdi_ has joined #openstack-ironic21:11
*** rama_y has quit IRC21:11
*** wajdi has quit IRC21:15
*** wajdi_ has quit IRC21:16
*** ametts has quit IRC21:17
*** garthb has joined #openstack-ironic21:17
*** jjohnson2 has quit IRC21:17
openstackgerritMerged openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/31375621:20
*** [1]cdearborn has joined #openstack-ironic21:20
openstackgerritDevananda van der Veen proposed openstack/ironic: Update the api-ref documentation for Ports  https://review.openstack.org/31318721:23
openstackgerritDevananda van der Veen proposed openstack/ironic: Update the api-ref documentation for Root and Nodes  https://review.openstack.org/31279521:23
*** mbound has joined #openstack-ironic21:30
openstackgerritSam Betts proposed openstack/ironic-python-agent: TinyIPA: Shave off some file size from tinyipa ramdisk  https://review.openstack.org/31367521:35
*** daemontool_ has joined #openstack-ironic21:35
*** cdearborn has quit IRC21:36
sambetts|afknight all21:38
*** ijw_ has quit IRC21:41
jrolljlvillal: so that nw info cache thing....21:42
jrollisn't an issue at that bit21:42
jrollbut rather21:42
jroll2016-05-09 19:59:57.781 | + /opt/stack/new/grenade/projects/60_nova/resources.sh:create:82 :   openstack server create --image cirros-0.3.4-x86_64-uec --flavor baremetal --security-group nova_grenade --nic net-id=cfd19d5d-7d90-4552-a40b-01e3ff74a463 nova_server1 --wait21:42
jroll2016-05-09 20:00:04.173 | Error creating server: nova_server121:42
jrollwhich on my latest appears to be issues building http://logs.openstack.org/65/311865/3/experimental/gate-grenade-dsvm-ironic/e635dec/logs/old/screen-n-cpu.txt.gz?level=WARNING21:43
*** milan has quit IRC21:44
*** milan has joined #openstack-ironic21:44
jrolland now that I look, grenade creates a new network for resource create21:46
jrollwhich I assume we don't tap the conductor into https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L65321:46
jrolljlvillal: we don't have a grenade plugin yet, right?21:47
jrolloh, we do, devstack/upgrade21:49
jrollbut we need to do this before nova resource create, after neutron resource create21:49
jrollLE SIGH21:49
NobodyCam:(21:50
*** ijw has joined #openstack-ironic21:51
*** sacharya has joined #openstack-ironic21:52
*** run_dfc has quit IRC21:52
mrdaMorning Ironic21:53
*** amotoki has joined #openstack-ironic21:53
NobodyCamhey hey morning mrda21:53
jrollhi mrda21:54
*** sacharya_ has quit IRC21:54
*** daemontool_ has quit IRC21:55
*** fragatina has joined #openstack-ironic21:57
*** rama_y has joined #openstack-ironic21:57
mrdahey NobodyCam and jroll o/21:57
NobodyCam:)21:57
*** rbudden has quit IRC22:11
*** piet has joined #openstack-ironic22:14
*** [1]cdearborn has quit IRC22:15
xavierrMorning mrda22:17
*** xavierr is now known as xavierr_pizza22:19
*** rloo has quit IRC22:22
*** raildo is now known as raildo-afk22:38
rama_yJayF: Hello?22:50
rama_yjroll: hello22:51
JayFrama_y: Is there a reason you just tried to DCC me a file? On most public networks that's considered abusive without asking first22:59
rama_yJayF: I wanted to chat.22:59
rama_yJayF: to be specific, ask a question.23:00
JayFIf there's something you'd like to ask, feel free to ask here and if someone (maybe me?) knows the answer I'm sure they'll try to help23:00
rama_yI have a question regarding the metrics patch:  #/c/301526/ and the spec for adding pluggable metrics backend for ironic & ipa.23:00
rama_yAny suggestions how OSprofiler can be used as additional backend(?) or engine?23:00
*** baoli_ has quit IRC23:01
*** Goneri has joined #openstack-ironic23:03
JayFrama_y: I mean, the code is up, it is pretty straightforward to implement a backend if the correct primitives exist23:03
JayFnot sure what the question is23:03
rama_yJayF: could you be more specific about the primitives?23:04
JayFI mean, it's all spelled out in the spec, not sure what value I'll add telling you in IRC as well :P23:05
JayFhttps://review.openstack.org/#/c/301526/13/ironic_lib/metrics.py line 9423:06
JayFyou'd just have to implement a metricslogger class for osprofiler23:06
rama_yJayF: thanks very much for the input.23:07
rama_yJayF: that means, it would be part of ironic-lib, am I correct?23:08
JayFyes23:08
rama_yJayF: thanks!23:08
JayFI'd suggest filing an RFE bug for it, and just having the patch depend on alineb's patch23:09
JayFalso by reviewing her patch you can help metrics as a whole get in23:09
rama_yJayF: I filed this RFE earlier: https://bugs.launchpad.net/ironic/+bug/156070423:13
openstackLaunchpad bug 1560704 in Ironic "[RFE] Ironic support for OSprofiler" [Wishlist,In progress] - Assigned to Ramamani Yeleswarapu (ramamani-yeleswarapu)23:13
rama_yJayF: thanks for the suggestions.23:17
JayFnp, good luck23:18
rama_yThanks!23:19
*** d0ugal has quit IRC23:21
*** absubram has quit IRC23:26
*** jaybeale has joined #openstack-ironic23:26
*** rcernin has quit IRC23:28
*** Goneri has quit IRC23:30
*** sacharya has quit IRC23:34
*** sacharya has joined #openstack-ironic23:35
*** spandhe has quit IRC23:36
*** d0ugal has joined #openstack-ironic23:36
*** spandhe has joined #openstack-ironic23:38
jlvillalJayF: Do you think 'OSProfiler' can be a 'backend'?  Is OSProfiler the same type of thing as 'statsd' for example?23:39
jlvillalI'm somewhat confused.23:39
JayFjlvillal: if it fits into the model, sure it can. That's why I caveated it with 'if it implements the proper primitives'23:40
jlvillalJayF: I guess I'm thinking of 'OSProfiler' as maybe an 'engine' and not a 'backend'.23:42
jlvillalFrom reading the code the 'backend' seems to be something that records the statistics.23:42
JayFjlvillal: I'm not an expert on OSProfiler23:43
* jlvillal is not either :(23:43
JayFthat's why I pointed the interested person at the code23:43
NobodyCamI seem to recall that using functions in function definition's was a bad thing?23:44
jlvillalNobodyCam: I'm not sure I understand.23:46
NobodyCamdef function(self, node_id, mac=utils.generate_mac_address(), params='')23:46
jlvillalNobodyCam: Hmmm. I'm not sure.  Not sure if it is like doing foo={}23:47
jlvillalAs in bad.23:47
NobodyCamI seem to recall something about it.. but didn't find anything with a quick google23:48
*** piet has quit IRC23:51
*** crinkle has quit IRC23:55
*** chlong has joined #openstack-ironic23:56
jlvillalNobodyCam: My gut feeling is that it should be okay. But I have nothing to back it up :)23:58
*** garthb has quit IRC23:59

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