Monday, 2016-08-15

*** hoangcx2 has joined #openstack-ironic00:01
*** baoli has quit IRC00:04
*** hoangcx has quit IRC00:05
*** baoli has joined #openstack-ironic00:12
*** greghaynes has quit IRC00:48
*** greghaynes has joined #openstack-ironic00:49
*** hoangcx has joined #openstack-ironic00:54
*** jhooker has quit IRC01:06
*** jhooker has joined #openstack-ironic01:06
*** zhangjl has joined #openstack-ironic01:14
*** xuao has joined #openstack-ironic01:19
*** hoangcx2 has quit IRC01:42
*** vishwanathj has joined #openstack-ironic01:43
*** vishwanathj has quit IRC01:43
*** hoangcx2 has joined #openstack-ironic01:45
*** baoli has quit IRC01:56
*** rbudden has joined #openstack-ironic02:47
*** amotoki has quit IRC02:54
*** amotoki has joined #openstack-ironic02:55
*** amotoki has quit IRC02:56
*** rbudden has quit IRC03:01
*** hoangcx2 has quit IRC03:03
*** zhangjl has left #openstack-ironic03:12
*** zhangjl has joined #openstack-ironic03:12
*** baoli has joined #openstack-ironic03:24
openstackgerritNguyen Hung Phuong proposed openstack/ironic: Clean imports in code  https://review.openstack.org/35528803:35
*** baoli has quit IRC03:39
*** amotoki has joined #openstack-ironic03:39
*** vikrant has joined #openstack-ironic03:40
*** vikrant has quit IRC03:51
*** vikrant has joined #openstack-ironic03:52
*** sabeen has joined #openstack-ironic03:58
*** sabeen1 has joined #openstack-ironic04:00
*** amotoki has quit IRC04:01
*** sabeen has quit IRC04:02
*** hshiina has joined #openstack-ironic04:07
*** vikrant is now known as vikrant|brb04:08
openstackgerritDarren Chan proposed openstack/ironic: Update readme file  https://review.openstack.org/35529004:09
*** amotoki has joined #openstack-ironic04:09
*** amotoki has quit IRC04:13
openstackgerritDarren Chan proposed openstack/ironic: Update readme file  https://review.openstack.org/35529004:15
*** amotoki has joined #openstack-ironic04:17
*** vikrant|brb is now known as vikrant04:20
*** chlong has joined #openstack-ironic04:21
*** vishwanathj has joined #openstack-ironic04:25
*** zackf has quit IRC04:34
*** vishwanathj has quit IRC04:55
*** rook has quit IRC05:15
*** vishwanathj has joined #openstack-ironic05:15
*** lucas-dinner has quit IRC05:15
*** vishwanathj is now known as vishwanathj_zzz05:16
*** rook has joined #openstack-ironic05:17
*** rook is now known as Guest1403905:18
*** lucasagomes has joined #openstack-ironic05:19
*** rcernin has joined #openstack-ironic05:25
*** jhooker has quit IRC05:40
*** jhooker has joined #openstack-ironic05:41
*** baoli has joined #openstack-ironic05:51
*** baoli has quit IRC05:56
*** jmccrory is now known as jmccrory_away06:12
*** moshele has joined #openstack-ironic06:18
*** david-lyle has joined #openstack-ironic06:36
*** pcaruana has joined #openstack-ironic06:36
*** david-lyle_ has quit IRC06:39
*** sabeen1 has quit IRC06:44
*** phuongnh has joined #openstack-ironic06:49
*** Lord_Phoenix has joined #openstack-ironic06:49
*** jhooker has quit IRC06:52
*** xuao has quit IRC06:55
*** rbartal has joined #openstack-ironic07:04
*** zhangjl1 has joined #openstack-ironic07:05
*** zhangjl has quit IRC07:07
*** chlong has quit IRC07:08
*** rbartal has quit IRC07:19
*** jpich has joined #openstack-ironic07:21
*** chlong has joined #openstack-ironic07:25
*** ifarkas_afk is now known as ifarkas07:26
*** rbartal has joined #openstack-ironic07:32
*** amotoki_ has joined #openstack-ironic07:32
pas-hamorning Ironicers :)07:32
*** amotoki has quit IRC07:35
openstackgerritTang Chen proposed openstack/ironic: Split API root unit and functional tests  https://review.openstack.org/34896507:36
openstackgerritTang Chen proposed openstack/ironic: Move api driver tests to functional dir  https://review.openstack.org/35036507:36
openstackgerritTang Chen proposed openstack/ironic: Prepare for functional tests  https://review.openstack.org/34942107:36
*** ohamada has joined #openstack-ironic07:43
*** xuao has joined #openstack-ironic07:49
*** praneshp has quit IRC07:49
*** zzzeek has quit IRC08:00
openstackgerritDao Cong Tien proposed openstack/ironic: Fix assertEqual(10, 10) in unit/api/v1/test_utils.py  https://review.openstack.org/35535808:01
*** sdake has joined #openstack-ironic08:01
*** zzzeek has joined #openstack-ironic08:02
*** chlong has quit IRC08:07
*** athomas has joined #openstack-ironic08:13
openstackgerritzhangjl proposed openstack/ironic: Make sure the hostname of conductors is unique  https://review.openstack.org/35536108:14
*** e0ne has joined #openstack-ironic08:15
*** sdake has quit IRC08:15
*** dtantsur|afk is now known as dtantsur08:18
dtantsurMorning Ironic08:18
xuaoMorning dtantsur~08:18
openstackgerritDao Cong Tien proposed openstack/ironic: Fix incorrect order of params of assertEqual() in test_objects.py  https://review.openstack.org/35536408:18
dtantsuro/08:18
lucasagomesmorning all08:18
zhangjl1hey all08:18
xuaoMorning lucasagomes~~~08:19
zhangjl1lucasagomes: it`s glad to meet you08:20
lucasagomeshey there :-)08:21
zhangjl1en , while, there was a problem bothered me, maybe, i need your help08:23
openstackgerritTang Chen proposed openstack/ironic: Split API root unit and functional tests  https://review.openstack.org/34896508:23
openstackgerritTang Chen proposed openstack/ironic: Move api driver tests to functional dir  https://review.openstack.org/35036508:23
openstackgerritTang Chen proposed openstack/ironic: Prepare for functional tests  https://review.openstack.org/34942108:23
*** strigazi is now known as strigazi_AFK08:26
*** ccamacho has joined #openstack-ironic08:26
*** ccamacho has quit IRC08:26
*** ccamacho has joined #openstack-ironic08:26
tangchenHi guys, any HP OneView expert here ?  Would you please help to review this patch ?  https://review.openstack.org/#/c/347127/08:27
tangchenThanks very much08:28
dtantsurI think HP OneView expers appear later08:30
lucasagomestangchen, I think they are sleeping at the moment (they are based in brazil)08:30
lucasagomestangchen, you can add then on the review (see the "add" button at the top right corner)08:31
tangchen<dtantsur>, <lucasagomes>: Thanks. But I don't really know who they are...Would you please tell me some names ?08:31
lucasagomestangchen, thiago paiva brito08:32
dtantsurtangchen, thiagop and xavier IIRC08:32
tangchenOK, Thanks. :)08:32
*** e0ne has quit IRC08:35
*** sdake has joined #openstack-ironic08:35
*** e0ne has joined #openstack-ironic08:37
*** tuanla_ has joined #openstack-ironic08:42
*** sdake has quit IRC08:43
aarefievmorning!08:46
zhangjl1lucasagomes: i deployed a physical node last week , but now, the node still cannot start up08:54
*** Goneri has joined #openstack-ironic08:55
*** hshiina has quit IRC08:55
zhangjl1the follows is the error information08:55
zhangjl1[ O K ]A start job is running for dev-disk••*cfe.device (72h min 43s / no limit)08:55
vdrokmorning pas-ha dtantsur lucasagomes and aarefiev !08:56
aarefievmorning vdrok08:56
dtantsurmorning aarefiev, vdrok, lucasagomes08:56
aarefievmorning dtantsur08:56
*** electrofelix has joined #openstack-ironic08:58
lucasagomesvdrok, dtantsur morning08:59
lucasagomeszhangjl1, oh, in Ironic the node is marked as active?08:59
xuaoIf anyone have met this problem? While deploy a physical node, it hangs here.08:59
zhangjl1yes08:59
zhangjl1no any error09:00
lucasagomesright it seems to be something with the tenant image, you sure this image boots on the target hardware?09:01
lucasagomesbut also, that error seems to be about mounting the root file system? It usually will find it by uuid09:01
lucasagomesin the message you might see "... dev-disk-by-uuid <uuid here>", is that uuid the uuid of the root filesystem? (if you don't know you can mount the image in a loop and check it with "blkid"09:02
zhangjl1yes, i am sure.09:02
lucasagomes)09:02
zhangjl1the uuid ?09:02
zhangjl1you means the uuid must be the user image uuid?09:03
lucasagomesthe filesystem uuid09:03
lucasagomese.g09:03
sambettsMorning all09:04
zhangjl1pls wait minutes09:04
lucasagomes$ sudo blkid -s UUID -o value /dev/sda109:04
lucasagomes68ccb940-ff05-44c7-8d0c-35d1b0f5725209:04
lucasagomessambetts, morning09:04
lucasagomeszhangjl1, anyway it should be right... (/me is thinking)09:05
*** xuao has quit IRC09:13
zhangjl1em....i cannot login to the target hardware system, and it seams that i cannot mount the image09:16
zhangjl1when i use command losetup /dev/loop0 f2eda2b4-d61c-4248-8256-7fe8c191a158 and mount /dev/loop0 -o loop /mnt/09:17
zhangjl1it hangs here09:17
*** xuao has joined #openstack-ironic09:17
lucasagomeszhangjl1, if it's a qcow2 image you need to use ndb09:18
lucasagomesor convert it to raw before09:18
zhangjl1ok09:18
lucasagomesbut honestly it think it may be something else :-/09:18
lucasagomeszhangjl1, is it booting from the network or local disk?09:18
zhangjl1from network09:18
zhangjl1use pxe09:18
zhangjl1my deploy image is centos09:19
zhangjl1and my user image is fedora2409:19
*** yuriyz_ has left #openstack-ironic09:19
zhangjl1now, my node boot from fedora2409:19
zhangjl1but it hangs09:19
*** yuriyz_ has joined #openstack-ironic09:19
*** yuriyz_ has quit IRC09:20
*** yuriyz has joined #openstack-ironic09:20
zhangjl1[root@controller fedora]#  blkid -s UUID -o value /dev/loop109:21
zhangjl18894815e-0b90-40a5-b000-3629cc15dcfe09:21
zhangjl1[root@controller fedora]#09:21
lucasagomeszhangjl1, does it matches with the one in the pxe template?09:22
zhangjl1this is my filesystem uuid09:22
zhangjl1yes09:23
zhangjl1http://paste.openstack.org/show/557477/09:24
zhangjl1this is my pex configuration file09:24
zhangjl1generated by ironic09:24
lucasagomeszhangjl1, :-/ hmm I wonder why it's hanging then09:26
zhangjl1http://paste.openstack.org/show/557481/09:26
lucasagomeszhangjl1, does the machine falls on an emergency shell or something so you can troubleshoot it?09:26
zhangjl1this is my node info09:26
xuaoIt just hangs there for several days...😂09:27
lucasagomeszhangjl1, ohhh is the image deployed on a 3TB disk?09:27
zhangjl1maybe, i am not sure09:29
zhangjl1i do  not kown which disk ironic would be chose09:30
openstackgerritAndy Yan proposed openstack/ironic-specs: bug/1613198  https://review.openstack.org/35538909:30
zhangjl1is there any problem?09:30
openstackgerritLucas Alvares Gomes proposed openstack/ironic: DevStack: No need to change the ramdisk filesystem type  https://review.openstack.org/35539009:30
lucasagomeszhangjl1, depends, the node is booting legacy BIOS mode or UEFI?09:31
lucasagomesbecause if it's the former, the partition table type MBR is limited to disks with 2TB09:32
lucasagomesyou can't create a partition bigger than that (and cloud-init might have expanded your's til the end of the disk)09:32
lucasagomescausing it to fail to deploy09:32
lucasagomessambetts, have seem that happened before in his local tests09:32
lucasagomeszhangjl1, for that reason we included a way to force the disk to be formatted with GPT09:33
lucasagomeszhangjl1, take a look here: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#choosing-the-disk-label09:33
sambettsright, the symtoms of cloud-init breaking things is if the first boot up was successful, but then when rebooted it fails09:33
openstackgerritAndy Yan proposed openstack/ironic-specs: bug/1613198  https://review.openstack.org/35538909:35
zhangjl1lucasagomes, we do not specify any boot type09:35
dtantsurmorning sambetts09:35
zhangjl1but we use gpt09:35
lucasagomeszhangjl1, right so BIOS mode09:35
* lucasagomes is confused then09:36
lucasagomeszhangjl1, is it a whole disk image that you are trying to deploy? (a image that contains a partition table already)09:36
zhangjl1we use partition images09:37
sambettso/ dtantsur09:37
zhangjl1not whole disk09:37
lucasagomeszhangjl1, right, so if the boot mode is not specified I think Ironic will deploy it with MBR (I need to check the code)09:41
*** hoangcx has quit IRC09:41
lucasagomeszhangjl1, you can force gpt using that link that I've sent to you09:41
lucasagomeszhangjl1, can you test it?09:41
zhangjl1yes09:42
zhangjl1before deploy, i set nova flavor with gpt, and update node info with gpt09:42
lucasagomeszhangjl1, yup09:42
zhangjl1but , it`s still hangs09:43
lucasagomesbtw it's just a guess, I don't know if that's the cause but may worth testing09:43
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/35452009:43
lucasagomeszhangjl1, oh :-/09:43
* lucasagomes thinks09:43
zhangjl1:'(09:43
*** sdake has joined #openstack-ironic09:46
*** hoangcx has joined #openstack-ironic09:49
openstackgerritYuiko Takada Mori proposed openstack/ironic: Add node serial console documentation  https://review.openstack.org/29387209:51
xuaolucasagomes, sambetts, could you teach us about  your commend configration of these? About, bios/uefi, partiton-images/whole-images...and so on~~09:54
openstackgerritAndy Yan proposed openstack/ironic-specs: bug/1613198  https://review.openstack.org/35538909:55
xuaoAnd if we can do something to have a test ?09:55
*** athomas has quit IRC09:57
*** athomas has joined #openstack-ironic10:02
lucasagomesxuao, what specifically about it? Cause things like the type of image to use is operators decisions, depends on their use case10:03
xuaoget√。10:03
openstackgerritMerged openstack/ironic: Fix assertEqual(10, 10) in unit/api/v1/test_utils.py  https://review.openstack.org/35535810:08
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844010:10
*** hoangcx has quit IRC10:10
*** xuao has quit IRC10:10
openstackgerritMerged openstack/ironic: Update renos for fix to ipmi's set-boot-device  https://review.openstack.org/35441810:14
*** xuao has joined #openstack-ironic10:14
*** xuao has quit IRC10:31
lucasagomesjlvillal, when you have time, I would be interested in your opinion on https://review.openstack.org/#/c/355361/10:34
openstackgerritVladyslav Drok proposed openstack/ironic: Operator documentation for multitenancy  https://review.openstack.org/22849610:42
*** mbound has joined #openstack-ironic10:47
*** mbound has quit IRC10:51
openstackgerritVladyslav Drok proposed openstack/ironic: Fix the description of inspection time fields  https://review.openstack.org/35541010:52
tangchenlucasagomes, <dtantsur>: Hi, would you please help to review these three functional test patches ?   I have fixed Milan Kováčik's comments, and make the experimental functional tests pass successfully.10:56
tangchenhttps://review.openstack.org/#/c/349421/1010:56
dtantsursure, later today10:57
tangchenhttps://review.openstack.org/#/c/348965/10:57
tangchenhttps://review.openstack.org/#/c/350365/10:57
tangchen<dtantsur>, <lucasagomes>: Thank you very much. :)10:57
*** milan has joined #openstack-ironic10:58
*** phuongnh has quit IRC10:58
*** xuao has joined #openstack-ironic11:15
*** lucasagomes is now known as lucas-hungry11:16
xuaolucasagomes, we just have meal, full now , ahhh. And I want to share you with our steps to build the node~~This is the paste!11:18
xuao lucas-hungry, we just have meal, full now , ahhh. And I want to share you with our steps to build the node~~This is the paste!11:19
xuaohttp://paste.openstack.org/show/557492/11:19
*** zhangjl1 has quit IRC11:27
*** zhangjl has joined #openstack-ironic11:31
*** amotoki_ has quit IRC11:31
*** bfournie has quit IRC11:34
*** amotoki has joined #openstack-ironic11:34
*** amotoki has quit IRC11:37
*** zhangjl1 has joined #openstack-ironic11:40
*** zhangjl has quit IRC11:42
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Refactoring code of generating temp_url  https://review.openstack.org/35228911:47
*** amotoki has joined #openstack-ironic11:48
*** mbound has joined #openstack-ironic11:51
*** dprince has joined #openstack-ironic11:52
*** mbound has quit IRC11:56
openstackgerritAnton Arefiev proposed openstack/python-ironicclient: Fail with more meaningful error while creating client  https://review.openstack.org/34479111:58
*** baoli_ has joined #openstack-ironic12:00
*** sdake has quit IRC12:02
*** ohamada has quit IRC12:04
*** ohamada has joined #openstack-ironic12:04
*** ohamada has quit IRC12:06
*** ohamada has joined #openstack-ironic12:06
*** sdake has joined #openstack-ironic12:06
jrollg'morning12:08
dtantsurmorning jroll12:10
*** bfournie has joined #openstack-ironic12:10
openstackgerritFatih Degirmenci proposed openstack/bifrost: Introduce support for kvm acceleration  https://review.openstack.org/35543112:10
TheJuliaGood morning12:13
sambettso/ TheJulia, jroll12:15
*** amotoki has quit IRC12:16
TheJuliammm coffeee12:17
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management  https://review.openstack.org/34894312:18
milanmorning Ironic! :)12:18
dtantsurmorning TheJulia, milan12:18
milanmorning dtantsur! :)12:20
*** ohamada has quit IRC12:20
*** ohamada has joined #openstack-ironic12:20
*** zhangjl1 has left #openstack-ironic12:21
openstackgerritJulia Kreger proposed openstack/bifrost: Set OS_AUTH_TOKEN to dummy string, instead of empty space  https://review.openstack.org/35461512:22
*** amotoki has joined #openstack-ironic12:23
*** MattMan has quit IRC12:23
*** MattMan has joined #openstack-ironic12:23
milanaarefiev, hi, FYI I've got some progress on the introspection state patch: enum_ gone, transition renamed to event, functional tests added  to check the event processing w/r the node state, states&transitions description comments added12:24
*** zhangjl1 has joined #openstack-ironic12:24
*** rbudden has joined #openstack-ironic12:25
*** ohamada has quit IRC12:26
*** ohamada has joined #openstack-ironic12:26
*** lucas-hungry is now known as lucasagomes12:27
zhangjl1lucasagomes, my node still hangs12:27
zhangjl1any advices?12:28
*** amotoki has quit IRC12:28
lucasagomeszhangjl1, :-( I'm thinking about I've no idea off the top of my head12:29
lucasagomeszhangjl1, maybe adding "debug" to the kernel cmdline12:29
aarefievmilan: hey, nice, it's on my today list12:29
lucasagomessee if systemd gives you some hint of what's going on12:30
milanaarefiev, thx :)12:30
lucasagomeszhangjl1, "systemd.log_level=debug"12:30
*** xuao has quit IRC12:30
aarefievmilan: btw, I'm not quite get last time, why do you need db version?12:30
zhangjl1could you review my steps at http://paste.openstack.org/show/557492/12:31
zhangjl1is there any error in my operations?12:31
milanaarefiev, helps keeping db.Node and node_info in sync12:32
lucasagomeszhangjl1, it seems fine. On step 3) you don't need to do this: "ironic node-update $NODE_ID add instance_info/capabilities='{"disk_label": "gpt"}'"12:33
vdrokmorning jroll TheJulia milan12:33
lucasagomeszhangjl1, since you're using ironic with nova, it will fill out the instance_info stuff12:33
milanaarefiev, see http://martinfowler.com/eaaCatalog/optimisticOfflineLock.html12:33
milanit's just to avoid specific type of races12:33
milanmorning vdrok! :)12:33
lucasagomeszhangjl1, another thing you can try is to use local boot instead of PXE after the image is deployed12:33
lucasagomeszhangjl1, but that would just workaround the problem, we still should be able to network boot the node after deployment12:34
*** thrash|wknd is now known as thrash12:34
zhangjl1No matter how, thanks for your advices!!!12:34
milanaarefiev, for instance, a stale request might appear if inspector got disconnected but if the version_id changed from what was expected, one would just drop it12:35
*** mtanino has joined #openstack-ironic12:35
*** tuanla_ has quit IRC12:36
openstackgerritJim Rollenhagen proposed openstack/ironic: Add nova scheduler_host_subset_size option to docs  https://review.openstack.org/35056912:37
*** sdake has quit IRC12:38
*** zhangjl1 has left #openstack-ironic12:38
*** zhangjl1 has joined #openstack-ironic12:38
*** zhangjl1 has left #openstack-ironic12:38
aarefievmilan: I'm not quite get it how you will get into such case if you have distributed lock12:39
milanaarefiev, since we don't have it atm I wanted the code to be safe-guarded by this even now12:40
milanaarefiev, it sort of is a distributed lock implementation, but an optimistic one :)12:41
*** vikrant has quit IRC12:46
milanaarefiev, I need to be running for an errand, back in ~2h, thanks12:48
*** mjturek1 has joined #openstack-ironic12:48
*** amoralej|off has quit IRC12:50
*** jhooker has joined #openstack-ironic12:52
*** hshiina has joined #openstack-ironic12:52
*** mjturek1 has quit IRC12:53
*** amoralej has joined #openstack-ironic12:53
*** rloo has joined #openstack-ironic12:54
*** ppiela has joined #openstack-ironic12:55
openstackgerritCornelio Hopmann proposed openstack/ironic-specs: [RFE] Support network switches provisioning  https://review.openstack.org/35057012:55
xavierr_afkgood morning Ironic12:57
xavierr_afkmorning lucasagomes aarefiev milan zhugaoxiao vdrok12:57
*** xavierr_afk is now known as xavierr12:57
lucasagomesxavierr, hi there, g'morning12:58
* xavierr is looking for some code to review12:58
xavierrlucasagomes: \o12:58
*** milan has quit IRC13:01
aarefievmilan: ok, I just trying to imagine this case, did you fetch it while testing? Because it's looks like lost update problem, and should be solved by setup db isolation? imo we need uuid lock here13:01
aarefievmorning xavierr13:02
*** rloo has quit IRC13:02
*** rloo has joined #openstack-ironic13:02
*** milan has joined #openstack-ironic13:04
*** ccamacho has quit IRC13:05
*** mjturek1 has joined #openstack-ironic13:07
*** jcoufal has joined #openstack-ironic13:13
vdrokmorning xavierr13:14
*** amotoki has joined #openstack-ironic13:19
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification  https://review.openstack.org/34843713:20
openstackgerritJim Rollenhagen proposed openstack/ironic-specs: Update implemented specs for 6.1  https://review.openstack.org/35546313:23
jrollwow, lots of specs done in 6.1 ^^13:23
dtantsur\o/13:29
openstackgerritJarrod Johnson proposed openstack/pyghmi: Implement identify workaround for ThinkServer SD350  https://review.openstack.org/35489413:30
openstackgerritVladyslav Drok proposed openstack/ironic: Operator documentation for multitenancy  https://review.openstack.org/22849613:30
jrollvdrok: so fast!13:35
jroll+213:35
vdrokthx jroll :)13:35
jrollthank you!13:36
xavierrthats great o/13:36
openstackgerritMerged openstack/ironic: Fix the description of inspection time fields  https://review.openstack.org/35541013:37
jrolldevananda: dtantsur: have you been reading the cinder driver removal thread?13:37
jrollit's... interesting timing13:37
lucasagomesjroll, w00t13:41
*** [1]cdearborn has joined #openstack-ironic13:43
lucasagomesjroll, which thread is that?13:43
lucasagomesI see a lot of "pending on * removal" threads from cinder13:43
jrolllucasagomes: [openstack-dev] [tc][cinder] tag:follows-standard-deprecation should be removed13:44
jrollit's a long one :)13:44
lucasagomesheh yeah13:45
lucasagomeswill skim through it. Erno, the guy who started the thread, works for RH here in ireland13:46
jrollyeah13:47
* jroll steps away for a bit13:47
TheJuliadtantsur: Question, did you see the email I sent to the mailing list regarding driver composition defaults?13:50
*** causten has joined #openstack-ironic13:51
*** chlong has joined #openstack-ironic13:52
*** dprince has quit IRC13:55
lucasagomesjroll, btw, if you have some time: https://review.openstack.org/#/c/354608/ The stable branch pxe_ssh jobs i broken13:58
lucasagomesI think that would fix it13:58
lucasagomesstable/mitaka*13:58
*** dims has quit IRC13:59
*** anush_ has joined #openstack-ironic14:00
*** dims has joined #openstack-ironic14:05
*** mtanino has quit IRC14:09
jrollthanks lucasagomes14:10
*** sdake has joined #openstack-ironic14:10
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Refactoring code of generating temp_url  https://review.openstack.org/35228914:11
lucasagomesjroll, I mean "is broken"14:12
lucasagomesheh14:12
sambettslucasagomes: I'm fine with you taking the blame ;)14:12
lucasagomessambetts, heh no point in blaming anyone, sh** happens :-)14:13
* lucasagomes gives sambetts a pint14:13
sambettstrue true, its so hard to test whats going to happen with the jobs too until you run them :-p14:14
sambettslucasagomes: ha thanks :)14:14
lucasagomesyeah, plus, we can't use the depends-on tag for project-config stuff14:14
sambettsexactly :(14:14
jrolllucasagomes: yeah, I figured it out14:15
*** jaosorior has joined #openstack-ironic14:15
openstackgerritMarkos Chandras proposed openstack/bifrost: Fix typo when querying the python version in scripts/env-setup.sh  https://review.openstack.org/35548514:15
*** hwoarang has joined #openstack-ironic14:18
*** causten_ has joined #openstack-ironic14:18
*** causten__ has joined #openstack-ironic14:19
rloomorning ironic'ers, jroll, sambetts, lucasagomes, TheJulia14:19
lucasagomesrloo, hi there14:20
jrollhey rloo :)14:20
rloolucasagomes: qq about https://review.openstack.org/#/c/355390/. Is it possible that someone might have something out-of-tree that needs this logic?14:20
*** fragatina has joined #openstack-ironic14:20
*** fragatina has quit IRC14:21
lucasagomesrloo, hmm hardly because that was done for the bash ramdisk which is not supported anymore (the endpoints in the vendor passhtru endpoints were removed IIRC)14:21
lucasagomesrloo, in any case, that's just devstack as well14:21
*** fragatina has joined #openstack-ironic14:21
rloolucasagomes: ok14:22
lucasagomesrloo, in real production, if needed, people would append that option to the ironic.conf file14:22
rloolucasagomes: I wasn't worried about real production, just if someone was using devstack to test14:22
* lucasagomes assumes devstack == testing, hope nobody uses it to deploy a production cloud14:22
*** causten has quit IRC14:22
jrollwow, that's the silliest if block there14:22
lucasagomesjroll, heh yeah14:22
*** causten_ has quit IRC14:23
*** jtomasek is now known as jtomasek|biab14:23
lucasagomesrloo, right, so no worry about it, because coreos needs + than 1GB of ram14:23
lucasagomesrloo, and tinyipa needs way less than that14:23
rloolucasagomes: +A'd. jroll, if you disagree, be quick about it :)14:23
jrollrloo: nope, sounds good to me14:23
vdrokmorning rloo14:25
openstackgerritVladyslav Drok proposed openstack/ironic: Use upper-constraints for all tox targets  https://review.openstack.org/35548814:25
rloohi vdrok14:25
*** rbrud has joined #openstack-ironic14:25
*** ametts has joined #openstack-ironic14:26
*** dprince has joined #openstack-ironic14:27
openstackgerritVladyslav Drok proposed openstack/ironic: Use upper-constraints for all tox targets  https://review.openstack.org/35548814:27
lucasagomesjroll, re cinder stuff, good thread. I like the idea of having a "supported=True/False" parameter on the drivers14:28
openstackgerritMerged openstack/ironic: DevStack guide: Bump IRONIC_VM_SPECS_RAM to 1280  https://review.openstack.org/35347414:29
*** xuao has joined #openstack-ironic14:30
TheJuliagood morning rloo14:30
*** pcaruana has quit IRC14:30
*** thiagop has joined #openstack-ironic14:31
thiagopGood morning, Ironic14:31
jrolllucasagomes: yeah, not sure it solves the problem though14:31
* jroll needs to think about it more14:32
*** Lord_Phoenix has quit IRC14:32
lucasagomesjroll, right, yeah I also need to think more about it. But, I don't necessarily like the idea of removing drivers without communication before14:32
lucasagomesI mean for ironic, the cinder community might have their reasons14:32
lucasagomesbut anyway, that may be a good topic to have in barcelona14:33
jrolllucasagomes: yeah, seems it's the only way vendors will put CI on things14:33
thiagopjroll: I think 355463 needs to have an index entry14:33
jrolllucasagomes: well, removing them is our plan for this cycle... barcelona is a bit late :P14:33
lucasagomesjroll, yeah I understand that that would give an "extra push" on people do act on something14:34
lucasagomesjroll, heh yeah, maybe for future drivers14:34
jrollthiagop: you're right, thanks14:34
*** Sylvester has joined #openstack-ironic14:34
openstackgerritJim Rollenhagen proposed openstack/ironic-specs: Update implemented specs for 6.1  https://review.openstack.org/35546314:35
*** vishwanathj_zzz is now known as vishwanathj14:40
*** krtaylor has quit IRC14:42
jrollthiagop: are you still working on removing our tempest tests from the tempest tree?14:43
thiagopjroll: It's on my queue, but haven't time to work on it in the past weeks14:43
jrollok, thanks14:43
thiagopjroll: CI stuff14:43
jrollthiagop: what are the chances you'll get to work on that this cycle?14:44
thiagopjroll: but I plan to get back to it yet today or tomorrow14:44
jrollthiagop: awesome, ty14:44
*** syed_ has joined #openstack-ironic14:45
openstackgerritMerged openstack/pyghmi: Implement identify workaround for ThinkServer SD350  https://review.openstack.org/35489414:47
*** sdake has quit IRC14:48
TheJuliagood morning thiagop14:48
jaosorioris booting ironic-python-agent with vmedia supported?14:50
jaosoriordon't find much options to configure that14:50
lucasagomesjaosorior, that's vendor specific14:50
lucasagomesbut yeah should be supported on drivers like ilo14:51
lucasagomesirmc14:51
lucasagomes(maybe others...)14:51
*** [1]cdearborn has quit IRC14:51
thiagopmorning TheJulia14:54
*** [1]cdearborn has joined #openstack-ironic14:54
*** michauds has joined #openstack-ironic14:56
dtantsurTheJulia, yes, but I didn't look into details14:57
*** dtantsur is now known as dtantsur|mtg14:57
*** wajdi has joined #openstack-ironic14:59
*** sabeen has joined #openstack-ironic14:59
*** xuao has quit IRC15:01
*** xuao has joined #openstack-ironic15:01
*** hshiina has quit IRC15:02
TheJuliadtantsur: Okay, it seems like there is a consensus.  Just wondering if anymore discussion is required, but it sounds like maybe not today :)15:03
*** mbound has joined #openstack-ironic15:05
*** rcernin has quit IRC15:06
*** dprince has quit IRC15:12
*** Sylvester has quit IRC15:13
*** sabeen1 has joined #openstack-ironic15:15
jrollsomeone tell sukhdev I may miss the network meeting today. I'll be here for our meeting though. :)15:15
* jroll bbl15:15
*** sabeen has quit IRC15:17
sambettsjroll: will do :)15:19
*** jtomasek|biab is now known as jtomasek15:20
*** phschwartz has quit IRC15:20
*** hatTip has quit IRC15:22
*** moshele has quit IRC15:25
jlvillalGood morning Ironic15:27
thiagopmorning jlvillal15:27
jlvillallucasagomes: Looks like that patch you pinged me about got abandoned15:27
jlvillallucasagomes: Thanks for letting me know15:27
jlvillalthiagop: :)15:28
lucasagomesjlvillal, alright, no worries :-)15:28
lucasagomesjlvillal, morning15:28
jlvillallucasagomes: You might be interested in this: https://review.openstack.org/#/c/355142/15:30
*** xuao has quit IRC15:31
lucasagomesjlvillal, in a call atm, will take a look after it15:31
*** aNupoisc has joined #openstack-ironic15:33
*** thiagop has quit IRC15:35
*** baoli_ has quit IRC15:35
*** thiagop has joined #openstack-ironic15:36
devanandag'morning, all15:37
lucasagomesdevananda, morning15:39
*** harlowja_at_home has joined #openstack-ironic15:40
jlvillalMorning devananda15:40
*** aNupoisc has quit IRC15:40
jlvillalJayF: Do you think we should abandon some of the older patch sets for IPA? I see some from 2014 and 2015.15:40
JayFjlvillal: I don't have an objection. I'd suggest doing something similar to what I did with specs, if there's negative feedback + it's older than X, abandon15:42
JayFbut email the list asking for a veto first :)15:42
thiagopmorning devananda15:42
jlvillalJayF: Thanks15:42
jaosoriorIs there a way to get IPA to do some custom actions? Or should I be doing that through other means?15:42
*** praneshp has joined #openstack-ironic15:44
TheJuliajaosorior: I think it all depends on what you mean by custom actions15:44
JayFjaosorior: What exactly do you mean by "custom actions", we have HardwareManagers which allow adding/modifying cleaning steps, and a few other methods15:44
openstackgerritMichael Turek proposed openstack/ironic: Add classes for Portgroups API enablement  https://review.openstack.org/34754915:44
JayFjaosorior: https://github.com/jayofdoom/ipa-example-hardware-managers has a few examples of what you can do with custom hardware managers, plus it's pretty well documented in IPA docs15:44
vdrokmorning thiagop jlvillal devananda and JayF15:45
jlvillaldobroye utro vdrok15:45
openstackgerritJames Slagle proposed openstack/ironic: Fix key word argument interface_type -> interface  https://review.openstack.org/35553715:46
vdrokjlvillal: you're progressing :D15:46
jlvillalvdrok: heh15:46
jaosoriorJayF, TheJulia: Basically the deal is that I don't want IPA to be communicating with Ironic via HTTP. So, I wanted to know if there's a way for me to enroll to an identity system by passing an OTP (probably through a kernel cmdline parameter of vmedia) and performing some enrollment commands. But this needs to run before IPA starts.15:47
JayFjaosorior: that'd be a major new feature :)15:48
jaosoriorI se15:49
jaosorior* see15:49
*** hshiina has joined #openstack-ironic15:50
mat128jaosorior, JayF: IIRC, this was vaguely discussed in https://review.openstack.org/#/c/212206/15:51
mat128in the alternatives I think15:52
sambettswe also discussed similar things at the OpenStack summit in Austin15:53
*** Sukhdev has joined #openstack-ironic15:53
mat128jaosorior: if you have a dynamic DHCP, implementing an OTP would work, but would require minor changes like generating a new OTP when doing multiple reboots during cleaning/deployment15:53
mat128jaosorior: nothing impossible :)15:54
sambettsthe part that I'm not sure about is the communicating with Ironic via HTTP, what alternative are you after? authenticated https or some other protocol to be used?15:56
mat128sambetts: if it's a OTP it doesnt matter if the password leaks15:56
mat128sambetts: client certificates could as well be hijacked15:56
mat128sambetts: and we shall not forget that I could just impersonate you while querying the TFTP15:57
mat128and get your OTP15:57
*** mtanino has joined #openstack-ironic15:57
sambettsmat128: sure, I was just trying to assertain what he's asking for, changing the way IPA talks to Ironic to a differnt protocol other than HTTP/S would be a much bigger change than just working to authenticate it15:57
mat128JayF: if you're around, this is missing a +W https://review.openstack.org/#/c/354821/15:57
mat128sure :)15:58
mat128sambetts: oh15:58
mat128sambetts: just re-read what jaosorior said and maybe you're right, maybe he just doesnt want cleartext15:58
*** jpich has quit IRC16:00
jaosoriormat128, sambetts: Would be HTTPS, but once enrolled to the identity system, I could get https authentication (or even kerberos)16:00
mat128jaosorior: how do you first authenticate the server?16:01
mat128jaosorior: prior to / during enrollment16:01
*** ifarkas is now known as ifarkas_afk16:01
*** mat128 is now known as mat128|afk16:02
*** dtantsur|mtg is now known as dtantsur16:02
dtantsurTheJulia, yeah, someone just has to submit a spec update, and I don't have time for it right now16:02
sambettsmat128|afk, jaosorior: I think having keystone authentication and policy on the IPA communication is a common desire, and things become easier with vmedia because we can inject information through mgnt, but we want to try to solve it for the PXE/TFTP case too16:04
devanandajaosorior: I believe HTTPS is already possible16:04
jaosoriordevananda: problem is for that we need to have the CA certificate in the image already16:04
devanandajaosorior: yes - which you can do when you build the IPA image16:05
devanandajaosorior: and if you are concerned about transferring the IPA ramdisk over iPXE (which is HTTP, not HTTPS) and having it tampered with in transit, you can use a secured out of band channel such as iLO Virtual Media16:06
jaosoriordevananda: injecting the cert at image building, however, becomes problematic when a vendor needs to support provided images. Once the vendor injects the CA cert, the hash will change and support will become an issue.16:06
devanandajaosorior: the hash of the image can be recalculated after the CA cert is added16:09
jaosoriordevananda: not sure how feasable is that as a vendor. Getting a hash from the client and accepting to support that.16:09
*** jtomasek is now known as jtomasek|afk16:10
devanandajaosorior: oh - I see.16:10
*** dims has quit IRC16:10
devanandathat's not about ironic's technical capability with the hash, but your support system's tracking / identification of images (and whether htey've been tampered with)16:10
jaosoriordevananda: FWIW that was my initial proposal (since it's way easier), but was told it's not a feasable option :/16:11
devanandajaosorior: and you can't, as the vendor, bake the CA cert into the image -- because your customer / the local operator must be able to change that... hmmm16:11
jaosoriordevananda: exactly16:11
devanandajaosorior: can the CA certs be provided via another means, without changing the image itself?16:11
devanandafor example, written to a USB device, or attached over virtual media?16:12
jaosoriordevananda: that is an option, I think16:12
NobodyCamGood Morning Ironicers16:12
thiagopmorning NobodyCam16:12
devanandajaosorior: interesting problem. I think others would benefit from / be interested in helping to solve it upstream too16:13
NobodyCammorning thiagop :)16:13
dtantsurmorning NobodyCam, thiagop, devananda16:14
NobodyCammorning dtantsur :)16:14
thiagopdtantsur vdrok o/16:14
devanandajaosorior: I think we often assume that the operator is going to customize the imgaes, including hte IPA image. I hadn't anticipated that causing such difficulty for image-vendors.16:14
TheJuliadtantsur: Well, if nobody volunteers to put it up, I'll try and do it in the next 24 hours.16:14
dtantsurTheJulia, would be awesome! then we'll have less iterations due to my vague English :)16:14
TheJulias/the next/about 24 hours/16:15
TheJuliadtantsur: I think your English is better than you think it is :)16:16
dtantsurlol maybe :)16:16
dtantsurI went to a grammar course this summer. I can't say it improved substantially after it, but I started seeing all my mistakes >_<16:16
*** e0ne has quit IRC16:17
*** lucasagomes is now known as lucas-afk16:17
* lucas-afk will be back for the meeting16:18
openstackgerritJames Slagle proposed openstack/ironic: Fix key word argument interface_type -> interface  https://review.openstack.org/35553716:21
openstackgerritJames Slagle proposed openstack/ironic: Fix key word argument interface_type -> interface  https://review.openstack.org/35553716:22
*** dprince has joined #openstack-ironic16:24
openstackgerritStephanie Miller proposed openstack/ironic: Add volume_connector table to save connector information  https://review.openstack.org/20098316:26
openstackgerritStephanie Miller proposed openstack/ironic: Introduce VolumeConnector object  https://review.openstack.org/21458416:26
openstackgerritStephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume connector operation  https://review.openstack.org/21458516:27
openstackgerritStephanie Miller proposed openstack/ironic: Add volume_targets table to save target information  https://review.openstack.org/28521816:27
openstackgerritStephanie Miller proposed openstack/ironic: Introduce VolumeTarget object  https://review.openstack.org/28521916:27
openstackgerritStephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume target operations  https://review.openstack.org/28522016:28
*** harlowja_at_home has quit IRC16:28
*** ametts_ has joined #openstack-ironic16:28
rloodtantsur: qq. those two points seem to be the same thing? https://review.openstack.org/#/c/353450/3/releasenotes/notes/update-live-port-ee3fa9b77f5d0cf7.yaml16:29
*** ametts has quit IRC16:30
openstackgerritStephanie Miller proposed openstack/ironic: [WIP] Add storage_interface DB field and object  https://review.openstack.org/34800516:30
dtantsurrloo, not exactly. one is about allowing the operation instead of HTTP 500, the other is about forbidding the operation with a correct error code16:30
dtantsurrloo, it might be fine to merge them, but I plan on backporting this patch, so it will diverge from stable branches16:31
rloodtantsur: thx for explaining. It was a bit hard to understand.16:34
*** rbrud has quit IRC16:36
*** aNupoisc has joined #openstack-ironic16:38
*** ohamada has quit IRC16:41
*** dims has joined #openstack-ironic16:42
NobodyCammorning rloo :)16:44
rloomorning NobodyCam!16:46
*** dims has quit IRC16:47
*** anush__ has joined #openstack-ironic16:47
jrolldevananda: jaosorior: one note on your conversation up there, iPXE can use HTTPS for image transfer if you build the iPXE ROM to do so (yes, I know this isn't the sticking point, just a heads up)16:48
devanandajroll: ah - thanks for the clarification16:50
devanandajroll: in that case, do you bake a CA cert into the ipxe rom?16:51
jrolldevananda: iirc yes16:51
devanandaneat!16:51
jrollor a list of them, I guess16:51
*** anush_ has quit IRC16:51
dtantsurlucas-afk, do you anticipate more changes to ironic for https://bugs.launchpad.net/ironic/+bug/1603411 ?16:52
openstackLaunchpad bug 1603411 in Ironic "[RFE] Wipe disks metadata cleaning step" [Wishlist,In progress] - Assigned to Lucas Alvares Gomes (lucasagomes)16:52
*** dims has joined #openstack-ironic16:53
*** rama_y has joined #openstack-ironic16:54
*** praneshp has quit IRC16:56
*** lucas-afk is now known as lucasagomes16:58
lucasagomesdtantsur, docs16:58
lucasagomes:D16:58
dtantsuraha, ok16:58
lucasagomesdtantsur, will put some up tomorrow tops16:59
dtantsurwe have a hell lot of open bugs :(16:59
lucasagomesyeah :-/16:59
*** anush__ has quit IRC17:00
*** rpioso has joined #openstack-ironic17:00
*** sdake has joined #openstack-ironic17:00
Sukhdevdevananda, jroll : we can take up the discussion here17:02
jrollSukhdev: having the normal ironic meeting now, sorry17:02
Sukhdevjroll : ah right - np17:03
*** jaosorior has quit IRC17:06
*** Goneri has quit IRC17:07
*** mat128|afk is now known as mat12817:07
*** harlowja has joined #openstack-ironic17:08
*** dprince has quit IRC17:09
*** shobhar_ has joined #openstack-ironic17:12
*** rajinir has joined #openstack-ironic17:12
*** sabeen1 has quit IRC17:15
*** krtaylor has joined #openstack-ironic17:22
lucasagomescalling it a day, have a great evening all17:24
*** lucasagomes is now known as lucas-dinner17:25
jlvillallucas-dinner: Good night!17:25
mat128cya17:25
vdroknight lucas-dinner17:25
dtantsurg'night!17:26
*** dtantsur is now known as dtantsur|afk17:26
NobodyCamnight lucas-dinner dtantsur|afk17:28
sambettsNight all17:28
jlvillaldtantsur|afk: Good night17:28
jlvillalGood night sambetts17:28
NobodyCamnight sambetts17:28
NobodyCammorning jlvillal17:28
jlvillalHi NobodyCam :)17:29
vdrokand good night dtantsur|afk17:29
NobodyCam:)17:29
thiagopnight lucas-dinner sambetts xf[[17:29
thiagopnight dtantsur|afk17:29
*** [1]cdearborn has quit IRC17:30
*** joprovost has joined #openstack-ironic17:31
* jroll -> lunch17:34
*** sambetts is now known as sambetts|afk17:35
*** dprince has joined #openstack-ironic17:36
*** rcernin has joined #openstack-ironic17:37
rloodevananda, jroll: did/are either of you going to reply to Anne's email: http://lists.openstack.org/pipermail/openstack-dev/2016-August/101458.html17:37
*** cdearborn has joined #openstack-ironic17:37
rlooTeam17:38
rlooironic can I change your api-ref-jobs: line to bare-metal instead of17:38
rloobaremetal?17:38
jrollrloo: we've been talking in the patch instead17:38
jrollhttps://review.openstack.org/#/c/354135/17:38
rloodevananda, jroll: ^^17:38
rloojroll: oh, didn't realize there was a patch. ok good.17:38
jroll:)17:38
*** e0ne has joined #openstack-ironic17:38
*** sabeen has joined #openstack-ironic17:39
rloojroll, devananda: have fun. I already did a round of 'bare metal' whatever awhile ago.17:39
*** milan has quit IRC17:39
*** joprovost1 has joined #openstack-ironic17:42
*** joprovost has quit IRC17:42
*** joprovost1 is now known as joprovost17:42
*** praneshp has joined #openstack-ironic17:43
*** athomas has quit IRC17:45
*** joprovost has quit IRC17:47
*** cdearborn has quit IRC17:48
*** [1]cdearborn has joined #openstack-ironic17:48
*** hshiina has quit IRC17:53
*** ijw has joined #openstack-ironic17:56
*** Sukhdev has quit IRC17:58
*** shobhar__ has joined #openstack-ironic17:58
rajinirsambetts: I see an ARP timeout, any idea?18:00
*** praneshp has quit IRC18:01
*** shobhar_ has quit IRC18:02
rajinirsambetts, thiago, mat128: ARP timeouts on pxe booting of the nodes. It isnot able to resolve the TFTP server's IP address into the corresponding MAC address via the ARP. Could be a routing issue? Any ideas?18:06
thiagopCan be routing, yes18:06
*** pester has quit IRC18:10
*** pester has joined #openstack-ironic18:11
lucas-dinnerjust a headsup, apparently the gate is broken!? https://review.openstack.org/#/c/355390/18:14
lucas-dinner2016-08-15 17:30:08.498 | No hosts found to map to cell, exiting.18:14
*** e0ne has quit IRC18:15
*** pester has quit IRC18:15
*** causten has joined #openstack-ironic18:15
*** mbound has quit IRC18:16
*** e0ne has joined #openstack-ironic18:17
*** causten__ has quit IRC18:18
*** [1]cdearborn has quit IRC18:19
*** pester has joined #openstack-ironic18:20
mjturek1lucas-dinner: yeah our CI system is seeing this too18:20
*** cdearborn has joined #openstack-ironic18:21
devanandajroll: i've been reading the cinder/tc thread ... very intersting indeed18:21
openstackgerritThiago Paiva Brito proposed openstack/ironic: [WIP] Testing tests removal from tempest  https://review.openstack.org/35559118:22
*** electrofelix has quit IRC18:27
openstackgerritMerged openstack/ironic: Fix incorrect order of params of assertEqual() in test_objects.py  https://review.openstack.org/35536418:32
rajinir<lucas-dinner: Any idea when the gate will be fixed18:33
*** rcernin has quit IRC18:42
*** ijw has quit IRC18:43
*** ijw has joined #openstack-ironic18:44
*** rbartal has quit IRC18:46
*** ijw has quit IRC18:49
*** ijw has joined #openstack-ironic18:50
jrolllucas-dinner: ouch :/18:54
jrolldansmith: hey, that cells/devstack thing broke our CI, I think because n-cpu hasn't registered yet when we run the cell create http://logs.openstack.org/90/355390/1/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa/b9e264a/logs/devstacklog.txt.gz#_2016-08-15_17_31_13_35818:55
jrollwe have n-cpu sit and spin until ironic comes up, http://logs.openstack.org/90/355390/1/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa/b9e264a/logs/screen-n-cpu.txt.gz#_2016-08-15_17_30_51_71018:55
dansmithjroll: the devstack one or the grenade one18:56
dansmith?18:56
dansmithdevstack I guess18:56
jrolldansmith: devstack18:56
dansmithalaski: ^18:56
dansmithwe can actually fast revert the devstack one if we need to18:56
jrollI'd say just start ironic before nova, but ironic is a devstack plugin, so bleh18:57
jrollnot sure if possible18:57
dansmithprobably just need to not fail there and then do the create cell late18:57
jrollI'm happy with a revert for now but we gotta figure out a path forward18:57
jrollyeah18:57
dansmithjroll: lemme look for a sec hang on18:57
alaskiyeah, the migration assumes your n-cpus are running when you migrate18:58
jrolldansmith: okay, reverted in 355599 if we want to go that route18:59
*** e0ne has quit IRC18:59
dansmithalaski: I'm thinking we just try moving it until after plugins would have run and see if that works and/or is acceptable19:00
alaskiwhat typically indicates that nova is up and ready for testing on an ironic devstack run?19:03
dansmithalaski: it's ironic being up, not nova19:03
alaskigotcha19:04
dansmithjroll: can we see if this works? https://review.openstack.org/35560219:04
dansmithjroll: mtreinish: I guess maybe the best thing is to just revert the original thing and work on a fix once ironic is unblocked19:05
alaskidansmith: what about moving it below the service_check on line 1360?19:05
openstackgerritJim Rollenhagen proposed openstack/ironic: DNM: testing dansmith gate fix  https://review.openstack.org/35560319:05
jrolldansmith: ^ that'll check it out19:05
dansmithalaski: good call19:05
jrollthough I guess the nv devstack job should show us too19:05
jrollyeah, ironic job failed on the original patch https://review.openstack.org/#/c/322311/19:06
jrollI LOVE being non-voting19:06
jroll`/b 11719:06
jrolloops19:06
dansmithjroll: yeah, sorry :/19:06
jrollit happens19:07
dansmithdansmith happens?19:07
JayFOur jobs are actually pretty reliable. Not sure what virtue there is in being not voting anywhere.19:07
dansmithalaski happens?19:07
JayFI guess it keeps us from running in the gate?19:07
jrollJayF: because people don't trust us not to break them, and when it does break it resets the entire integrated gate19:07
alaskidansmith: heh19:08
jrollwhich at high load can cause like 30 patches to have to recheck19:08
mtreinishdansmith: I can +2 the revert if you think that's the best path forward (ie a fix isn't gonna be immediate)19:10
dansmithmtreinish: ask jroll :)19:10
dansmithI can put the rest of the original patch in mine and we can add it back in after the revert lands19:11
jrolleither way, if you aren't confident with the fix let's revert19:11
dansmithmtreinish: I'm not positive, so might as well revert I suppose19:12
jrolldansmith: cool, thank you19:12
openstackgerritStephanie Miller proposed openstack/ironic: [WIP] Add storage_interface DB field and object  https://review.openstack.org/34800519:12
mtreinishjroll, dansmith: ok, +2 on the revert19:12
jrollthanks mtreinish19:13
*** ametts_ has quit IRC19:13
*** mbound has joined #openstack-ironic19:17
*** ametts_ has joined #openstack-ironic19:18
*** ametts_ has quit IRC19:18
*** ametts has joined #openstack-ironic19:19
*** mbound has quit IRC19:22
*** Sukhdev has joined #openstack-ironic19:25
openstackgerritStephanie Miller proposed openstack/ironic: [WIP] Add storage_interface to base driver class  https://review.openstack.org/34800619:30
rloojroll etc. Our gate is broken now, right?19:32
jrollrloo: yes, the revert is on its way https://review.openstack.org/#/c/355599/19:32
jrollI should put this on the whiteboard eh19:32
rloojroll: yup, i was going to suggest that. thx for volunteering :)19:33
jrolland done :)19:33
rloo:)19:33
* jroll always forgets that19:33
*** causten_ has joined #openstack-ironic19:36
*** causten has quit IRC19:39
*** narasimha_SV has joined #openstack-ironic19:39
narasimha_SVwhen I am starting the ironic compute service I am getting below error19:39
narasimha_SVhttp://paste.openstack.org/show/557617/19:39
narasimha_SVI am unable to understand what exactly the version here19:39
thiagopnarasimha_SV: your ironicclient is requesting version 1.21 of the API but your ir-api service doesn't support this version19:40
narasimha_SVok19:41
narasimha_SVhow to resolve this ?19:42
jrolldeploy a newer ironic :)19:47
*** ijw has quit IRC19:48
thiagopnarasimha_SV: or use --ironic-api-version 1.18 on your ironicclient call19:48
jrolllooks like you're using a newer nova than ironic, which isn't supported - ironic always needs to be upgraded before nova19:48
jrollthiagop: that's in nova code19:48
thiagopjroll: oww19:48
narasimha_SVok19:48
narasimha_SVI had deployed the same before last week once19:48
narasimha_SVI am using same code now19:49
narasimha_SVlast week I was able to deploy it duccessfully19:49
narasimha_SV*successfully19:49
openstackgerritAlexander Bashmakov proposed openstack/ironic: Rewrite devstack/tools/ironic/scripts/create-node.sh in Python.  https://review.openstack.org/34683219:50
jrollnarasimha_SV: I wonder if kolla doesn't upgrade ironic or something19:50
narasimha_SVno I am using local source code which I have taken some days back19:50
narasimha_SVkolla uses that local source code until I change19:51
jrollah19:51
jrollyeah, pull latest ironic19:51
* jroll thinks we should make that error better19:51
narasimha_SVis this error because of the nova code which is more lastest than ironic ?19:52
JayFnarasimha_SV: yes19:52
jrollyep19:52
JayFnarasimha_SV: You must always upgrade Ironic before Nova.19:52
narasimha_SVjroll,jayf: but this worked as the same last week19:52
narasimha_SVi am using the same code19:52
JayFnarasimha_SV: we merged something to Nova within the last week that requires a newer Ironic.19:53
openstackgerritJulia Kreger proposed openstack/ironic: [WIP] Add base cinder common interface  https://review.openstack.org/35562519:53
jrollnarasimha_SV: the ironic code you're using is definitely older than last week19:53
*** fragatina has quit IRC19:53
narasimha_SVjroll: yes it is older than the last week . may be 3 weeks before code19:54
narasimha_SVeven nova also the same19:55
jrollnarasimha_SV: seems like nova is newer, this is the change that makes it request 1.21 https://review.openstack.org/#/c/345082/19:55
jrollmerged 3 days ago19:56
rlooyou guys! jroll: when is a feature done? I just moved https://trello.com/c/37YuKIB8/28-promote-agent-vendor-passthru-to-core-api to done, but i see there is api-ref outstanding. that patch doesn't have the same topic as the rest. Are we considering api-ref updates needed for feature completion?19:57
jrollrloo: yeah, I try to consider docs for 'done', that's why I didn't move that to done yet19:57
jroll(but it got lumped in with other api-ref updates)19:57
JayFI should probably add deploy docs for metrics as well, hmm...19:58
rloojroll: ok, i can move it back. the subteam report also said it was done so i deleted that subteam too.19:58
rloojroll: i mean, i deleted the subteam from our etherpad.19:58
jrollrloo: yeah, I think removing the subteam is fine19:58
jrolldon't need reports on it :)19:58
narasimha_SVthanks guys19:59
narasimha_SVthat was the issue19:59
jrollyou're welcome19:59
narasimha_SVour local repo was updated couple of hours ago19:59
*** Goneri has joined #openstack-ironic20:00
*** shobhar__ has quit IRC20:02
*** causten_ has quit IRC20:02
*** causten_ has joined #openstack-ironic20:03
openstackgerritMerged openstack/bifrost: Change Bifrost to TinyIPA as the default  https://review.openstack.org/35291920:06
openstackgerritMerged openstack/bifrost: Fix DHCP test scenario  https://review.openstack.org/35309920:06
*** shobhar__ has joined #openstack-ironic20:09
rajinirjroll: Can we fast track this please? https://review.openstack.org/#/c/355599/120:14
jrollrajinir: it's been approved, waiting for CI, nothing I can do20:14
rajinirjrol: ok, thanks :)20:15
*** Goneri has quit IRC20:15
openstackgerritJulia Kreger proposed openstack/ironic: [WIP] Add base cinder common interface  https://review.openstack.org/35562520:16
openstackgerritMichael Turek proposed openstack/ironic: Add classes for Portgroups API enablement  https://review.openstack.org/34754920:18
jrollany specs cores want to review what was implemented in 6.1? https://review.openstack.org/#/c/355463/20:19
JayFI've got it open, will review in a bit20:22
JayFthanks20:22
jrollnp, ty20:23
*** ijw has joined #openstack-ironic20:25
*** sdake has quit IRC20:32
openstackgerritJulia Kreger proposed openstack/bifrost: Change Vagrant VM to mirror memory/cpu in CI  https://review.openstack.org/35292120:39
*** shobhar__ has quit IRC20:41
openstackgerritjxiaobin proposed openstack/ironic: Keep config drive in swift until server deletion  https://review.openstack.org/35514220:41
*** fragatina has joined #openstack-ironic20:41
*** fragatina has quit IRC20:42
*** fragatina has joined #openstack-ironic20:43
*** shobhar__ has joined #openstack-ironic20:52
*** sdake has joined #openstack-ironic20:57
jrolldansmith: is this the grenade thing you mentioned? :) http://logs.openstack.org/03/355603/1/check/gate-grenade-dsvm-ironic/c39957a/logs/grenade.sh.txt.gz#_2016-08-15_20_29_05_76220:59
dansmithgdi20:59
*** dprince has quit IRC20:59
jrolldansmith: strangely enough, same error in a non-grenade context http://logs.openstack.org/03/355603/1/check/gate-tempest-dsvm-ironic-multitenant-network/d7f36b8/logs/devstacklog.txt.gz#_2016-08-15_20_08_16_01020:59
dansmithjroll: not sure that should fail in the same way because there should be a registered host already right?20:59
jrollthis is the patch depending on your possible fix for this21:00
jrolldansmith: yeah, it fails differently21:00
jrollmaybe unrelated to grenade21:00
*** aNupoisc has quit IRC21:00
jrollbleh, meeting21:00
dansmithhmm, that one I don't get21:00
dansmithI have to run too, but back in a bit21:01
jrollyeah, it's weird right?21:01
jrollcool, no worries21:01
*** aNupoisc has joined #openstack-ironic21:05
*** sdake has quit IRC21:08
thiagopCalling it a day21:13
thiagopsee you tomorrow21:13
jlvillalthiagop: Good night21:14
*** sdake has joined #openstack-ironic21:17
*** snizar has joined #openstack-ironic21:23
*** snizar_ has joined #openstack-ironic21:23
openstackgerritAlexander Bashmakov proposed openstack/ironic: Rewrite devstack/tools/ironic/scripts/create-node.sh in Python.  https://review.openstack.org/34683221:27
*** ametts has quit IRC21:28
openstackgerritMerged openstack/ironic: Document retrieving logs from the deploy ramdisk  https://review.openstack.org/35248321:31
openstackgerritRuby Loo proposed openstack/ironic-specs: Update implemented specs for 6.1  https://review.openstack.org/35546321:35
*** jcoufal has quit IRC21:35
*** sdake has quit IRC21:39
openstackgerritAlexander Bashmakov proposed openstack/ironic: Rewrite devstack/tools/ironic/scripts/create-node.sh in Python.  https://review.openstack.org/34683221:39
openstackgerritjxiaobin proposed openstack/ironic-python-agent: Deactivate LVM before write image to disk  https://review.openstack.org/35444021:40
aNupoiscjroll: Hi Jim, i have a question regarding bringing ironic up with agent_ipmitool driver, are you there?21:44
JayFIt's generally considered polite to direct questions at the channel rather than a single person; you should just ask and if someone can help they will.21:45
dansmithalaski: so, we're only running that cell setup command once in the ironic grenade job, but we fail with a uniq constraint in mappings21:47
dansmithjroll: this should not be doing anything other than a single ironic compute, right?21:48
*** amotoki has quit IRC21:48
openstackgerritMerged openstack/ironic-specs: Update implemented specs for 6.1  https://review.openstack.org/35546321:49
jrolldansmith: yes21:49
jrollaNupoisc: what JayF said, I'd like to help but I'm working on something else and leaving soon21:49
jrolldansmith: if you get stuck somewhere, ping me in the morning and I can help21:50
mrdaMorning21:50
JayFdansmith: I'm here until 5 pst and also know things, lmk if I can help21:51
dansmithjroll: JayF: I'm nearly positive that this means we somehow have two compute hosts in the db at this point21:51
JayFany way to just get the contents of that table dumped during the job21:52
jrolldansmith: OH21:52
jrolllol21:52
dansmithjroll: JayF two ironic nodes?21:52
jrollso we have multiple.. yep21:52
jrollsame host, different node21:52
jrollcompute_nodes table will have two entries21:53
jrollso tl;dr tell jaypipes to hurry up with the resource providers? :)21:53
dansmithokay I think I know the change we need then21:53
JayFI think we have three nodes in devstack by default, actually21:53
jrollnot in most jobs21:53
jrollbut yeah, I think 3+ in the two jobs that failed21:53
*** sdake has joined #openstack-ironic21:54
jrollok, gotta bounce now, thanks dansmith21:54
dansmithi/21:54
dansmither o/ even21:54
JayFo/21:54
dansmithi/ is like a leg kick or something21:55
*** harlowja has quit IRC21:55
JayFTheJulia: -1'd your rename bifrost job patch21:55
JayFTheJulia: it's soft though, so tell me "no" and I'll change the vote :)21:56
TheJuliaJayF: It is actually a really valid point that is being discussed a little in infra now21:57
aNupoiscJayF jroll: Yeah you are right, actually I was googling and found jroll as contact person related to ipmitool. So thought.21:59
*** amotoki has joined #openstack-ironic22:00
aNupoiscSo actually while bringing up I am getting "No conductor service registered which supports driver agent_ipmitool."22:00
openstackgerritDan Smith proposed openstack/ironic: DNM: testing dansmith gate fix  https://review.openstack.org/35560322:00
aNupoiscbut I am enabling that through local.conf (as using devstack)22:00
dansmithJayF: ^22:00
*** amotoki has quit IRC22:00
JayFaNupoisc: those kind of changes require a full unstack and restack to take effect.22:01
aNupoischere is the pastebin http://pastebin.com/7yhumNLd22:01
dansmithJayF: gonna run off yet again for a bit but will come back and check on that when I'm done22:01
aNupoiscI ran unstack and then trying stack.sh again22:01
JayFdansmith: so we actually found something broken in the code? \o/22:01
JayFaNupoisc: If you follow our guide to the letter, on how to setup devstack, you should not have this problem. I'd suggest starting over and ensuring you are following every step exactly, from the devstack guide here: http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack22:02
*** mtanino has quit IRC22:03
JayFdansmith: that's always more pleasant then it just being a silly devstack thing ;)22:03
dansmithJayF: well I found it broken in the code you found a non-working job :P22:03
aNupoisci am enabling the driver using IRONIC_ENABLED_DRIVERS22:03
dansmithJayF: broken in nova actually, just now tickling it because of devstack though22:03
dansmithJayF: best part is it's not my code, so yeah, happy day :P22:04
dansmithbiab22:04
aNupoiscOkay JayF, shall i then start again with scratch?, I had devstack running but due to some problems I want to tear it down and start it again22:04
aNupoisc*i teared it down22:04
*** wajdi_ has joined #openstack-ironic22:05
*** wajdi has quit IRC22:08
*** wajdi has joined #openstack-ironic22:08
*** fragatin_ has joined #openstack-ironic22:10
openstackgerritFatih Degirmenci proposed openstack/bifrost: Introduce support for kvm acceleration  https://review.openstack.org/35543122:10
*** cdearborn has quit IRC22:11
openstackgerritMerged openstack/ironic: Use upper-constraints for all tox targets  https://review.openstack.org/35548822:11
*** wajdi_ has quit IRC22:11
*** wajdi has quit IRC22:12
*** fragatina has quit IRC22:13
*** fragatin_ has quit IRC22:14
*** fragatina has joined #openstack-ironic22:15
*** jistr has quit IRC22:18
*** ijw has quit IRC22:19
*** jistr has joined #openstack-ironic22:19
*** sdake has quit IRC22:20
*** ijw has joined #openstack-ironic22:22
*** bfournie has quit IRC22:22
*** thrash is now known as thrash|g0ne22:28
*** sabeen has quit IRC22:32
*** rpioso has quit IRC22:33
*** ijw has quit IRC22:34
*** sdake has joined #openstack-ironic22:35
*** krtaylor has quit IRC22:38
*** ijw has joined #openstack-ironic22:39
openstackgerritjxiaobin proposed openstack/ironic: Keep config drive in swift until server deletion  https://review.openstack.org/35514222:44
*** mbound has joined #openstack-ironic22:47
*** mbound has quit IRC22:52
openstackgerritFatih Degirmenci proposed openstack/bifrost: Fix release notes formatting issue  https://review.openstack.org/35566822:53
*** shobhar__ has quit IRC22:56
*** sdake has quit IRC22:57
openstackgerritFatih Degirmenci proposed openstack/bifrost: Fix release notes formatting issue  https://review.openstack.org/35566822:58
*** narasimha_SV has quit IRC22:59
*** michauds has quit IRC23:02
*** Goneri has joined #openstack-ironic23:08
*** baoli has joined #openstack-ironic23:16
*** harlowja has joined #openstack-ironic23:36
*** rama_y has quit IRC23:40
*** rama_y has joined #openstack-ironic23:41
*** rama_y has quit IRC23:46
*** fumie has joined #openstack-ironic23:48
*** baoli has quit IRC23:54
openstackgerritMerged openstack/ironic-python-agent: Minor updates to metrics documentation  https://review.openstack.org/35449023:55

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