Wednesday, 2016-05-25

cineramaaNupoisc, did you source the "env-vars" file?00:00
aNupoiscyes00:00
*** rama_y has quit IRC00:00
cineramaaNupoisc, what is the value for auth_strategy in /etc/ironic/ironic.con00:02
*** lhcheng has quit IRC00:03
aNupoiscit is keystone00:04
*** _vsaienko has quit IRC00:04
aNupoisccinerama: it is keystone00:04
cineramaaNupoisc aha so you are trying to use keystone there. did you have ironic installed on your VM before maybe?00:04
*** _vs has quit IRC00:05
aNupoiscnope00:05
aNupoisccinerama:so what the auth_strategy should be00:05
aNupoisccinerama: keystone is by default00:06
cineramaaNupoisc, so if you installed ironic using bifrost and didn't change anything it will default to noauth00:07
cineramaaNupoisc, anyway, can you try changing auth_strategy to noauth in your conf file and restarting ironic-api? then try doing ironic node-list and see what happens00:09
aNupoisccinerama: ah okay. Yes i will do that00:10
*** wajdi has joined #openstack-ironic00:11
aNupoisccinerama: oh yes now it see an empty table00:13
cineramaaNupoisc, so assuming that works you can try doing your deployment again.00:13
aNupoisccinerama: yes thnx :)00:13
oomichiyuikotakadamori: can you take a look at your nova-spec related to ironic? https://review.openstack.org/#/c/31950700:14
yuikotakadamorioomichi, sure00:14
oomichiyuikotakadamori: that contains a -1, and can you get helps from ironic team?00:15
JayFThere was a mailing list thread about those console specs, asking which ones won from the design summit.00:15
oomichiyuikotakadamori: the feature freeze of non-prio is 3rd Jun on Nova side according to http://releases.openstack.org/newton/schedule.html00:16
JayFYou might want to have a look and reply to that, as until the ironic-spec merges you're unlikely to get movement on the nova-spec.00:16
yuikotakadamorioomichi, we've got basic consensus from Ironic folks at design summit00:16
aNupoisccinerama: it is giving the same error upon running enroll-dynamic00:16
yuikotakadamoriJayF, thanks, ah, I've forgot it.00:16
JayFyuikotakadamori: there was some question on the ML about what was decided, you should have a look and possibly clear up any confusion00:16
yuikotakadamoriJayF, yes, I will do so00:16
cineramaaNupoisc: do you get the same messages in your logs?00:17
oomichiyuikotakadamori: from nova side, the corresponding ironic-spec needs to be approved before the nova-spec as the consensus of ironic team00:18
aNupoisccinerama: yeah00:18
aNupoisccinerama: somehow keystone is getting called00:19
yuikotakadamorioomichi, right00:20
cineramaaNupoisc, can you do "ps fauxww | grep ironic" for me?00:21
*** ChrisAusten has joined #openstack-ironic00:22
aNupoisccinerama: here is it http://pastebin.com/z7TKVm0n00:25
cineramaaNupoisc, and you're running bifrost from a git checkout?00:27
aNupoisccinerama: yes00:29
cineramaaNupoisc, can you do "git diff" and just see if it says anything?00:31
oomichiyuikotakadamori: as JayF's advice, it is nice to make clear summary on -dev ml as a basic consensus00:31
JayFThere's already a thread with an open question. For everyone's sanity please don't make a new one :)00:31
* yuikotakadamori is writing a mail00:31
oomichiyuikotakadamori: after that, is it possible to get approval on ironic side in this week?00:32
JayFI'd strongly doubt you'll have your spec approved by the end of the week.00:32
aNupoisccinerama: i am just wondering ironic is installed as standalone so authentication should not be present00:32
JayFBut if you keep it updated, with little negative feedback, and answer the open questions on the mailing list it'll at least be moving at all :)00:32
aNupoisccinerama: yes i will check the diff00:32
oomichiJayF: is it difficult to get approval on ironic-specs in general ?00:33
yuikotakadamorioomichi, I'm not sure because basically spec approval takes long time, and it's same in Nova or almost all projects also, right?00:33
aNupoisccinerama: do you know where we configure WSGI component00:33
JayFIt's not nearly as bad as other projects with Ironic in my experience00:34
JayFthe fact many of the open questions were answered at the summit will help too00:34
JayFI just think three working days to get anything merged is difficult :)00:34
yuikotakadamoriJayF, thanks, I will do my best and hope get approval in this week00:34
oomichiyuikotakadamori: that is case by case. If we(nova side) have gotten consensus on the summit, we can easily get approval on nova side00:34
oomichiyuikotakadamori: based on basic direction without deep tech thing00:35
cineramaaNupoisc, that's not something bifrost will set up for you. for ironic in general there's information in the docs: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#configuring-ironic-api-behind-mod-wsgi00:37
oomichiJayF: hehe, yeah. 3 days seem very short ;)00:37
yuikotakadamoriJayF, you think so? me, too, a little >_<00:37
aNupoisccinerama: oh okay.00:38
yuikotakadamorioomichi, BTW, does my proposal need spec? I've heard that simple feature doesn't need a spec, just a simple blueprint00:39
JayFyuikotakadamori: we don't use blueprints anymore ,we do RFE bugs00:39
JayFyuikotakadamori: as long as you aren't changing any APIs or interfaces, an RFE is usually fine. Being more detailed in the bug also makes it less likely you'll be asked to write a spec.00:39
yuikotakadamoriJayF, sorry, I want to talk about in nova. You know something??00:40
JayFah, that's for Ironic00:40
JayFI don't know the process for Nova.00:40
* JayF off for the evening o/00:40
yuikotakadamoriJayF, yes, I don't know well, too00:41
yuikotakadamoriJayF, good night ;)00:41
aNupoisccinerama: hey i am about head out. Is it fine if i ping you tomorrow? I will google more later today and see00:41
oomichiyuikotakadamori: JayF: yeah, basically the nova-spec would be necessary in this case00:41
oomichiyuikotakadamori: JayF: but this seems a general progress issue: We discuss each tech thing at the summit and we need to get approval on each project side before nova feature freeze if the feature is related to nova00:42
oomichiand in this case, the spec approval requires 3 weeks in general, then time is over.. :-(00:43
cineramaaNupoisc, sure, many people will be able to help you tomorrow00:43
*** praneshp has quit IRC00:44
aNupoisccinerama: thanks a lot00:44
*** yosshy has joined #openstack-ironic00:45
yuikotakadamorioomichi, what can i do?00:46
*** yosshy has quit IRC00:47
oomichiyuikotakadamori: is it possible to get approval as basic consensus on ironic side without deep detail thing?00:48
*** ijw has quit IRC00:50
yuikotakadamorioomichi, I'm note sure because discussing about tech thing deeply in spec is usual thing00:50
*** ijw has joined #openstack-ironic00:52
*** yuanying has quit IRC00:55
yuikotakadamorijroll: is there?00:57
*** Egyptian[Home] has quit IRC01:00
*** Sukhdev has quit IRC01:01
*** amotoki has quit IRC01:08
*** baoli has joined #openstack-ironic01:09
*** Egyptian[Home] has joined #openstack-ironic01:13
*** ijw has quit IRC01:24
*** ijw has joined #openstack-ironic01:27
*** jcoufal has quit IRC01:29
*** aNupoisc has quit IRC01:29
*** hoangcx has joined #openstack-ironic01:29
*** rloo has joined #openstack-ironic01:29
*** rloo has quit IRC01:30
*** ijw has quit IRC01:31
*** ElCoyote__ has quit IRC01:38
*** sdake has joined #openstack-ironic01:40
openstackgerritMerged openstack/python-oneviewclient: OneView client with managers  https://review.openstack.org/28680301:43
*** amotoki has joined #openstack-ironic01:48
*** sdake_ has joined #openstack-ironic01:52
*** sdake has quit IRC01:53
*** amotoki has quit IRC01:56
*** ptx0 has quit IRC01:59
*** yuanying has joined #openstack-ironic01:59
*** amotoki has joined #openstack-ironic01:59
*** sabeen has quit IRC02:03
*** sabeen has joined #openstack-ironic02:04
*** wajdi_ has joined #openstack-ironic02:07
*** sdake_ has quit IRC02:09
openstackgerritJohn L. Villalovos proposed openstack/ironic: Patch to test Grenade.  https://review.openstack.org/31775102:09
*** wajdi has quit IRC02:09
*** amotoki has quit IRC02:09
openstackgerritMerged openstack/ironic-python-agent: Use constraints for all the things  https://review.openstack.org/31866902:17
*** amotoki has joined #openstack-ironic02:18
*** ptx0 has joined #openstack-ironic02:22
*** ptx0 has joined #openstack-ironic02:22
openstackgerritRuby Loo proposed openstack/ironic: Mention RFEs in README  https://review.openstack.org/32074702:33
*** joprovost has quit IRC02:37
*** joprovost1 has joined #openstack-ironic02:37
*** joprovost1 is now known as joprovost02:40
*** penick has quit IRC02:45
*** vishwanathj has joined #openstack-ironic02:52
*** rama_y has joined #openstack-ironic02:55
*** vishwanathj has quit IRC02:58
*** Nisha_away has joined #openstack-ironic03:04
*** absubram has joined #openstack-ironic03:06
*** absubram_ has joined #openstack-ironic03:10
*** absubram has quit IRC03:11
*** absubram_ is now known as absubram03:11
*** yuanying has quit IRC03:14
*** sdake has joined #openstack-ironic03:15
*** Sukhdev has joined #openstack-ironic03:20
*** sdake has quit IRC03:27
*** stendulker has joined #openstack-ironic03:39
*** kromanenko__ has joined #openstack-ironic03:39
*** kromanenko_ has joined #openstack-ironic03:39
*** ayoung has quit IRC03:42
*** phuongnh has joined #openstack-ironic03:43
*** sdake has joined #openstack-ironic03:44
*** Egyptian[Home] has quit IRC03:52
*** aparnav has joined #openstack-ironic03:54
*** links has joined #openstack-ironic04:01
*** rbrud has quit IRC04:02
openstackgerritTan Lin proposed openstack/ironic: Make use of oslo-config-generator  https://review.openstack.org/24733104:05
*** praneshp has joined #openstack-ironic04:08
*** sacharya has quit IRC04:16
*** joprovost has quit IRC04:20
*** Nisha_away has quit IRC04:28
*** Nisha_away has joined #openstack-ironic04:28
*** yuanying has joined #openstack-ironic04:33
*** _vs has joined #openstack-ironic04:55
*** _vsaienko has joined #openstack-ironic04:55
*** wajdi_ has quit IRC04:59
*** wajdi has joined #openstack-ironic05:01
*** ishant has joined #openstack-ironic05:01
*** Sukhdev has quit IRC05:02
*** rama_y has quit IRC05:03
*** sdake_ has joined #openstack-ironic05:11
*** sdake has quit IRC05:13
*** sacharya has joined #openstack-ironic05:17
*** baoli has quit IRC05:19
openstackgerritYuiko Takada Mori proposed openstack/ironic-specs: Nova compatible serial console support  https://review.openstack.org/31950505:20
*** sacharya has quit IRC05:22
jlvillallintan, On https://review.openstack.org/#/c/247331/05:22
jlvillallintan, I saw that the oslo patch to fix the deprecated group thing got merged. Do we have to wait for a release for the fix to show up?05:22
jlvillallintan, Or is it available now?05:22
lintanhi jlvillal, the patch is landed but it miss the last released which release today morning.  So I prefer to land in the oslo-config-generator first if we can and use a follow-up patch to correct when it is available05:26
jlvillallintan, Sounds good. I was just curious. Thanks05:27
lintan:)05:27
*** _vs has quit IRC05:31
*** _vsaienko has quit IRC05:31
*** ChubYann has quit IRC05:35
*** jaybeale has joined #openstack-ironic05:39
*** jaybeale has quit IRC05:43
*** jaybeale has joined #openstack-ironic05:45
*** wajdi has quit IRC05:45
*** wajdi has joined #openstack-ironic05:45
*** jaybeale has quit IRC05:50
*** wajdi_ has joined #openstack-ironic05:51
*** wajdi has quit IRC05:51
*** daemontool has joined #openstack-ironic05:54
*** ChrisAusten has quit IRC05:59
*** ig0r_ has joined #openstack-ironic06:01
*** Nisha_away has quit IRC06:04
*** ig0r_ has quit IRC06:17
*** sacharya has joined #openstack-ironic06:18
*** Nisha_away has joined #openstack-ironic06:20
*** rcernin has joined #openstack-ironic06:21
*** sacharya has quit IRC06:23
*** mjura has joined #openstack-ironic06:25
*** moshele has joined #openstack-ironic06:25
*** mjura has quit IRC06:31
*** mjura has joined #openstack-ironic06:43
*** tesseract has joined #openstack-ironic06:47
*** aparnav has quit IRC06:51
*** e0ne has joined #openstack-ironic06:52
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: [WIP] [POC] Callback from ramdisk support for Ansible driver  https://review.openstack.org/31081606:57
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: [WIP] [POC] Ansible deploy driver alt impl  https://review.openstack.org/31004206:57
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: [PoC] Make streaming_write to be Ansible module  https://review.openstack.org/31860906:57
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: [PoC] Add callback for logging into Ironic log  https://review.openstack.org/31861006:57
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: [PROOF OF CONCEPT] Ansible deploy driver  https://review.openstack.org/23818306:57
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: [PoC] Gather facts by default  https://review.openstack.org/32080206:57
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: [PoC] Add Ansible v1.9.x compatibility  https://review.openstack.org/32080306:57
*** ifarkas has joined #openstack-ironic07:10
*** athomas has joined #openstack-ironic07:10
*** moshele has quit IRC07:20
*** jtomasek has joined #openstack-ironic07:20
*** moshele has joined #openstack-ironic07:21
*** hoonetorg has quit IRC07:43
*** chlong has quit IRC07:46
*** moshele has quit IRC07:50
*** moshele has joined #openstack-ironic07:51
*** sdake_ has quit IRC07:56
*** hoonetorg has joined #openstack-ironic07:56
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: [PoC] Add Ansible v1.9.x compatibility  https://review.openstack.org/32080307:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** itamarl has joined #openstack-ironic08:02
*** daemontool_ has joined #openstack-ironic08:07
*** mgoddard has joined #openstack-ironic08:07
*** daemontool_ has quit IRC08:09
*** daemontool_ has joined #openstack-ironic08:09
*** phuongnh has quit IRC08:09
*** daemontool has quit IRC08:10
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844008:10
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844008:10
*** lucas-afk is now known as lucasagomes08:19
lucasagomesmorning all08:19
yuikotakadamorilucasagomes, morning!08:19
*** ig0r_ has joined #openstack-ironic08:20
openstackgerritTan Lin proposed openstack/ironic-specs: Support rolling upgrades  https://review.openstack.org/29924508:24
*** dmk0202 has joined #openstack-ironic08:28
*** _fortis has quit IRC08:32
*** derekh has joined #openstack-ironic08:34
*** fragatina has quit IRC08:36
*** dtantsur|afk is now known as dtantsur08:37
dtantsurMorning Ironic, morning (ugt) lucasagomes, yuikotakadamori08:37
*** mgould|afk is now known as mgould08:37
yuikotakadamoridtantsur, good ugt morning!08:37
mgouldmorning lucasagomes yuikotakadamori dtantsur and everyone else!08:38
yuikotakadamorimgould, morning!08:38
dtantsurhey mgould08:38
lucasagomeshey hey yuikotakadamori dtantsur mgould, good morning!08:38
* dtantsur recovers from sudden beers yesterday08:38
lucasagomes#pixiesay beer -m happy08:39
PixieBootsʕ^ᴥ^ʔ: beer08:39
yuikotakadamoriwho is PixieBoots?? cute! caracters are corrupting, though08:40
mgouldyuikotakadamori: https://github.com/umago/pixiebot/08:41
mgoulddtantsur: beers often attack suddenly and in packs08:42
lucasagomesyuikotakadamori, it a font that supports unicode08:42
yuikotakadamorilucasagomes, I see, bear's face is so cute :D08:43
* mgould looks up the characters08:43
lucasagomes#pixiesay -m likeaboss08:43
PixieBootsʕ▀̿ᴥ▀̿ʔ08:43
mgouldLATIN LETTER PHARYNGEAL VOICED FRICATIVE, apparently08:43
*** sdake has joined #openstack-ironic08:44
*** wajdi has joined #openstack-ironic08:44
*** wajdi_ has quit IRC08:45
mgouldoh, it's an IPA symbol08:45
mgouldhow appropriate :-)08:45
yuikotakadamori#pixiesay -m confused08:46
PixieBootsʕ๏ᴥ๏ʔ08:46
mgouldand ʔ is glottal stop, which is the sound Londoners traditionally use instead of a T08:46
mgouldyuikotakadamori: International Phonetic Alphabet08:47
lucasagomesmgould, cool08:47
mgouldhttps://en.wikipedia.org/wiki/Voiced_pharyngeal_fricative08:47
dtantsur#pixiesay Everyone likes me -m happy08:47
PixieBootsʕ^ᴥ^ʔ: Everyone likes me08:47
yuikotakadamorimgould, thanks :D08:49
yuikotakadamorisuper kawaii!08:49
dtantsur:)08:49
mgould:-)08:49
*** sdake has quit IRC08:49
hoangcxAbsolutely, everyone loves you PixieBoots :-p08:50
dtantsurwe need to make it respond to mentions :)08:51
lucasagomes:-D would be cool to get him to reply some messages08:51
*** sdake has joined #openstack-ironic08:51
lucasagomesdtantsur, exactly08:51
dtantsurlucasagomes, now to less funny things :) could you please take a look at https://review.openstack.org/320295 (and if you have some more time also https://review.openstack.org/308292)?08:52
lucasagomesdtantsur, yup08:52
dtantsurI hope to make at least some progress with this damned disk problems08:52
lucasagomesdtantsur, oh the other one was reverted already?08:52
* lucasagomes looks08:52
dtantsurlucasagomes, yep. this is a new version that does not count as a feature, so we can even backport it!08:53
lucasagomeseven better08:53
*** praneshp has quit IRC08:55
*** mbound has joined #openstack-ironic08:55
*** amotoki has quit IRC09:00
vdrokmorning ironic, lucasagomes dtantsur mgould and yuikotakadamori !09:00
Nisha_awayhello Ironic09:00
Nisha_awayvdrok, o/09:00
vdrokhi Nisha_away09:00
yuikotakadamorivdrok, Nisha_away, morning!09:01
dtantsurmorning vdrok, Nisha_away09:01
Nisha_awayI am hitting this error in iLO drivers CI from this Monday. The same deploy ISO was working till friday. http://paste.openstack.org/show/505337/09:01
lucasagomesvdrok, Nisha_away morning09:01
Nisha_awayanyone know about it?09:01
Nisha_awaylucasagomes, o/09:01
Nisha_awaylucasagomes, vdrok dtantsur mgould ^^^^09:03
vdrokNisha_away: it's the first time I see such thing09:03
* lucasagomes looks09:03
Nisha_awayvdrok, :(09:03
openstackgerritTan Lin proposed openstack/ironic: Add a doc about how to append kernel parameters to boot instances  https://review.openstack.org/32084809:04
Nisha_awayI am just thinking it might be some devstack settings due to which it is failing09:04
vdrokdo you have vm logs?09:04
Nisha_awaybut not able to figure that out.09:04
vdrokI mean, ipa logs09:04
lucasagomesNisha_away, strange, it seems like something in the deploy ramdisk09:04
Nisha_awaylucasagomes, yeah but same ramdisk was working till friday09:05
*** sdake has quit IRC09:05
Nisha_awayso i am left clueless how suddenly it started failing09:05
lucasagomesvery strange, because it's looking for /dev/disk/by-path09:05
lucasagomesand that's controlled by udev AFAIK09:05
lucasagomesit should be populated, perhaps udev is just taking some time ? ( I wonder if we need a udevadm trigger for block devices)09:05
mgouldmorning vdrok Nisha_away09:06
lucasagomesNisha_away, indeed, very odd to suddenly starting happening09:06
lucasagomesNisha_away, same hardware as well?09:06
Nisha_awaydeploy ramdisk also says the the iSCSI connection couldnt be found09:06
sergekmorning all o/09:06
Nisha_awayvdrok, no i dont have IPA logs09:06
Nisha_awaylucasagomes, yes09:06
Nisha_awaythe CI doesnt capture that09:06
lucasagomesright, it happens 100% of the time?09:07
lucasagomesmaybe the disk is failing?09:07
Nisha_awaynow yes, just once yesterday it passed09:07
Nisha_awaythat also i dont know how09:07
Nisha_awaylucasagomes, what do u mean by "disk is failing"09:07
lucasagomesit could be some hardware failure, may worth checking09:07
Nisha_awaylucasagomes, i am not sure what to check if its hardware failure09:08
*** yuanying has quit IRC09:08
*** vmud213 has joined #openstack-ironic09:09
lucasagomesthe deploy ramdisk logs as vdrok said may give us some tips on why it couldn't create the iscsi target09:09
lucasagomes(assuming it couldn't create the iscsi target)09:09
lucasagomesNisha_away, can you try again and collect the logs from the ramdisk?09:09
lucasagomeseven if manually via a serial console idk09:09
*** yuanying has joined #openstack-ironic09:10
Nisha_awaylucasagomes, ok09:10
vsaienkomorning licasagomes, vdrok,  Nisha_away, sergk, dtantsur and all09:11
Nisha_awayvsaienko, o/09:11
openstackgerritVladyslav Drok proposed openstack/ironic: Add support for API microversions in Tempest tests  https://review.openstack.org/26035809:14
*** Waltz has joined #openstack-ironic09:17
WaltzHi everyone, quick question for you all, is it possible to use whole disk image with pxe_ilo in uefi boot mode?09:17
vmud213morning vdrok,lucasagomes,vsaienko,Nisha_away,dtantsur and everyone else not listed09:19
*** daemontool_ has quit IRC09:19
*** daemontool_ has joined #openstack-ironic09:19
Nisha_awayvmud213, o/09:20
stendulkerWaltz: yes09:20
Nisha_awayWaltz, yes09:21
Waltzstendulker: I must have mis-conigured then I am seeing a failure with "Whole disk image with netboot is not supported in UEFI boot mode"09:21
Waltz*configured09:21
stendulkerWaltz: Have you set boot_option as 'netboot?09:22
*** sambetts|afk is now known as sambetts09:23
sambettsMorning all09:23
Waltzstendulker:  I used this "ironic node-update <node-uuid> replace properties/capabilities='boot_option:local,boot_mode:uefi,secure_boot:false' "09:23
openstackgerritVladyslav Drok proposed openstack/ironic: Add support for API microversions in Tempest tests  https://review.openstack.org/26035809:23
vdrokmorning vmud213 stendulker  and sambetts09:24
stendulkermorning vdrok09:24
*** deray has joined #openstack-ironic09:25
vmud213morning sambetts09:25
yuikotakadamorisambetts, o/09:25
sambettsHey vmud213 yuikotakadamori :D09:25
sambettso/ vdrok09:25
stendulkerWaltz: I hope your image do not have kernel_id and ramdisk_id parameters09:25
stendulkerWaltz: That message is logged from here https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L254-L25809:26
derayhello Ironicers!09:27
vmud213deray o/09:27
derayvmud213, :)09:28
* deray in the process of setting up standalone Ironic09:29
Waltzstendulker: I used "glance image-create --name centos7-wd --visibility public --disk-format qcow2  --container-format bare < /tmp/centos7-wd.qcow2" so know kernal and ram disk. the nodes where set to legacy boot and whole image just before this, I will try deleting them and try again. I can see from the comments in the code that this should work as you say.09:29
stendulkerWaltz: ok09:32
openstackgerritMerged openstack/ironic-python-agent: Wait for at least one suitable disk to appear on start up  https://review.openstack.org/32029509:42
dtantsuroh good, this one even cherry-picks cleanly to mitaka ^^09:43
*** wajdi has quit IRC09:43
*** wajdi has joined #openstack-ironic09:44
*** tczekajlo has joined #openstack-ironic09:46
*** amotoki has joined #openstack-ironic09:47
sambetts\o/09:47
mgouldNisha_away: I know nothing about iSCSI, but I'd start by checking that it's possible to connect to the iSCSI server from a working machine09:50
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Get root device hints from the node object  https://review.openstack.org/31853809:53
sambettsNisha_away: I've seen failures to setup the iSCSI endpoint if either the ramdisk didn't have the drivers needed for my hardware or when my RAID controller broke and so my machine didn't have any disks to use09:55
*** fragatina has joined #openstack-ironic09:55
openstackgerritSam Betts proposed openstack/ironic-python-agent: Consolidate IPA configuration into a config module  https://review.openstack.org/32086709:55
Nisha_awaysambetts, hmm ...the ramdisk was a working one till Friday.... :( i am trying to debug to see why it suddenly started failing09:58
Nisha_awayit fails even for another hardware with the same error09:58
*** fragatina has quit IRC09:59
*** YiboCai has quit IRC10:02
mgouldsambetts: RAID controller on the node being booted, or on the node serving iSCSI?10:05
sambettsmgould: node being booted, so there isn't a disk for it to copy the image onto10:05
mgouldright, gotcha10:05
mgouldsambetts: I'd be surprised to see the error "iSCSI connection was not seen by the file system" in that case, though10:06
sambettshmmm :/ yeah normally you get a Can not find a disk big enough error from that case10:07
mgouldoh wait, that's an Ironic error; yeah, I can see how that would happen10:07
dtantsurI wonder if the patch we just merged could help10:08
mgouldhttps://github.com/openstack/ironic/blob/master/ironic/drivers/modules/deploy_utils.py#L17910:08
tczekajloHi, I have issue with number of spawning instances. I can't spawn more than 10 instances simultaneously, instances above 10 has task state 'scheduling'. When some instance's finished spawning then next queued instance move on. Is there any limit for number of simultaneously spawns?10:09
sambettstczekajlo: With Ironic?10:10
mgouldsambetts: if Nisha_away is seeing the same problem with another BM node, it's probably not a hardware problem on the BM node10:11
* mgould would try mounting the iSCSI volume on a known-good machine, to check that the problem isn't at that end10:13
*** milan has joined #openstack-ironic10:16
*** hoangcx has quit IRC10:17
milanmorning Ironic!10:17
sambettso/ milan10:18
*** sacharya has joined #openstack-ironic10:19
milansambetts, hi :)10:21
*** sacharya has quit IRC10:24
*** guest has quit IRC10:26
*** yuikotakadamori has quit IRC10:29
openstackgerritVladyslav Drok proposed openstack/ironic: Add support for API microversions in Tempest tests  https://review.openstack.org/26035810:30
vdrokmorning milan10:30
milanmorning vdrok :)10:31
*** electrofelix has joined #openstack-ironic10:35
vdroktczekajlo: https://github.com/openstack/nova/blob/master/nova/conf/compute.py#L16110:35
*** ankit__ has joined #openstack-ironic10:36
ankit__Hi all, can someone please take a look at https://review.openstack.org/31849710:37
*** divya_ has joined #openstack-ironic10:41
*** divya_ has quit IRC10:42
*** divya_ has joined #openstack-ironic10:43
*** appu103_ has joined #openstack-ironic10:49
*** appu103_ is now known as aparnav10:50
*** szu has joined #openstack-ironic10:54
*** fragatina has joined #openstack-ironic10:55
*** szu has left #openstack-ironic10:56
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Add boot information into the inventory  https://review.openstack.org/32044810:56
*** Egyptian[Home] has joined #openstack-ironic10:57
*** fragatina has quit IRC11:00
jrollmgould | and ʔ is glottal stop, which is the sound Londoners traditionally use instead of a T <- this is one of my favorite things, TIL11:00
jrollmorning everyone11:00
sambettso/ jroll!11:02
dtantsurmorning jroll11:02
dtantsurjroll, when you get some coffee, could you please take a look at stable backport https://review.openstack.org/320862 ?11:02
*** jralph has quit IRC11:02
lucasagomesjroll, morning11:03
openstackgerritSam Betts proposed openstack/ironic-python-agent: Consolidate IPA configuration into a config module  https://review.openstack.org/32086711:03
openstackgerritSam Betts proposed openstack/ironic-python-agent: Support LLDP data as part of interfaces in inventory  https://review.openstack.org/32058411:03
*** Nisha_away has quit IRC11:03
sambettsJayF: ^^11:04
vmud213lucasagomes: what is your opinion on https://review.openstack.org/#/c/302143/ (local boot with fedora uefi)11:08
vmud213it's pending for long time11:08
lucasagomesvmud213, will look11:08
*** stendulker has quit IRC11:10
*** Goneri has quit IRC11:13
vdrokmorning jroll11:14
*** amotoki has quit IRC11:17
*** openstackgerrit has quit IRC11:18
*** openstackgerrit has joined #openstack-ironic11:18
openstackgerritMerged openstack/ironic: Mention RFEs in README  https://review.openstack.org/32074711:19
dtantsurjroll, https://review.openstack.org/#/c/260358/ (microversion testing) actually lgtm, please take a look11:24
dtantsurlucasagomes, ^^^11:24
jrolldtantsur: sure, just a minute11:24
lucasagomesdtantsur, ++ 1 sec11:24
*** thrash|g0ne is now known as thrash11:33
jrolldtantsur: one thing there...11:33
dtantsurjroll, I think it's using fake drivers, otherwise how do it pass now?11:34
jrolldtantsur: note that I don't think these run yet /me double checks11:34
*** amotoki has joined #openstack-ironic11:34
dtantsurI saw them passing in the console log... Oo11:34
jrolloh wow they do run11:34
jrollhuh. TIL.11:34
jrollI just assumed they use the driver devstack is configured with11:35
jroll+2, I'll give lucasagomes time to look unless he wants me to go ahead and merge11:35
dtantsurjroll, https://github.com/openstack/tempest/blob/master/tempest/config.py#L1081-L108311:35
jrollaha11:36
jrollthanks11:36
lucasagomesjroll, 1 sec finishing the review11:36
jrollno rush :)11:36
sambettsthe tempest APIs tests use the FakeDriver, which means that running the full tempest suite actually fails if you remove fake drivers from the enabled drivers list in devstack11:36
sambettsI discovered that when I was setting up my CI11:36
dtantsurright. this patch does not change it fwiw11:37
*** fragatina has joined #openstack-ironic11:37
jrollsambetts: unless you configure tempest to use your driver, apparently :)11:38
sambettsalso the API tests register and delete a ton of nodes in Ironic using the Fakedriver and that plays havok with nova which is how I found the race condition between the schedular and the resource tracker, and its also why we have to turn concurrency to 111:38
jrollright11:38
divya_hi all11:39
sambettsjroll: the API tests will try to use the Fakedriver even if you turn all drivers except say the Cisco one off :-P11:39
jrollsambetts: we should move to one job for our api tests, and many jobs for booting instances with different configs and such11:39
sambetts++11:39
jrollsambetts: even if you have the cisco driver here? https://github.com/openstack/tempest/blob/master/tempest/config.py#L1081-L108311:39
divya_getting error during nova boot--Fixed interval looping call 'nova.virt.ironic.driver.IronicDriver._wait_for_active' failed^[[00m11:40
divya_can anybody help to analysis11:40
sambettsjroll: ah no, but we have no way to override that in our devstack script right now11:40
jrolldivya_: if you read the code there, you'll see the deploy didn't finish before nova's timeout, check ironic logs11:40
lucasagomes+a on microversion in tempest11:41
jroll\o/11:41
lucasagomesjroll, btw, i've added https://bugs.launchpad.net/ironic/+bug/158559511:41
openstackLaunchpad bug 1585595 in Ironic "[RFE] Handling SIGHUP on Ironic services" [Wishlist,New] - Assigned to Lucas Alvares Gomes (lucasagomes)11:41
sambettsjroll: actually hold on, it might even to it with that set :/11:41
lucasagomesmaybe is not needed for the rolling upgrades anymore (dan smith comment on the patch)11:41
jrollsambetts: you're right, missing it here https://github.com/openstack-dev/devstack/blob/master/lib/tempest#L47011:41
lucasagomesbut still a good feature on it's on, disabling and enabling debug mode w/o restarting the services for example11:41
lucasagomesown*11:42
*** rbudden has joined #openstack-ironic11:42
divya_jroll: The deployment failed..File "/opt/stack/nova/nova/virt/ironic/driver.py", line 407, in _wait_for_active ^[[01;31m2016-05-25 16:31:12.214 TRACE oslo.service.loopingcall ^[[01;35m^[[00m    raise exception.InstanceDeployFailure(msg)11:42
divya_how to come out of this error11:42
*** fragatina has quit IRC11:42
jrolldivya_: you'll need to read ironic logs11:43
jrolllucasagomes: dan comment on which patch? I do agree it's nice to have anyway11:43
lucasagomesjroll, on the rolling upgrade spec11:43
lucasagomes1 sec, will get the link11:44
lucasagomesjroll, https://review.openstack.org/#/c/299245/811:44
jrolllucasagomes: yeah I've got it, thanks11:45
lucasagomesjroll, because on that version (#8 and below) SIGHUP seemed to be a fundamental part of our strategy, so I went out to test it and found out as well that nova does not support it at the moment11:45
lucasagomesjroll, so he commented explaning how they do it11:45
*** tczekajlo has quit IRC11:45
jrolllucasagomes: cool, works for me11:46
lucasagomesack ty11:46
*** ankit__ has quit IRC11:49
jrolldtantsur: so now we need to make a list of microversion tests to add :)11:50
* jroll puts that on todo list11:50
divya_ERROR ironic.drivers.modules.ipmitool [^[[00;36m-^[[01;31m] ^[[01;35m^[[01;31mIPMI Error while attempting "ipmitool -I lanplus -H 172.26.24.104 -L ADMINISTRATOR -U USERID -R 12 -N 5 -f /tmp/tmpQIKqfT power off"for node 27d8fab9-653b-481b-9a33-0a77e8d488d7. Error: Unexpected error while running command11:51
divya_this error is observed in ir-con logs jroll11:51
dtantsurjroll, oh yeah, there will be a lot11:51
*** vmud213 has quit IRC11:52
jrolldivya_: so something is wrong with your BMC or the network there, maybe there is additional logs that tell you what the error with ipmitool was11:52
jrolldtantsur: there can't be too many, we only have 16 versions or so and a few are already covered :)11:54
*** amotoki has quit IRC11:54
divya_which logs shd i refer for ipmitool.. since ir logs doesnt have much details11:54
dtantsurjroll, ideally we should also run integration tests on at least a couple of versions, not only 1.111:55
*** nico-afk is now known as nicodemos11:58
nicodemosmorning, ironic11:58
*** athomas has quit IRC11:58
*** Egyptian[Home] has quit IRC12:00
*** ishant has quit IRC12:01
*** aparnav has quit IRC12:01
*** lucasagomes is now known as lucas-hungry12:03
*** athomas has joined #openstack-ironic12:03
*** caiobo has joined #openstack-ironic12:04
vsaienkomorning nicodemos12:06
jrolldivya_: the output of the ipmitool commands should be in ironic-conductor logs, if you're running in debug mode12:07
jrolldtantsur: well, nova hardcodes ironic api version, so not sure it's very useful12:07
*** fragatina has joined #openstack-ironic12:08
caiobogood morning jroll, JayF, dtantsur, vdrok, nicodemos, lucas-hungry  and everybody else :)12:08
dtantsurmorning caiobo, nicodemos12:09
dtantsurjroll, yeah, but I would still like to have at least something post-enroll12:09
jrolldtantsur: so you want devstack to test different versions?12:09
jrolls/test/run12:09
dtantsurjroll, essentially yes12:10
jrollhmm12:11
jrolldtantsur: idk if I see much use in that, we just create a chassis, a node, and a port12:12
*** fragatina has quit IRC12:12
jrollFYI, there's a town hall on the new summit format thing: http://www.openstack.org/blog/2016/05/faq-evolving-the-openstack-design-summit/12:14
jroll(and an faq)12:14
dtantsurjroll, anyway, inspector gate is currently running with "latest" :) so we might cover that12:16
jrolldtantsur: heh, cool12:17
*** xavierr_afk is now known as xavierr12:17
*** fragatina has joined #openstack-ironic12:18
vdrokmorning nicodemos and caiobo :)12:19
*** derekh has quit IRC12:19
xavierrmorning all12:20
nicodemoshey caiobo, vsaienko, vdrok, dtantsur, xavierr  o/12:20
vdrokhi xavierr12:20
xavierrmorning caiobo nicodemos vdrok dtantsur12:20
*** dprince has joined #openstack-ironic12:20
*** sacharya has joined #openstack-ironic12:21
caiobomorning xavierr12:21
xavierrvdrok: hi man :)12:21
*** thiagop has joined #openstack-ironic12:21
openstackgerritMerged openstack/ironic: Add support for API microversions in Tempest tests  https://review.openstack.org/26035812:22
vdrokdtantsur, jroll how do you feel about this one? https://review.openstack.org/318497 I see the hacking being referenced, but do we actually care in this particular case?12:23
vsaienkojroll, dtantsur do we need this https://review.openstack.org/#/c/306445/? I'm going to abandon...12:23
dtantsurvdrok, I don't care much either way12:24
jrollvdrok: I really don't care about little patches like that, to be honest, I see no benefit in it12:24
*** fragatina has quit IRC12:24
jrollvsaienko: it can't hurt but I don't mind abandoning12:25
*** sacharya has quit IRC12:25
*** lindycoder has joined #openstack-ironic12:26
vdrokyeah, me too12:26
lazy_princeHi All.. trying to use VMware ESXi VM as baremetal node in Ironic.. When performing nova boot, observing "Failed to set boot device for node virt_type vmware does not support this"12:28
lazy_princeis there a way to move forward with ESX VMs..?12:28
thiagopGood morning Ironic12:28
*** fragatina has joined #openstack-ironic12:29
dtantsurlazy_prince, maybe try the libvirt driver from ironic-staging-drivers?12:29
dtantsurI don't think our pxe_ssh supports vmware really well12:29
dtantsurmorning thiagop12:29
xavierrlazy_prince, which driver you are trying to use?12:30
lazy_princexavierr: i ma using pxe_ssh12:31
sambettsjroll: reading that Q and A about the summit stuff, it sounds like a much better plan than the one I had heard at the summit12:32
*** fragatina has quit IRC12:33
*** jjohnson2 has joined #openstack-ironic12:33
lazy_princeif vmware is not supported, may be we should remove vmware from pxe_ssh driver otherwise we should fix it..12:33
dtantsurlazy_prince, the whole *_ssh family is going away relatively soon12:34
vdrokbut, it's going to the same place libvirt is :)12:35
lazy_princeI thought it would be there to ease dev work..12:35
vdroklazy_prince: we'll be using virtualbmc and ipmitool for that12:36
vsaienkolazy_prince have you tried libvirt driver from ironic-staging-drivers?12:36
vsaienkoaccording to documentation it supports esx hypervisors https://libvirt.org/drvesx.html12:36
sambettslazy_prince: we now provide virtualBMC as a way of translating ipmi to libvirt calls, so we can actually run the real pxe_ipmitool driver for testing12:36
lazy_princevsaienko: i have not tried.. will be trying that now..12:36
lazy_princeaha.. but that is only kvm i guess.. is there a way to use ESX VMs by any chance..?12:37
mat128Anyone knows which vmware version this supports? https://github.com/openstack/ironic/blob/56380f0e12587618b71c0347518324a50cf2769e/ironic/drivers/modules/ssh.py#L16812:37
mat128Which is likely the problem lazy_prince is encountering12:38
*** mat128 is now known as mat128|brb12:38
lazy_princefor the record, I am using ESXi 5.512:39
*** wajdi has quit IRC12:39
lazy_princebut https://github.com/openstack/ironic/blob/56380f0e12587618b71c0347518324a50cf2769e/ironic/drivers/modules/ssh.py#L74612:40
lazy_princelooks like it was never working and is broken since some refactor..12:40
*** wajdi has joined #openstack-ironic12:40
lazy_princeor is it only that VMware Workstation is supported but no ESXi ??12:40
dtantsurlazy_prince, vbmc is not only KVN12:41
dtantsurKVM, damn12:41
dtantsurit's anything that libvirt supports12:41
dtantsureven if we don't configure it right now, it must be very easy to add12:41
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add CLI tests to check JSON response body  https://review.openstack.org/29123912:42
*** baoli_ has joined #openstack-ironic12:45
vsaienkolazy_prince: could you please provide feedback about libvirt driver once you tried it?12:45
lazy_princevsaienko: let me read about how to enable it and then once I try, I will let you know..12:46
*** absubram has quit IRC12:47
openstackgerritAndre keedy proposed openstack/ironic-inspector: Replace node data via uuid/data/unprocessed API  https://review.openstack.org/30837912:47
*** wajdi has quit IRC12:48
*** absubram has joined #openstack-ironic12:50
*** Goneri has joined #openstack-ironic12:56
*** rloo has joined #openstack-ironic13:01
*** ayoung has joined #openstack-ironic13:01
*** jjohnson2 has quit IRC13:01
*** lucas-hungry is now known as lucasagomes13:03
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add tests for --wrap option for driver-properties output  https://review.openstack.org/29707713:03
*** alexpilotti has joined #openstack-ironic13:04
*** ig0r__ has joined #openstack-ironic13:06
*** ptx0 has quit IRC13:07
*** trown|outtypewww is now known as trown13:08
*** ig0r_ has quit IRC13:08
*** mjura has quit IRC13:08
*** lindycoder has quit IRC13:11
*** ptx0 has joined #openstack-ironic13:13
jlvillalvsaienko, vdrok: Not sure if the extra TIMEOUT is required for Grenade: https://review.openstack.org/#/c/317751/13:17
vsaienkojlvillal, I would prefer to set them, since I received an timeout errors during testing13:19
vsaienkojlvillal, smoke tests uses up to 3 active instances, so default timeout may not be enough13:20
*** joprovost has joined #openstack-ironic13:23
*** rloo has quit IRC13:24
vsaienkojlvillal there is an example https://bugs.launchpad.net/ironic/+bug/158099413:24
openstackLaunchpad bug 1580994 in Ironic "increase unprovision_timeout for Ironic grenade job" [High,Confirmed] - Assigned to Vasyl Saienko (vsaienko)13:24
*** rloo has joined #openstack-ironic13:24
*** fragatina has joined #openstack-ironic13:29
*** [1]cdearborn has joined #openstack-ironic13:31
*** rloo has quit IRC13:32
*** rloo has joined #openstack-ironic13:32
*** fragatina has quit IRC13:35
NobodyCamGood morning Ironicers13:37
TheJuliaGood morning13:37
thiagopmorning NobodyCam TheJulia13:37
xavierrmorning NobodyCam :)13:38
xavierrmorning TheJulia13:38
NobodyCamgood morning TheJulia thiagop xavierr :)13:38
krtaylormorning Ironic13:39
krtaylorjlvillal, just saw your message, yes, I'll run the QA meeting today, no prob13:40
NobodyCammorning krtaylor13:40
TheJuliadtantsur: question for you, I seem to remember there was a bug in IPA with inspection where if IPA gave up... say because it didn't know the ironic api url to heartbeat to.... that inspection might fail?  By chance, do you remember anything like this?13:40
*** wajdi has joined #openstack-ironic13:41
*** derekh has joined #openstack-ironic13:41
*** milan has quit IRC13:42
*** itamarl has quit IRC13:42
*** mgoddard_ has joined #openstack-ironic13:44
*** mgoddard has quit IRC13:48
*** links has quit IRC13:48
dtantsurTheJulia, morning. no, does not ring any bells13:50
*** ametts has joined #openstack-ironic13:51
TheJuliaI distinctly remember the discusison of the class of issue here in IRC, I'll go dig through the bugs and file a new bug since we actually had some internal folks attempting to deploy ironic and use inspector for discovery hit it head on yesterday.  Once we figured out basically what was occuring, we got them moving forward thought \o/13:52
*** xek has quit IRC13:54
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add sanity tests for testing actions with Port  https://review.openstack.org/24094413:55
dtantsurTheJulia, I don't quote get the essence of the bug.. what exactly fails?13:55
TheJuliadtantsur: I think, I've not looked at the code (since I got a LOT of email in two weeks...) yet, but the essence is that ipa appears to have exited or failed it's self prior to inspection actually being able to post data.  I distinctly remember a similar discussion around if no disk storage is found by IPA about a month ago... maybe two.13:57
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add sanity tests for testing actions with Port  https://review.openstack.org/24094413:58
TheJuliaonce we gave ipa enough info to find a valid api to try and heartbeat against for discovery operations, inspection worked perfectly.  Then again, I also had no direct access to the system :\13:58
NobodyCammorning dtantsur :)13:58
dtantsurNobodyCam, morning13:58
dtantsurTheJulia, note that inspection/discovery does not require heartbeating, hence my question13:59
*** fragatina has joined #openstack-ironic14:01
TheJuliadtantsur: that is what I thought as well, but  it is what they are reporting that what they had to change to make it work.... tl;dr I need to sift through the code of IPA again becuase this also feels like something related that we may have already fixed14:01
*** joprovost1 has joined #openstack-ironic14:02
*** joprovost has quit IRC14:03
*** joprovost1 is now known as joprovost14:03
deraykrotscheck, hi14:04
*** sabeen has quit IRC14:04
deraykrotscheck, I was trying to connect to an ironic setup (with noauth and cors enabled) from webclient. It fails ..14:06
*** sabeen has joined #openstack-ironic14:06
*** fragatina has quit IRC14:08
lucasagomesjroll, dtantsur if you guys have a time https://bugs.launchpad.net/ironic/+bug/1585595 , I've a PoC on the conductor running which reloads the config on SIGHUP and configure logs14:08
openstackLaunchpad bug 1585595 in Ironic "[RFE] Handling SIGHUP on Ironic services" [Wishlist,New] - Assigned to Lucas Alvares Gomes (lucasagomes)14:08
lucasagomesdoes it need a spec?14:09
lucasagomesit's a fairly small change14:09
lucasagomesmaybe I can clean the code and submit it so you can see14:09
openstackgerritRuby Loo proposed openstack/ironic: minor changes to security documentation  https://review.openstack.org/32056614:09
NobodyCamoh lucasagomes is that like a clean shutdown :) ... and good morning14:09
lucasagomesNobodyCam, nop, it's reloading (some config options, "mutable" ones according to oslo.config) without having to restart the service14:10
lucasagomesNobodyCam, e.g, I can enable or disable the debug logs14:10
lucasagomeswithout restarting ir-conductor14:10
*** _vsaienko has joined #openstack-ironic14:12
*** _vs has joined #openstack-ironic14:12
NobodyCamnice14:13
rloohi and morning everyone, lucasagomes, NobodyCam, deray, TheJulia, dtantsur, krtaylor, xavierr, thiagop, vsaienko, jlvillal14:13
*** mgoddard_ has quit IRC14:13
lucasagomesrloo, morning14:13
sambettso/ rloo14:13
thiagophello rloo !14:13
derayrloo, g'morning :)14:14
rloolucasagomes: didn't tan need that config reloading stuff for his rolling upgrade?14:14
rloohi sambetts14:14
lucasagomesrloo, it doesn't. I mean that's how I started looking into it14:14
lucasagomesI saw that we didn't support that so I just checked out of curiosity14:14
rloolucasagomes: cool14:14
*** _vsaienko has quit IRC14:14
NobodyCammorning rloo and sambetts14:15
lucasagomesrloo, but also, I noticed that nova does not support it at the moment and the ironic spec for rolling upgrade was saying that we needed14:15
sambettshi NobodyCam14:15
NobodyCam:)14:15
lucasagomesrloo, so I've asked around, and dan smith from nova comment on the spec (the rolling upgrade ones in ironic) explaining how they do it14:15
lucasagomesso really nice of 'em14:15
*** _vs has quit IRC14:15
rloolucasagomes: i am behind wrt that spec, will try to look at it today. i saw some comments from dan so thx for pinging him about it.14:16
lucasagomesrloo, right on, np!14:16
xavierrrloo, hi there :)14:16
dtantsurmorning rloo14:19
rloooh dtantsur, wanted to ask you about https://review.openstack.org/#/c/320295/. Would it make sense to have config options for disk-wait-attempts, disk-wait-delay?14:20
*** sacharya has joined #openstack-ironic14:21
dtantsurrloo, I thought I will do if someone asks for it :)14:23
rloodtantsur: i'll file a low-hanging-fruit for that then. no need for you to do it.14:23
dtantsurrloo, thanks!14:23
*** joprovost has quit IRC14:24
*** mgoddard has joined #openstack-ironic14:25
vsaienkorloo morning14:25
*** _vsaienko has joined #openstack-ironic14:25
*** _vs has joined #openstack-ironic14:26
*** sacharya has quit IRC14:26
*** joprovost has joined #openstack-ironic14:29
*** jaybeale has joined #openstack-ironic14:29
dtantsurmy X server crashed after an attempt to increase brightness Oo are you sure it's not Monday today?14:34
dtantsur#pixiesay Feels like Monday... -m meh14:34
PixieBoots¯\_(ツ)_/¯: Feels like Monday...14:34
*** Goneri has quit IRC14:35
*** Goneri has joined #openstack-ironic14:36
jrolllucasagomes: did you just say "nice of dansmith"? he might not like you talking that way about him14:38
NobodyCammorning jroll and :p14:38
xavierrraise IncreaseBrightnessException("I'm afraid you can't do that")14:38
jrollhey NobodyCam :)14:39
NobodyCam:)14:39
TheJuliagood morning rloo14:41
dansmithjroll: I'm not so curmudgeonly that I don't enjoy making people happy :)14:43
*** Sukhdev has joined #openstack-ironic14:43
thiagopLOL14:44
jrolldansmith: I thought that required bribes14:46
krotscheckderay: Heya! Do you have log files that you can share?14:46
* jroll adjusts strategy14:46
dansmithheh14:47
*** _vsaienko has quit IRC14:54
*** _vs has quit IRC14:54
*** xhku has joined #openstack-ironic14:55
*** deray has quit IRC14:56
*** _vsaienko has joined #openstack-ironic14:56
*** _vs has joined #openstack-ironic14:57
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add tests for testing node-delete command  https://review.openstack.org/25145814:57
*** elfosardo has joined #openstack-ironic14:59
*** milan has joined #openstack-ironic15:01
vdrokmorning jlvillal NobodyCam TheJulia and rloo !15:01
TheJuliagood morning vdrok15:01
* TheJulia should just change her nick to TheZombieJulia for the rest of the week :)15:02
sambetts:-P15:02
NobodyCamgood morning vdrok :)15:02
*** amotoki has joined #openstack-ironic15:03
*** fragatina has joined #openstack-ironic15:03
thiagop#pixiesay TheJulia braaains.... -m dead15:04
PixieBootsʕxᴥxʔ: TheJulia braaains....15:04
rloohi vdrok, TheJulia15:05
TheJuliapretty much how I'm feeling sifting through two weeks of emails and getting back into the swing of things15:06
*** fragatina has quit IRC15:08
*** sdake has joined #openstack-ironic15:09
*** yolanda_ has joined #openstack-ironic15:10
lucasagomesjroll, hah I did15:11
vdrokrloo: TheJulia NobodyCam lucasagomes could you take a look at https://review.openstack.org/#/c/319183/ ? gathering opinions :)15:11
lucasagomesc15:11
lucasagomesvdrok, yes will do15:11
vdrokthanks15:11
*** _vsaienko has quit IRC15:12
*** _vs has quit IRC15:12
*** mgoddard_ has joined #openstack-ironic15:12
*** sdake_ has joined #openstack-ironic15:13
*** sdake has quit IRC15:13
lucasagomesvdrok, the only problem there is the level of debug?15:15
vdroklucasagomes: seems so :)15:15
*** mgoddard has quit IRC15:16
*** daemontool_ has quit IRC15:16
lucasagomesvdrok, done15:18
*** lindycoder has joined #openstack-ironic15:18
dtantsurvdrok, lucasagomes, oh, I've found a serious problem with this patch15:20
dtantsurlemme comment15:20
rloovdrok: so at that point, you know for sure that it is due to an older/mitaka-or-older ramdisk?15:20
*** yolanda has quit IRC15:20
lucasagomesJayF, if you have some time, I've rebased https://review.openstack.org/#/c/318538/ after a merge conflict15:21
*** amotoki_ has joined #openstack-ironic15:21
lucasagomesty in advance15:21
*** Sukhdev has quit IRC15:21
dtantsurso tl;dr: when we fallback to the default iscsi port we still try to use the configured one15:21
jrollaha15:21
jrollharsh15:21
*** xavierr is now known as xavierr_afk15:22
dtantsurI'm inclined to say that one must always update IPA first15:22
dtantsurand don't do this patch15:22
dtantsurjroll, lucasagomes, vdrok ^^15:22
vdrokrloo: most likely, yes :) maybe I can be more specific to check that type is TypeError15:22
lucasagomesdtantsur, I think the idea was that ipa would be backward compat15:22
dtantsurlucasagomes, IPA is, you can use it with older Ironic15:23
lucasagomesbut, well do we have releases for it right?15:23
rloodo we have a story wrt versioning of ipa and ironic?15:23
dtantsurI don't think so15:23
jrollwe don't15:23
rloomaybe we should?15:23
jrollthe story used to be "it always works"15:23
lucasagomesdtantsur, oh right yeah, understand it's the otherway around. Ironic not playing well with old IPA's15:23
jrolland then someone said we needed stable branches15:23
lucasagomes:-/15:23
jrollso I did it because packaging reasons15:23
lucasagomesrloo, we don't have versioning for IPA commands15:23
*** amotoki has quit IRC15:24
jrollidk if we would have caught this in the old world though15:24
jrollso, meh15:24
rlooyeah i think it might have been a problem in the old world. but given that we're in this new world, we should probably clarify/have a story...15:24
dtantsurjroll, no, because everything would use the master IPA15:25
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: [WIP] Remove support for the old bash ramdisk  https://review.openstack.org/31922715:26
lucasagomesjroll, dtantsur if we do not apply that patch, would a note in the release notes saying to upgrade IPA when upgrading ironic be suffice ?15:26
vdrokso in case of grenade that would mean we should update all the nodes to use the new ramdisk15:26
vdrokand in case of any upgrade15:26
dtantsurlucasagomes, I think we should even document to always do it, but dunno15:26
rloodtantsur: in the old world, we never cut a package for ipa?15:26
dtantsurrloo, iirc yes15:26
sambettsthe problem is that its a data migration to do it right, e.g. update all the nodes kernel / ramdisk driver_infos15:26
jrollwhich can't be done with sql commands other than REPLACE() :(15:27
jrollbecause json :(15:27
* rloo goes back to more sane stuff15:27
dtantsurin can be done with ironicclient though15:27
lucasagomesjroll, a command to update the deploy ramdisk / kernel is not a bad idea tho15:28
sambettsand we have to upload something to glance as part of the upgrade processes15:28
lucasagomesexternal ofc15:28
jrollstill makes for a horrible upgrade process15:28
lucasagomes:-/15:28
jrolland if we ever get long-running ramdisks, an even more painful one15:28
*** poll0172 has joined #openstack-ironic15:29
lucasagomesjroll, we can fix by design, if we start actually versioning the commands15:29
jrollI don't have another good answer for this other than versioning15:29
jrollwhich gets real complex real fast15:29
lucasagomesright15:29
jrollI need to step away and slam lunch down before meetings, sorry :/15:29
lucasagomesbut still better than never being able to extend a given command15:29
lucasagomesotherwise it's just ossified15:29
lucasagomesjroll, enjoy15:30
lucasagomesdtantsur, sambetts so I think it comes down to: 1. have a version negotiation between ironic and ipa. 2. have tools to facilitate the upgrade. 3. do as that patch is doing, but may be more in the future15:31
sambettsI suggest we use this patch as a bandaid, while we implement either a versioning system or a better API discovery process for IPA, e.g. send a request to IPA and ask what parameters it accepts15:31
lucasagomesvdrok, ^15:31
*** Romanenko_K has joined #openstack-ironic15:31
vdrokif we need it fast #3 is the way I think, and I don't like #2 :)15:32
*** amotoki_ has quit IRC15:34
*** sdake_ has quit IRC15:34
*** rama_y has joined #openstack-ironic15:36
vdrokdtantsur: not sure I get your comment, don't we need to update the default here https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_client.py#L134 to be CONF.iscsi.portal_port?15:36
vdrokin the second try we don't pass portal_port at all, in the first try it is CONF.iscsi.portal_port15:37
*** Sukhdev has joined #openstack-ironic15:37
dtantsurvdrok, yeah, and what port will Ironic use after the second try?15:38
dtantsurstill CONF.iscsi.portal_port, even though IPA uses the standard one15:38
vdrokah, youre right15:38
vdrokthanks15:38
*** praneshp has joined #openstack-ironic15:38
*** watanabe_isao has joined #openstack-ironic15:42
openstackgerritSam Betts proposed openstack/ironic-inspector: Add generic LLDP processing hook  https://review.openstack.org/32108215:45
sambettsdtantsur: ^^15:45
dtantsurcool!15:45
sambettsdtantsur: its based on receiving the data from that IPA patch I pushed15:46
*** Nisha_away has joined #openstack-ironic15:46
* sambetts -> home15:49
*** sambetts is now known as sambetts|afk15:49
*** dmk0202 has quit IRC15:49
sambetts|afknight all15:49
dtantsurg'night sambetts|afk15:50
*** sacharya has joined #openstack-ironic15:51
rlooalineb, are you around? let me know if you want to discuss the metrics patch.15:51
vdrokgood night sambetts|afk15:51
jrollI agree with sambetts|afk, bandaid with that patch and figure out the future15:52
jrollprobably allow listing possible commands and their arguments15:52
*** rpioso has joined #openstack-ironic15:55
*** MattMan has quit IRC15:56
*** MattMan has joined #openstack-ironic15:57
lucasagomessambetts|afk, see ya15:57
thiagopsambetts|afk: good night15:57
*** Sukhdev has quit IRC15:59
*** Sukhdev has joined #openstack-ironic16:00
vsaienkolucasagomes: once you have a time could you please have look on https://review.openstack.org/#/c/299229.16:01
*** fragatina has joined #openstack-ironic16:02
lucasagomesvsaienko, sure thing!16:04
lucasagomes1 sec16:04
*** rcernin has quit IRC16:04
*** _vsaienko has joined #openstack-ironic16:04
*** _vs has joined #openstack-ironic16:04
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add tests for testing node-delete command  https://review.openstack.org/25145816:05
*** fragatina has quit IRC16:06
*** athomas has quit IRC16:08
*** athomas has joined #openstack-ironic16:09
*** ipukha has quit IRC16:10
*** athomas has quit IRC16:10
*** praneshp has quit IRC16:10
*** links has joined #openstack-ironic16:10
*** daemontool has joined #openstack-ironic16:10
*** ifarkas has quit IRC16:12
*** tesseract has quit IRC16:13
*** _vsaienko has quit IRC16:14
*** _vsaienko has joined #openstack-ironic16:14
vdrokdtantsur: I see even more problems - https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/iscsi_deploy.py#L371 portal port is not passed to continue_deploy, so here https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/iscsi_deploy.py#L263 it will get16:16
vdrokhttps://github.com/openstack/ironic/blob/master/ironic/drivers/modules/iscsi_deploy.py#L20516:16
*** _vsaienko has quit IRC16:16
*** _vs has quit IRC16:16
*** _vsaienko has joined #openstack-ironic16:16
*** _vs has joined #openstack-ironic16:16
*** mbound has quit IRC16:18
dtantsuroh16:18
vdrokdtantsur: it seems to me that we need to change https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/iscsi_deploy.py#L205 this line to something like driver_internal_info['port_override'] or CONF.iscsi.portal_port16:19
vdrokport_override being set in start_iscsi_target16:19
dtantsuryeah16:19
*** daemontool has quit IRC16:20
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: [WIP] Remove support for the old bash ramdisk  https://review.openstack.org/31922716:20
dtantsurtime to go, see you tomorrow16:20
*** dtantsur is now known as dtantsur|afk16:21
*** derekh has quit IRC16:22
*** aNupoisc has joined #openstack-ironic16:23
*** athomas has joined #openstack-ironic16:24
*** penick has joined #openstack-ironic16:25
vdroknight dtantsur|afk16:25
*** kromanenko_ has quit IRC16:29
*** Romanenko_K has quit IRC16:29
*** sdake has joined #openstack-ironic16:29
*** kromanenko__ has quit IRC16:29
*** aNupoisc has quit IRC16:30
lucasagomesvdrok, dtantsur|afk see ya16:34
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add SIGHUP support for ironic-conductor service  https://review.openstack.org/32111116:34
*** _vsaienko has quit IRC16:34
*** _vs has quit IRC16:35
*** moshele has quit IRC16:35
vdrokI'm still here for a while :)16:36
*** _vsaienko has joined #openstack-ironic16:36
*** _vs has joined #openstack-ironic16:36
*** _vsaienko has quit IRC16:41
*** _vs has quit IRC16:42
*** rama_y has quit IRC16:46
*** Sukhdev has quit IRC16:48
*** Sukhdev has joined #openstack-ironic16:49
*** Sukhdev has quit IRC16:54
*** aNupoisc has joined #openstack-ironic16:55
*** xavierr_afk is now known as xavierr16:57
lucasagomesvdrok, oh sorry, thought you were going too16:58
vdroklucasagomes: heh, it's not the thing to be sorry about :D16:59
alinebrloo: are you still here? (sorry, i was in a meeting)17:00
*** athomas has quit IRC17:00
*** e0ne has quit IRC17:00
vdrokjlvillal: meeting?17:01
vdrokoh, sorry, youre away17:02
*** rloo has quit IRC17:02
*** rloo has joined #openstack-ironic17:03
*** fragatina has joined #openstack-ironic17:03
jrollkrtaylor: I question all of those CI systems that say they're running on every patch: https://review.openstack.org/#/c/318497/17:03
jrollthat's been up since may 1917:03
* jroll brb17:04
jrolloh right, meeting, I'll mention it there17:04
thiagopjroll: ours is not.17:04
thiagopjroll: just "/ironic/drivers"17:04
jrollthiagop: why, you don't think other changes can break you?17:04
thiagopjroll: we just don't have the hardware to run that much patches17:05
jrollhmm17:05
rlooalineb: here now17:05
thiagopjroll: and frankly, our tests are taking too much time. But I'm about to deploy a new infra where I cut the time to nearly half17:06
*** fragatina has quit IRC17:07
*** mgoddard_ has quit IRC17:07
alinebrloo: ok, so if i understand correctly from your review, you would prefer to remove the delimiter and let the caller handle it if one is needed?17:07
*** mgoddard_ has joined #openstack-ironic17:07
rlooalineb: no. the delimiter is fine. to delimit whatever the format was, global-prefix-prefix-host-name.17:08
*** jaybeale has quit IRC17:08
rlooi just don't think we should have methods that take 'prefix_parts' and generates a prefix string from that.17:08
rlooalineb: what did i write that makes you/jay think I don't like the delimiter?17:09
JayFrloo: we misunderstood independently fwiw, hehe17:10
rlooJayF: right, so 2 people misunderstood. I was wondering what I wrote that led to that misunderstanding. Maybe i need to communicate differently or something.17:11
alinebrloo: so no delimiter in "prefix" or "metric name" but betwen them, it's ok?17:11
alineband i just think my problem is that i saw the delimiter either as completely there or not at all17:12
lucasagomescalling it a day folks :-)17:12
rlooalineb: yes, that is how i see the delimiter.17:12
lucasagomessee you all tomorrow17:13
rlooalineb: i could be wrong though. haven't seen any justification otherwise.17:13
rloobye lucasagomes17:13
jrollnight lucas17:13
lucasagomesbye bye17:13
*** lucasagomes is now known as lucas-dinner17:13
alinebrloo: no, i think it makes sense, that's just not how i understood things at first :) i will change that17:14
rlooalineb: thx17:14
jrollvdrok: well, grenade-partial might not quite be configured right yet, to upgrade only -conductor17:15
vdrokjroll: yeah, seems so, it just shuts down both api and conductor I guess17:15
*** trown is now known as trown|lunch17:16
*** poll0172 has quit IRC17:17
alinebrloo: the other question i had concerning your review is about the naming of the decorator and context managers: in statsd client, they don't need to differentiate them since they implement it differently17:17
*** ChrisAusten has joined #openstack-ironic17:18
rlooalineb: I am not a metrics expert, I don't know the/any accepted terminology for metric-related things.17:19
rlooalineb: i questioned what I saw though, cuz timer_d and timer() just doesn't do it for me.17:19
jrollvdrok: heh17:20
alinebrloo: ok, i will try to see what i can find then :) thanks a lot for your help!17:20
rlooalineb: thx. I'd like to get that stuff merged sooner rather than later and I know I am a major blocker of it...17:21
JayFrloo: "timer" is the pretty common term used for that kind of metric17:24
JayFrloo: I'm kinda on board with timer_c and timer_d being awkward, but don't have a better idea and those were explicitly approved in the spec :/17:25
rlooJayF: 'timer' to emit some timing. or 'timer' to time something?17:25
JayFa timer metric is typically used to indicate how long it took to do something17:25
JayFi.e. timing to see if slow_method() is slow17:25
rlooJayF: so the decorator actually times something and emits the timing. then there is some timer() method that just emits some timing value.17:26
JayFAlthough full disclosure, most of my metrics experience is emitting to statsd, so that may be why I am used to that terminology17:26
rlooJayF: so it seems odd to me that both of them would be called 'timer..'17:26
rlooJayF: look, if it is just me, then you can ignore my dumb questions.17:27
*** rama_y has joined #openstack-ironic17:27
JayFI mean, I think in this case you represent someone less familiar with the domain of metrics17:27
JayFbut I think that makes you a more effective reviewer :)17:27
JayFso I want to make sure I understand the concerns well17:28
JayFrloo: so you're saying that maybe we shouldn't expose a method that you can just pass in a random timing value?17:28
JayFrloo: that we should restrict sending timer metrics to only if you use the decorator or context manager?17:28
rlooJayF: so yeah, if within the domain of metrics, that is understandable terminology, then fine, just tell me that it is.17:28
JayFa timer is a type of metrics17:29
JayFlike statsd handles a "timer" differently than a "gauge"17:29
rlooJayF: no, I'm not saying to get rid of the methods. I am questioning the naming of the methods and decorators. Are they clear?17:29
JayFthe thing I'm not 100% sure about is if it's statsd terminology or metrics terminology :)17:29
JayFoh absolutely17:29
rlooJayF: I thought I saw that statsd handled 'timing' data.17:29
JayFit's like 500 lines of nodejs, I'll check that code directly :P17:29
JayFhttps://github.com/etsy/statsd/blob/master/stats.js#L7917:30
JayFthat uses the term "timers"17:30
rlooJayF: so if I were to do this, I'd look at a few metrics related s/w or whatever, and see what terminology they use.17:30
JayFI know for statsd, graphite, and the proprietary rackspace monitoring stuff I worked on, this makes sense17:31
JayFI honestly don't know many other tools people use to track these kind of software metrics. New Relic, maybe?17:31
rlooJayF: what makes sense? that they have a timer_d() decorator and a timer() method that don't do the same thing?17:31
JayFHuh?17:32
JayFtimer_d decorator does provide a timer() metric/17:32
*** ChubYann has joined #openstack-ironic17:32
*** sdake_ has joined #openstack-ironic17:33
*** sdake has quit IRC17:35
rlooJayF: so maybe it is just me (besides the '_d' which makes me think of fortran), that timer_d will actually time how long something takes and emit that timing, and timer() just emits a timing. but they both are called timerX17:35
*** poll0172_ has joined #openstack-ironic17:35
JayFtimer() # you have already timed something and need to emit that value17:36
JayFtimer_d() # you have a function that you want to measure timing on and want the details handled17:36
*** poll0172_ has quit IRC17:37
JayFtimer_c() # you have a section of code that you want to measure timing on and want the details handled17:37
JayFare you thinking timer() should be something like timer_raw()?17:37
*** poll0172 has joined #openstack-ironic17:37
JayFfwiw we did dictate these names for the interfaces in the spec: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/add-pluggable-metrics-backend-for-ironic-and-ipa.html17:37
*** piet has joined #openstack-ironic17:37
jrollI feel like these names are weird too17:39
*** poll0172 has quit IRC17:39
jrollwhat if we just namespaced them - metrics.context_mgr.timer, metrics.decorators.timer, metrics.timer17:39
jrollor something17:39
JayFI mean, those are better than the existing ones17:41
JayFbut honestly I feel like it's a little bikesheddy at this point if that's the only remaining objection17:42
rlooJayF: you're right. it is shown in the spec. How did I miss it then? Sigh.17:42
JayFLike I agree the methods are strangely named. I couldn't think of anything better than or now17:42
jrollJayF: from a developer standpoint it isn't the best API :(17:42
*** mgoddard_ has quit IRC17:42
*** poll0172 has joined #openstack-ironic17:42
JayFbut feels strange to be talking about this as the only real objection to the code landing17:42
rlooJayF: it isn't the only real objection. I believe I've had quite a few objections.17:43
JayFreal was a bad word to use there.17:43
JayFunresolved?17:43
JayFlike the other things we have a path to fixing, right?17:43
rlooJayF: so far, the only unresolved. I think.17:43
*** Goneri has quit IRC17:43
jrolldo we not have a path to fixing this? I just suggested one :)17:43
JayFjroll: you wanna propose the update to the spec then or shall I?17:44
rlooJayF: honestly, I think I've made suggestions and comments. And I haven't had any real good reasons for using the things I objected to. There have been suggestions.17:44
JayFYeah, I'm just frustrated that this didn't move for a long time (nobody working on it) and now it feels like going backwards a little17:45
*** Goneri has joined #openstack-ironic17:45
*** _vsaienko has joined #openstack-ironic17:46
*** _vs has joined #openstack-ironic17:46
rlooJayF: so I can't comment on the no one working on it. but I have been reviewing it and I am somewhat frustrated by it and wondering how some of it could have been done more effectively. Maybe I need to spend more time getting more details/thinking about it, at spec level.17:46
JayFI know, the nobody working on it is not anyone's fault other than circumstances17:46
JayFI can understand wanting to get the naming right before we're locked in17:47
openstackgerritVladyslav Drok proposed openstack/ironic: Gracefully degrade start_iscsi_target for Mitaka ramdisk  https://review.openstack.org/31918317:47
openstackgerritVladyslav Drok proposed openstack/ironic: Use correct iscsi portal port in continue_deploy  https://review.openstack.org/32114517:47
JayFI'll push an update to the spec using the names jroll suggested17:47
jrollJayF: thanks17:50
*** ig0r__ has quit IRC17:51
*** poll0172 has quit IRC17:51
*** links has quit IRC17:52
*** e0ne has joined #openstack-ironic17:54
*** ChrisAusten has quit IRC17:54
devanandarunning off to a lunch meeting - bbl17:58
*** swatson has joined #openstack-ironic17:58
openstackgerritStephanie Miller proposed openstack/bifrost: Make booleans in templates explicit  https://review.openstack.org/32115118:00
openstackgerritJay Faulkner proposed openstack/ironic-specs: Update metrics spec to reflect review comments  https://review.openstack.org/32115318:01
JayFrloo: jroll alineb ^18:01
*** fragatina has joined #openstack-ironic18:03
*** aNupoisc has quit IRC18:04
*** ricardoas has joined #openstack-ironic18:05
*** e0ne has quit IRC18:06
*** _vs has quit IRC18:07
*** _vsaienko has quit IRC18:07
*** fragatina has quit IRC18:08
*** e0ne has joined #openstack-ironic18:11
*** aNupoisc has joined #openstack-ironic18:12
*** ChrisAusten has joined #openstack-ironic18:13
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Tempest: add basic test  https://review.openstack.org/29392618:13
*** e0ne has quit IRC18:14
*** swatson has quit IRC18:14
*** fragatina has joined #openstack-ironic18:17
*** penick has quit IRC18:18
*** _vsaienko has joined #openstack-ironic18:18
*** _vs has joined #openstack-ironic18:18
*** vishwanathj has joined #openstack-ironic18:19
*** electrofelix has quit IRC18:21
*** mjturek1 has joined #openstack-ironic18:21
*** moshele has joined #openstack-ironic18:21
*** trown|lunch is now known as trown18:22
*** sdake_ is now known as sdake18:23
*** fragatina has quit IRC18:24
*** moshele has quit IRC18:25
*** moshele has joined #openstack-ironic18:25
*** watanabe_isao has quit IRC18:26
openstackgerritVladyslav Drok proposed openstack/ironic: Gracefully degrade start_iscsi_target for Mitaka ramdisk  https://review.openstack.org/31918318:27
openstackgerritVladyslav Drok proposed openstack/ironic: Use correct iscsi portal port in continue_deploy  https://review.openstack.org/32114518:27
*** watanabe_isao has joined #openstack-ironic18:28
*** watanabe_isao has quit IRC18:28
*** penick has joined #openstack-ironic18:29
*** fragatina has joined #openstack-ironic18:30
NobodyCamjroll: ElCoyote_: question on 311278... would it be better to name the option use_headless instead of vbox_use_headless, especially the comment of other drivers making use of it..18:32
jrollNobodyCam: meh18:33
ElCoyote_Several other hypervisors can/could make use of 'headless' mode.. but only vbox currently implements it18:33
jrollup to you I guess18:33
ElCoyote_So I'm in favor of keeping 'use_headless' but it's only being toggled if vbox_use_headless is defined in driver_info.18:34
*** fragatina has quit IRC18:34
vdrokgood night everyone!18:34
*** devananda has quit IRC18:35
*** sdake_ has joined #openstack-ironic18:35
NobodyCamjust afraid at some point we'll have [vbox|virsh|vmware|parallels]_use_headless opts18:35
NobodyCamnight vdork18:35
ElCoyote_I know you can run 'headless' with Parallels and VMware workstation so I guess it could be implemented elsewhere.18:35
rloonight vdrok18:35
ElCoyote_NobodyCam: Yes, I see your point.18:35
NobodyCamI saw that with lines 153/154 ofssh.py18:36
*** sdake has quit IRC18:37
ElCoyote_NobodyCam: Don't know what to tell you.. the vbox_ prefix was requested by some others here.18:38
jrollyeah, I'd prefer it unless other types support that18:38
NobodyCamoh let me look at the other comments18:39
jrollprefer vbox_, that is18:39
*** piet has quit IRC18:39
jrollNobodyCam: mostly for the output of required/optional properties18:39
jrollor to keep people from askign why parallels pops open a window when they have use_headless=true18:39
jrollif this was a production driver I'd look much harder at it though :)18:40
NobodyCamack18:41
NobodyCam+A'd18:46
*** sdake_ is now known as sdake18:48
*** piet_ has joined #openstack-ironic18:51
*** mgoddard has joined #openstack-ironic19:00
*** joprovost has quit IRC19:04
*** fragatina has joined #openstack-ironic19:07
*** fragatina has quit IRC19:07
openstackgerritJarrod Johnson proposed openstack/pyghmi: Fix KeyError due to lazy use of raw_command  https://review.openstack.org/32119119:07
*** fragatina has joined #openstack-ironic19:08
*** jaypipes has joined #openstack-ironic19:08
openstackgerritMerged openstack/ironic: Make use of oslo-config-generator  https://review.openstack.org/24733119:08
*** _vsaienko has quit IRC19:11
*** _vs has quit IRC19:11
*** ChrisAusten has quit IRC19:13
*** aNupoisc has quit IRC19:13
openstackgerritMerged openstack/ironic: Make the ssh driver work on headless VirtualBox machines  https://review.openstack.org/31127819:24
*** _vs has joined #openstack-ironic19:26
*** _vsaienko has joined #openstack-ironic19:26
*** ricardoas has quit IRC19:27
*** jaypipes has quit IRC19:30
*** jaypipes has joined #openstack-ironic19:31
*** _vs has quit IRC19:32
*** _vsaienko has quit IRC19:32
*** caiobo has quit IRC19:41
*** Nisha_away has quit IRC19:46
openstackgerritMilan Kováčik proposed openstack/ironic: Introduce provision states: AVAILABLE, ENROLL  https://review.openstack.org/32120419:50
*** jaybeale has joined #openstack-ironic19:51
*** _vsaienko has joined #openstack-ironic19:52
*** _vs has joined #openstack-ironic19:52
*** jaybeale has quit IRC19:54
*** ElCoyote_ has quit IRC19:55
*** piet_ has quit IRC19:56
*** ElCoyote_ has joined #openstack-ironic20:02
openstackgerritRuby Loo proposed openstack/ironic: Follow-up patch of 8e5e69869df476788b3ccf7e5ba6c2210a98fc8a  https://review.openstack.org/31414920:02
*** ayoung has quit IRC20:04
*** moshele has quit IRC20:04
*** _vsaienko has quit IRC20:14
*** _vs has quit IRC20:14
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Tempest: add basic test  https://review.openstack.org/29392620:17
*** jcoufal has joined #openstack-ironic20:23
*** _vs has joined #openstack-ironic20:25
*** _vsaienko has joined #openstack-ironic20:25
openstackgerritJarrod Johnson proposed openstack/pyghmi: Fix FPC detection for Lenovo Nextscale  https://review.openstack.org/32122220:30
*** ChrisAusten has joined #openstack-ironic20:32
*** Waltz has quit IRC20:36
*** rbrud has joined #openstack-ironic20:38
*** _vs has quit IRC20:45
*** _vsaienko has quit IRC20:45
*** sdake_ has joined #openstack-ironic20:46
*** sdake has quit IRC20:48
*** aNupoisc has joined #openstack-ironic20:49
openstackgerritJarrod Johnson proposed openstack/pyghmi: Raise incurred exception if appropriate  https://review.openstack.org/32123420:50
*** dprince has quit IRC20:54
jrollmariojv: safe to assume there's more patches coming after https://review.openstack.org/#/c/298461/ ?20:55
*** _vsaienko has joined #openstack-ironic20:55
*** _vs has joined #openstack-ironic20:55
mariojvjroll: yeah, i have code for a power state notif that i took down, unit tests weren't passing20:57
mariojvtrying to get that fixed and up this week20:57
jrollmariojv: okay, just wanted to make sure that wasn't everything20:57
jrollthanks20:57
*** penick has quit IRC21:03
*** baoli_ has quit IRC21:04
*** daemontool has joined #openstack-ironic21:07
*** mjturek1 has quit IRC21:10
*** jaypipes has quit IRC21:11
*** jaybeale has joined #openstack-ironic21:12
*** jaypipes has joined #openstack-ironic21:13
*** jaybeale has quit IRC21:13
*** clenimar has quit IRC21:14
*** jaybeale has joined #openstack-ironic21:15
*** trown is now known as trown|outtypewww21:23
*** mgoddard has quit IRC21:24
*** [1]cdearborn has quit IRC21:25
*** penick has joined #openstack-ironic21:25
*** penick has quit IRC21:26
*** lindycoder has quit IRC21:26
*** ametts has quit IRC21:28
jrollsee y'all tomorrow21:28
xavierrjroll see ya21:29
*** Goneri has quit IRC21:30
*** daemontool has quit IRC21:31
openstackgerritMerged openstack/ironic: Allow to specify node arch  https://review.openstack.org/31738921:33
openstackgerritMerged openstack/ironic-python-agent: Add a configurable sleep before IPA starts working  https://review.openstack.org/31874521:34
*** penick has joined #openstack-ironic21:37
openstackgerritRuby Loo proposed openstack/ironic: Add 'How to get a decision on something' to FAQ  https://review.openstack.org/32124621:37
*** Egyptian[Home] has joined #openstack-ironic21:38
*** rpioso has quit IRC21:41
*** Egyptian[Home] has quit IRC21:45
*** sdake_ has quit IRC21:45
*** penick has quit IRC21:50
*** sdake has joined #openstack-ironic21:51
*** ChrisAusten has quit IRC21:54
*** xhku has quit IRC21:54
*** piet has joined #openstack-ironic21:58
*** penick has joined #openstack-ironic22:00
*** aNupoisc has quit IRC22:09
openstackgerritThiago Paiva Brito proposed openstack/python-oneviewclient: Fixing __getattribute__ def and refactor from_json  https://review.openstack.org/32126222:12
*** ChrisAusten has joined #openstack-ironic22:15
*** sdake has quit IRC22:25
*** Egyptian[Home] has joined #openstack-ironic22:27
*** jcoufal has quit IRC22:27
openstackgerritStephanie Miller proposed openstack/bifrost: Unify testing scripts  https://review.openstack.org/29742422:28
*** ayoung has joined #openstack-ironic22:30
*** _vsaienko has quit IRC22:36
*** _vs has quit IRC22:36
*** ChrisAusten has quit IRC22:36
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [console]  https://review.openstack.org/29594622:36
openstackgerritMerged openstack/pyghmi: Fix KeyError due to lazy use of raw_command  https://review.openstack.org/32119122:36
*** rloo has quit IRC22:48
*** rloo has joined #openstack-ironic22:48
*** Egyptian[Home] has quit IRC22:55
*** rbrud has quit IRC22:56
*** thiagop has quit IRC22:56
*** chlong has joined #openstack-ironic23:05
*** jaybeale has quit IRC23:09
*** jaybeale has joined #openstack-ironic23:11
*** harlowja has quit IRC23:12
*** rloo has quit IRC23:12
*** vishwana_ has joined #openstack-ironic23:21
openstackgerritMerged openstack/pyghmi: Fix FPC detection for Lenovo Nextscale  https://review.openstack.org/32122223:21
*** vishwanathj has quit IRC23:23
*** Goneri has joined #openstack-ironic23:24
*** alexpilotti has quit IRC23:25
*** sdake has joined #openstack-ironic23:28
*** jaypipes has quit IRC23:29
*** piet has quit IRC23:32

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