Monday, 2016-05-09

*** sacharya has joined #openstack-ironic00:23
*** gabriel-bezerra has quit IRC00:23
*** gabriel-bezerra has joined #openstack-ironic00:24
*** sacharya has quit IRC00:29
*** Guest460 is now known as mfisch00:32
*** mfisch is now known as Guest8170400:32
*** elfosardo_ has joined #openstack-ironic00:35
*** baoli has joined #openstack-ironic00:38
*** elfosardo has quit IRC00:38
*** catintheroof has joined #openstack-ironic00:55
*** baoli has quit IRC01:43
*** Nisha has joined #openstack-ironic01:52
*** Nisha has quit IRC01:57
*** Fdaisuke has joined #openstack-ironic01:58
*** hoangcx has joined #openstack-ironic02:07
*** Haomeng has joined #openstack-ironic02:08
*** sacharya has joined #openstack-ironic02:27
*** amit213 has quit IRC02:34
*** amit213 has joined #openstack-ironic02:34
*** smoriya has joined #openstack-ironic02:41
*** baoli has joined #openstack-ironic02:44
*** baoli has quit IRC02:49
*** Guest81704 is now known as mfisch03:03
*** mfisch is now known as Guest3001803:04
*** amotoki has joined #openstack-ironic03:07
*** Sukhdev has joined #openstack-ironic03:09
*** amotoki has quit IRC03:18
*** amotoki has joined #openstack-ironic03:40
*** links has joined #openstack-ironic03:44
*** Sukhdev has quit IRC03:55
*** catintheroof has quit IRC03:56
*** fragatina has quit IRC03:58
*** appu103_ has joined #openstack-ironic04:01
*** sacharya has quit IRC04:02
*** appu103_ is now known as aparnav04:03
*** itamarl has quit IRC04:05
*** vmud213 has joined #openstack-ironic04:10
*** fragatina has joined #openstack-ironic04:13
*** fragatina has quit IRC04:18
*** fragatina has joined #openstack-ironic04:29
*** fragatina has quit IRC04:29
*** fragatina has joined #openstack-ironic04:30
*** hoangcx has quit IRC04:45
*** hoangcx has joined #openstack-ironic04:45
*** amotoki_ has joined #openstack-ironic04:46
*** amotoki has quit IRC04:49
*** yonglihe has joined #openstack-ironic05:04
*** ChubYann has quit IRC05:29
*** linuxgeek has quit IRC05:29
*** amotoki_ has quit IRC05:35
*** rcernin has joined #openstack-ironic05:40
*** amotoki has joined #openstack-ironic05:51
*** vinbs has joined #openstack-ironic05:54
*** yolanda has joined #openstack-ironic05:58
*** vinbs_ has joined #openstack-ironic06:00
*** vinbs has quit IRC06:01
*** vinbs_ is now known as vinbs06:03
*** sacharya has joined #openstack-ironic06:03
*** sacharya has quit IRC06:08
*** vinbs has quit IRC06:16
*** vinbs has joined #openstack-ironic06:20
*** vinbs has quit IRC06:30
*** vinbs has joined #openstack-ironic06:36
*** itamarl has joined #openstack-ironic06:42
*** vinbs has quit IRC06:42
*** xek__ is now known as xek06:43
*** openstackgerrit has quit IRC06:47
*** openstackgerrit has joined #openstack-ironic06:48
*** chihhsin has quit IRC06:50
*** tesseract has joined #openstack-ironic07:01
*** sacharya has joined #openstack-ironic07:04
*** fragatina has quit IRC07:04
*** irf has joined #openstack-ironic07:06
*** mkovacik__ has quit IRC07:06
*** sacharya has quit IRC07:08
*** ohamada has joined #openstack-ironic07:11
*** yolanda has quit IRC07:17
*** hshiina has joined #openstack-ironic07:22
*** yolanda has joined #openstack-ironic07:23
*** ifarkas has joined #openstack-ironic07:29
*** vinbs has joined #openstack-ironic07:33
*** vinbs has quit IRC07:34
*** daemontool has joined #openstack-ironic07:45
*** Haomeng has quit IRC07:58
*** Haomeng has joined #openstack-ironic07:58
*** derekh has joined #openstack-ironic08:07
*** athomas has joined #openstack-ironic08:11
*** irf has quit IRC08:11
*** irf has joined #openstack-ironic08:12
irfMorning Ironic !!08:12
*** elfosardo_ is now known as elfosardo08:12
*** elfosardo has quit IRC08:13
*** lintan has quit IRC08:16
*** lintan has joined #openstack-ironic08:16
*** openstackgerrit has quit IRC08:17
*** openstackgerrit has joined #openstack-ironic08:18
*** lucas-afk has quit IRC08:19
*** daemontool has quit IRC08:21
*** yuanying has quit IRC08:23
*** chihhsin has joined #openstack-ironic08:24
*** lucasagomes has joined #openstack-ironic08:24
*** stendulker has joined #openstack-ironic08:25
*** mkovacik__ has joined #openstack-ironic08:26
lucasagomesmorning all :-)08:26
lucasagomeshey mrda irf08:27
*** athomas has quit IRC08:29
*** athomas has joined #openstack-ironic08:30
irfas continuation to the issue which i was facing last week, i was referring to the portion : http://docs.openstack.org/developer/ironic/deploy/install-guide.html#configure-networking-to-communicate-with-the-bare-metal-server08:30
irfdo i need to create the flat network for baremetal ? to launch the instances?08:31
vmud213morning lucasagomes,irf08:31
*** jistr has joined #openstack-ironic08:32
lucasagomesirf, for the moment, yes08:32
lucasagomesthere are patches in the queue for ironic and nova (I believe neutron ones are merged already) that allow you to have network isolation08:33
lucasagomesstill work in progress08:33
irfok08:33
irfi was thinking of using the existing network for bare metal node08:34
openstackgerritAparna proposed openstack/ironic: Adds RAID interface for 'iscsi_ilo'  https://review.openstack.org/31401608:34
*** fragatina has joined #openstack-ironic08:36
irfok.. moving further , while configuring the drivers for bare metal service , i have followed the steps for PXE setup as i am using pxe_wol driver .. is it enough ? Or do i need to folow the steps for UEFI and iPXE also ?08:39
lucasagomesirf, that depends on what you want, if you want iPXE to boot the machine you should follow the iPXE section08:41
lucasagomesfor UEFI, pxe_wol is not able to flip the boot mode at deploy time so, it probably need to be manually configured if you want to have it08:41
lucasagomes(flip by adding the boot_mode:uefi capabilitiy I mean)08:42
irfok so probably i will use just PXE then08:42
*** e0ne has joined #openstack-ironic08:42
*** Nisha has joined #openstack-ironic08:43
irfi have used the boot options as local08:46
irfusing this command : ironic node-update <node-uuid> add properties/capabilities="boot_option:local"08:46
irfi havent use : UEFI08:48
lucasagomesright08:50
lucasagomesthat's fine08:50
lucasagomes(as a note, uefi is boot_mode not boot_option)08:50
irfok08:55
*** mkovacik__ has quit IRC08:59
*** milan has joined #openstack-ironic09:00
*** linuxgeek has joined #openstack-ironic09:00
milanmorning ironic!09:00
* milan changes nick from mkovacik09:00
*** moshele has joined #openstack-ironic09:01
*** e0ne has quit IRC09:03
openstackgerritLucas Alvares Gomes proposed openstack/pyghmi: DO NOT REVIEW, TESTING GATE  https://review.openstack.org/31363809:03
irfia m still getting : No valid host was found. There are not enough hosts available. I think Ironic service and Nova are not able to comunicate properly ... in the nova conductor logs, i am seeing above error but dont know where to look for now ...09:03
irfi referd this as well : http://docs.openstack.org/developer/ironic/deploy/troubleshooting.html#nova-returns-no-valid-host-was-found-error09:04
*** sacharya has joined #openstack-ironic09:05
*** sacharya has quit IRC09:10
irfplease help09:10
lucasagomesirf, that's a tricky error, have you checked n-scheduler logs see if there's something there?09:17
*** dtantsur|afk is now known as dtantsur09:19
dtantsurMorning Ironic, happy monday! :-P09:19
lucasagomesdtantsur, hey ya, morning09:19
dtantsurseems like I've finally recovered from Austin \o/09:20
irfyes .. i have checked the logs and there also same error .....09:20
lucasagomesJayF, quick updated on the vbmc thingy, apparently disabling the rsp timeout in pyghmi (https://github.com/openstack/pyghmi/blob/master/pyghmi/ipmi/bmc.py#L147) makes it way more reliable09:20
lucasagomesJayF, problem being not configurable by default :-) (/me will submit a patch to pyghmi)09:21
lucasagomesirf, right, but it doesn't say anything else? Can I see the output of nova hypervisor-stats?09:21
irfyes sure ...09:21
dtantsurlucasagomes, is our gate fine? the whiteboard says so, but I've seen strange errors over the weekend09:23
irfhere it is : http://pastebin.com/isB1eWmd09:23
lucasagomesdtantsur, I'm not sure to be honest, lemme check09:24
dtantsurno worries, I'll figure out09:24
* lucasagomes was looking more into pyghmi's gate now09:24
dtantsurjust thought maybe you already know09:24
dtantsurcause it was nova failed to connect to ironic with http 40109:24
lucasagomesthere's one suspicious failure here http://logs.openstack.org/16/314016/1/check/gate-tempest-dsvm-ironic-pxe_ipa/9a11ac8/logs/devstacklog.txt.gz#_2016-05-09_08_44_02_42109:25
Nishadtantsur, ++ even i am facing the same error for ilo ci09:25
dtantsuroooops09:25
Nishafrom today09:25
* dtantsur updates the whiteboard09:25
Nishadtantsur, till friday it was working09:26
dtantsurlucasagomes, this one is fine, I'm rather talking about https://review.openstack.org/31373009:26
dtantsurNisha, a link handly?09:27
dtantsur* handy09:27
Nishahttp://logs.openstack.org/52/285852/44/check/gate-tempest-dsvm-ironic-pxe_ssh/153769f/logs/screen-n-cpu.txt.gz09:28
dtantsuryeah, the same as in inspector gate. thanks09:28
Nishadtantsur, yeah looks like nova has pulled off the support from v2 of keystone09:29
dtantsurwonderful09:29
Nishawhile ironic uses keystone v309:29
*** hshiina has quit IRC09:29
Nishasorry09:29
Nishav209:29
lucasagomesstrange yes failing waiting for hyp-stats hmm09:30
Nishadtantsur, i was strugglingon same from morning, then i checked even the stable ironci gates are failing jenkins due to this09:30
irfyes.09:30
Nishayeah because n-cpu doesnt start09:30
dtantsurmeh..09:30
lucasagomesirf, is it a hybrid (vm + baremetal) cloud?09:31
dtantsurhttps://github.com/openstack-dev/devstack/commit/251b870e859e1a03b57c8fe24c7692a5e2a10fde ?09:31
irfyes09:31
*** electrofelix has joined #openstack-ironic09:31
lucasagomesirf, but apparently you have only 1 nova-compute right?09:31
irfjust to mention again ... all openstack services are deployed on VM along with Ironic09:31
irfyes09:32
lucasagomescount=1 there in the hypervisors-stats09:32
irfyes09:32
irfmy bare metal server is just one normal desktop machine09:32
lucasagomesirf, so that doesn't work, cause you need > n-computes, at least 1 to manage baremetal (with the ironic virt driver loaded) and the other one for your vms09:33
Nishadtantsur, lucasagomes so we need to revert back this fix09:33
lucasagomesirf, which driver is loaded on that nova compute?09:33
dtantsurNisha, on it right now09:33
irfthis is writeen in nova.conf file in controller node : compute_driver=ironic.IronicDriver09:34
* lucasagomes jumps on a quick meeting09:34
irfon the compute node as well same line in nova.conf09:35
irfbut, compute_manager line is commented ....09:35
dtantsurrevert: https://review.openstack.org/#/c/314024/09:36
*** daemontool has joined #openstack-ironic09:45
*** baoli has joined #openstack-ironic09:50
Nishadtantsur, i started the ilo gate manually with the above patch. will update by my evening ( next 2-3 hrs)09:54
*** baoli has quit IRC09:54
irfare u back from meeting ?09:55
dtantsurgreat! we also have a (non-voting unfortunately) gate on devstack, so we'll know if the problem is fixed09:56
* Nisha leaving for home right now. will connect from home later09:56
*** sambetts|afk is now known as sambetts09:57
dtantsurok folks, get ready for some ugly rant09:58
dtantsurhow could they seriously merge https://review.openstack.org/#/c/300151/2/lib/nova_plugins/hypervisor-ironic if these options are not even present in https://github.com/openstack/nova/blob/master/nova/conf/ironic.py ???09:58
sambettsdtantsur: is that whats blown everything up ?!10:00
dtantsuryep10:01
* dtantsur is sad at devstack reviews..10:01
*** Nisha has quit IRC10:01
sambettsDon't we co-gate devstack? or are we non-voting?10:01
*** hshiina has joined #openstack-ironic10:02
sambettswhere are you seeing the errors from that? The main errors I'm seeing is the nova hypervisor >= 1 one which points at the race condition I've highlighted10:03
* lucasagomes is back now10:04
lucasagomesirf, ok, but I see that you have some used resources over there10:05
lucasagomes memory_mb_used       | 614410:05
irfok ..10:05
lucasagomesbecause for each baremetal machine it will actually count as a hypervisor to nova10:05
irfnot gettig sorry10:06
lucasagomesso where does the used resources coming from?10:06
dtantsursambetts, we're non-voting, and people ignored the failed gate10:06
irfmay be the case that VMs are using thise MBs?10:06
dtantsursambetts, hypervisor stats error is because of http://logs.openstack.org/30/313730/1/check/gate-ironic-inspector-dsvm-ipa/27bc161/logs/screen-n-cpu.txt.gz#_2016-05-08_15_43_49_38010:06
irfcan this be happen ?10:06
lucasagomesirf, it may yes... Because, for baremetal what happens is that10:06
lucasagomesonce you spawn a baremetal node, it will take up all the resources for node10:07
lucasagomesand each node is seems as a hypervisor on it's own in nova10:07
lucasagomesthat's what I mean10:07
lucasagomesso once you deploy it, it resource tracker 100% of it10:07
lucasagomessince you have some resources used (idk how yet) it may be failing there10:07
aparnav Hey lucasagomes, I have a query about your comment on this patch https://review.openstack.org/#/c/30722510:08
aparnavIs the check for task.is_take_over() necessary?10:08
irfok so how shud i rectify that ... any command to check that or sometineg ?10:08
lucasagomesaparnav, lemme read the patch, cause I don't remember my comment10:09
aparnavlucasagomes, sure10:09
lucasagomesirf, hmm not that I know of :-/10:09
irfi am sure , there is some issue in nova configuration but not sure what is that exactly10:09
lucasagomesperhaps the filters you've been using?10:10
lucasagomeswe have some docs about hw to configure nova10:10
sambettsdtantsur: ah, I'm also seeing the hypervisor stats error because of the race condition I identified, results in similar devstack errors10:10
lucasagomesirf, http://docs.openstack.org/developer/ironic/deploy/install-guide.html#configure-compute-to-use-the-bare-metal-service10:10
dtantsursambetts, ETOOMANYERRORS :D10:10
sambettsquite ...10:10
irfi did this already ...10:11
irfhow about this : http://docs.openstack.org/developer/ironic/deploy/install-guide.html#configure-networking-to-communicate-with-the-bare-metal-server10:11
*** daemontool_ has joined #openstack-ironic10:11
sambettsirf: are you running two nova cpus?10:11
irfu mean nova compute ?10:12
sambettsyes10:12
*** e0ne has joined #openstack-ironic10:12
irfno only one nova compute .. as per the command : nova hypervisor-stats10:12
*** daemontool has quit IRC10:14
sambettsirf: if you want to run BM and VMs then you need to run 2 nova computes, 1 on your compute host configured with libvirt for hosting VMs, and another either on your controller or somewhere else configured with the Ironic compute driver etc to communicate with Ironic and provide BMs10:14
*** hoangcx has quit IRC10:15
irfok so one nova compute is already running ... which is a nova compute node ...10:15
irfnova-compute service is running on this10:16
sambettsright and that is going to host VMs10:16
sambettsnow you need a new nova compute, configured to talk to Ironic to provide the BM10:16
irfok .. just to clear ... the current nova compute is running on VM. so it cant provision the bare metal node .. ?10:17
sambettsno, the fact its running in a VM has nothing to do with it, the problem is each nova compute service can only provide VMs or BM not both10:18
sambettsyou need multiple nova compute service to do that10:18
lucasagomesaparnav, problem being, how do we implement such is_take_over() if not by listening the states10:19
irfohh ok10:19
irfand i am trying to provision the both VM and BM as well10:19
lucasagomesaparnav, that shoulds like that, when _do_take_over run in the manager we should somehow pass a flag (via a parameter) to the prepare() function ?10:19
lucasagomesI haven't looked much into it10:19
lucasagomesbut wondering how we could implement that10:19
irfso where i shud install new nova compute .. ?10:20
sambettsit can run on the controller or somewhere else10:20
lucasagomesaparnav, or maybe somehow say it when you get the lock? Cause I don't think that checking the purpose string is a good way of doing that10:20
irfand i understand that , controller can have new nova compute even if it is on the VM...10:21
sambettsif your VM is powerful enough to run that extra service10:21
aparnavlucasagomes, My question is do we really need to check for take_over? Cant we just check the provision_state since it the check will be only need for deploying and cleaning, also for inspection once it is added10:21
irfok10:22
lucasagomesaparnav, yeah I was good with it as well10:22
irfbut another dowbt ,,, nova schedulear and nova compute can be on same macbine ? any issue of haveing them on single VM ?10:23
sambettsno10:23
irfok10:23
sambettsthey can be on the same machine the defautl devstack setup runs every service on the same machine10:24
irfso whatever changes i have done in the nova.conf file on the nova compute .. i will remove that and do the same changes in the new nova compute once it is running on the controller node ....10:25
sambettsyup :)10:25
aparnavlucasagomes, the process of making the _do_takeover to send pass a flag or getting the lock seems complex. So I will proceed with checking the states.10:25
lucasagomesaparnav, +110:25
lucasagomesyeah, it requires some plumbing10:25
irfawesome !!! thanks much .. will update once i configure that ...10:26
aparnavlucasagomes, thank you :)10:26
irffurther, is this by design that single nova compute service provisions only VMs or BMs but not both ?10:31
*** daemontool__ has joined #openstack-ironic10:31
irfcan u plz help me in redirecting the documentation ...10:31
sambettsirf: yes, each nova compute can only be configured with one type of hypervisor, e.g. kvm, xen, or baremetal(ironic)10:31
lucasagomesaparnav, thank YOU for the fix10:32
lucasagomesirf, yup, that's how nova works10:32
*** daemontool_ has quit IRC10:34
*** e0ne has quit IRC10:35
*** daemontool has joined #openstack-ironic10:36
*** daemontool__ has quit IRC10:37
*** e0ne has joined #openstack-ironic10:39
openstackgerritLucas Alvares Gomes proposed openstack/pyghmi: Allows BMCs to configure the event loop timeout  https://review.openstack.org/31404510:39
*** mjturek1 has quit IRC10:41
*** daemontool_ has joined #openstack-ironic10:41
irfok ... thanks much10:41
*** MattMan has quit IRC10:41
*** MattMan has joined #openstack-ironic10:42
lucasagomesJayF, ^10:42
lucasagomesI can confirm that tunning that timeout makes it way more stable10:43
*** daemontool__ has joined #openstack-ironic10:43
irfafter configuration of new nova compute service , hypervisor-stats should show two hypervisor rite ?10:43
lucasagomesyup10:43
irfok10:43
*** daemontool has quit IRC10:44
*** daemontool_ has quit IRC10:46
*** daemontool_ has joined #openstack-ironic10:51
*** baoli has joined #openstack-ironic10:51
*** marios has quit IRC10:52
*** baoli has quit IRC10:55
*** daemontool__ has quit IRC10:55
*** mgould has joined #openstack-ironic10:59
*** daemontool__ has joined #openstack-ironic10:59
milanhey guys, we've got +2x2 on the Inspector HA spec, should we possibly workflow+1 it?  https://review.openstack.org/#/c/253675/2010:59
sambettsmilan: done ;)10:59
mgouldmorning Ironicers!11:00
sambettsHey mgould!11:00
* mgould checks the clock11:00
milansambetts, that's fast! thanks :D11:00
mgoulddamn, afternoon :-)11:00
mgouldsambetts, milan hi!11:00
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: Restructure the repository according to OpenStack  https://review.openstack.org/31302511:00
milanmorning sambetts and moguld :)11:00
sambettsmgould: according to my clock you were 1 minute still in the morning haha11:00
mgould\o/11:01
dtantsurmilan, heh, forgot about i, sorry11:01
dtantsurafternoon, mgould :)11:01
milandtantsur, no prob11:01
openstackgerritMerged openstack/ironic-inspector-specs: High Availability for Ironic Inspector  https://review.openstack.org/25367511:02
mgoulddtantsur, afternoon :-)11:02
mgouldmilan, high-five!11:02
milanmgould, yay :D11:03
milanhig-five!11:03
*** daemontool_ has quit IRC11:03
milanmgould, now the fun will start :D11:04
*** daemontool has joined #openstack-ironic11:04
mgouldwait, you mean that was the easy bit?11:05
* mgould cowers in a corner11:05
*** sacharya has joined #openstack-ironic11:05
*** e0ne has quit IRC11:06
*** daemontool_ has joined #openstack-ironic11:07
*** daemontool__ has quit IRC11:07
milanmgould, seems I've got to hack it now :D11:08
dtantsurhappy hacking :D11:08
*** daemontool__ has joined #openstack-ironic11:08
milanso, I'd say that was the easy bit :D11:08
dtantsurthat reminds me to submit a release request for inspector11:08
milandtantsur, thanks :D11:08
milandtantsur, submit a preservation request and a solid fork maybe too ;)11:09
dtantsurhmm, I'd love to land https://review.openstack.org/#/c/276331/ before the release, so I'll update it today11:10
*** daemontool has quit IRC11:10
*** sacharya has quit IRC11:10
sambetts+111:11
*** daemontool has joined #openstack-ironic11:11
*** daemontool_ has quit IRC11:12
lucasagomesmgould, mat128 if you guys are happy with https://review.openstack.org/#/c/313025/ I will go ahead and merge it11:12
mgouldlucasagomes, go for it11:12
lucasagomesI want to start doing some work on adding unittests for vbmc and things like that11:12
lucasagomesmgould, cool, thanks a lot for all the reviews! Very appreciated11:13
lucasagomesmgould, fyi, I've uncluded the umask + chdir when detaching the process11:13
mgouldlucasagomes, cool11:14
*** daemontool_ has joined #openstack-ironic11:14
dtantsurlucasagomes, checking now11:15
*** daemontool__ has quit IRC11:15
lucasagomesdtantsur, ack11:15
*** thrash|wknd is now known as thrash11:15
* lucasagomes waits11:15
dtantsurdone11:16
*** smoriya has quit IRC11:17
*** daemontool has quit IRC11:18
jrollmorning everyone11:19
jrollYAY GATE11:20
lucasagomesjroll, heh down again :-/11:20
dtantsurmorning jroll, yes :(11:20
dtantsurhappy monday11:20
jrollyah, pinged some folks11:20
lucasagomesjroll, if that makes you more comfortable, the new job on pyghmi's gate detected its first failure: http://logs.openstack.org/12/313012/4/check/gate-tempest-dsvm-ironic-pxe_ipmitool-pyghmi-src-nv/c5be74f/logs/screen-ir-cond.txt.gz#_2016-05-09_08_46_37_07311:20
lucasagomes(which happens prior to the waiting for hypervisor-stats thingy)11:21
jrolllucasagomes: woot11:21
jrollso, is there any change we need to make in nova for keystone v3 to work with our client?11:25
jrollpas-ha: vdrok ^ I think you know things about this11:25
odyssey4mejroll I think we're already using Keystone v3 for the OSA implementation - let me check11:26
jrollyeah, I think it should just work, but it's unclear to me if it actually does :)11:27
odyssey4mejroll hmm, ok so we have done it in the neutron section: https://github.com/openstack/openstack-ansible-ironic/blob/master/templates/ironic.conf.j2#L100-L11311:28
odyssey4mebut I'm not seeing a nova section, unless I'm blind11:28
odyssey4meshould there be a nova section?11:29
jrollodyssey4me: well, there's ironic configuration done in nova.conf11:29
jrollso the virt driver can talk to ironic11:30
*** stendulker has quit IRC11:30
odyssey4meah yes11:30
odyssey4meyuk, this is keystone v2 https://github.com/openstack/openstack-ansible-os_nova/blob/master/templates/nova.conf.j2#L245-L25311:31
jrolleh, I'm still not so sure. the options look like v2, however I think our client can handle v2 or v311:31
jrollbut that todo makes me unsure11:31
odyssey4memrda ^ I see your note - please test whether changing that up to Keystone v3 will work?11:31
odyssey4mejroll yeah, if your client is doing that, then as I understand it you're including logic that you shouldn't be... the keystone auth library already has that logic11:32
vmud213lucasagomes:I didn't get your comment on https://review.openstack.org/#/c/31161311:32
jrollodyssey4me: yeah, it's hard to tell, I'm not super familiar11:32
odyssey4meif you're shimming additional logic in front of that then you're walking into tech debt and causing an inconsistent operator experience11:32
vmud213lucasagomes: bios local boot support for GPT partition table11:32
jrollodyssey4me: changing those nova.conf things directly to v3 options certainly doesn't work, devstack did that and broke our gate :)11:33
odyssey4menot you personally, of course11:33
vmud213should i keep the Bios boot partition configurable or hardcoded?11:33
lucasagomesvmud213, on the release note?11:33
lucasagomesvmud213, I'm good with it being configurable (I added a comment in the patch-set, not per line about it)11:33
jrollodyssey4me: we do have quite a mess here, though https://github.com/openstack/python-ironicclient/blob/master/ironicclient/client.py#L2011:33
lucasagomesvmud213, my -1 was about the release note11:33
vmud213I think it'd always be better to keep it as configurable parameters11:33
lucasagomesvmud213, me too11:33
lucasagomesvmud213, my -1 was because of the release note11:34
vmud213ok..so i see some commnet even on that..No problem.11:34
lucasagomesvmud213, see my comment at the bottom of the patch11:34
vmud213i just make changes to release note and other nits.11:34
lucasagomesvmud213, (at "history")11:34
odyssey4mejroll yeah, I've seen those a few times - what a mess11:34
lucasagomesvmud213, yup11:34
vmud213lucasagomes, got it.thanks11:35
lucasagomescool, thanks for the patch11:35
odyssey4mejroll I would recommend having a chat with dolphm / lbragstad (or generally in #openstack-keystone) about what needs to be done to make it work right... and perhaps they can point you at a client that's doing it the right way11:35
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: Restructure the repository according to OpenStack  https://review.openstack.org/31302511:35
*** e0ne has joined #openstack-ironic11:36
jrollodyssey4me: sure, I agree11:36
*** amotoki has quit IRC11:36
*** aparnav has quit IRC11:36
*** frontrunner has quit IRC11:37
jrolldtantsur: congrats on EOL :)11:39
*** appu103_ has joined #openstack-ironic11:39
mgouldvmud213, not sure about this specific parameter, but in general: hardcoded if possible, to reduce code and ops complexity, but configurable if every possible answer will break some subset of users11:41
*** e0ne has quit IRC11:42
vmud213mgould, i understand. But in this context i hope better to leave it as a configurable parameter.11:42
* lucasagomes goes grab some food11:42
*** lucasagomes is now known as lucas-hungry11:43
mgouldvmud213, yeah, sounds like you're in situation #2 here11:43
*** e0ne has joined #openstack-ironic11:46
openstackgerritMerged openstack/virtualbmc: Restructure the repository according to OpenStack  https://review.openstack.org/31302511:49
*** nathan-h has joined #openstack-ironic11:53
*** fragatina has quit IRC11:54
jroll11:55:09  openstackgerrit | Jim Rollenhagen proposed openstack-infra/project-config: Give ironic grenade jobs 7 "nodes" running tinyipa  https://review.openstack.org/31406611:55
jrollshould be useful11:55
sambettssweet :D11:55
jrollI realized smoke tests must be passing for people if they are getting to the resource create bits11:56
jrollI thought the order was the other way for some reason11:56
sambettsI think it is the other way around :/11:57
*** yolanda has quit IRC11:57
sambettsat least thats how I understood it11:57
*** e0ne has quit IRC11:58
jrolllooking at the gate it runs smoke tests first11:58
*** baoli has joined #openstack-ironic11:58
jrollwell, and the code https://github.com/openstack-dev/grenade/blob/master/grenade.sh#L241-L26211:58
*** dprince has joined #openstack-ironic12:00
sambetts:/ oh ... well thats pretty good then12:01
dtantsurjroll, heh, thanks :)12:01
*** yolanda has joined #openstack-ironic12:02
*** baoli has quit IRC12:03
openstackgerritvinay kumar muddu proposed openstack/ironic-lib: Add support for BIOS local boot for GPT label  https://review.openstack.org/31161312:06
dtantsurjroll, lucas-hungry, I think we should keep an eye on https://review.openstack.org/#/c/300154/12:08
jrolldtantsur: heh, so a missing depends-on is what broke us, eh12:09
dtantsuryep12:09
jrollcool12:11
sambetts\0\12:12
sambetts\-/12:12
sambettsffs...12:12
sambetts\0/12:12
jrolllol12:13
*** early` has quit IRC12:14
*** baoli has joined #openstack-ironic12:14
*** mjturek1 has joined #openstack-ironic12:24
openstackgerritMathieu Mitchell proposed openstack/ironic: Allow configuring shred's final overwrite with zeros  https://review.openstack.org/30410112:24
*** appu103_ has quit IRC12:25
irfhello , i am back ..12:27
*** jjohnson2_ has joined #openstack-ironic12:27
irfconfigured the nova compute service on the controoler node and now hypervisor-stats command shoes two compute nodes12:27
irfdo i need to foloow any specific steps to use the baremetal hypervisor for the second compute service ?12:28
sambettsirf: the ones in the install guide, e.g. setting the right virt driver etc12:28
*** mgould has quit IRC12:29
irfyes i did that ..12:30
irfi set virt_type=qemu in the nova-compute conf file12:30
irffull path of the file : /etc/nova/nova-compute.conf12:30
irfboth the hypervisors are enabled and up12:31
sambettsqemu? thats a virtual machine hypervisor, you need to set it to the ironic virt driver, like in the install guide http://docs.openstack.org/developer/ironic/deploy/install-guide.html#configure-compute-to-use-the-bare-metal-service compute_driver=ironic.IronicDriver12:31
irfohhh12:32
irfsorrry12:32
*** daemontool__ has joined #openstack-ironic12:41
*** early has joined #openstack-ironic12:43
*** daemontool_ has quit IRC12:45
openstackgerritMerged openstack/pyghmi: Add Hardware inventory from Lenovo Agentless  https://review.openstack.org/31256812:46
*** Nisha has joined #openstack-ironic12:47
irfok .. i have added compute_driver=ironic.IronicDriver in the default section of nova-compute file12:47
irfremoved the entry : [libvert]12:48
*** marios has joined #openstack-ironic12:50
irfok now my Ironic node is being listed as hypervisor host for the command : nova hypervisor-list is it rite ?12:52
sambettsYes :D12:52
Nishadtantsur, your patch fixed the stack issue12:52
Nisha:)12:52
irfok gre812:52
dtantsurNisha, awesome, thanks for checkingf12:52
*** lucas-hungry is now known as lucasagomes12:54
*** mgould has joined #openstack-ironic12:54
lucasagomesdtantsur, ++ will take a look12:55
*** vmud213 has quit IRC12:56
*** irf has quit IRC12:56
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: [inspection] wait for the PXE DHCP by default and remove the carrier check  https://review.openstack.org/31351112:57
dtantsursambetts, mat128, this one should be fine by you hopefully :) ^^^12:57
*** zzzeek has joined #openstack-ironic12:58
*** yolanda has quit IRC12:58
*** ppiela has joined #openstack-ironic13:00
Nishadtantsur, do we have any code patches for https://review.openstack.org/#/c/305864/13:00
mat128dtantsur: works for me :) I thought we had discussed options tho (c.f. "folks not proposing anything")13:01
sambettsdtantsur: https://review.openstack.org/#/c/314086/ I've just pushed this btw, could you maybe try it and see if it helps too? my concern is that dhcp-all-interfaces won't actually  configure an interface for dhcp if carrier != 1 so I don't know how the node in your bug zilla is getting an IP if carrier is wrong13:01
dtantsurNisha, not yet. it was on my radar, but I got distracted by bugs. I want a buy-in from at least one core though13:01
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Support Ironic node names in our API  https://review.openstack.org/27633113:01
dtantsurmat128, well, not something from the top of my head :(13:01
Nishadtantsur, :) I was trying to use the code patches for my inband inspection support in ilo drivers13:02
mat128dtantsur: no problem, I remember suggesting something like "trust_sysfs=false" or something, but don't worry about it :) your current patch makes it even better13:02
*** links has quit IRC13:02
dtantsursambetts, DIB magic, yeah... it definitely gets an IP *somehow* otherwise IPA would not come back13:02
Nishadtantsur, as this is the base for any code changes i do for inband inspection supprot there13:03
dtantsurmat128, yeah, sorry. My problem with trust_sysfs is that we're shifting the decision to a user, and a user/operator might not actually know13:03
dtantsurmat128, so instead I suggest them to specify what they expect us to return *for sure*. I think it's more fair to them.13:03
*** rloo has joined #openstack-ironic13:03
mat128yep, your patch is perfect as is and works for every use case I can think of :)13:04
*** yolanda has joined #openstack-ironic13:04
dtantsurgreat!13:04
dtantsurNisha, I'll come to it soon. Please review the spec in the meanwhile.13:04
NishaYeah thats in my list...13:04
Nishadtantsur, lucasagomes regarding comments on https://review.openstack.org/#/c/302403/13:06
dtantsursambetts, I need to say that building tinyipa is substantially faster than the coreos one!13:06
* dtantsur uses it in his testing now13:06
sambettsdtantsur: :D13:06
*** sacharya has joined #openstack-ironic13:06
* sambetts so happy he's made a significant difference to Ironic with tinyipa hehe13:07
*** jcoufal has joined #openstack-ironic13:07
dtantsuryeah, cheers :)13:07
* dtantsur is looking forward to dropping coreos here13:08
* dtantsur wishes he could avoid DIB just as easily.....13:08
Nishadtantsur, ilo drivers need to use DIB13:10
sambettsthat DIB patch I added will hopfully make it better for you, not sure though, are you stuck on DIB because you have to build a RH based IPA ramdisk?13:10
sambettsNisha: for making an ISO?13:10
dtantsurNisha, I know... I'd prefer we use something different to build images, e.g. libguestfs13:10
Nishasambetts, yeah as DIB only supports uefi13:10
Nishaand secureboot13:10
*** sacharya has quit IRC13:11
Nishaand frankly building images thru coreos/tinyipa doesnt work here in the servers i am trying...:( couldnt make that work suceesfully...anyway i need DIB for testing it on gate13:12
sambettshmm, I wonder what changes I need to make to tinyipa to make it support those :/ there is "make all" or "make then make iso" in tinyipa that'll spit out an iso btw :)13:12
*** krtaylor has quit IRC13:12
Nishasambetts, its not those steps, generally it fails at puuling the coreos/tinyipa initially step only13:13
Nishait doesnt go thru the process i would say13:13
Nishaso i dont say that it doesnt work, but i would i am not able to make it work13:13
mat128Nisha: could you post a log by any chance?13:13
sambettsNisha: I'd love to know whats missing / what error you see with tinyipa on your end13:13
sambettsso we can get it working for you13:14
Nishamat128, i would do once i get chance to do, i need to make ilo CI public asap13:14
Nishasambetts, ilo CI needs to use DIB for UEFI and secureboot13:14
Nishathose are not supported by coreos and tinyipa yet13:15
*** jtomasek_ has joined #openstack-ironic13:15
Nishadtantsur, could you look into https://review.openstack.org/#/c/302403/ and let me know ur opinion on the comments13:15
*** zzzeek has quit IRC13:16
sambettsNisha: can I ask what elements you use to build IPA through DIB so I can check out the code and work out what needs adding to tinyipa to make it work with those13:16
*** amotoki has joined #openstack-ironic13:17
*** Nisha_away has joined #openstack-ironic13:17
*** zzzeek has joined #openstack-ironic13:18
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: "vbmc list" to sort the result by domain name  https://review.openstack.org/31409513:18
*** krtaylor has joined #openstack-ironic13:19
*** Nisha_brb has joined #openstack-ironic13:20
*** Nisha has quit IRC13:20
*** KennethWilke has quit IRC13:21
*** Nisha_away has quit IRC13:22
*** jtomasek_ has quit IRC13:22
dtantsursambetts, I guess tinyipa does not have journalctl?13:23
sambettsdtantsur: no, its not systemd13:23
mat128dtantsur: CoreOS does13:23
mat128if thats the only thing thats blocking you13:23
lucasagomesjjohnson2_, around? when you have some time, mind taking a look at https://review.openstack.org/#/c/314045/  ?13:23
dtantsurjust checking, not important right now13:23
dtantsurthnx13:23
*** rbudden has joined #openstack-ironic13:24
jjohnson2_lucasagomes, so weird, I still don't get notified about pyghmi reviews naturally13:24
lucasagomesjjohnson2_, btw, it would be great to have unittests in pyghmi (I'm filling a bug about it)13:25
*** marios has quit IRC13:25
lucasagomesjjohnson2_, oh, yeah notifications are handy13:25
lucasagomesthere's also a problem with the python 3 patch, I left a link to the error in the review13:25
jjohnson2_lucasagomes, yeah, they don't even appear on incoming reviews...13:25
*** marios has joined #openstack-ironic13:25
jjohnson2_lucasagomes, yeah, I mentioned it don't work... ;)13:25
lucasagomesoh heh13:25
lucasagomesjjohnson2_, at least we can detect it on gate now :-)13:25
dtantsursambetts, aha. maybe we need to fix the 'logs' inspection collector to support whatever is for logging there13:26
jjohnson2_lucasagomes, the virtual bmc code is broken by the changes13:26
jjohnson2_lucasagomes, so it just hangs out13:26
jjohnson2_I mean the review just hangs out13:26
lucasagomesright on13:26
jjohnson2_the error is an error13:26
lucasagomesI can take a look if needed13:26
jjohnson2_it's too early on a monday for me to speak good13:26
openstackgerritMerged openstack/pyghmi: Allows BMCs to configure the event loop timeout  https://review.openstack.org/31404513:27
sambettsdtantsur: yeah, might be a plan, I think by default even syslog is disabled because it takes more RAM XD13:27
*** keedya has joined #openstack-ironic13:27
jjohnson2_lucasagomes, if you don't mind, it's a matter of the whole bytes/bytearray/string stuff being fixed13:27
jjohnson2_lucasagomes, actually, I didn't look at the gate failure yet, just based on my recollection of tests before even running git review13:27
dtantsursambetts, ouch :D13:27
*** vishwanathj has joined #openstack-ironic13:28
*** Nisha_away has joined #openstack-ironic13:28
lucasagomesjjohnson2_, yeah no problems!13:28
jjohnson2_lucasagomes, one would think integration tests by starting a fake bmc and then running commands against the fake bmc would be handy13:29
lucasagomesjjohnson2_, ++ totally13:29
lucasagomesand in parallel we should start working on unittests for pyghmi13:30
jjohnson2_of course, I need to upload some sample FRU, SPD, and so on and so forth for the parser fuctions to hit13:30
jlvillallucasagomes, I was wondering if you had a chance to test Grenade anymore?13:30
jjohnson2_and now to work my other job...13:30
jlvillalmgould, Also wondering if you had made any progress on Grenade? :)13:31
*** KennethWilke has joined #openstack-ironic13:31
mgouldjlvillal, nope13:31
*** Nisha_brb has quit IRC13:31
mgouldyou?13:31
lucasagomesjlvillal, I haven't runned it today, on friday I passed the network problem using jroll's patch13:31
lucasagomesthen hit one about not having enough loop devices13:32
lucasagomesso I manually created 200 of 'em heh now I'm hitting something different, lemme run it again13:32
lucasagomessee what I get13:32
jlvillallucasagomes, Thanks. I think you got farther than me. I am dying elsewhere. I will update information when I get to work.13:32
*** rloo has quit IRC13:32
*** mtanino has joined #openstack-ironic13:32
*** rloo has joined #openstack-ironic13:33
jlvillalmgould, Thanks. I've been running it. Getting an error. I will update etherpad in next few hours13:33
mgouldjlvillal, cool13:33
mgouldare you going to be around for the meeting today?13:33
jlvillalYep13:33
lucasagomesjlvillal, cool, I'm trying to split the time between vbmc/pyghmi's and grenade cause both needs a lot of attention at least for now13:33
jlvillallucasagomes, Thanks!13:33
*** yolanda has quit IRC13:36
dtantsursambetts, you are fine with backport https://review.openstack.org/#/c/313932/ ?13:37
sambettsdtantsur: yup +113:37
dtantsursambetts, please +2 then ;)13:39
sambettssure :F13:41
sambetts:D13:41
*** yolanda has joined #openstack-ironic13:43
dtantsurjroll, wdyt about https://bugs.launchpad.net/ironic/+bug/157894713:45
openstackLaunchpad bug 1578947 in Ironic "No need to do node clean if it deleted from a deploy failed state" [Undecided,Opinion] - Assigned to Zhenguo Niu (niu-zglinux)13:45
jrolldtantsur: totally agree with you, posted as such13:46
sambettswe about to respond we exactly the same thing13:47
NobodyCamgood morning Ironic :)13:47
jrollI for one know that my deployment needs a reclean if a deploy fails13:47
dtantsurdevananda, hey, I've see your (expired) bug about us not fully wiping the GPT backup. I suspect https://bugs.launchpad.net/ironic-python-agent/+bug/1576526 might be a case of it.13:48
openstackLaunchpad bug 1576526 in ironic-python-agent "Issue in reteieving UUID of the root partition during boot loader installation" [Undecided,Incomplete]13:48
*** david-lyle_ has joined #openstack-ironic13:48
*** mtanino has quit IRC13:49
*** milan has quit IRC13:49
*** david-lyle has quit IRC13:50
*** xhku has joined #openstack-ironic13:51
*** ametts has joined #openstack-ironic13:53
dtantsurjroll, an RFE for our attention: https://bugs.launchpad.net/ironic/+bug/157593513:54
openstackLaunchpad bug 1575935 in Ironic "[RFE] Rebuild should also accept a configdrive" [Wishlist,Confirmed] - Assigned to Stephane Miller (stephaneeee)13:54
jrolldtantsur: my opinion on that, without clicking, is "we should do whatever VMs do"13:56
* dtantsur has no idea what VM's do13:56
sambetts+1 for VM convergence13:57
mgouldNobodyCam, monring13:57
mgould*morning13:57
sambettso/ NobodyCam13:57
dtantsuroh, morning NobodyCam, jlvillal13:57
dtantsurand morning jroll (in case I didn't say it today)13:57
jrolldtantsur: going by api docs it looks like nova supports this, so sure!13:57
jrollheh13:57
jrollmorning :)13:57
jlvillalGood morning dtantsur and everyone else! :)13:58
NobodyCammorning mgould sambetts dtantsur jroll13:58
NobodyCamjlvillal: too13:58
*** Goneri has joined #openstack-ironic13:59
dtantsurjroll, sambetts, this sounds like a lot of fun: https://bugs.launchpad.net/ironic/+bug/157609813:59
openstackLaunchpad bug 1576098 in Ironic "[RFE] Support assign different types of network cards to different networks" [Undecided,New] - Assigned to Zhenguo Niu (niu-zglinux)13:59
jrollO_o13:59
sambettsdtantsur: yup, thats the port mapping issue that we've got a follow up spec for14:00
jrolloh, this one is so played out14:00
dtantsurI mean, I see point. But it's gonna be tricky14:00
jrollcmon LP looooad14:00
jrolldtantsur: there's a dupe for it14:00
dtantsurgood to know. a link handy?14:00
sambettsthe only way is supporting a cababilitys like thing for portsa14:00
sambettsman I can't type todaty14:01
sambetts-,-14:01
dtantsur:D14:01
jrolldtantsur: I think https://bugs.launchpad.net/ironic/+bug/154416914:01
openstackLaunchpad bug 1544169 in Ironic "[RFE] Advanced network configuration in ironic" [Wishlist,Confirmed] - Assigned to Vasyl Saienko (vsaienko)14:01
dtantsurthat's monday14:01
*** catintheroof has joined #openstack-ironic14:03
dtantsurjroll, do you think https://bugs.launchpad.net/ironic/+bug/1575249 should be a clean step? do we need a spec here?14:04
openstackLaunchpad bug 1575249 in Ironic "Add image caching support to agent driver" [Wishlist,In progress] - Assigned to Clif Houck (f-te-u)14:04
*** piet has joined #openstack-ironic14:04
dtantsurjroll, also tell me to shut up at any moment if I distract you :) just going through bug notifications14:04
jrolldtantsur: I don't, because going manageable->available will wipe the image right off the disk :)14:07
*** d0ugal has quit IRC14:07
dtantsurjroll, so it should be a clean step with lower priority, right?14:07
jrollfor automated cleaning... maybe, but I like the given model14:07
jrollfor example, it allows some service to do statistics to cache the right amount of different images14:08
jroll(or a person)14:08
*** links has joined #openstack-ironic14:08
dtantsurso when and how should it be called?14:08
rloohello and a good monday morning to everyone, including YOU (jroll, dtantsur, sambetts, NobodyCam, jlvillal, mgould, lucasagomes14:09
jrolldtantsur: while a node is in available, via the API14:09
lucasagomesrloo, hello there, good morning :-)14:09
sambettshey rloo14:09
dtantsurmorning rloo :)14:09
jrollmorning rloo :)14:09
NobodyCamhey hey morning rloo :)14:09
dtantsurjroll, directly via IPA API? or via a new Ironic API? I guess this RFE needs a spec :)14:09
jrolldtantsur: it has one, no?14:09
jrollmaybe not in the bug itself14:10
jrolldtantsur: https://review.openstack.org/#/c/310594/14:10
dtantsurouch, yeah :(14:10
* dtantsur is probably too sleepy14:10
jrolldtantsur: disclaimer with my comments here: we do this in production today14:11
jroll2 minute 'nova boot' ftw14:11
dtantsurI see14:11
*** links has quit IRC14:11
mgouldmorning rloo14:12
rloodtantsur: wrt https://bugs.launchpad.net/python-ironicclient/+bug/156864514:12
openstackLaunchpad bug 1568645 in python-ironicclient "Improve baremetal unittest framework of the OSC " [Undecided,Invalid] - Assigned to xiexs (xiexs)14:12
dtantsurmmm?14:12
*** fragatina has joined #openstack-ironic14:13
rloodtantsur: is it true that we don't track refactorings via our bug tracking system? as an rfe/wishlist maybe?14:13
rloodtantsur: am asking because i know we let some? refactorings get approved w/o a bug i think.14:13
rloodtantsur: but i also think it is useful to have something track the refactoring. depends on how much of a refactoring too.14:13
rloodtantsur: cuz what if we don't want the proposed refactoring.14:14
dtantsurrloo, that's my take on it. if it's a huge refactoring (e.g. futurist switch), it's worth tracking. if it's small tests refactoring (like here), it's not IMO14:14
dtantsurrloo, if we don't want the proposed refactoring, we should -2 the patch. closing the bug wont' help :)14:14
rloodtantsur: 'huge' refactoring is ... questionable. that bug mentions two steps; it seems useful to have the bug that describes the overall intent. anyway, just my opinion.14:16
rloodtantsur: I mean 'huge' is kinda hand waving. not sure how big 'huge' is.14:17
dtantsurrloo, imagine you saying "omg it's huge" :D my problem is that I don't want every small change posted to the bug tracker. I'll go mad figuring them out14:17
rloodtantsur: the other use for the bug tracking is if there is more than one person that intends to do the same thing; at least this is an indication to others that someone is planning on doing it.14:18
dtantsuri.e. I'll have to assess validness of every one of them. I'd prefer it to be done during reviews instead14:18
dtantsurrloo, I'd prefer bug tracking to stay bug tracking (+ rfe) for reasons stated above :) if we can make the whole core team triage the bugs regularly (just as reviews), I can change my mind14:19
rloodtantsur: oh. i'd rather assess validness earlier in the process; seems like that would save people time. instead of someone doing all the code changes etc and then finding out that we don't want it.14:19
dtantsurrloo, that would save their time, but waste mine14:19
dtantsuralso it will make the future of their proposal depend on one person triaging their bug14:19
*** Nisha_away has quit IRC14:19
dtantsurwhich is contrary to our 2x +2 rule in gerrit14:20
rloodtantsur: ah. I see. True. waste your/someone's time now, or waste someone's time (and maybe more than one person's time) later.14:20
rloodtantsur: the problem is that you are the only one that is doing 99.99% of the bug triaging.14:21
rloodtantsur: ok, until we can solve that, i guess you can do what works for you :)14:22
dtantsur:)14:22
dtantsuryou can bring it to the meeting, if you feel it's important14:23
rloodtantsur: well, i think it is important, but note that i didn't volunteer to help you with bug triaging.14:23
dtantsurLOL14:23
dtantsurthat's how it ends up ;)14:23
*** ChrisAusten has joined #openstack-ironic14:23
*** keedya has quit IRC14:25
rloodtantsur: i was also asking, cuz when I see patches, i sometimes ask for a bug to be associated with it. so would like to be consistent wrt what we expect bugs/rfes for.14:25
dtantsuryeah, good point14:25
dtantsurI still treat bugs as bugs (in a sense of something broken) with an explicit exception for RFE's (which can be treated as missing functionality)14:26
*** itamarl_ has joined #openstack-ironic14:27
*** itamarl has quit IRC14:29
*** itamarl_ is now known as itamarl14:29
*** keedya has joined #openstack-ironic14:30
rloohey lucasagomes, when you get a chance, i had some questions on https://review.openstack.org/#/c/308965/. wrt how to deal with it in the future.14:32
lucasagomesrloo, hello there, lemme take a look14:33
*** amotoki has quit IRC14:34
*** hshiina has quit IRC14:34
lucasagomesrloo, ++ for the optimization by generating the list only once + the two other nits. Re: moving from pecan/wsme14:35
sambettsrloo, lucasagomes: didn't we talk about making the API more RESTful, and the URLs safer e.g. /nodes?detail=true14:36
*** keedya has quit IRC14:36
sambettsinstead of having "fake" resources14:36
lucasagomesrloo, I believe that once that happens we can make the list static again as part of that work?14:36
lucasagomessambetts, right, but we can't change it now14:37
*** dprince has quit IRC14:37
rloosambetts, so you're saying that whatever we move to, we won't have the sam problem we have with wsme/pecan?14:37
sambettsbut for new APIs going forward we should do it that way14:37
sambettsno more fake resources on new things14:37
lucasagomessambetts, ++ agreed, yeah we should rethink the design of the API once v2 start being planned14:37
lucasagomesfwiw, nova also has the /detail and such endpoints14:38
lucasagomeswe copied it over14:38
sambettsyeah, I hate it :-P14:38
rloosambetts: AH, ok, got it. So that list of invalid reserved words really is static and should not change.14:38
rloosambetts, lucasagomes: oh wait. you mean only when we go to v2?14:38
sambettsI'd hope we shouldn't have to add to it14:38
lucasagomesrloo, yeah, we have to keep the endopoints for v1, because removing it might potentially break people14:39
*** mtanino has joined #openstack-ironic14:39
rloolucasagomes: we have to keep the existing endpoints for v1. but if we add new endpoints in v1, they will also have to be reserved, or we can do what sam suggested and not treat them as fake resources?14:40
lucasagomesrloo, or if you mean the static list, yeah I believe we will keep pecan/wsme for v1 no? I first thought about translating it using something else14:40
lucasagomesbut that's going to be a lot of work :-/14:40
*** stendulker has joined #openstack-ironic14:40
sambettstbh I think we should add nodes?detail=true endpoints in parallel with the existing ones, and document them but without remove the old ones, then any new ones should be that styler14:40
lucasagomesrloo, oh for new ones we probably could do as sambetts said yes14:40
rloosambetts, lucasagomes: OK, got it. so for v1, that reserved list IS static. and for v2, there won't be a reserved list.14:41
lucasagomestho, the api will be a tad more inconsistent heh14:41
lucasagomesrloo, hope we won't need such list for v214:41
sambetts++14:41
lucasagomescause that's a design flaw14:41
rloolucasagomes: in that case, we should just make the list static instead of generating it. that way, if the unit test fails, we know that someone added a new endpoint.14:41
sambettslucasagomes: we could fix the inconsistancy by supporting both styles for the old endpoints14:41
lucasagomesrloo, yeah, even tho I generate it inspecting the class I've added a unittest for that (see test_utils.py)14:42
lucasagomessambetts, right, yeah, honestly I would love to start focusing on v2 if we are going to do that14:42
*** fragatina has quit IRC14:42
lucasagomesv1 already looks like a pig with lipstick heh14:42
rloolucasagomes: the problem with that code is future direction is lacking. someone looking at it may think, i'll just add a new endpoint and things will work properly. if we don't want to add new endpoints similar to what we have, we need to reflect that somehow outside of irc :)14:43
sambettshahahaha great way to describe it14:43
NobodyCamno one liked the V2 idea when I brought it up :(14:43
lucasagomesNobodyCam, I like it14:43
dtantsurI did14:43
rloolucasagomes, sambetts: hey, don't make fun of ms piggy.14:43
NobodyCam:)14:43
NobodyCamrloo: lol14:43
lucasagomesrloo, right... I can make it static again, I mean, I don't mind because both for now works. I see your point and also deva's point14:44
rloolucasagomes, sambetts: sorry, it is Miss Piggy: https://en.wikipedia.org/wiki/Miss_Piggy14:44
lucasagomesbut I hope that bits of the api won't change until EOL of v114:44
lucasagomesrloo, you want me to put a patch making it static again and then we can keep the discussion there?14:44
rloolucasagomes: I think we should add comments in the code (assuming we all agree with doing as sam said and the inconsistency in the API)14:45
rloolucasagomes: we don't know when we will end v1 and i'd rather we document things. too many things to remember now as it is. i'm not going to remember.14:45
lucasagomesrloo, agree, just wondering if we need to do it only on that class14:46
lucasagomesor all classes with subcontrollers14:46
rloolucasagomes: adding a follow up patch should work.14:46
lucasagomesI believe that class only since the "ident" is some word14:46
rloolucasagomes: it is only for names right?14:46
lucasagomesyeah14:46
lucasagomesrloo, will do that, and fix the opmization thingy you pointed out too14:46
rloolucasagomes: so only for nodes for now. but maybe we should make that change that sam suggested anyway.14:46
lucasagomes(and the nits)14:46
lucasagomesack, 1 sec will cook a patch up for it14:47
lucasagomesrloo, btw, thanks for pointing it out14:47
rloolucasagomes: yw. i debated about approving/not approving it.14:47
rloolucasagomes: i figured we still needed to fix it so i approved it.14:48
lucasagomesfair enough14:48
rloolucasagomes: i am going to try to focus more on documenting any decisions/thoughts we have, to reduce technical debt (or at least make it easier to remember later why we did whatever)14:48
sambettsrloo: +1 too many conversations are lost to the blackhole of IRC14:49
lucasagomes+114:49
lucasagomessome things may fall into our FAQ (which currently is quite poor)14:49
rloosambetts: exactly. and i realize, even emails aren't sufficient. who is going to scroll back and look.14:49
lucasagomesref: http://docs.openstack.org/developer/ironic/dev/faq.html14:50
rloolucasagomes: yeah, faq is good too. but i think if it is something that is localized, can be documented in code, that would be easier for developers.14:50
sambettsrloo: I'd love to see something like the #info bot thingy added to our IRC which could be used to indicate decisions/conversations made in IRC14:51
sambettsso there could be an automated minutes pushed out on the mailer at the end of each day for example14:51
lucasagomessambetts, PixieBot :-)14:51
rloosambetts: i think that'd be useful, but i'd rather that info be elsewhere. code or faq seems like the best places where newbies would think to look.14:51
lucasagomesNobodyCam, you sitll working on that bot?14:51
rloosambetts: not just newbies. folks like me with short memories too :)14:52
* lucasagomes has short memory14:52
NobodyCamoh poop I need to get back on that14:52
rloosambetts: make it easier to review code if it is right there in your face :)14:52
sambettsMaybe it could automatically build a wiki page or something? With that days highlights :/ and a link to that point in the openstack evesdrop for more context14:53
rloodtantsur: did you see https://etherpad.openstack.org/p/requirements-cruft, line 50 -- ironic-discoverd14:54
*** d0ugal has joined #openstack-ironic14:55
dtantsurrloo, yeah, my patch is on line 52 :)14:55
dtantsurbut thanks for raising it!14:56
*** Goneri has quit IRC14:56
rloodtantsur: ah, ok. let me update. cuz line 51 said 'i would check w/ironic folks ...14:56
dtantsursure14:58
*** sacharya has joined #openstack-ironic14:58
rloodtantsur: so you had a patch to remove it, but it hasn't been approved yet. and then https://review.openstack.org/#/c/313826/ was proposed?14:58
rloodtantsur: shouldn't we abandon one of them?14:58
dtantsurcorrect14:58
*** ijw_ has quit IRC14:58
rloodtantsur: ugh. ok, someone -1'd the second one cuz it is a dup of yours.14:58
*** Guest30018 is now known as mfisch14:59
rloodtantsur: is it a no-no to edit stuff other people put in etherpads?14:59
*** mfisch is now known as Guest5645914:59
dtantsurrloo, I thought that's what etherpads are for14:59
dtantsurthey would use email otherwise15:00
rloodtantsur: ok. am going to update that etherpad then.15:00
rloodtantsur:updated and now i know that you've dealt with it :)15:02
dtantsur:)15:02
mjturek1does anyone know if there's a ironic-neutron meeting today? the wiki says it's now in meeting-4 but it's a different team there15:04
*** clenimar has joined #openstack-ironic15:04
sambettsmjturek1: the time is in UTC so its in an hour15:05
mjturek1sambetts: ahhh thanks. Must've been looking at the wrong timezone by accident15:06
sambetts:)15:06
mjturek1:)15:06
*** d0ugal has quit IRC15:08
*** yolanda has quit IRC15:09
*** Goneri has joined #openstack-ironic15:11
openstackgerritImre Farkas proposed openstack/ironic: [WIP] DRAC RAID configuration  https://review.openstack.org/17631915:14
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Follow-up patch of 8e5e69869df476788b3ccf7e5ba6c2210a98fc8a  https://review.openstack.org/31414915:16
lucasagomesrloo, sambetts ^ lemme know if that's good enough in the comments15:16
rloolucasagomes: will look in a few...15:17
lucasagomescool, no rush15:18
*** itamarl has quit IRC15:18
*** Nisha has joined #openstack-ironic15:19
*** yolanda has joined #openstack-ironic15:19
*** davideagnello has joined #openstack-ironic15:24
*** d0ugal has joined #openstack-ironic15:25
*** jistr has quit IRC15:33
* dtantsur relocates for the meeting, brb15:37
*** dtantsur is now known as dtantsur|bbl15:37
NobodyCamrloo: I don't get the comment on the commit message for 31414915:40
rlooNobodyCam: sec15:40
NobodyCamoh never mind I see it now15:41
rlooNobodyCam: ok :)15:41
*** daemontool__ has quit IRC15:41
openstackgerritSam Betts proposed openstack/ironic-python-agent: TinyIPA: Shave off some file size from tinyipa ramdisk  https://review.openstack.org/31367515:41
openstackgerritSam Betts proposed openstack/ironic-python-agent: TinyIPA: Precompile python code for faster load  https://review.openstack.org/31310315:41
*** harlowja_at_home has joined #openstack-ironic15:43
*** rama_y has joined #openstack-ironic15:45
openstackgerritAndre keedy proposed openstack/ironic-inspector: Replace node data via uuid/data/unprocessed API  https://review.openstack.org/30837915:46
*** ohamada has quit IRC15:46
*** ohamada has joined #openstack-ironic15:46
*** ohamada has quit IRC15:47
*** thrash is now known as thrash|lunch15:47
*** Sukhdev has joined #openstack-ironic15:47
*** fragatina has joined #openstack-ironic15:50
*** d0ugal has quit IRC15:51
*** wajdi has joined #openstack-ironic15:51
*** fragatina has quit IRC15:53
*** fragatin_ has joined #openstack-ironic15:53
*** jistr has joined #openstack-ironic15:54
stendulkerjroll: Hi15:57
*** davideagnello has quit IRC15:58
*** hshiina has joined #openstack-ironic15:58
*** dprince has joined #openstack-ironic15:58
jrollstendulker: hey, what's up?15:59
stendulkerjroll: this is regarding adding proliantuils to ironic governance https://review.openstack.org/#/c/313667/16:00
stendulkerjroll: This is wrt CI discussion we had at summit, that CI information is fetched from stackalytics16:00
jrollstendulker: yes, what about it?16:00
jrollrather, what's your question?16:01
stendulkerjroll: Was following dracclient for the same16:01
*** tesseract has quit IRC16:01
stendulkerand have raised thsi review... Had raised another patch https://review.openstack.org/#/c/311277/ in driverlog that is merged16:01
*** fragatin_ has quit IRC16:02
jrollstendulker: sure, I've seen it, I put it up for discussion in the meeting today?16:02
jrollstendulker: if this is just for stackalytics CI things, not sure it's needed16:02
stendulkerjroll: not sure on the impact of the same16:02
stendulkerjroll: ok...16:02
stendulkerjroll: if its incorrect, i can abandon the same..16:03
jrollstendulker: well, it's a discussion I want to have with the community anyway, so I'll leave it until the meeting16:04
stendulkerjroll: ok :)16:04
*** milan has joined #openstack-ironic16:05
NobodyCamjroll: as I understand release management would then become (or is going to become) the responsibility of the main projects PTL???16:05
*** rcernin has quit IRC16:06
*** afaranha has joined #openstack-ironic16:06
stendulkerjroll: thank you16:06
*** d0ugal has joined #openstack-ironic16:06
*** keedya has joined #openstack-ironic16:06
jrollNobodyCam: everything in the ironic governance will become managed by the release team. approval of those releases is up to the PTL *or* a *single* release liaison16:06
jrollNobodyCam: the release team wants one point of contact per "project" (project meaning e.g. everything under the ironic governance)16:07
jrollNobodyCam: that said, anyone can still propose a release, I just need to +1 it16:07
*** fragatina has joined #openstack-ironic16:07
NobodyCamahh okay, Thank you that's helps my understanding16:08
jrollnp16:08
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: Add unittests for the utils.py module  https://review.openstack.org/31419716:11
*** piet has quit IRC16:13
*** jjohnson2_ has quit IRC16:14
*** jjohnson2_ has joined #openstack-ironic16:14
openstackgerritMerged openstack/ironic-specs: Place priorities list in reverse chronological order  https://review.openstack.org/31279416:15
rloofolks, I forgot to send email out last week about it. If you are a primary contact for any of the newton priority items, please update the subteam reports in whiteboard. And others that are on subteams already -- you don't need this reminder :)16:15
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Follow-up patch of 8e5e69869df476788b3ccf7e5ba6c2210a98fc8a  https://review.openstack.org/31414916:20
lucasagomesrloo, NobodyCam ^ updated the commit message and the added the missing period16:20
rloothx lucasagomes16:21
NobodyCamlol awesome lucasagomes Thank you16:22
lucasagomes:-)16:22
* jlvillal goes to update the whiteboard...16:22
lucasagomesty for the review16:22
jlvillaldtantsur|bbl: I change 09.05.2016 to 09-May-2016.  Since Europe and USA use different formatting :)  To USA it looks like September 5, 201616:24
jlvillalFor the whiteboard, gate broken status16:24
NobodyCamlucasagomes: +2 will hit +a as soon as we get test results16:24
lucasagomescool ty16:24
NobodyCamjlvillal: did 314024 land?16:25
jlvillalNobodyCam: I don't know...16:25
jlvillalNobodyCam: I was only changing the date.16:25
jlvillalNobodyCam: I've been heads down this morning doing my (late) status report. And then Grenade stuff. So I haven't been paying attention :(16:26
NobodyCamjlvillal: I thought we were waiting for https://review.openstack.org/#/c/314024 to land16:27
jrollyes we are waiting for that to land16:31
jlvillaljroll: What would you say is the status of tempest smoke?16:34
jrolljlvillal: I haven't touched it since posting the one patch16:35
jlvillalthanks16:35
jrolljlvillal: I did have a question for you about that, I see smoke is ran before resource create16:35
jrollso is smoke passing for you, if you're getting that far?16:35
jroll(on the "old" version)16:35
jlvillalSo in my devstack-gate-test I am having it do 'baremetal' for the tempest part.16:35
jlvillalSo that can get farther along. Of course we need tempest smoke working at some point.16:36
jrolloh16:36
jrollwhich runs nothing16:36
*** wajdi has quit IRC16:36
jlvillalIt runs about 70 tests or so.16:36
jlvillalI think16:36
jlvillalThe same as we run in our normal Ironic gate.16:37
jrolloh, baremetal instead of smoke, not baremetal combined with smoke (which have no overlap)16:37
jrollright16:37
jrollokay, I didn't realize that wasn't representative of the typical grenade job16:37
* jroll will keep hacking on it then16:37
*** rcernin has joined #openstack-ironic16:37
jlvillalAwesome :)16:37
jrolljlvillal: fyi, I did deploy this so we can start making progress on the gate side https://review.openstack.org/#/c/314066/16:39
* jlvillal looks16:39
jlvillaljroll: Awesome! :)16:41
jlvillalsambetts: We all love tinyipa :)16:41
sambettsjlvillal: there is a couple more optimisation patches up, I've got tinyIPA down to 49mb16:42
jlvillaljroll: I will update the devstack-gate-test to try 7 tinyipa vms16:42
jlvillalafter meeting16:42
jrolljlvillal: well, these are needed for smoke tests, resource create isn't really affected by it16:43
*** athomas has quit IRC16:43
jrollsambetts: have you tried running with 128 or 256 MB of ram yet? :)16:43
* jlvillal is working on getting it running on his new home system. To avoid proxy hell...16:43
sambettsjroll: no, but I really need to because getting rid of the squashfs mounting in the boot process might have helps to get the ram requirement down even further16:44
*** moshele has quit IRC16:45
*** david-lyle_ is now known as david-lyle16:45
*** nathan-h has quit IRC16:46
*** wajdi has joined #openstack-ironic16:47
jrollsambetts: yeah, would love to see that :)16:47
sambettsjroll: yeah that would be a huge improvement16:48
*** ijw has joined #openstack-ironic16:49
*** derekh has quit IRC16:50
*** ijw has quit IRC16:56
*** dtantsur|bbl is now known as dtantsur16:57
*** penick has joined #openstack-ironic16:57
dtantsurjlvillal, heh, thanks :)16:57
dtantsursambetts, 49mb wow. man you rock16:57
*** xhku has quit IRC16:59
dtantsursambetts, speaking of tinyipa. we can partially work around missing logs by setting up a python logging handler to dump IPA logs into a file and send it to inspector as well. wdyt?16:59
dtantsur(this is for case when journald and/or syslog are not present)16:59
sambettsdtantsur: teeing IPA logs to a file makes sense to me17:00
*** keedya has quit IRC17:00
sambettswouldn't get the boot logs like you would from journald or syslog though :/17:02
*** jistr has quit IRC17:02
dtantsursambetts, yes, yes... better than nothing, I guess17:03
*** harlowja_at_home has quit IRC17:04
*** ijw has joined #openstack-ironic17:05
*** thrash|lunch is now known as thrash17:05
*** yolanda has quit IRC17:06
dtantsura stable backport if someone has some time: https://review.openstack.org/#/c/314147/17:06
*** daemontool__ has joined #openstack-ironic17:09
*** ijw has quit IRC17:10
*** nicodemos has joined #openstack-ironic17:10
mgoulddtantsur, +117:11
dtantsurthnx17:11
*** yolanda has joined #openstack-ironic17:12
*** cdearborn has joined #openstack-ironic17:15
*** wajdi has quit IRC17:32
*** davideagnello has joined #openstack-ironic17:32
*** ijw has joined #openstack-ironic17:32
*** ijw has quit IRC17:33
mgouldI have an Ironic node which failed to deploy properly17:34
mgouldnow Nova thinks that instance is holding the node, so it won't let me boot another baremetal node17:34
*** ijw has joined #openstack-ironic17:34
mgouldhow do I get it to release the lock?17:34
dtantsurmgould, did you try 'nova delete' on the instance?17:35
mgoulddtantsur, yes17:35
*** ppiela has quit IRC17:35
mgouldand it succeeded17:35
lucasagomesmgould, the ironic node still have the instance_uuid field set?17:36
mgouldlucasagomes, yes17:38
*** ijw has quit IRC17:39
lucasagomesmgould, right, so try ironic node-update <node uuid> remove instance_uuid17:41
lucasagomesthat should free it up17:41
*** davideagnello has quit IRC17:43
mgouldlucasagomes, thanks!17:44
openstackgerritDevananda van der Veen proposed openstack/ironic: Update the api-ref documentation for Drivers  https://review.openstack.org/31370817:45
openstackgerritDevananda van der Veen proposed openstack/ironic: Update the api-ref documentation for Ports  https://review.openstack.org/31318717:45
lucasagomesyw :-)17:45
*** elfosardo has joined #openstack-ironic17:45
*** harlowja has joined #openstack-ironic17:48
*** Sukhdev has quit IRC17:50
*** garthb has joined #openstack-ironic17:51
*** wajdi has joined #openstack-ironic17:56
*** penick has quit IRC17:56
*** Sukhdev has joined #openstack-ironic17:57
*** Nisha_away has joined #openstack-ironic17:57
*** davideagnello has joined #openstack-ironic17:58
*** ChubYann has joined #openstack-ironic17:59
*** penick has joined #openstack-ironic18:00
*** Nisha has quit IRC18:00
* mgould -> home; good night!18:02
*** mgould has quit IRC18:02
NobodyCamnight mgould18:02
lucasagomesI'm also going to call it a day18:02
*** penick has quit IRC18:02
*** ppiela has joined #openstack-ironic18:02
lucasagomeshave a great evening all, see you tomorrow :-)18:02
dtantsurg'night folks18:02
dtantsurg'night lucasagomes18:02
NobodyCamnight dtantsur18:02
NobodyCamnight lucasagomes18:03
*** dtantsur is now known as dtantsur|afk18:03
*** lucasagomes is now known as lucas-dinner18:04
jrollbbiab18:04
stendulkerjroll: A query, being in ironic governance would allow making a relase of proliantutils off-cycle of openstack release, say wit defect fixes or so...18:07
jrollstendulker: yes, you'll be able to release at any time (other than during release freezes), via a patch to the openstack/releases repo with a +1 from me18:07
*** ifarkas has quit IRC18:08
stendulkerjroll: Are there any additional processes that we may have to follow? I could not get any link, hence these questions. If there is any lik i would go through the same18:08
jrollstendulker: don't believe so18:08
*** hshiina has quit IRC18:10
stendulkerjroll: thank you18:10
jrollnp18:11
*** wajdi_ has joined #openstack-ironic18:16
*** penick has joined #openstack-ironic18:17
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/31373018:18
*** wajdi__ has joined #openstack-ironic18:19
*** wajdi has quit IRC18:19
*** electrofelix has quit IRC18:21
*** Nisha_away has quit IRC18:21
*** wajdi_ has quit IRC18:22
*** ijw has joined #openstack-ironic18:29
*** raildo has joined #openstack-ironic18:31
raildojroll: hey, if you have some free time, can you take a look on this patch later? https://review.openstack.org/#/c/313690 :)18:32
*** openstackgerrit has quit IRC18:33
*** openstackgerrit has joined #openstack-ironic18:33
*** jcoufal has quit IRC18:47
*** mtanino has quit IRC18:47
*** keedya has joined #openstack-ironic18:49
*** yolanda has quit IRC18:51
*** dprince has quit IRC18:53
*** piet has joined #openstack-ironic18:53
*** irf has joined #openstack-ironic18:53
*** nicodemos has quit IRC18:53
irfhello, i am back ..18:54
*** AJaeger has joined #openstack-ironic18:54
irfnova hypervisor-list command shows total 5 hypervisors ...18:55
*** AJaeger has left #openstack-ironic18:55
irfbut in reality only two nova compute services are runnimg ....18:55
irfhow come command is showing 5 hypervisors ...18:56
*** baoli has quit IRC18:56
*** cloudnul- has joined #openstack-ironic18:56
*** cloudnul- has quit IRC18:57
*** stendulker has quit IRC18:58
*** ijw has quit IRC18:58
*** ijw has joined #openstack-ironic18:59
*** jtomasek_ has joined #openstack-ironic19:00
*** dprince has joined #openstack-ironic19:00
*** jcoufal has joined #openstack-ironic19:01
openstackgerritAndre keedy proposed openstack/ironic-inspector: Replace node data via uuid/data/unprocessed API  https://review.openstack.org/30837919:01
sambettsirf: every ironic node appears as a hypervisor that can support exactly 1 VM19:06
sambettsso 4 Ironic nodes == 4 hypervisors + your virtual machine one == 5 totla19:07
sambettstotal19:07
*** garthb has quit IRC19:07
irfok19:08
*** spandhe has joined #openstack-ironic19:08
*** Sukhdev has quit IRC19:08
irfok.. now as i have configured the nova compute service for baremetal node ... it seems there is still an issue ...19:09
irfstill getting same error : no valid hosst was found...19:10
irfagain some link is missing i guess.19:11
irfhere is the output for hypervisor-stats command19:12
irfhttp://pastebin.com/kSGyMAZQ19:12
sambettsdo you know if those stats have increased since the Ironic nodes have appeared as hypervisors?19:14
*** Sukhdev has joined #openstack-ironic19:15
irfyes .. it has increased19:15
irfbut one thing is there ,, only one hypervisor is valid among all 4 ironic hypervisos ...19:16
irfshud we delete the rest of the hypervisors ?19:16
sambettsno, having 5 hypervisors is right if you have 1 VM hypervisor and 4 ironic nodes19:18
irfironic node means bare metal machine rite ?19:20
sambettsyes19:20
irfok so i have just one ironic node19:21
sambettswhen you pasted the output of ironic node-list the other day I saw more than one19:22
irfhere is teh output of ironic node-list : http://pastebin.com/eYKjjKPy19:24
irfand only last entry is valid19:24
*** jjohnson2_ has quit IRC19:25
*** jjohnson2 has joined #openstack-ironic19:25
irfshud i remove the nodes which are not valid ?19:25
sambettsall ironic nodes will show up in nova once the nova compute is talking to Ironic, but the hypervisor stats will only increase by the number of nodes that are availiable and pass node validate19:26
sambettsor are in mainteance19:26
sambettsaren't in mainteance19:26
irfok.. so i think i shud remove the nodes which are not valid19:26
sambettsthat would help to debug :)19:28
devanandarloo: your email this morning is formatted very strangely19:28
rloodevananda: :-(, in meeting now. trying different mail system.19:29
jrolldevananda: rloo: looks fine to me in plain text19:36
*** mbound has joined #openstack-ironic19:36
irfok.. i have removed the invalid nodes .. shud i restart the services and try again ?19:37
rloojroll, devananda: looking. it seems ok although there are lots of blank space. I sent that email via Outlook instead of google.19:38
jrollyeah agree19:38
*** baoli has joined #openstack-ironic19:44
*** baoli has quit IRC19:47
*** mbound has quit IRC19:47
*** baoli has joined #openstack-ironic19:47
*** Sukhdev has quit IRC19:47
*** [1]cdearborn has joined #openstack-ironic19:49
*** afaranha has quit IRC19:50
sambettsirf: you shouldn't need to restart the services, they should pick up the changes automatically19:50
*** [2]cdearborn has joined #openstack-ironic19:54
*** [1]cdearborn has quit IRC19:54
*** irf has quit IRC19:57
*** irf has joined #openstack-ironic19:58
*** ijw has quit IRC20:00
irfok20:00
*** ijw has joined #openstack-ironic20:00
*** jcoufal has quit IRC20:01
jrollTheJulia: cinerama: devananda: NobodyCam: there's a bifrost+kolla thread on the ML, in case you haven't seen it20:02
*** fragatina has quit IRC20:04
*** cdearborn has quit IRC20:05
*** [2]cdearborn is now known as cdearborn20:05
*** lucas-dinner has quit IRC20:06
jrollbetherly: krotscheck: https://review.openstack.org/#/c/313270/ needs someone to do the needful20:08
NobodyCamcool Ty jroll :)20:09
betherlyjroll: +2'd it :)20:10
*** spandhe has quit IRC20:11
jrollcools, thanks20:12
*** baoli has quit IRC20:13
*** baoli has joined #openstack-ironic20:14
krotscheckjroll: Have pushed the button20:14
jrollty20:14
*** baoli has quit IRC20:14
*** baoli has joined #openstack-ironic20:15
devanandajroll: yea, replying now20:15
*** rcernin has quit IRC20:15
devanandajroll: do you know sean mooney's irc nic?20:15
*** lucasagomes has joined #openstack-ironic20:16
jrolldevananda: not off the top of my head, no20:16
*** ChrisAusten has quit IRC20:17
*** lucasagomes is now known as lucas-dinner20:17
*** hurgleburgler has joined #openstack-ironic20:20
*** hurgleburgler has left #openstack-ironic20:20
*** baoli has quit IRC20:21
*** cdearborn has quit IRC20:23
*** fragatina has joined #openstack-ironic20:23
*** cdearborn has joined #openstack-ironic20:24
*** sambetts is now known as sambetts|afk20:29
harlowjajroll  https://review.openstack.org/#/c/313699/ if u get some time20:32
harlowjaits the imho much more sane version of that spec matcher stuff20:33
harlowjawith an actual well defined grammar and all20:33
jrollharlowja: I don't know that code well at all, so changes to it scare the heck out of me20:34
*** dprince has quit IRC20:34
jrollbut tests do pass, well done :)20:34
harlowjathat should mean this new change is better, cause u can at least view the grammar :-P20:34
jrolltbh though, that code scares most people already, and imo this looks scarier20:34
harlowjahttps://review.openstack.org/#/c/313699/8/oslo_utils/specs_matcher.py (See make grammer function)20:34
harlowjahmmmm, a well defined grammar scares people ?20:35
jrollat any rate, I'd have nova folks look20:35
jrollthe code is the scary part, not the fact it's well defined20:35
harlowjahmmm, confused, lol20:35
jrollit's much more difficult to reason about by reading20:36
jroll(imo)20:36
jrollunless you've written a grammar before :)20:36
harlowjaright, don't create informal DSLs is the story of that one20:36
harlowjaalways create formal ones20:36
jrollanyway, I would see what nova folks think about it20:36
harlowjacause informal ones u get half-made-up-stuff20:37
harlowjaat least this one has a grammar that can be read by folks who can read it :-P20:37
harlowjajroll  who in nova knows that stuff :-P20:37
*** ChrisAusten has joined #openstack-ironic20:38
jrollharlowja: great question :)20:38
jrollvish? :P20:38
jrollooo I know ask termie20:38
harlowjaany people that are still around :-P20:38
harlowjaand/or not in secret companies20:38
harlowjalol20:38
*** baoli has joined #openstack-ironic20:38
jrollI have no clue20:39
jrollthat's part of the reason nobody wants to touch this20:39
jrollit also isn't well tested outside of unit tests20:39
*** e0ne has joined #openstack-ironic20:39
jrollbecause this is primarily for arbitrary flavor<->compute node matching based on json blobs20:39
harlowjalol20:39
harlowjawell at least now there is a real grammar :-P20:40
harlowjawith a real evaulator20:40
jrollsure, I just don't see how useful that is20:40
*** baoli has quit IRC20:40
jrollalso20:40
jrollhow would this work on top of your patch? do I need to pass an entire grammar in? https://review.openstack.org/#/c/308399/220:40
* jroll assumes so20:41
*** baoli has joined #openstack-ironic20:42
harlowjashouldn't be to hard to do that20:42
harlowjawithout whole grammer20:42
harlowjaso nothing drastic for that20:42
jrollnot really seeing how at a glance20:42
harlowjak, one sec20:42
*** ChrisAusten has quit IRC20:42
* harlowja cooks up a review20:43
jrollharlowja: I mean, you don't need to show me20:43
harlowjanope to late20:43
harlowjai'm already cooking20:43
jrollrealistically I think we need nova buy in on completely re-writing that20:43
jrollI'm apathetic as long as zero deployments break :P20:44
*** fragatina has quit IRC20:44
harlowjaso that means we need to find vish, lol20:44
*** baoli has quit IRC20:47
jrollheh20:48
*** baoli has joined #openstack-ironic20:48
harlowja https://review.openstack.org/314274 (how to override some things)20:48
harlowjaanyways, i'll find vish lol20:50
jrollheh20:50
* harlowja heads over to the oracle buildings20:50
jrollwas also going to ask, what about _make_grammar(), that's where my concern was20:51
*** jcoufal has joined #openstack-ironic20:51
harlowjaah, u want to chop pieces out of the grammar20:51
harlowjasounds like a bad idea, lol20:51
jrollI want to be able to use a subset, yes20:52
jrollor superset20:52
jrollor just my own set of methods20:52
jrollI mean, if you asked anyone today, they'd probably say all of this is a bad idea20:53
jrollbut this is the hand we were dealt20:53
harlowjax:-P20:53
harlowja:-P i mean20:53
harlowjaya, so at that point i'd wonder if its better to define your own make_grammar and leave the evaluation as is20:53
harlowjaand that means figuring out the grammar u want20:53
jrollor just leave the code as-is >.>20:54
jrolllike, an informal DSL thing is bad, but it isn't *that* bad20:54
harlowjain a unsupportable/scary to people state :-P20:54
harlowjabut ya, i get the viewpoint20:55
harlowjato me having a DSL that is informal is worse than having one thats formal (at least it forces u to be explicit about the actual DSL)20:55
harlowjabut ya, point taken20:56
*** baoli has quit IRC20:59
*** wajdi__ has quit IRC21:00
*** wajdi has joined #openstack-ironic21:01
*** Sukhdev has joined #openstack-ironic21:01
*** wajdi_ has joined #openstack-ironic21:02
*** ChrisAusten has joined #openstack-ironic21:04
*** rloo has quit IRC21:05
*** rloo has joined #openstack-ironic21:05
*** baoli has joined #openstack-ironic21:06
*** wajdi has quit IRC21:06
*** Goneri has quit IRC21:06
*** ChrisAusten has quit IRC21:07
*** fragatina has joined #openstack-ironic21:11
*** ChrisAusten has joined #openstack-ironic21:11
*** Guest56459 is now known as mfisch21:15
*** mfisch is now known as Guest8708521:15
*** spandhe has joined #openstack-ironic21:17
*** Guest87085 is now known as mfisch21:19
*** mfisch has quit IRC21:20
*** mfisch has joined #openstack-ironic21:20
*** mjturek1 has quit IRC21:22
*** jjohnson2 has quit IRC21:25
*** d0ugal has quit IRC21:26
persiaI ran into an interesting use case today: wanting to remotely drive network-attached hardware that can neither run linux nor has a BMC, but does have an API to load code from the network.  Is this something that is Ironic?  If so, which bit should I look at?21:33
persia(more specifically, these are PCI-attached coprocessor cards with high-speed network interfaces, mostly only using PCI for power, rather than being PCI-passthrough candidates)21:34
*** irf has quit IRC21:34
*** jtomasek_ has quit IRC21:38
*** jcoufal has quit IRC21:41
devanandapersia: can you describe a use case for this hardware?21:42
*** e0ne has quit IRC21:42
mrdaMorning all21:43
devanandapersia: I'm curious if our logical modelling maps to it or not. you might look at http://docs.openstack.org/developer/ironic/api/ironic.drivers.base.html#ironic.drivers.base.BaseDriver21:44
*** mbound has joined #openstack-ironic21:46
*** thrash is now known as thrash|g0ne21:48
*** mbound has quit IRC21:48
*** mbound has joined #openstack-ironic21:48
*** wajdi has joined #openstack-ironic21:50
mrdajroll, odyssey4me: So just on keystone v2/v3 thing, we did have trouble getting the right collection of params together working (even after talking to a keystone SME) which ended us settling on what we have today.  I can re-test if we think things have changed here.21:51
*** wajdi_ has quit IRC21:53
jrollmrda: yeah, okay. thanks21:59
*** wajdi has quit IRC21:59
jrollfyi, I recap'd the summit http://jroll.ghost.io/newton-summit-recap/21:59
*** rama_y has quit IRC22:00
*** Sukhdev has quit IRC22:00
*** catintheroof has quit IRC22:01
*** amotoki has joined #openstack-ironic22:03
mrdajroll: excellent!22:05
*** cdearborn has quit IRC22:08
persiadevananda: one use case would be to run FPGA kernels to accelerate analytic workloads (the current common practice is to use PCI pass through and Nova to access FPGAs in an OpenStack environment)22:08
persiaIndeed, I think one would be able to compose a BaseDriver interface to do that which is needed.  I will need to dig more to check, but that seems a sensible place to start.  Thanks!22:11
*** rbudden has quit IRC22:16
*** ChrisAusten has quit IRC22:23
jrollsent out a poll about the midcycle22:24
jrollsee y'all tomorrow22:24
*** piet has quit IRC22:30
*** sacharya has quit IRC22:37
*** sambetts|afk has quit IRC22:43
*** keedya has quit IRC22:45
*** sambetts has joined #openstack-ironic22:47
*** ChrisAusten has joined #openstack-ironic22:52
*** ametts has quit IRC22:58
*** ijw has quit IRC23:07
*** ijw has joined #openstack-ironic23:07
*** yuanying has joined #openstack-ironic23:14
*** mbound has quit IRC23:18
*** rloo has quit IRC23:20
openstackgerritMichael Still proposed openstack/ironic: Cleanup unused conf variables  https://review.openstack.org/28049323:23
*** mbound has joined #openstack-ironic23:25
*** penick has quit IRC23:28
*** chlong has joined #openstack-ironic23:33
*** ChrisAusten has quit IRC23:33
*** ijw has quit IRC23:35
*** ijw has joined #openstack-ironic23:35
*** sacharya has joined #openstack-ironic23:37
*** spandhe has quit IRC23:39
*** sacharya has quit IRC23:42
*** mbound has quit IRC23:46
*** gabriel-bezerra has quit IRC23:47
*** rloo has joined #openstack-ironic23:54
*** ijw has quit IRC23:56
*** ijw_ has joined #openstack-ironic23:56
*** ChrisAusten has joined #openstack-ironic23:56
*** Sukhdev has joined #openstack-ironic23:59

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