Friday, 2015-03-06

devanandaNobodyCam: I've put it on the agenda for monday. we need to sort it out ASAP00:00
NobodyCamyes. reading it now00:02
*** achanda has quit IRC00:04
*** penick has joined #openstack-ironic00:11
*** Marga_ has quit IRC00:18
jlvillalBadCub: devananda: I was looking at the spreadsheet line 63.  It says AMT power is merged.  Isn't this patch part of that?  https://review.openstack.org/#/c/152858/00:26
devanandajlvillal: oh snap. you're correct. commit 918d4e4 added only the common library00:28
BadCubI don't see any notes on line 63 on spreadsheet00:28
devanandathanks for catching that00:28
jlvillalBadCub: It said 'merged'00:28
jlvillalBadCub: Someones just changed it ;)  Not me though!00:29
jlvillaldevananda: You're welcome00:29
* devananda updates the spreadsheet00:29
*** Marga_ has joined #openstack-ironic00:29
BadCubI just saw devananda change it00:29
*** absubram has quit IRC00:32
jlvillaldevananda: I assume the answer to this is no.  But I've been asked if https://blueprints.launchpad.net/ironic/+spec/bare-metal-trust-using-intel-txt can be approved for Kilo.00:34
*** penick has quit IRC00:37
*** achanda has joined #openstack-ironic00:41
devanandajlvillal: no. today is/was the deadline for feature proposals, and we're already over-committed for reviews in the next few weeks, and it looks like we're a long way from having enough reviews on that spec00:41
jlvillaldevananda: Thanks.  I thought so and already tried to set expectations low.  We will try for Liberty.  Thanks!00:41
devanandajlvillal: if it helps, a lot of the work this cycle ws foundational to enable that sort of work i the future00:42
jlvillaldevananda: It helps :)  I will let them know that.00:42
*** anderbubble has joined #openstack-ironic00:43
*** penick has joined #openstack-ironic00:43
devanandajlvillal: have a look at the work related to uefi, if you have'nt already, and the addition of the management interface00:44
jlvillaldevananda: Will do.00:44
*** achanda has quit IRC00:46
NobodyCamand the effort around the boot interface00:46
jlvillalNobodyCam: Thanks00:46
*** ijw has quit IRC00:49
*** zhenguo has joined #openstack-ironic00:53
openstackgerritTan Lin proposed openstack/ironic: Add AMT-PXE driver doc  https://review.openstack.org/15285900:58
*** romcheg1 has quit IRC01:00
*** lucas-dinner has quit IRC01:03
*** mikedillion has joined #openstack-ironic01:03
*** jmccrory has quit IRC01:04
devanandarloo: there was a lengty discussion just now in -infra re: microversions01:04
devanandarloo: I started drafting my reply to you, but wanted to check with them that my understanding of the gate was accurate01:05
*** mikedillion has quit IRC01:05
*** mikedillion has joined #openstack-ironic01:06
openstackgerritJosh Gachnang proposed openstack/ironic: Implement cleaning/zapping for the agent driver  https://review.openstack.org/16145301:09
*** BadCub has quit IRC01:16
* NobodyCam heads to the gym for an hour or so01:16
*** rwsu is now known as rwsu-afk01:17
*** mtanino has quit IRC01:19
openstackgerritAdam Gandelman proposed openstack/python-ironicclient: Adds basic ironicclient functional testing  https://review.openstack.org/16197601:22
rloodevananda: glad there are discussions going on wrt microversions.01:29
*** r-daneel has quit IRC01:31
*** stendulker has joined #openstack-ironic01:37
*** penick has quit IRC01:39
openstackgerritShivanand Tendulker proposed openstack/ironic: Common changes for secure boot support  https://review.openstack.org/15397401:40
*** wshao has quit IRC01:40
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for pxe_ilo driver  https://review.openstack.org/15480801:40
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for iscsi_ilo driver  https://review.openstack.org/15481401:40
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for agent_ilo driver  https://review.openstack.org/15481601:41
openstackgerritShivanand Tendulker proposed openstack/ironic: Ilo drivers sets capabilities:boot_mode in node  https://review.openstack.org/15573101:41
*** achanda has joined #openstack-ironic01:46
*** BB_ has quit IRC01:54
*** PaulCzar has quit IRC01:57
*** Haomeng|2 has quit IRC01:59
*** mikedillion has quit IRC02:02
*** ChuckC has quit IRC02:04
*** ChuckC has joined #openstack-ironic02:05
*** wshao has joined #openstack-ironic02:06
*** anderbubble has quit IRC02:12
*** anderbubble has joined #openstack-ironic02:13
*** oomichi has quit IRC02:14
*** anderbubble has quit IRC02:28
*** stendulker has quit IRC02:38
*** chenglch has joined #openstack-ironic02:50
*** jerryz_ has joined #openstack-ironic02:50
*** ChuckC_ has joined #openstack-ironic02:56
*** ChuckC has quit IRC02:59
*** wshao has quit IRC03:05
*** rloo has quit IRC03:18
*** ChuckC_ is now known as ChuckC03:27
*** stendulker has joined #openstack-ironic03:33
*** Haomeng has joined #openstack-ironic03:34
*** spandhe has quit IRC03:37
openstackgerritTan Lin proposed openstack/python-ironicclient: Enable ironicclient with --ironic-api-version 1.x  https://review.openstack.org/15562403:42
*** egon has joined #openstack-ironic03:43
*** david-lyle is now known as david-lyle_afk03:44
*** rameshg87 has joined #openstack-ironic03:46
*** Haomeng has quit IRC03:46
rameshg87good morning ironic03:46
*** oomichi has joined #openstack-ironic03:51
*** harlowja is now known as harlowja_away03:57
*** achanda has quit IRC03:57
*** Haomeng has joined #openstack-ironic04:06
rameshg87Haomeng: hi04:10
*** rameshg87 is now known as rameshg87-brb04:11
*** naohirot_ has joined #openstack-ironic04:12
*** sabah has joined #openstack-ironic04:12
*** alexpilotti has quit IRC04:22
*** Marga_ has quit IRC04:24
openstackgerritSyed Ismail Faizan Barmawer proposed openstack/ironic: Add support for partition images in agent driver  https://review.openstack.org/16200804:25
*** stendulker has quit IRC04:27
*** rameshg87-brb has left #openstack-ironic04:30
*** rameshg87-brb has joined #openstack-ironic04:30
*** achanda has joined #openstack-ironic04:38
*** achanda has quit IRC04:54
*** DCWilliams_VA has joined #openstack-ironic05:09
Haomengrameshg87-brb: thanks for your comments, got your point, we should reraise the exception from continue_deploy() method call, and I will check more unittest issue and commit new patch to cover your comments, welcome your comments:)05:11
*** DCWilliams_VA has quit IRC05:13
*** oomichi has quit IRC05:20
Haomengrameshg87-brb: :)05:30
*** Marga_ has joined #openstack-ironic05:39
*** Marga_ has quit IRC05:40
*** Marga_ has joined #openstack-ironic05:41
*** absubram has joined #openstack-ironic05:49
*** achanda has joined #openstack-ironic05:56
*** rameshg87-brb is now known as rameshg8705:57
rameshg87Haomeng: sure05:57
Haomengrameshg87: :)05:58
*** absubram has quit IRC05:59
*** wshao has joined #openstack-ironic06:06
*** coolsvap_ has quit IRC06:11
*** Marga_ has quit IRC06:21
*** yog__ has joined #openstack-ironic06:37
*** pradipta has joined #openstack-ironic06:37
*** wshao has left #openstack-ironic06:38
*** lazy_prince is now known as killer_prince06:58
*** killer_prince is now known as lazy_prince07:03
*** rameshg87 is now known as rameshg87-lunch07:03
openstackgerritJamie Lennox proposed openstack/ironic: Convert neutronclient to use session  https://review.openstack.org/16203707:03
openstackgerritShivanand Tendulker proposed openstack/ironic: Enable uefi boot mode for agent_ilo driver  https://review.openstack.org/16204307:35
*** stendulker has joined #openstack-ironic07:36
*** sabah has quit IRC07:40
*** achanda has quit IRC07:44
*** achanda has joined #openstack-ironic07:44
*** achanda has quit IRC07:48
*** jerryz_ has quit IRC07:58
*** jerryz_ has joined #openstack-ironic07:59
*** jcoufal has joined #openstack-ironic08:01
dtantsurMorning folks!08:02
dtantsurHaomeng, hi, mind reviewing https://review.openstack.org/#/c/156562 if you have some time?08:04
dtantsurI'll be on PTO next week, don't want this stuff to just hang around08:04
*** achanda has joined #openstack-ironic08:05
*** sandhya has joined #openstack-ironic08:07
Haomengdtantsur: good morning:) sure, let me take a look08:08
*** chlong has quit IRC08:11
*** erwan_taf has joined #openstack-ironic08:13
*** ndipanov has joined #openstack-ironic08:17
*** sabah has joined #openstack-ironic08:19
openstackgerritMerged openstack/ironic-specs: Change clean_steps to a dict  https://review.openstack.org/16051508:20
*** ifarkas has joined #openstack-ironic08:21
*** yog__ has quit IRC08:22
*** sandhya has quit IRC08:22
*** krtaylor has quit IRC08:22
*** soren has quit IRC08:25
Haomengdtantsur: quickly reviewed your patch, excluding some nits from Chris and John, others LGTM but I did not test this patch,and think we can merge it if no new patch is required08:32
Haomengdtantsur: +2d08:35
Haomengdtantsur: :)08:35
*** krtaylor has joined #openstack-ironic08:35
dtantsurHaomeng, thanks, I'll be fixing nits in a separate patch today08:40
Haomengdtantsur: thank you!08:41
dtantsurHaomeng, and I tested it myself, so if you feel like +A'ing it, go ahead :)08:41
Haomengdtantsur: yes, if you have tested it, that is fine:)08:41
Haomengdtantsur: :)08:42
dtantsur:)08:43
dtantsurifarkas, hey, quick ques: will we use discoverd inspection for DRAC drivers in Kilo?08:47
dtantsurok, will catch up later08:48
*** dtantsur is now known as dtantsur|bbl08:48
*** jgrimm is now known as zz_jgrimm08:49
*** achanda has quit IRC08:54
*** krtaylor has quit IRC08:54
*** pas-ha has joined #openstack-ironic08:55
*** romcheg has joined #openstack-ironic08:55
rameshg87-lunchHaomeng: dtantsur|bbl: when you get some time, can you please have a look at https://review.openstack.org/15523008:56
rameshg87-lunchHaomeng: dtantsur|bbl: waiting for eyes for some many days :(08:56
*** andreykurilin_ has joined #openstack-ironic08:56
*** achanda has joined #openstack-ironic08:57
ifarkasdtantsur|bbl, yes08:57
ifarkasdtantsur|bbl, and also for figuring out the root device during raid configuration08:58
*** sandhya has joined #openstack-ironic08:59
Haomengrameshg87-lunch: sure, will go home and take look your patch tonight:)09:01
rameshg87-lunchHaomeng: thanks09:01
*** rameshg87-lunch is now known as rameshg8709:01
*** jistr has joined #openstack-ironic09:02
*** romcheg has quit IRC09:07
*** krtaylor has joined #openstack-ironic09:10
*** achanda has quit IRC09:12
*** achanda has joined #openstack-ironic09:13
*** achanda has quit IRC09:14
*** achanda has joined #openstack-ironic09:14
KinnisonGood morning everyone09:18
*** achanda has quit IRC09:19
*** lucasagomes has joined #openstack-ironic09:23
*** mgoddard1 has joined #openstack-ironic09:29
*** romcheg has joined #openstack-ironic09:30
*** mgoddard has quit IRC09:31
stendulkerlucasgomes, dtantsur: can you please review this when you get some time https://review.openstack.org/#/c/15573109:32
*** sabah has quit IRC09:33
*** jistr is now known as jistr|biab09:36
openstackgerritNisha Agarwal proposed openstack/ironic: iLO implementation for hardware inspection  https://review.openstack.org/15159609:38
openstackgerritMerged openstack/ironic: Add module for in-band inspection using ironic-discoverd  https://review.openstack.org/15656209:43
*** chenglch has quit IRC09:46
*** naohirot_ has quit IRC09:53
*** lazy_prince has quit IRC10:00
*** killer_prince has joined #openstack-ironic10:00
*** killer_prince is now known as lazy_prince10:01
*** MattMan has quit IRC10:02
*** rameshg87 has left #openstack-ironic10:05
*** mgoddard1 has quit IRC10:13
*** MattMan has joined #openstack-ironic10:14
*** yog__ has joined #openstack-ironic10:15
*** andreykurilin_ has quit IRC10:18
*** andreykurilin__ has joined #openstack-ironic10:18
*** yuanying has quit IRC10:26
*** sandhya has quit IRC10:38
*** andreykurilin__ has quit IRC10:38
*** jistr|biab is now known as jistr10:39
*** jiangfei|3 has quit IRC10:39
*** jiangfei has joined #openstack-ironic10:40
*** mgoddard has joined #openstack-ironic10:40
*** Haomeng|2 has joined #openstack-ironic10:45
*** athomas has quit IRC10:46
*** Haomeng has quit IRC10:47
*** athomas has joined #openstack-ironic10:52
*** lazy_prince is now known as killer_prince10:56
*** igordcard has joined #openstack-ironic11:04
openstackgerritMerged openstack/ironic: Local boot support for IPA  https://review.openstack.org/15772311:08
*** stendulker has quit IRC11:09
*** stendulker has joined #openstack-ironic11:10
*** yog__ has quit IRC11:12
*** stendulker has quit IRC11:15
*** Haomeng has joined #openstack-ironic11:16
*** Haomeng|2 has quit IRC11:18
*** chlong has joined #openstack-ironic11:22
*** pas-ha has quit IRC11:23
*** jcoufal_ has joined #openstack-ironic11:28
*** jcoufal has quit IRC11:31
*** dtantsur|bbl is now known as dtantsur11:39
*** romcheg has quit IRC11:51
bauzasndipanov: minutes to discuss the bug you reported ?11:51
bauzasndipanov: ie. https://bugs.launchpad.net/nova/+bug/142777211:52
openstackLaunchpad bug 1427772 in OpenStack Compute (nova) "Instance that uses force-host still needs to run some filters" [Low,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza)11:52
bauzasoops wrong window11:52
bauzas*facepal*11:52
ndipanovbauzas, over at nova?11:52
dtantsurifarkas, ok, so I'll update https://review.openstack.org/#/c/161132/ with drac drivers too, wdyt?11:53
ifarkasdtantsur, good idea! thanks for doing that11:56
*** pradipta has quit IRC11:59
*** vdrok_afk is now known as vdrok12:04
vdrokafternoon ironic12:05
*** Marga_ has joined #openstack-ironic12:07
*** alexpilotti has joined #openstack-ironic12:13
*** sambetts has quit IRC12:17
*** sambetts has joined #openstack-ironic12:17
*** romcheg has joined #openstack-ironic12:17
openstackgerritDmitry Tantsur proposed openstack/ironic: Start using in-band inspection  https://review.openstack.org/16113212:22
dtantsurifarkas, ^^^12:22
dtantsurifarkas, ugh, now manual merge is required with RAID patch >_<12:24
dtantsurifarkas, and finally: could you review https://review.openstack.org/#/c/157448/ (and it's parent) please? If you want to test, it's in our downstream RPM.12:31
*** lucasagomes is now known as lucas-hungry12:37
ifarkasdtantsur, sure, will do12:37
*** DCWilliams_VA has joined #openstack-ironic13:04
*** humble_ has joined #openstack-ironic13:06
*** dprince has joined #openstack-ironic13:07
jrollmorning all :)13:14
*** romcheg has quit IRC13:21
*** Haomeng has quit IRC13:27
*** Haomeng has joined #openstack-ironic13:27
dtantsurjroll, morning!13:29
jrollhey dtantsur, happy friday13:29
dtantsuroh yeah :)13:29
*** romcheg has joined #openstack-ironic13:32
*** lucas-hungry is now known as lucasagomes13:35
lucasagomesjroll, morning13:35
jrollmorning lucasagomes :)13:38
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: CLI: give user a hint to power on the node when setting IPMI creds  https://review.openstack.org/16215413:43
*** htrmeira has joined #openstack-ironic13:44
htrmeiraHi guys. Is there an Ubuntu repository for a ironic version compatible with Juno?13:44
htrmeiraThe trusty-updates/juno/main seems to not have ironic packages...13:45
*** Marga_ has quit IRC13:53
openstackgerritMerged openstack/ironic: Add server-supported min and max API version to HTTPNotAcceptable(406)  https://review.openstack.org/16075813:56
*** DCWilliams_VA has quit IRC13:57
*** Marga_ has joined #openstack-ironic13:58
*** stendulker has joined #openstack-ironic13:59
openstackgerritShivanand Tendulker proposed openstack/ironic: Enable uefi boot mode for agent_ilo driver  https://review.openstack.org/16204314:01
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Enable overwrite_existing by default  https://review.openstack.org/16216014:01
*** kkoski has joined #openstack-ironic14:02
openstackgerritSyed Ismail Faizan Barmawer proposed openstack/ironic-lib: Add .gitreview file and some minor changes  https://review.openstack.org/16216214:03
*** zz_jgrimm is now known as jgrimm14:07
*** Marga_ has quit IRC14:19
*** stendulker has quit IRC14:25
*** romcheg has quit IRC14:25
openstackgerritMerged stackforge/ironic-discoverd: None is a valid default value for base_url and auth_token in the client  https://review.openstack.org/16172814:26
*** chlong has quit IRC14:34
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Add python-openstackclient plugin for ironic-discoverd  https://review.openstack.org/15744814:36
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: CLI: give user a hint to power on the node when setting IPMI creds  https://review.openstack.org/16215414:36
*** htrmeira has left #openstack-ironic14:40
*** mtanino has joined #openstack-ironic14:51
*** absubram has joined #openstack-ironic14:56
NobodyCamgood morning Ironicers14:57
*** bauzas is now known as bauwser14:58
NobodyCamand TGIF!!!!14:58
NobodyCam:)14:58
*** absubram has quit IRC14:58
jrolltgif NobodyCam :)14:58
dtantsurNobodyCam, TGIF!!14:58
dtantsurand morning14:58
*** pas-ha has joined #openstack-ironic14:58
NobodyCammorning jroll and dtantsur :)14:59
dtantsuroh for me it's TGIF and TGTPTO14:59
*** absubram has joined #openstack-ironic14:59
openstackgerritMerged stackforge/ironic-discoverd: Add python-openstackclient plugin for ironic-discoverd  https://review.openstack.org/15744814:59
dtantsurwhich stands for "Tomorrow starts PTO" :)14:59
openstackgerritMerged stackforge/ironic-discoverd: CLI: give user a hint to power on the node when setting IPMI creds  https://review.openstack.org/16215414:59
NobodyCamoh dtantsur how long is your pto?14:59
dtantsurfor the next week15:00
NobodyCamawesome :) enjoy :)15:00
NobodyCamdo you have any plans or just bum around the house for a week?15:00
*** persia has quit IRC15:01
NobodyCamoh coffee is ready brb15:01
dtantsurNobodyCam, going to Paris :)15:05
NobodyCamdtantsur: oh that will be fun :)15:06
NobodyCamjust don't go to any summits15:06
NobodyCam:-p15:06
dtantsuroh sure :)15:06
NobodyCamlol15:06
dtantsurIt's the 4th time I go to Paris this year btw15:06
dtantsurthey should give me a residence permit already :D15:06
NobodyCamoh wow :)15:06
victor_lowtherGood morning Ironic15:07
*** igordcard has quit IRC15:08
*** BadCub has joined #openstack-ironic15:11
dtantsurvictor_lowther, morning15:12
*** persia_ has joined #openstack-ironic15:13
*** persia_ has quit IRC15:13
*** persia_ has joined #openstack-ironic15:13
NobodyCamood morning victor_lowther :)15:14
NobodyCamgood even15:14
BadCubmorning Ironic15:14
NobodyCamgood morning BadCub15:14
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Enable overwrite_existing by default  https://review.openstack.org/16216015:17
*** PaulCzar has joined #openstack-ironic15:18
*** r-daneel has joined #openstack-ironic15:20
lucasagomesNobodyCam, BadCub good morning15:24
*** anderbubble has joined #openstack-ironic15:25
jlvillalGood morning: NobodyCam BadCub lucasagomes dtantsur victor_lowther and the rest of Ironic :)15:25
dtantsuro/15:25
jlvillalNobodyCam: I did another update to https://review.openstack.org/161803 yesterday.  Adding a finally block to delete the file.15:26
jlvillalI think it is done :)15:26
lucasagomesmorning :D15:26
*** stendulker has joined #openstack-ironic15:26
*** kkoski has quit IRC15:27
*** hj-hp has joined #openstack-ironic15:28
*** kkoski has joined #openstack-ironic15:30
*** kkoski has quit IRC15:31
*** kkoski has joined #openstack-ironic15:31
*** pas-ha has quit IRC15:35
*** absubram has quit IRC15:35
* victor_lowther would like some eyes on https://review.openstack.org/#/c/150488/15:36
NobodyCammorning lucasagomes and jlvillal15:39
NobodyCamahh a second try :)15:39
NobodyCamjlvillal: why the second try with out any catches?15:40
jlvillalNobodyCam: So it will do the finally block15:41
jlvillalNobodyCam: Delete the file, and then if there was an exception, it will get raised after the finally block15:41
openstackgerritShivanand Tendulker proposed openstack/ironic: Common changes for secure boot support  https://review.openstack.org/15397415:41
*** ChuckC_ has joined #openstack-ironic15:41
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for pxe_ilo driver  https://review.openstack.org/15480815:41
jlvillalNobodyCam: So it allows any exceptions which get raised to pass through, but still delete the temporary file.15:41
*** ChuckC has quit IRC15:42
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for iscsi_ilo driver  https://review.openstack.org/15481415:42
jlvillalNobodyCam: Hopefully that makes sense :)15:42
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for agent_ilo driver  https://review.openstack.org/15481615:42
openstackgerritShivanand Tendulker proposed openstack/ironic: Ilo drivers sets capabilities:boot_mode in node  https://review.openstack.org/15573115:43
*** ChuckC_ has quit IRC15:46
*** anderbubble has quit IRC15:48
*** pas-ha has joined #openstack-ironic15:50
lucasagomescheck-tempest-dsvm-ironic-pxe_ipa-nv: SUCCESS (non-voting)15:51
lucasagomesour PXE + IPA job :)15:51
NobodyCamlucasagomes: w00t15:51
NobodyCamnice15:51
lucasagomes(run it as "check experimental")15:51
lucasagomesI'm now thinking about make it run on every commit15:52
lucasagomesas non-voting ofc15:52
*** anderbubble has joined #openstack-ironic15:52
lucasagomesinstead of the experimental pipeline, should I?15:52
NobodyCamI'd +1 that15:52
lucasagomescool :) I will put a patch up soonish15:53
jroll+115:54
* dtantsur just got provision_state=available from client not using microversions Oo WTF?15:57
*** lazy_prif has joined #openstack-ironic15:58
*** lazy_prif is now known as lazy_prince15:58
*** killer_prince has quit IRC15:58
NobodyCamO.o15:59
*** humble_ has quit IRC16:03
dtantsurreinstalled RPM's and problem is gone. let's assume I've screwed my environment...16:06
lucasagomeshere we go https://review.openstack.org/#/c/162204/16:08
dtantsurI wonder why it's not called pxe_ssh_ipa :)16:09
* lucasagomes is bad with names16:10
*** ChuckC_ has joined #openstack-ironic16:12
*** romcheg has joined #openstack-ironic16:21
*** romcheg has left #openstack-ironic16:23
*** krtaylor has quit IRC16:23
*** penick has joined #openstack-ironic16:26
*** anderbubble has quit IRC16:29
dtantsurfolks, https://review.openstack.org/#/c/161132 finishes in-band inspection code-wise :) tested it with pxe_ipmitool driver here, ifarkas may want to test with DRAC driver (though it's pretty complex due to lack of client support).16:31
dtantsuranyone wants to have a look?16:32
*** ParsectiX has joined #openstack-ironic16:32
*** romcheg1 has joined #openstack-ironic16:33
*** chlong has joined #openstack-ironic16:33
NobodyCambrb16:35
*** krtaylor has joined #openstack-ironic16:35
*** penick has quit IRC16:36
*** romcheg1 has quit IRC16:38
*** romcheg has joined #openstack-ironic16:39
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/16181116:45
*** Marga_ has joined #openstack-ironic16:49
devanandag'morning, all16:49
NobodyCamgood morning devananda :)16:49
*** hemna has joined #openstack-ironic16:49
NobodyCamoh devananda got a sec to toss a +1 on https://review.openstack.org/#/c/16220416:51
*** mdbooth_ has joined #openstack-ironic16:52
dtantsurdevananda, morning16:53
*** stendulker has quit IRC16:54
jlvillalNobodyCam: I was thinking about updating my patch.  My question is that I was thinking about updating the commit message to either reference or close the bug: https://bugs.launchpad.net/bugs/1428722  What do you think?16:55
openstackLaunchpad bug 1428722 in Ironic "Ironic drivers that utilize temp files should verify they can actually create them" [Low,In progress] - Assigned to Chris Krelle (nobodycam)16:55
*** sambetts has quit IRC16:55
*** purp is now known as purp_away16:55
*** romcheg has quit IRC16:55
jlvillalNobodyCam: patch: https://review.openstack.org/16180316:56
*** mgoddard has quit IRC16:56
*** mdbooth has quit IRC16:56
*** mdbooth_ is now known as mdbooth16:56
*** mgoddard has joined #openstack-ironic16:56
*** sambetts has joined #openstack-ironic16:57
*** anderbubble_ has joined #openstack-ironic16:57
*** jerryz_ has quit IRC16:58
NobodyCamhummm, jlvillal but does that patch verify a driver can actually create a temp file? or clarify the error message when the driver can not create a temp file16:58
*** mgoddard1 has joined #openstack-ironic16:58
jlvillalNobodyCam: My belief is that the bug is not about failing to create a temporary file.16:59
jlvillalNobodyCam: My belief is that the call to ipmitool failed, but then got re-raised as the PasswordFileFailedCreate (or whatever it is called ;) )16:59
jlvillalNobodyCam: In reality the exception that was raised was ProcessExecution.17:00
jlvillalNobodyCam: That's what I think.  But maybe I'm wrong :)17:01
openstackgerritMerged stackforge/ironic-discoverd: Enable overwrite_existing by default  https://review.openstack.org/16216017:01
*** mgoddard has quit IRC17:01
*** mgoddard1 has quit IRC17:03
*** mgoddard has joined #openstack-ironic17:03
NobodyCamjlvillal: how about Partial-Bug or Related-Bug?17:04
*** ifarkas has quit IRC17:04
jlvillalNobodyCam: Works for me.  I see it is on its way to being merged.  I wouldn't mind it getting merged as is.  My only change would have been to the commit message and a comment.  What do you think I should do?17:05
NobodyCamoh well then I would leave as is17:05
devanandaNobodyCam: hmm. in looking at that, I found the test run where it passed -- https://review.openstack.org/#/c/15864417:05
NobodyCamlucasagomes: approved it17:05
jlvillalNobodyCam: Woo hoo!  I was hoping you would say that :)17:06
devanandaNobodyCam: so yea, I'll +1. However, it looks like we have two other experimental jobs failing?17:06
NobodyCam:(17:06
NobodyCamya lucasagomes just got that test going17:06
devanandaadam_g: I haven't been tracking check-grenade-dsvm-[partial-]ironic ... looks like they're failing?17:06
* lucasagomes reads17:07
lucasagomesoh which patch is that the one that add a finally block?17:07
devanandalucasagomes: i'm just checking through the logs on http://logs.openstack.org/44/158644/8/experimental/check-tempest-dsvm-ironic-pxe_ipa-nv/bd20459/logs/ before +1'ing the patch to move it to check queue17:07
lucasagomesI don't see the problem with that17:07
devanandanot ht epatch17:07
devanandathe test runs, since you ticked check-experimental17:07
* lucasagomes is confused17:08
NobodyCamdevananda: refferencing the tests17:08
lucasagomeswait lemme read the backlogs17:08
NobodyCamyou've been working on17:08
lucasagomesdevananda, right yeah the other experimental jobs are failing17:08
lucasagomesbut that's not related to the patch17:08
devanandaright17:08
lucasagomesthat's why I also want to add it to the check gate17:09
devanandatotally17:09
lucasagomesbecause we don't often run check experimental17:09
devanandai'm jus curious why the're failing - because, yea, I didn't know17:09
lucasagomesI just added to experimental at first to be safe17:09
lucasagomeswas my first patch adding a job on gate so17:09
lucasagomesyeah lemme take a look17:09
lucasagomesat the other logs17:09
lucasagomesother jobs logs*17:10
devanandalucasagomes: yup. I wasn't asking about your patch -- just that, in looking at your patch, I noticeed the other failures17:10
*** jmccrory has joined #openstack-ironic17:10
lucasagomesdevananda, gotcha17:10
lucasagomesyeah sorry my brain is almost dead as well at this time of the day :)17:10
NobodyCam:)17:10
devanandalucasagomes: woot - testing with a config drive. nice. http://logs.openstack.org/44/158644/8/experimental/check-tempest-dsvm-ironic-pxe_ipa-nv/bd20459/logs/screen-ir-cond.txt.gz#_2015-03-06_15_33_41_90517:11
devanandaalso, that really shouldn't log at INFO level17:11
lucasagomesdevananda, yup :) we enable it by default17:11
* devananda files a bug17:11
lucasagomesoh indeed17:11
lucasagomesthat should be DEBUG17:12
lucasagomesI mean nova enables it by default*17:12
devanandaI'm going to mark this bug high and target17:13
devanandabecause, if someone uses config drive, this could break the conductor host17:13
devanandaby logging 64MB config drives17:13
devanandaand overwhelming syslog .... :(17:14
NobodyCam:(17:14
devanandahttps://bugs.launchpad.net/ironic/+bug/142920817:14
openstackLaunchpad bug 1429208 in Ironic "continuing deploy log message contains config_drive contents" [High,Confirmed]17:14
lucasagomesfair enuff17:14
*** kkoski has quit IRC17:14
lucasagomeswill fix it in a sec17:14
devanandacool, ty17:15
lucasagomesthe granade experimetal checks are failing on tempest.api.compute.admin.test_baremetal_nodes.BaremetalNodesAdminTestJSON.test_list_baremetal_nodes17:15
* lucasagomes looks what it does17:15
devanandalucasagomes: problem with your job17:17
devanandado not put it in check queue yet17:17
* lucasagomes mark the patch as WIP17:17
devanandalucasagomes: downloads, especially of large files, should be done in d-g prepare phase and pre-cached, not during devstack's run17:18
devanandalucasagomes: http://logs.openstack.org/44/158644/8/experimental/check-tempest-dsvm-ironic-pxe_ipa-nv/bd20459/logs/devstacklog.txt.gz#_2015-03-06_15_19_52_26717:18
devanandalucasagomes: any downloads at this stage are statistically likely to fail17:18
lucasagomesdevananda, right, this is what we have for agent_ssh as well17:18
lucasagomesbecause we don't build the image17:18
lucasagomeswe could17:18
devanandasure. but it should be pre-cached17:18
devanandaboth of them17:18
devanandalucasagomes: hm. let me restate -- you could put it in check queue, but still as -nv17:19
devanandabut we're probably going to get failures from the download17:19
lucasagomessee https://review.openstack.org/#/c/161790/4/jenkins/jobs/devstack-gate.yaml L 90217:19
devanandathat will skew our tracking of its success rate17:19
lucasagomesoh def, it will be -nv17:20
lucasagomeshow can I pre-cache it?17:20
* lucasagomes search for some examples17:20
lucasagomesdevananda, I will only move it out of -nv after a month or so from now17:20
lucasagomesif it continues to pass17:20
devanandak k17:20
NobodyCambbt brb17:21
*** pas-ha has quit IRC17:21
lucasagomeswill ask on -infra if it's possible to cache it17:21
devanandaanteaya: you might know -- how would we have devstack-gate pre-cache a fairly large file for ironic jobs, rather than download during the devstack run?17:24
*** mgoddard has quit IRC17:25
devanandamy hunch is it'd go somewhere around here: https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate.sh#L18717:25
anteayaI'm looking at lucasagomes' question in infra17:25
anteayaI do not know17:25
anteayabut it is a good question17:25
devanandak k, thanks17:25
lucasagomesanteaya, thanks, I'm searching for examples as well17:25
lucasagomesI will fix it for both pxe_ipa and agent_ssh17:26
anteayalucasagomes: kk17:26
*** mgoddard has joined #openstack-ironic17:26
*** kkoski has joined #openstack-ironic17:27
*** Marga_ has quit IRC17:29
dtantsursee you in 1 week (PTO), g'night!17:30
*** dtantsur is now known as dtantsur|afk17:30
lucasagomesdtantsur|afk, see u, enjoy PTO17:30
*** dtantsur|afk is now known as dtantsur|pto17:30
dtantsur|pto<-- a bit better :)17:30
NobodyCamhave a great week dtantsur|pto :)17:30
*** igordcard has joined #openstack-ironic17:30
*** jcoufal_ has quit IRC17:33
*** rwsu-afk is now known as rwsu17:33
*** arif-ali has quit IRC17:34
*** jistr has quit IRC17:37
*** kkoski has quit IRC17:38
*** kkoski has joined #openstack-ironic17:39
*** david-lyle_afk is now known as david-lyle17:43
*** anderbubble_ has quit IRC17:44
*** kkoski has quit IRC17:45
*** kkoski has joined #openstack-ironic17:45
*** anderbubble has joined #openstack-ironic17:46
*** kkoski has quit IRC17:48
*** igordcard has quit IRC17:49
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Make sure we don't log the full content of the config drive  https://review.openstack.org/16225217:50
lucasagomesforgot the bug tag17:50
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Make sure we don't log the full content of the config drive  https://review.openstack.org/16225217:50
*** bnemec is now known as beekneemech17:52
*** purp_away is now known as purp17:53
*** penick has joined #openstack-ironic17:53
*** jcoufal has joined #openstack-ironic17:54
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Make sure we don't log the full content of the config drive  https://review.openstack.org/16225218:00
lucasagomesfolks I will call it a day18:03
lucasagomeshave a good night everyone, enjoy the weekend18:04
*** lucasagomes is now known as lucas-beer18:04
lucas-beerdevananda, I will take a look at the cache thing on monday18:05
* lucas-beer wrote it down18:05
NobodyCamhave a great weekend lucas-beer18:06
*** igordcard has joined #openstack-ironic18:06
devanandag'night lucas-beer o/18:08
*** spandhe has joined #openstack-ironic18:08
*** penick_ has joined #openstack-ironic18:10
*** penick has quit IRC18:10
*** penick_ is now known as penick18:10
adam_gdevananda, yea, our parallel tests are broken since the IPA-by-default change. waiting on a devstack change for that18:12
adam_g(the grenade jobs run tempest parallel by default)18:12
lucas-beeradam_g, oh that's the 512MB ram18:13
lucas-beerI've +1 that patch, maybe asking someone on -qa to take a look at it18:14
adam_glucas-beer, ya18:14
jlvillallucas-beer: Enjoy the weekend! :)18:14
lucas-beerthanks :)18:15
* lucas-beer runs now18:15
*** ndipanov has quit IRC18:17
*** igordcard has quit IRC18:18
*** purp is now known as purp_away18:18
*** athomas has quit IRC18:19
*** achanda has joined #openstack-ironic18:23
*** Marga_ has joined #openstack-ironic18:23
*** coolsvap has joined #openstack-ironic18:29
openstackgerritAdam Gandelman proposed openstack/python-ironicclient: Adds basic ironicclient functional testing  https://review.openstack.org/16197618:31
*** hj-hp has quit IRC18:31
*** thiagop has joined #openstack-ironic18:33
*** erwan_taf has quit IRC18:38
*** mgoddard has quit IRC18:41
*** athomas has joined #openstack-ironic18:44
*** kkoski has joined #openstack-ironic18:44
*** kkoski has quit IRC18:46
*** kkoski1 has joined #openstack-ironic18:46
*** kkoski1 has quit IRC18:47
*** arif-ali has joined #openstack-ironic18:50
*** achanda has quit IRC18:52
*** absubram has joined #openstack-ironic18:52
*** gridinv has joined #openstack-ironic18:55
*** achanda has joined #openstack-ironic18:57
*** arif-ali has quit IRC18:59
*** arif-ali has joined #openstack-ironic19:00
*** igordcard has joined #openstack-ironic19:00
*** kkoski has joined #openstack-ironic19:00
*** kkoski has quit IRC19:01
*** kkoski has joined #openstack-ironic19:01
*** mgoddard has joined #openstack-ironic19:10
*** igordcard has quit IRC19:16
devanandaanyone else want to review lintan's latest rev of client support for microversions before I land it?19:17
NobodyCamdevananda: I have on a open tab19:22
devanandak19:22
NobodyCamlooking now19:23
devanandai've tested, works for me19:23
devanandaalready has a +2 from dtantsur|pto19:23
*** thiagop has quit IRC19:23
*** coolsvap has quit IRC19:25
NobodyCamyep19:27
NobodyCamand now I am about to +2 any one else or go ahead with +a?19:28
devanandago for it19:28
devanandai'll rebase my logical names client patch in a minute19:28
NobodyCamawesome. :)19:28
NobodyCamdevananda: I'm going to put up a follow up to address the nits on that one now19:29
devanandaNobodyCam: k19:31
devanandayea, so, our client sucks for versioning right now19:32
devanandaI just spun up stable/juno and pointed the current client + lintan's patch at it19:32
devanandait's doing exactly what I expect19:33
devanandamost things work, but then there are oddities19:33
*** gridinv has quit IRC19:34
*** gridinv has joined #openstack-ironic19:34
openstackgerritNisha Agarwal proposed openstack/ironic: ironic port deletion fails even if node is locked by same process  https://review.openstack.org/16186119:36
*** MattMan has quit IRC19:37
openstackgerritChris Krelle proposed openstack/python-ironicclient: Fix typeo on patch 155624  https://review.openstack.org/16228019:38
NobodyCamsuper easy one char change review !!! ^^^^^^19:39
JayFs/typeo/typo/19:39
NobodyCamgah19:40
NobodyCamdoh19:40
adam_gLOL19:40
openstackgerritChris Krelle proposed openstack/python-ironicclient: Fix typo on patch 155624  https://review.openstack.org/16228019:40
NobodyCamand mornig JayF and adam_g :)19:41
adam_gfor python-ironicclient functional testing, curious what people think of just modifying the existing unit tests so that they can be run against a fake api server with mocked responses for unit tests, or a live server for functional? rather than write a second test suite19:41
*** rlr0329 has quit IRC19:41
devanandasome areas where we can improve our client usability -- http://paste.openstack.org/show/u91DtCf0fwRyv0auQWpx/19:42
devanandathis sort of stuff makes users really really sad ....19:43
devanandaadam_g: live server19:43
devanandayou can fake everything else. no need for devstack19:43
devanandasee the paste above -- all that was run in a venv19:43
devanandaironic stable/juno in one venv, python-iornicclient in another. I've also got rabbitmq and mysql running. but that's all that is needed19:44
adam_gdevananda, i think it'd be easier from an infra POV to just use devstack to spin all that up19:44
adam_g(with keystone)19:44
devanandaadam_g: fair. just don't start any other os_ services19:45
adam_gdevananda, yeah, thats the client job here https://review.openstack.org/#/c/161483/5/jenkins/jobs/ironic.yaml#3819:45
devanandacool19:47
*** spandhe_ has joined #openstack-ironic19:51
*** ijw has joined #openstack-ironic19:51
*** spandhe has quit IRC19:53
*** spandhe_ is now known as spandhe19:53
*** Marga_ has quit IRC19:59
*** gridinv has quit IRC20:04
*** hj-hp has joined #openstack-ironic20:30
*** achanda has quit IRC20:37
*** hj-hp has quit IRC20:48
*** hj-hp has joined #openstack-ironic20:49
*** dprince has quit IRC20:59
*** achanda has joined #openstack-ironic21:00
*** ijw has quit IRC21:05
*** ijw has joined #openstack-ironic21:08
*** mgoddard has quit IRC21:12
*** lucas-beer has quit IRC21:16
*** absubram has quit IRC21:17
openstackgerritJohn L. Villalovos proposed openstack/ironic: Make sure we don't log the full content of the config drive  https://review.openstack.org/16225221:19
*** absubram has joined #openstack-ironic21:23
jlvillalUhh.  Does uploading a patch revision magically make me the owner of a bug?  I had uploaded a revision of https://review.openstack.org/162252 and I just got an email that the bug associated with it was assigned to me.21:32
*** persia_ is now known as persia21:32
JayFYeah, that happens21:32
JayFnow if you don't fix it, you're in big trouble mister ;)21:33
*** harlowja_away has quit IRC21:34
jlvillalJayF: Okay.  Thanks.  I don't have the power to assign it back to lucas.  I will see what happens :)21:34
* jlvillal thinks the merge gate is very slow today...21:35
JayFs/today//21:36
jlvillal:D21:36
jlvillalJayF: Well my job has been going for 4 hours and 36+ minutes....21:37
JayFyou know how to lookup and follow status for such things yet?21:37
jlvillalJayF: I'm just going to the Zuul page.  http://status.openstack.org/zuul/  Is there something else I should look at?21:38
JayFis this for a change to *merge* (i.e. gate) or for it to get a +1 from jenkins (check)21:38
jlvillalJayF: It is a *merge*21:38
JayFwhat's the patch you're talking about?21:38
JayFhttps://review.openstack.org/#/c/161803/ ?21:38
jlvillalJayF: Almost done... https://review.openstack.org/16180321:38
jlvillalYep21:38
JayFso yeah, that works on a cascading dependency system21:38
jlvillalJayF: I'm not blocked by it, but I was watching it...21:39
JayFso if right now, neutron-fwaas change that's just ahead of us failed21:39
*** jcoufal has quit IRC21:39
JayFwe'd have to rerun all the checks again, except this time with your change at the head21:39
JayFif that makes sense?21:39
jlvillalJayF: That makes sense.21:39
JayFmy hunch is you got in a small handful of failing attempts21:39
jlvillalJayF: My former job was working on a buildbot based CI system.21:39
JayFoh nice21:39
JayFI bet you're glad to be escaped from that :)21:40
jlvillalJayF: It wasn't too bad.  Python code :)  But we also did the thing where the patch would be built and tested and then merged by the CI system.21:40
jlvillalJayF: Though Zuul is a bit fancier than the one we had.21:40
JayFheh21:41
JayFjust thinking about how the only difference between "complex" and "fancy" is how you feel about the fancy complexity21:41
openstackgerritMerged openstack/ironic: Make try block shorter for _make_password_file  https://review.openstack.org/16180321:42
jlvillalJayF: It seems like a good idea to do the testing and merging against tip.  And smart to therefore do it against the 'predicted tip' as it allows more parallelization (sp?)21:42
jlvillalJayF: Woot!  It merged :)21:43
*** bigjools has quit IRC21:46
*** igordcard has joined #openstack-ironic21:50
*** bigjools has joined #openstack-ironic21:51
*** bigjools has joined #openstack-ironic21:51
*** penick has quit IRC21:52
*** kkoski has quit IRC22:00
*** romcheg has joined #openstack-ironic22:01
*** hj-hp has quit IRC22:04
*** penick has joined #openstack-ironic22:04
*** jgrimm is now known as zz_jgrimm22:06
*** ijw has quit IRC22:08
*** ijw has joined #openstack-ironic22:08
*** Marga_ has joined #openstack-ironic22:14
*** r-daneel has quit IRC22:23
*** chlong has quit IRC22:24
*** andreykurilin_ has joined #openstack-ironic22:24
*** ijw has quit IRC22:28
openstackgerritMerged openstack/python-ironicclient: Enable ironicclient with --ironic-api-version 1.x  https://review.openstack.org/15562422:33
*** igordcard has quit IRC22:42
*** anderbubble has quit IRC22:45
*** igordcard has joined #openstack-ironic22:58
*** ParsectiX has quit IRC23:00
*** athomas has quit IRC23:00
*** ParsectiX has joined #openstack-ironic23:00
*** r-daneel has joined #openstack-ironic23:02
*** achanda_ has joined #openstack-ironic23:03
*** achanda has quit IRC23:03
*** igordcard has quit IRC23:03
*** anderbubble has joined #openstack-ironic23:03
*** andreykurilin_ has quit IRC23:04
*** r-daneel_ has joined #openstack-ironic23:06
*** r-daneel has quit IRC23:07
*** achanda_ has quit IRC23:07
*** achanda has joined #openstack-ironic23:10
*** r-daneel_ has quit IRC23:10
*** r-daneel_ has joined #openstack-ironic23:10
*** hj-hp has joined #openstack-ironic23:10
*** ijw has joined #openstack-ironic23:11
*** igordcard has joined #openstack-ironic23:20
*** ijw_ has joined #openstack-ironic23:21
*** ijw_ has quit IRC23:22
*** ijw_ has joined #openstack-ironic23:23
*** ijw has quit IRC23:23
devanandaNobodyCam: you should read the comments on https://review.openstack.org/#/c/15826923:25
*** ChuckC_ has quit IRC23:25
devanandaregarding the relationship between n-cpu and ironic, and running multiple n-cpu processes23:25
*** anderbubble has quit IRC23:29
*** hj-hp has quit IRC23:33
*** ijw_ has quit IRC23:39
*** ChuckC_ has joined #openstack-ironic23:43
*** ChuckC_ is now known as ChuckC23:44
*** romcheg has left #openstack-ironic23:55
openstackgerritTan Lin proposed openstack/ironic: Add AMT-PXE-Driver Power&Management&Vendor Interface  https://review.openstack.org/15285823:56
openstackgerritTan Lin proposed openstack/ironic: Add AMT-PXE driver doc  https://review.openstack.org/15285923:57

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