Thursday, 2015-03-26

NobodyCamwow amd lost over 6 bucks a share today00:00
NobodyCamoh sorry 6 % today00:01
NobodyCamdoh00:01
jrollcan I get an FFE if I upstream everything today?00:01
jrolllike, run git review on my downstream branch00:01
jrolljust blind +A00:01
jrollthat would make my life much easier00:01
*** ijw has joined #openstack-ironic00:01
jrollthanks in advance BadCub00:01
* BadCub has no +A power lol00:02
jroll:P00:02
BadCubhave to prove myself worthy of such responisibility :)00:02
NobodyCamintel is very near their 52 week low00:03
jrollorly00:03
BadCuband learn how to spell re-spon-sib-ility00:03
* NobodyCam dbl checks BadCub's suggestion for spelling00:03
NobodyCamlol00:03
jrollaw man, $29 and change00:04
jrollI have $28 cash in my etrade account right now00:04
jroll:P00:04
jrollSO CLOSE00:04
NobodyCamlol /me as 20000:04
NobodyCamlol00:04
*** naohirot has joined #openstack-ironic00:04
* BadCub uses sharebuilder00:04
NobodyCam++00:04
jrollI should add some and buy intel but meh00:04
NobodyCamjroll: with sharebuilder you can buy % of a hare00:05
NobodyCamshare*00:05
BadCubI haven;t hear any chatter about microbloat buying intel00:05
jrollhuh00:05
jrollNobodyCam: interesting00:05
jrollBadCub: I heard it from NobodyCam so it must be true00:05
*** ijw has quit IRC00:06
* BadCub remembers trusting NobodyCam when he said buy Enron, Qwest, and WorldCom lol00:06
NobodyCam:-p00:06
NobodyCamdoh00:06
jrolllmao00:06
jrollBadCub: actual question, when do specs open for L00:07
BadCubjroll: I spoke with Deva about that. He does not want to open specs until rc1 is done00:07
jrollok00:08
jrolljust gotta know when to push the spam button >:)00:08
BadCublol00:08
*** krtaylor has joined #openstack-ironic00:08
*** korekhov has joined #openstack-ironic00:09
jrollmostly serious00:09
*** davide has joined #openstack-ironic00:11
NobodyCamjroll: you do know BadCub has worked for the legal compliance dept of a affilate marketer (in the past)00:12
* BadCub has very painful memories of SPAM litigation00:12
TheJuliaworking anywhere near litigiation brings painful memories00:13
BadCubTheJulia: True that!00:13
NobodyCamlol BadCub remember http://www.danhatesspam.com00:13
* TheJulia twitches thinking of 8 years of her life00:13
*** korekhov has quit IRC00:13
* BadCub cringes, cries, and twitches at NobodyCam 's post00:13
* NobodyCam duck (litterally)00:14
* BadCub can still write C&Ds for Affiliate Marketing in his sleep00:14
jrollNobodyCam: lol00:15
jrollthis is gerrit spam, not email spam00:15
jrollso it's ok00:15
*** davide has quit IRC00:15
NobodyCamlol00:15
BadCubspam is spam, at least according to the Federal Can-Spam Act and the California Internet Communications Act lol00:16
*** Marga_ has quit IRC00:16
TheJuliaBadCub: In my case, it was RCAs and downtime notices that would be handed to lawyers possibly judges00:16
NobodyCamhttps://www.youtube.com/watch?v=anwy2MPT5RE00:16
BadCubTheJulia: Mine was all revolving around intelectual property, witness prep and writing legal compliance policies *groans* Then there were the hundreds of C&Ds we had to reply to and/or write weekly and out-of-court settlement contracts00:18
* TheJulia twitches00:19
BadCub"opt-in" marketing is a never-ending cycle of eating your young every morning00:19
*** mtanino has quit IRC00:19
*** Marga_ has joined #openstack-ironic00:19
BadCubwell folks.. It is time to go build_a_bed then uncover more of the mysterious brick walkway we unearthed in the latest excavation experiment00:21
NobodyCamhumm I falied pep8 on a rst file00:22
NobodyCamhuh00:22
BadCubroflmao00:22
NobodyCam:-p00:22
NobodyCamhttps://jenkins06.openstack.org/job/gate-ironic-python-agent-pep8/133/console00:22
* BadCub beats Jenkins with a stick and runs00:22
NobodyCamjroll: JayF: JoshNang: have you seen that b4 ^^^^^^00:23
* NobodyCam will dig into it in the morning :-p00:25
* NobodyCam heads off to build a bed and dig in the dirt00:26
NobodyCamg'night ya'll00:26
jrollNobodyCam: I have seen this00:26
jrollwaaaaaaait00:26
jrollwhat patch is that?00:26
NobodyCamoh ok00:26
jrollI think it's when your commit message ends with a period or something00:26
NobodyCamhttps://review.openstack.org/16784100:26
NobodyCamoh00:26
openstackgerritJim Rollenhagen proposed openstack/ironic-python-agent: Update README file  https://review.openstack.org/16784100:26
jrollboom00:26
jroll:D00:26
jrollworst bug ever00:27
NobodyCamlol00:27
NobodyCamnice00:27
NobodyCamty00:27
jrollnp00:27
BadCubhave a good evening folks!00:29
*** BadCub is now known as BadCub_Hiding00:29
TheJuliagoodnight BadCub_Hiding00:29
*** ijw has joined #openstack-ironic00:29
*** davide has joined #openstack-ironic00:35
*** ijw has quit IRC00:39
*** davide has quit IRC00:41
*** Marga_ has quit IRC00:41
*** Marga_ has joined #openstack-ironic00:45
*** penick has quit IRC00:47
*** oomichi_ has joined #openstack-ironic00:52
*** anderbubble has joined #openstack-ironic00:52
*** kkoski has joined #openstack-ironic00:55
*** kkoski has quit IRC00:57
*** achanda has quit IRC00:59
*** korekhov has joined #openstack-ironic01:11
*** korekhov_ has joined #openstack-ironic01:12
*** achanda has joined #openstack-ironic01:13
*** jcoufal has joined #openstack-ironic01:14
*** korekhov has quit IRC01:15
*** saripurigopi has joined #openstack-ironic01:20
*** Marga_ has quit IRC01:26
*** saripurigopi has quit IRC01:34
*** saripurigopi has joined #openstack-ironic01:37
*** kkoski has joined #openstack-ironic01:39
*** jcoufal has quit IRC01:39
*** jerryz has joined #openstack-ironic01:43
*** kkoski has quit IRC01:43
*** igordcard_ has quit IRC01:45
*** chenglch has joined #openstack-ironic01:55
*** achanda has quit IRC02:09
*** saripurigopi has quit IRC02:16
*** kan__ has joined #openstack-ironic02:19
*** davide has joined #openstack-ironic02:30
*** davide has quit IRC02:34
*** jmanko has joined #openstack-ironic02:36
*** korekhov has joined #openstack-ironic02:38
*** ramineni has joined #openstack-ironic02:40
*** jmank has quit IRC02:40
*** korekhov_ has quit IRC02:43
*** enikanorov__ has joined #openstack-ironic02:43
*** korekhov has quit IRC02:44
*** enikanorov has quit IRC02:45
*** harlowja is now known as harlowja_away02:53
*** achanda has joined #openstack-ironic03:10
*** anderbubble has quit IRC03:16
*** achanda has quit IRC03:17
*** Marga_ has joined #openstack-ironic03:33
*** Marga_ has quit IRC03:33
*** Marga_ has joined #openstack-ironic03:34
*** achanda has joined #openstack-ironic03:39
*** kan__ has quit IRC03:41
*** Sukhdev has joined #openstack-ironic03:47
*** kalpase has joined #openstack-ironic03:48
*** kan__ has joined #openstack-ironic03:49
*** yuanying has joined #openstack-ironic03:50
*** saripurigopi has joined #openstack-ironic03:59
*** jerryz has quit IRC04:04
*** jerryz has joined #openstack-ironic04:05
openstackgerritShivanand Tendulker proposed openstack/ironic: Deploys post whole disk image deploy fails  https://review.openstack.org/16763604:05
*** davide has joined #openstack-ironic04:07
*** achanda has quit IRC04:30
*** achanda has joined #openstack-ironic04:34
*** Nisha has joined #openstack-ironic04:47
*** saripurigopi has quit IRC04:47
*** Nisha has quit IRC04:49
*** Nisha has joined #openstack-ironic04:50
openstackgerritShivanand Tendulker proposed openstack/ironic: pxe deploy fails for whole disk images in UEFI  https://review.openstack.org/16788204:52
*** kalpase1 has joined #openstack-ironic04:54
*** kalpase has quit IRC04:58
*** pradipta has joined #openstack-ironic05:04
*** davide has quit IRC05:04
*** saripurigopi has joined #openstack-ironic05:09
*** korekhov has joined #openstack-ironic05:10
*** korekhov has quit IRC05:14
*** korekhov has joined #openstack-ironic05:26
openstackgerritMerged stackforge/proliantutils: Refactor code in RIS  https://review.openstack.org/16711705:32
*** r-daneel has joined #openstack-ironic05:36
*** r-daneel has quit IRC05:41
openstackgerritShivanand Tendulker proposed openstack/ironic: pxe deploy fails for whole disk images in UEFI  https://review.openstack.org/16788205:43
*** jerryz has quit IRC05:43
*** korekhov has quit IRC05:43
*** Sukhdev has quit IRC05:47
*** jamielennox is now known as jamielennox|away06:16
*** achanda has quit IRC06:31
openstackgerritShivanand Tendulker proposed openstack/ironic: grub2 bootloader support for uefi boot mode  https://review.openstack.org/16619206:35
*** Nisha has quit IRC06:38
*** achanda has joined #openstack-ironic06:39
*** takadayuiko has joined #openstack-ironic06:39
*** chlong has quit IRC06:45
*** absubram has quit IRC06:47
*** davide has joined #openstack-ironic06:52
*** davide has quit IRC06:57
*** alexpilotti has joined #openstack-ironic06:57
openstackgerritAnusha Ramineni proposed stackforge/proliantutils: Fallback to RIBCL  https://review.openstack.org/16715106:58
*** achanda has quit IRC07:13
*** achanda has joined #openstack-ironic07:15
*** saripurigopi has quit IRC07:22
*** achanda has quit IRC07:24
*** ukalifon has joined #openstack-ironic07:34
*** ifarkas has joined #openstack-ironic07:40
openstackgerritShivanand Tendulker proposed openstack/ironic: pxe deploy fails for whole disk images in UEFI  https://review.openstack.org/16788207:46
openstackgerritShivanand Tendulker proposed openstack/ironic: pxe deploy fails for whole disk images in UEFI  https://review.openstack.org/16788207:49
*** chapman_r has joined #openstack-ironic07:53
*** ekarlso has quit IRC07:59
*** chapman_r has quit IRC07:59
*** dlpartain has joined #openstack-ironic07:59
*** chapman_r has joined #openstack-ironic08:00
*** dlpartain has left #openstack-ironic08:04
*** saripurigopi has joined #openstack-ironic08:06
*** yog__ has joined #openstack-ironic08:11
*** saripurigopi has quit IRC08:20
*** jistr has joined #openstack-ironic08:27
*** kalpase has joined #openstack-ironic08:37
*** kalpase1 has quit IRC08:41
*** andreykurilin_ has joined #openstack-ironic08:46
*** dlpartain has joined #openstack-ironic08:46
*** oomichi_ has quit IRC08:50
*** bradjones has quit IRC08:51
*** romcheg has joined #openstack-ironic08:52
*** bradjones has joined #openstack-ironic08:55
*** bradjones has quit IRC08:55
*** bradjones has joined #openstack-ironic08:55
*** takadayu_ has joined #openstack-ironic09:00
*** yuanying has quit IRC09:06
*** ekarlso has joined #openstack-ironic09:09
*** andreykurilin_ has quit IRC09:11
*** Marga_ has quit IRC09:12
*** alexpilotti has quit IRC09:16
*** lucasagomes has joined #openstack-ironic09:20
lintan_hi lucasagomes,09:21
lucasagomeslintan_, hello there09:22
*** derekh has joined #openstack-ironic09:22
lintan_hi lucasagomes, I want to try the localboot,  Failed to do get a partition image with grub2 by using diskimage-builder09:24
lucasagomeslintan_, to install grub2?09:24
lintan_lucasagomes, yes09:25
* lucasagomes thought they had a grub2 element upstream09:25
lucasagomeslintan_, right... hmm 1 sec I think we have one element somewhere. perhaps we should put it upstream on tripleo09:25
lintan_lucasagomes, they have a localboot element?09:25
*** tiagogomes_ has joined #openstack-ironic09:25
openstackgerritRamakrishnan G proposed openstack/ironic: Add boot interface in Ironic  https://review.openstack.org/16651209:26
*** enikanorov__ has quit IRC09:26
lucasagomeslintan_, the deploy-ironic element already contains the bits to install grub2 on the image (but that depends on the the guest image)09:26
*** dtantsur|afk is now known as dtantsur09:26
dtantsur(Late) morning, folks!09:27
lucasagomeslintan_, https://github.com/rdo-management/instack-undercloud/tree/adade940a517f6c6fe4d67f056b118ae3759e89c/elements/grub209:27
* dtantsur is seriously under weather today09:27
lintan_lucasagomes, thanks, I will have a try on that09:27
lucasagomesdtantsur, oh hope u get better09:27
*** saripurigopi has joined #openstack-ironic09:28
lucasagomeslintan_, cool I will propose it upstream09:28
lucasagomeslintan_, I thought they had something on dib09:28
dtantsurlucasagomes, thanks, now I'm trying to solve it with coffee :)09:28
lucasagomesdtantsur, heh grab some coffee, find a sofa, work from there09:28
dtantsurlucasagomes, I'm working from my bed today :D09:29
lucasagomesdtantsur, that works as well!09:29
*** ndipanov has joined #openstack-ironic09:31
lintan_lucasagomes, they have a localboot'09:32
lintan_lucasagomes, they have a 'localboot' element but it doesn't work for me :(09:32
lucasagomeslintan_, what u mean?09:32
lucasagomeslintan_, that's the deploy-ironic element09:32
lucasagomesit already contains the code to do the local boot09:33
lucasagomesyou don't need any extra element apart from deploy-ironic and grub209:33
openstackgerritAnusha Ramineni proposed stackforge/proliantutils: Fallback to RIBCL  https://review.openstack.org/16715109:33
*** lazy_prince has joined #openstack-ironic09:34
lintan_lucasagomes, yes, so I only need a guest image with grub2, that's my problem...09:34
*** jerryz has joined #openstack-ironic09:34
lucasagomeslintan_, right, so you can use that grub2 element I gave you when generating the image09:35
lucasagomeslintan_, https://review.openstack.org/167929 (just proposed upstream on tripleo)09:35
lintan_lucasagomes, thanks09:36
*** kalpase1 has joined #openstack-ironic09:36
lucasagomeslintan_, no problem, lemme know if it works09:36
*** chapman_r has quit IRC09:37
*** kalpase has quit IRC09:39
dtantsurifarkas, morning Imre! feeling better today? ready for some discoverd patches? :D09:39
ifarkasdtantsur, morning! thanks, I am feeling much better!09:40
ifarkasdtantsur, I will take a look on those patches ;-)09:40
dtantsurifarkas, https://review.openstack.org/167640 https://review.openstack.org/167685 https://review.openstack.org/16691509:40
dtantsurthanks09:40
ifarkasdtantsur, np09:41
openstackgerritAnusha Ramineni proposed stackforge/proliantutils: Fallback to RIBCL  https://review.openstack.org/16715109:45
*** bradjones has quit IRC09:47
*** bradjones has joined #openstack-ironic09:51
*** bradjones has quit IRC09:52
*** bradjones has joined #openstack-ironic09:52
openstackgerritLucas Alvares Gomes proposed openstack/python-ironicclient: Add support for generating a config drive  https://review.openstack.org/16772509:53
*** pelix has joined #openstack-ironic09:54
*** naohirot has quit IRC09:57
*** mgoddard has joined #openstack-ironic09:57
*** lazy_prince has quit IRC09:57
ifarkasdtantsur, what does ENEEDTESTS do?09:58
dtantsurifarkas, usually it's mark for myself and reviewers that tests are not written :) I think I've already written them though10:00
*** pas-ha has joined #openstack-ironic10:00
ifarkasdtantsur, yeah, there are tests already. I just didn't know what does it stand for in the comments :-)10:01
dtantsurifarkas, kind of joke based on usual names for *nix errors :)10:01
openstackgerritMerged stackforge/ironic-discoverd: Invalidate cached node information after hooks run  https://review.openstack.org/16768510:03
openstackgerritMerged stackforge/ironic-discoverd: Add INSPECTFAIL as a valid state to start introspection  https://review.openstack.org/16764010:04
openstackgerritMerged stackforge/ironic-discoverd: Add option to delete ports after introspection  https://review.openstack.org/16691510:07
*** romcheg has quit IRC10:09
openstackgerritRamakrishnan G proposed openstack/ironic: Add boot interface in Ironic  https://review.openstack.org/16651210:18
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: [WIP] Add DevStack plugin for ironic-discoverd  https://review.openstack.org/16478210:19
*** takadayu_ has quit IRC10:23
*** pas-ha has quit IRC10:23
*** pshchelo has joined #openstack-ironic10:23
openstackgerritMerged openstack/ironic: Allow node.instance_uuid to be removed during cleaning  https://review.openstack.org/16703310:32
*** kalpase1 has quit IRC10:33
*** romcheg has joined #openstack-ironic10:34
*** chenglch has quit IRC10:35
*** kalpase has joined #openstack-ironic10:37
openstackgerritVladyslav Drok proposed openstack/ironic: Removed hardcoded IDs from "chassis" test resources  https://review.openstack.org/13320510:43
*** vdrok_afk is now known as vdrok10:44
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/16646811:01
*** ramineni has quit IRC11:08
*** ChuckC_ has joined #openstack-ironic11:17
*** tteggel_ has joined #openstack-ironic11:19
*** pelix has quit IRC11:19
*** mordred_ has joined #openstack-ironic11:20
*** omnibus7 has joined #openstack-ironic11:21
*** saripurigopi has quit IRC11:21
*** lifeless1 has joined #openstack-ironic11:21
*** pelix has joined #openstack-ironic11:21
*** rsFF has quit IRC11:24
*** rsFF has joined #openstack-ironic11:24
openstackgerritShivanand Tendulker proposed openstack/ironic: boot_mode is overwritten in node properties  https://review.openstack.org/16795211:24
*** romcheg has quit IRC11:25
*** hj-hp has quit IRC11:25
*** lsmola has quit IRC11:25
*** arif-ali has quit IRC11:25
*** davidlenwell has quit IRC11:25
*** lifeless has quit IRC11:25
*** tteggel has quit IRC11:25
*** logan2_ has quit IRC11:25
*** derekh has quit IRC11:25
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: [WIP] Add DevStack plugin for ironic-discoverd  https://review.openstack.org/16478211:25
*** logan2_ has joined #openstack-ironic11:26
*** chlong has joined #openstack-ironic11:26
*** absubram has joined #openstack-ironic11:26
openstackgerritShivanand Tendulker proposed openstack/ironic: boot_mode is overwritten in node properties  https://review.openstack.org/16795211:26
openstackgerritShivanand Tendulker proposed openstack/ironic: Deploys post whole disk image deploy fails  https://review.openstack.org/16763611:27
openstackgerritShivanand Tendulker proposed openstack/ironic: pxe deploy fails for whole disk images in UEFI  https://review.openstack.org/16788211:27
*** arif-ali has joined #openstack-ironic11:29
*** derekh has joined #openstack-ironic11:29
*** ChuckC has quit IRC11:30
*** dhellmann has quit IRC11:30
*** mordred has quit IRC11:30
*** anteaya has quit IRC11:30
*** jerryz has quit IRC11:30
*** ekarlso has quit IRC11:30
*** morgabra has quit IRC11:30
*** Nick_A has quit IRC11:30
*** rbogorodskiy has quit IRC11:30
*** sambetts has quit IRC11:30
*** bauzas has quit IRC11:30
*** agordeev has quit IRC11:30
*** naotokl has quit IRC11:30
*** mordred_ is now known as mordred11:30
*** zhenguo has joined #openstack-ironic11:30
*** lsmola has joined #openstack-ironic11:31
*** dhellmann has joined #openstack-ironic11:33
*** jerryz has joined #openstack-ironic11:33
*** ekarlso has joined #openstack-ironic11:33
*** rbogorodskiy has joined #openstack-ironic11:33
*** morgabra has joined #openstack-ironic11:33
*** Nick_A has joined #openstack-ironic11:33
*** sambetts has joined #openstack-ironic11:33
*** bauzas has joined #openstack-ironic11:33
*** agordeev has joined #openstack-ironic11:33
*** naotokl has joined #openstack-ironic11:33
*** anteaya has joined #openstack-ironic11:38
*** pradipta has quit IRC11:40
*** hj-hp has joined #openstack-ironic11:41
*** romcheg has joined #openstack-ironic11:47
*** takadayuiko has quit IRC11:47
*** romcheg has quit IRC11:48
jrollmorning all :)11:48
jrolllucasagomes: dtantsur: I can't decide if we've come to an agreement here https://review.openstack.org/#/c/167801/2/ironic/drivers/modules/agent_base_vendor.py11:48
jrollI tend to think that if a node heartbeats, we should record that, whether we do anything or not11:48
lucasagomesjroll, morning11:49
lucasagomesjroll, right I think it should not record heartbeats when it's in maintenance11:49
dtantsurjroll, morning, I no longer have opinion on this :)11:49
jroll(this is a downstream hack we've had for a bit now that has been super useful)11:49
lucasagomesjroll, as the comment, I think heartbeats are useful for timeout11:49
lucasagomesif it's maintenance we don't know when the operator will release the node, can take days, weeks11:49
lucasagomesso heartbeats are useless there11:49
lucasagomesno problem in logging a debug message if a heartbeat comes in tho11:50
jrollI'm not sure they are useless, though11:50
lucasagomesjroll, unless you have something in mind, like booting a custom ramdisk when in maintenance11:50
lucasagomesthat does some operations11:50
lucasagomesand that would timeout11:50
jrollto me, recording the heartbeat is more useful than a debug message11:51
jrolldebug message could get spammy, not sure11:51
lucasagomesright11:51
lucasagomeshmm yeah I'm divided I don't have a strong opnion against not putting it on the db11:52
lucasagomesbut if we can avoid I would11:52
lucasagomeswhy would we use the heartbeats for if no operations is going to be triggered, it's no-op11:52
jrollI'm not sure I have a strong argument for it, either :)11:52
lucasagomes?11:52
lucasagomeslet's brainstorm then :D11:52
jrollso that ironic knows its up11:52
jrollyeah, working on coffee, still early :)11:53
lucasagomesheh fair enuff, yeah I;m going for lunch pretty soon too11:53
* lucasagomes is hungry11:53
jrollI could really use that client --json option right now ;D11:53
jrollgoing to find out how many heartbeating nodes we have in maintenance11:54
lucasagomesone thing I think it's fair is that... if something happens in the deployment and the agent is booted11:54
lucasagomesdo we power off the node?11:54
lucasagomesmaybe we should leave it powered on so that we can look at the logs11:54
lucasagomesand, put the node in maintenance11:55
lucasagomesso I think this would be a valid case for adding heartbeat on the db11:55
*** saripurigopi has joined #openstack-ironic11:55
jrollright, I think we do shut it off today11:55
lucasagomeslook deployment fail and you still have logs there11:55
lucasagomesyeah11:55
lucasagomeswe do11:55
jrolldownstream we keep them running as much as possible11:55
lucasagomesbut I think that's a valid/useful case for that11:55
lucasagomeswell... anyway it doesn't do any harm, let's keep it on the db11:56
*** absubram has quit IRC12:00
jrolllucasagomes: ok, cool, thanks12:01
jrollstill want a debug log?12:01
jrollalso... for some data, 63% of our maintenanced nodes are still heartbeating12:01
lucasagomeshah cool12:01
jrollthat took way too long to find out, not sure if tooling or lack of caffeine12:02
lucasagomesI will change my vote there, but if u will the log I will wait the new patch set12:02
* lucasagomes goes change his vote12:03
jrollidk, I don't care if there's a log there, but if other folks want it I can add12:03
lucasagomesjroll, it's fine, as you said it can get spammy and it's already on the db12:06
lucasagomesso12:06
*** romcheg has joined #openstack-ironic12:06
lucasagomesno strong case for that12:06
jrollheh, ok :)12:06
* jroll pokes dtantsur12:06
lucasagomesI like the return instead of pass tho12:06
TheJuliagood morning12:06
lucasagomesbut... no reason to do a new patch-set for that12:06
dtantsurjroll, meow?12:07
jrolldtantsur: I'll sne dyou soup and drugs in return for a vote on https://review.openstack.org/#/c/167801/ :D12:07
dtantsurTheJulia, morning12:07
jrolldtantsur: (also, stop working and go get some rest, get better)12:07
jrollmorning TheJulia :)12:07
dtantsurjroll, aren't you going to add logging? I +2 provided that at least DEBUG is there12:08
jrolldtantsur: idk, I don't care if there's a log there, but if other folks want it I can add12:08
dtantsurjroll, I'm in bed and slowly debugging devstack support for discoverd (say hello to devstack's insane networking configuration)12:08
dtantsur:)12:08
jrolldtantsur: we're recording the heartbeat in the db, could get spammy12:09
jrolllol12:09
*** jcoufal has joined #openstack-ironic12:09
jrollcould get spammy for no reason*12:09
lucasagomesTheJulia, good morning12:09
*** saripurigopi has quit IRC12:10
dtantsurjroll, hmm... my gut feeling is to have more debug logs rather than less...12:10
jrolldtantsur: I'm fine either way, I have grep -v12:10
jroll:)12:10
jrollI'll add it quickly12:11
dtantsurack12:11
* dtantsur will read some tutorial on openvswitch, because hell.....12:12
TheJuliayou guys aren't going to make it easy for me to get the none dhcp driver fixed are you?12:13
jroll:|12:13
jrollTheJulia: I've been neglecting that patch, got a link?12:13
TheJuliahttps://review.openstack.org/#/c/16690312:14
TheJulia"fixed so nodes can deploy when its used"12:14
jrollgerritbot where is you12:14
jrolldtantsur: lucasagomes: updated https://review.openstack.org/#/c/167801/12:14
TheJuliajroll: I suspect we can blame badcub for the bot's disappearance :)12:14
lucasagomesTheJulia, :(12:14
lucasagomesloads of indecisions there, sorry for that12:15
TheJulialucasagomes: I actually agree tasks would be better overall, but the interface has basically been committed to, rewriting both at this point seems like feature work to me12:15
lucasagomesright, yeah we already screwed the interface, but I feel bad to screwing it a bit more now :/12:17
*** priteau has joined #openstack-ironic12:17
lucasagomesand all this time constraints due rc112:17
lucasagomesmaybe it's ok to have vifs= there for the release, so that we don't break the none provider12:18
lucasagomesbut :'(12:18
jrollyeah, what I'm thinking is lets think harder and do it right next cycle12:19
lucasagomesjroll, lemme know what you think12:19
lucasagomesright12:19
jrolljust posted the same on the review12:19
jrollI don't want to rush it12:19
lucasagomesright I'm ok with that12:19
*** davide has joined #openstack-ironic12:19
lucasagomesthe good thing is that at least vifs has a default12:19
* TheJulia goes and makes coffee12:20
lucasagomesjroll, I'm good with it, if u want to approve12:20
lucasagomesrame<tab>12:20
jrollshould we give ramesh a chance to agree?12:20
lucasagomesyeah12:20
jrollok12:20
lucasagomesI was about to ping him12:20
jrollI already tried :P12:20
jrollTheJulia: hey, so real question: in your environment do you put deploy_kernel/ramdisk in driver_info, and let ironic build the pxe configs, even though it isn't used?12:21
jrollTheJulia: we hack around it with `if CONF.dhcp.dhcp_provider != 'none'` in some plaves but curious what you do12:22
TheJuliajroll: We do set deploy_kernel/ramdisk in driver info, but I thought the ipxe template writes it out12:23
* lucasagomes goes lunch12:23
*** lucasagomes is now known as lucas-hungry12:23
jrollTheJulia: but if you're using no dhcp provider, then the ipxe template isn't used right?12:23
jroll(even though it's written)12:23
*** davide has quit IRC12:24
TheJuliajroll: it gets used, we've inserted boot.ipxe in, and directly configure dnsmasq to tell the booting machines to read boot.ipxe after loading ipxe12:24
TheJuliawhich then goes and reads the config file that is written out with the ipxe template12:25
jrollhow does it know the path for that config file? does boot.ipxe know the nodes' uuid?12:25
TheJuliamac address, boot.ipxe uses a feature of the firmware to send the client to a specific config folder by mac address, and ironic puts the file structure on disk as it is expected12:26
*** kalpase has quit IRC12:26
jrollaha12:27
jrollinteresting12:27
jrollthanks :)12:27
TheJuliano problem12:28
* TheJulia retrieves the sacred coffee12:28
jrolllucas-hungry: you're still -1 on https://review.openstack.org/#/c/166903 ?12:28
jrolloh lunch12:28
*** kalpase has joined #openstack-ironic12:30
*** trown|outttypeww is now known as trown12:42
*** kalpase has quit IRC12:46
*** toabctl has quit IRC12:50
*** toabctl has joined #openstack-ironic12:54
*** alexpilotti has joined #openstack-ironic12:57
*** dprince has joined #openstack-ironic12:58
pshchelohi all - is it somehow possible to assign neutron security groups to baremetal instances?12:59
*** athomas has joined #openstack-ironic13:01
*** aswadr has joined #openstack-ironic13:02
*** dlpartain has quit IRC13:03
devanandapshchelo: not afaik, because neutron isn't controlling a hardware firewall ...13:05
pshchelodevananda, so the user instances on baremetal have unrestricted access or obey the default security group?13:06
devanandapshchelo: a bare metal instance has no hypervisor, therefor no local neutron agent or local firewall on that host13:07
*** rloo has joined #openstack-ironic13:08
devanandaunless the user sets up their own firewall inside the instance 913:08
pshchelobut the network configuration is still made by Neutron - might neutron's FWaaS plugin work?13:09
jrolldoes that not depend on a local agent?13:09
devanandapshchelo: it depends13:09
dtantsurdevananda, morning13:09
jrollalso, good morning devananda and pshchelo :)13:09
*** dtantsur is now known as dtantsur|brb13:09
devanandagood afternoon :)13:10
* devananda is ready for a nap13:10
jrollheh13:10
pshchelojroll, not sure, just asking around. preparing a feature comparison list between standard VMs and Ironic baremetal instances from Nova perspective :)13:10
jrollpshchelo: aha, ok. that helps. I think it's safe to say we don't really support any neutron-y things... ironic requires a flat network13:11
pshchelook, will keep in mind13:11
jrolldevananda may have other commentary on that13:12
-openstackstatus- NOTICE: gerrit stopped emitting stream events around 11:30 utc and has now been restarted. please recheck any changes currently missing results from jenkins13:12
jrollpshchelo: the ironic driver creates neutron ports on the network it's given (which must be flat), we use those ports for dhcp for provisioning, and for tenant networking13:13
jrollpshchelo: but the traffic doesn't flow through neutron or whatever AIUI13:13
jroll(except dhcp)13:14
pshcheloyes, I supposed so.13:14
devanandajroll: it can be set up as a router, but yea, it's not really a good idea13:14
jrolldevananda: well, I learned something today then :)13:14
devanandagreghaynes: ^ isn't that how some things in the tripleo ci cloud were done? or am I misremembering?13:15
devanandajroll: also, I think Ironic should work better with Neutron as an SDN layer, but we dont have a lot of support on the neutron side right now13:16
devanandajroll: your mechanism driver notwithstanding :)13:17
jroll:P13:17
jrollI totally agree13:17
*** krtaylor has quit IRC13:17
jrolldevananda: I think this is probably our number one feature request right now13:17
jrolland then raid13:17
devanandajroll: they're both in the top of our requests, yes13:18
devanandajroll: boot from volume, too13:18
jrollyep.13:18
jrollalso mounting volumes13:18
devanandaI'd like to also see multiple vendors supporting pxe-less booting of the agent ramdisk13:19
jrollI wonder if we can ever effectively do that automatically13:19
jrollI'm fine with just passing the iscsi reference back to the user13:19
jrollas a start, anyhow13:19
devanandairmc had a proposal for that, but needs more work. I'd like to see dell add it, too. hopefully in liberty? :)13:20
jrollagree13:20
devanandajroll: volume stuff will get better in future hardware revs, but right now, it's just a limitation of the hardware13:20
jrolldevananda: sure, I want it to work for all hardware13:20
jrollwhich will require the user to mount it manually and/or an agent13:20
* devananda has spent the last couple days meeting with HP firmware architects, learning about what's coming through the DMTF standardization of out-of-band management systems13:21
jrollDMTF?13:21
devanandagoogle it13:21
jrollthat's so much more effort13:21
jroll:P13:21
jrollwow, so many standards13:22
devanandai know13:22
jrollat any rate, I'd like volume support without bmc cooperation13:22
jrolland I'm gonna do it13:22
jroll:)13:22
jrollthe manual step gives me a sad, but so do agents13:22
jrolland so do BMCs13:23
devanandajroll: so, rather than use the BMC, you'll use an in-band method to config the BIOS or FC card to make the volume available to the boot loader?13:24
jrolldevananda: I'm starting at just mounting a volume13:24
jrollnot booting13:24
jrollfor booting, ipxe can do it :)13:24
devanandaomg, your BMC will soon support UPnP ...13:25
jrollಠ_ಠ13:26
devanandafor, you know, discovering new hardware13:26
*** dlpartain has joined #openstack-ironic13:26
jrollI think I'll stick to my crappy basic BMCs :|13:27
*** dlpartain has left #openstack-ironic13:27
*** kkoski has joined #openstack-ironic13:30
*** rameshg87 has joined #openstack-ironic13:31
rameshg87hello ironic13:31
jrollmorning rameshg87 :)13:31
jrollor evening13:32
rameshg87jroll: morning :)13:32
jrollrameshg87: consider changing your vote on https://review.openstack.org/#/c/166903 ? :D13:32
rameshg87jroll: :)13:33
rameshg87jroll: actually one question13:33
jrollmhm13:33
rameshg87jroll: can we avoid changes to base.py ?13:33
rameshg87jroll: because then we will breaking the api contract twice, no ?13:33
jrollrameshg87: adding optional parameters doesn't break the contract IMO13:33
jrollit can still be called without those13:34
rameshg87jroll: okay, we would make it **kwargs in update_dhcp_opts in https://review.openstack.org/#/c/166903/6/ironic/dhcp/base.py ?13:34
rameshg87jroll: and may be consider passing task to update_port_address ?13:35
rameshg87if we are changing things to kwargs13:35
rameshg87and update_port_dhcp_opts too13:35
rameshg87does that make sense ?13:35
jrollrameshg87: I don't think we should change it this late in the cycle, did you see my comments there?13:35
rameshg87yeah i am not saying we should change any code for that13:36
rameshg87to the api contract in dhcp/base.py, we can just pass in the task argument13:36
rameshg87and do the same in neutron and none and then just don't use it13:36
openstackgerritShivanand Tendulker proposed openstack/ironic: grub2 bootloader support for uefi boot mode  https://review.openstack.org/16619213:36
rameshg87in liberty, we can make the change without breaking the api contract again13:36
rameshg87that's what i meant13:36
jrollrameshg87: that will end with breaking the contract, though13:37
jrollsure we will13:37
jrollif we drop parameters it will break the contract13:37
jrollI think we need to think harder about this13:37
* devananda reads13:37
jrolland I don't think this time in the cycle is the right time to do it, we'll end up rushing13:37
rameshg87yeah, i agree on leaving this to liberty13:37
jrollbasically any non-neutron provider is broken today, we should provide the minimal fix13:37
rameshg87but may be i am just thinking we can do enough not to break the api contract again in liberty13:37
jrollthe api contract is broken already13:38
rameshg87jroll: as i said, can't the minimal fix be just port the same thing on none.py as in neutron.py ?13:38
rameshg87jroll: without touching dhcp/base.py ?13:38
*** Marga_ has joined #openstack-ironic13:38
rameshg87jroll: just one thing i am thinking - not to break the api contract twice13:39
rameshg87jroll: if we make change now, and revisit again in L, we would be breaking twice, no ?13:39
jrollrameshg87: but then the base is still broken, anyone that adheres to that interface is broken13:39
rameshg87jroll: hmm .. yeah13:39
jrollrameshg87: we pass vifs to those methods today, any out of tree drivers that work, must already have that argument13:40
devanandarameshg87: base defines the API. the fact that we landed dtantur's patch which deviated from our own API is our fault13:40
*** davide has joined #openstack-ironic13:40
rameshg87devananda: i agree.13:40
devanandaeither we agree to hchange the base.py and none.py modules as we should have done in efd63cbf53813:40
devanandaor we revert it13:40
jrollrameshg87: devananda: btw, we already convinced lucas-hungry on this, he just hasn't changed the vote13:40
rameshg87devananda: jroll: here is my proposal, 1 sec putting on paste.openstack.org13:40
*** davide has quit IRC13:40
jrollso I say feel free to land13:40
devanandait's too late i nthe cycle to do design work on this13:40
* rameshg87 writes13:42
rameshg87give me a min13:42
devanandaoh, right, and then JoshNang broke the API contract in 534d9ee9 too13:43
rameshg87http://paste.openstack.org/show/196873/13:43
devanandawe did it twice this cycle :(13:43
rameshg87forget the docstrings :)13:43
devanandarameshg87: yes, **kwargs is also possible. that opens the door for any subclass to extend with their own args13:44
rameshg871) pass task to all the methods - it gives enough information for doing anything - that's what we have found out in all the interfaces that we define13:44
devanandait fixes this error, but changes the API13:44
jrollrameshg87: ... adding task breaks the contract yet again13:44
devanandarameshg87: and the change TheJulia proposed does not prevent us from doing your change, to use kwargs, later13:44
rameshg87jroll: but we will end up breaking in kilo anyway, right ?13:45
devanandawheras task needs to be a mandatory parameter if we add it13:45
rameshg87oh kwargs means we won't break. is that what you meant ?13:45
jrollI still am of the opinion we should land this change now and fix it for real in liberty13:45
rameshg87devananda: yeah..13:45
devanandajroll: ++13:45
* devananda pushes the button13:45
jrollheh13:45
jrollI guess that settles it13:45
rameshg87:)13:45
TheJuliayay13:46
*** ifarkas has quit IRC13:46
rameshg87devananda: have you pushed the correct button ? :)13:46
rameshg87i still can't see it ..13:46
*** krtaylor has joined #openstack-ironic13:47
* devananda was typing a message13:48
jrollnot allowed!13:48
*** ifarkas has joined #openstack-ironic13:49
* rameshg87 prepares to bug jroll again13:50
rameshg87jroll: https://review.openstack.org/#/c/167801/313:50
*** foexle has joined #openstack-ironic13:50
rameshg87jroll: why do we consider recording the timestamp of heartbeat from a node on maintanence ? i still didn't get :)13:51
jrollrameshg87: so that ironic knows the agent is alive13:51
jrollrameshg87: whether it does anything with that info is TBD, but I don't see any harm in it13:51
rameshg87jroll: was just thinking if ironic really cares about what's happening to the node on maintenance - that includes agent is alive or not as well, right ?13:53
rameshg87jroll: yeah may be ironic will never use that info13:53
rameshg87jroll: but one question13:53
jrollrameshg87: yes, I think it's valuable that ironic knows if the agent is up13:53
rameshg87jroll: just one more question - do you track agent heartbeat timeouts for long running ramdisks in downstream ?13:54
jrollrameshg87: yes, we track the heartbeat, we don't do anything automatic around it thuogh13:54
jrollrameshg87: they just show differently on our dashboard13:54
rameshg87jroll: something like put automatically to maintanence if it doesn't heartbeat in x seconds, no ?13:55
jrollrameshg87: (though I think soon I will add a periodic task to check it)13:55
rameshg87okay13:55
jrollrameshg87: right, we don't do that yet. it would be valuable since nova will still schedule to it. however, usually if it isn't heartbeating it's in cleaning mode13:55
jrollit hasn't been painful enough to prioritize13:56
rameshg87jroll: okay.13:57
*** ChuckC_ has quit IRC13:58
rameshg87the only reason why i added a question there was why to record the heartbeat info if ironic doesn't care about it - since it's in driver_internal_info - the operator can't see it too.13:58
jrolldriver_internal_info is visible to the operator, no?13:58
jrolljust read-only13:59
rameshg87jroll: oh, i *assumed* its invisible13:59
rameshg87jroll: oh okay13:59
jrollI believe it's read only13:59
jrollif it's not, I'm going to fix that13:59
jroll:)13:59
rameshg87fix showing up driver_internal_info ?13:59
jrollyes13:59
rameshg87:)13:59
* jroll deploying a rebase from master today13:59
devanandait was visible and read-only last i checked13:59
jroll(or trying to)14:00
jrollcool, thanks deva14:00
rameshg87okay14:00
rameshg87anyone else would want to take a look at https://review.openstack.org/#/c/167801/ ?14:03
rameshg87or i can +A it :)14:03
*** ChuckC_ has joined #openstack-ironic14:03
rameshg87one question - is it theoretically possible to have a node in maintanence and state = DEPLOYWAIT or DEPLOYING ?14:04
rameshg87i assume we don't allow node-update in any of these states14:04
jrolloh, you might be right14:04
rameshg87and a failure in any of the states, we don't set the maintanence flag either14:04
jrollwel14:04
jrolll14:04
jrollif the bmc died during deploy, it could go to maintenance14:05
rameshg87i would assume it would goto DEPLOYFAIL instead14:05
jrollalso, idk if we block set-provision-state active while in maintenance mode14:05
jrolldepends14:05
rameshg87hehe14:05
jrollif the power state loop catches it first, it will just put in maintenance mode14:05
devanandajroll: i believe we do14:06
jrollyou can certainly end up with a node in maintenance trying to go through cleaning, though, that's the main goal here14:06
jrolloh?14:06
jrolldevananda: so we can't tear down an active node with maintenance flag?14:06
rameshg87sync power state doesn't consider DEPLOYWAIT nodes - https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1071-L107314:06
jrolldevananda: seems bad14:07
devanandajroll: we block deploying, not deleting14:07
devanandaor we used to14:07
jrolldevananda: ok :)14:07
*** lucas-hungry is now known as lucasagomes14:07
jrollso really this guard only needs to be for cleaning states, though it doesn't change anything for deploy* states14:07
rameshg87jroll: so the test case over there might be actually testing what might be theoretically possible today :)14:07
jrollyeah14:08
jrollI still tend to think it's in a valid place14:08
lucasagomesjroll, sorry was having lunch14:08
jrollalternative is add the check to both of the if statements for cleaning14:08
lucasagomesits approved already so14:08
lucasagomesit's ok :)14:09
rameshg87yeah, fine with me too ..14:09
jrolllucasagomes: no worries, I forgot and tab complete doesn't tell me unless I look :P14:09
rameshg87lucasagomes: talking about https://review.openstack.org/#/c/167801/3 ?14:09
rameshg87lucasagomes: someone needs to push +A :)14:09
rameshg87oh waiting for jenkins14:09
rameshg87okay14:09
lucasagomesrameshg87, done14:09
lucasagomesjenkins won't merge if it fails test14:10
lucasagomesso it's ok14:10
rameshg87okay14:10
jrollwoot, thanks y'all14:10
* jroll one less hack14:10
NobodyCamGood morning Ironic14:11
rameshg87morning NobodyCam14:11
jrollheya NobodyCam :)14:11
NobodyCammornign rameshg87 jroll lucasagomes14:11
NobodyCam:)14:11
lucasagomesNobodyCam, morning14:11
rameshg87anyone free over there has some quick thoughts on using torrent to transfer image to the baremetal ?14:12
rameshg87i am just doing some experiments14:12
jroll+114:12
*** r-daneel has joined #openstack-ironic14:12
rameshg87use case is provision 25 nodes at once with same image14:12
jrollrameshg87: I'm all about it, would love14:12
rameshg87ironic conductor would create the torrent, send it to the baremetal14:13
rameshg87operator could run a tracker14:13
rameshg87all conductors would use the same tracker (or >1 trackers)14:13
jrollrameshg87: even if you wanted to provision one node, imagine if you had a bunch of long-running agents seeding it :)14:13
rameshg87so irrespective of when deploys start, all nodes from all conductors can seed14:13
rameshg87jroll: that would be too good14:13
rameshg87jroll: :)14:13
rameshg87jroll: only one thing is initial handshakes might take time14:14
*** yog__ has quit IRC14:14
rameshg87jroll: to find out seeds from the tracker and start requesting pieces from them14:14
lucasagomespxe_ipa-src job has been +a in -infra :)14:14
rameshg87jroll: it might not immediately start utilizing the full nic bandwidth of the bare metal14:15
jrolllucasagomes: nice!14:15
*** yog has joined #openstack-ironic14:15
*** yog is now known as Guest9818114:15
rameshg87lucasagomes: is it pxe_ssh + ipa ramdisk ?14:15
*** rsalevsky has quit IRC14:15
jrollrameshg87: I would think it should be fairly quick, certainly for large images it should be faster than http14:15
jrollespecially when deploying many nodes with the same image14:16
rameshg87jroll: yeah, it would break the load on the conductors too14:16
lucasagomesrameshg87, yup, that's on openstack/ironic-python-agent gate check tho14:16
rameshg87jroll: may be some experimental results would be nice to have :)14:16
lucasagomesit's building ipa from source and testing it14:16
rameshg87lucasagomes: oh okay14:16
lucasagomesso we can detect if some changes in ipa breaks the iscsi methodology14:16
jrollrameshg87: indeed, I think it would be good :)14:16
*** rsalevsky has joined #openstack-ironic14:16
rameshg87okay. will keep folks posted ..14:17
jrollrameshg87: also, there's been work on swift to support bittorrent14:17
jrollnot sure if it's done yet14:17
rameshg87oh14:17
jrollpresumably that would take care of the need for a tracker and conductor seeds14:17
rameshg87jroll: oh okay14:18
openstackgerritChris Krelle proposed openstack/ironic-python-agent: Update README file  https://review.openstack.org/16784114:18
rameshg87jroll: and may be building the torrent too14:18
jrollbut I'm not sure :)14:18
jrollyeah possibly14:18
rameshg87jroll: seems so14:18
rameshg87jroll: https://wiki.openstack.org/wiki/Swift/APIFeatureComparison14:18
jrollat any rate, it would be good to allow it to work without swift14:18
rameshg87jroll: GET Object torrent14:18
jrollnice14:18
rameshg87jroll: yeah ironic can be standalone too. we have seen enough interest for that in kilo :)14:19
devanandarameshg87: can we decouple the ilo vmedia boot from swift?14:20
jrollrameshg87: wait, that swift doc is about AWS compatibility, and says that call isn't supported in swift :P14:20
*** romcheg has quit IRC14:20
*** rsalevsky has quit IRC14:20
NobodyCamgood morning devananda :)14:20
devanandaalternatively, TheJulia, how hard would it be for bifrost to start up a single-node swift instance?14:20
rameshg87devananda: i am thinking about that ..14:20
rameshg87devananda: we can use an http server as alternative14:21
rameshg87devananda: but i missed to propose anything for kilo :(14:21
devanandarameshg87: perfect. bifrost already configures an http(s) server14:21
rameshg87devananda: yeah we can put the floppy image generated by ilo drivers in that http server14:21
devanandarameshg87: but I think the ilo code rihgt now is hard-coded to use swift w/ temp-urls14:21
jrollrameshg87: fwiw xenserver has bittorrent support in nova, idk if that's helpful14:21
jrollhttps://review.openstack.org/#/c/13401/14:22
*** kkoski has quit IRC14:22
rameshg87devananda: with non-glance image support, vdrok made changes for everything else except the floppy image14:22
rameshg87devananda: right now floppy image needs swift14:22
devanandarameshg87: even the ISO ?14:22
rameshg87devananda: : iso can be an http url14:22
devanandaah14:22
devanandacool. so yah. let's fix that for the vfat image too14:22
rameshg87devananda: vdrok made those changes in ilo deploy driver too14:23
rameshg87devananda: yeah will definitely pursue in early L14:23
* devananda wants full pxe-less boot using HTTPS from a single stand-alone ironic instance, running underneath apache or nginx14:23
devanandaoh, and I want that across hardware vendors14:23
rameshg87devananda: +2 :)14:23
* devananda doesn't really want that much :)14:23
NobodyCam:)14:23
rameshg87jroll: yes14:23
rameshg87jroll: the page said swift doesn't support14:24
rameshg87:)14:24
rameshg87jroll: but eucalyptus supports it14:24
jrollrameshg87: I'm not interested in eucalyptus dependencies :P14:24
devanandajroll: re: bittorrent, are you thinking about optimizing massive parallel deploys?14:24
rameshg87hehe14:24
rameshg87devananda: i just started that discussion. was trying out some experiments on it14:25
jrolldevananda: for me, I have all these agents sitting around doing nothing. they should seed some images. :)14:25
rameshg87devananda: was checking with jroll on his thoughts14:25
devanandajroll: I wouldn't exect ya'll to be working on HPC workloads (but I think we're going to see cray, ibm, and hp working on that soon)14:25
devanandause image multicast, not bittorrent14:25
devanandathough there' sno reason we can't do both14:26
rameshg87devananda: last time when i was in summit i got a chance to talk to lifeless114:26
rameshg87about this14:26
devanandabut in terms of optimizing time-to-deploy a very large nmber of instances with the same image -- multicast will be far less impact on the network14:26
rameshg87devananda: he was suggesting mulitcast too14:26
devanandajroll's case (there are idle agnets. let's use them!) also makes sense, but for a different use case14:26
jrolldevananda: right, we don't care about massive deploys of one image14:27
devanandacut down o nthe bandwidth usage of image downloads14:27
devanandaby caching locally on idle resources14:27
*** zhenguo has quit IRC14:27
*** BadCub_Hiding is now known as BadCub14:27
BadCubMorning folks14:27
rameshg87devananda: he shared https://etherpad.openstack.org/p/glance-multicast-image-transfer14:27
devanandarameshg87: yes. that's the general idea14:27
devanandarameshg87: we don't strictly need glance to do it, though14:28
devananda(but within openstack, that would be the ideal place to implement it)14:28
rameshg87devananda: if not glance, the sync up is another trouble in multicast14:28
rameshg87devananda: if deploying 10 nodes, all 10 nodes might not be managed by same conductor14:29
NobodyCammorning BadCub14:29
TheJuliagood morning BadCub, NobodyCam14:29
devanandarameshg87: this sort of optimization only makes sense if deploying 100's of nodes or more at the same time14:29
rameshg87devananda: so we might end using multiple multicasts - one from each conductor, no ?14:30
*** rsalevsky has joined #openstack-ironic14:30
devanandarameshg87: I hope not ... but let's revisit this design discussion after RC14:30
*** Marga_ has quit IRC14:30
BadCubMorning NobodyCam and TheJulia14:30
NobodyCammorning TheJulia :)14:30
rameshg87devananda: yeah okay ..14:31
NobodyCambrb14:31
*** rameshg87 is now known as rameshg87-brb14:31
*** romcheg has joined #openstack-ironic14:32
*** Guest98181 has quit IRC14:33
*** dtantsur|brb is now known as dtantsur14:33
* rameshg87-brb will leave for the day (was on leave anyway today)14:33
rameshg87-brbgood night ironic, see you tomorrow14:33
*** stendulker has joined #openstack-ironic14:33
dtantsurmorning NobodyCam, BadCub14:33
*** rameshg87-brb has quit IRC14:33
dtantsurrameshg87-brb, g'night14:33
BadCubg'morning dtantsur14:34
openstackgerritZhenguo Niu proposed openstack/ironic: Use node UUID in logs instead of node ID  https://review.openstack.org/16762314:35
*** Marga_ has joined #openstack-ironic14:37
*** ukalifon has quit IRC14:37
*** mdbooth has quit IRC14:37
*** mdbooth has joined #openstack-ironic14:38
*** kkoski has joined #openstack-ironic14:38
*** zz_jgrimm is now known as jgrimm14:39
openstackgerritZhenguo Niu proposed openstack/ironic: Add attempt of max retries to power state sync logs  https://review.openstack.org/16712214:41
*** saripurigopi has joined #openstack-ironic14:41
*** absubram has joined #openstack-ironic14:43
*** absubram has quit IRC14:43
*** absubram has joined #openstack-ironic14:43
NobodyCammorning dtantsur14:46
NobodyCamnight rameshg8714:48
*** achanda has joined #openstack-ironic14:48
openstackgerritMerged openstack/ironic: Correct update_dhcp_opts methods  https://review.openstack.org/16690314:51
*** anderbubble has joined #openstack-ironic14:54
* BadCub grumbles at the internets for being slower than a carrier pigeon14:54
*** absubram has quit IRC14:54
*** absubram has joined #openstack-ironic14:54
*** kkoski has quit IRC14:59
*** achanda has quit IRC15:01
*** absubram has quit IRC15:01
TheJuliaBadCub: your internets can't possibly be slower than IPoCP using printed packets on paper...15:02
BadCubTheJulia: wanna bet? lol15:03
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Clean up firewall rules on exit  https://review.openstack.org/16802515:03
BadCubwe just rebooted everything, seems better now *grumbles*15:03
TheJuliaBadCub: http://www.blug.linux.no/rfc1149/15:04
NobodyCamlol IPoCP15:04
* devananda coffees then looks for a place to nap15:04
NobodyCami love it15:04
NobodyCamTheJulia: thats great15:04
*** absubram has joined #openstack-ironic15:04
BadCublol15:04
NobodyCamTheJulia: we know a couple of people into racing pigeons, we could use their birds as the avian carriers that might get us more bandwidth15:08
BadCubNobodyCam: They just got rid of all their racing pigeons and put the house up for sale :(15:09
NobodyCamdevananda: enjot the nap15:09
NobodyCamdoh15:09
*** kkoski has joined #openstack-ironic15:09
BadCub*for way to much, so it will sit on market for ever and ever amen*15:09
NobodyCam:-p15:10
*** jjohnson2 has joined #openstack-ironic15:10
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Clean up firewall rules on exit  https://review.openstack.org/16802515:10
* BadCub is very happy they did not buy the house next door.....15:10
NobodyCam:)15:11
*** pradipta has joined #openstack-ironic15:11
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Stop console when updating node's driver  https://review.openstack.org/16803015:13
*** kkoski has quit IRC15:21
*** kkoski has joined #openstack-ironic15:23
*** jistr is now known as jistr|mtg15:23
NobodyCamoh looks like level 3 is having a few issues :-p http://www.internetpulse.com/Main.aspx?Metric=PL15:25
*** jistr|mtg is now known as jistr15:25
*** achanda has joined #openstack-ironic15:27
*** subscope has quit IRC15:33
*** ifarkas has quit IRC15:35
*** achanda has quit IRC15:40
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: [WIP] Add DevStack plugin for ironic-discoverd  https://review.openstack.org/16478215:46
*** ekarlso has quit IRC15:46
*** saripurigopi has quit IRC15:46
*** jmccrory has joined #openstack-ironic15:49
*** mtanino has joined #openstack-ironic15:50
*** saripurigopi has joined #openstack-ironic15:51
JayFNobodyCam: around? https://review.openstack.org/#/c/167449/ should be an easy one to land if you have a sec15:54
JayFhttps://review.openstack.org/#/c/167700 as well15:55
JayFalthough ^ that doesn't have any +2s yet15:55
*** hemna has joined #openstack-ironic15:57
NobodyCammorning JayF ..  will take a look see15:58
JayFNobodyCam: also put a comment on your IPA readme with a suggested replacement for the para lucasagomes commented on15:59
JayFheh, not yet, I drafted it15:59
* JayF been switching between github prs and gerrit a lot lately15:59
lucasagomes:)15:59
NobodyCamJayF: awesome ty will address the comment in a little bit.. seems I have a couple of review ahead of that task ... lol16:00
*** kozhukalov has joined #openstack-ironic16:00
JayFlucasagomes: when did IPA PXE support land? Should I just say Kilo in the IPA readme?16:00
openstackgerritMerged openstack/ironic: Deploys post whole disk image deploy fails  https://review.openstack.org/16763616:00
lucasagomesJayF, yeah it landed this cycle16:01
lucasagomesso informating that starting with the kilo release, IPA can be used with pxe_* drivers is correct16:01
JayF"The ironic-python-agent works with the agent driver in Ironic to provision the node. Starting with ironic-python-agent running on a ramdisk on the unprovisioned node, Ironic makes API calls to ironic-python-agent to provision the machine. This allows for greater control and flexibility of the entire deployment process.16:02
JayFThe ironic-python-agent may also be used with the original Ironic pxe drivers as of the Kilo OpenStack release."16:02
JayF^ is my drafted paragraph to replace the 2nd paragraph in NobodyCam's pr16:02
NobodyCam:)16:02
lucasagomeslooks good to me16:02
NobodyCamJayF: ++16:02
*** alexpilotti has quit IRC16:04
BadCubwhen would folks, be available to do a bug squash?16:06
NobodyCamJayF: 167449 +a'd16:06
NobodyCamI thought there was a hacking rule to check that # NOTE lines had (nameHere) on them16:11
JayFthat got removed some point, I think last cycle?16:12
NobodyCamahh16:12
lucasagomesI think it's a pep8 error16:12
lucasagomesoh names sorry16:13
lucasagomesthought was #comment (without a space)16:13
NobodyCamahh16:13
*** romcheg has left #openstack-ironic16:14
NobodyCamnope was just looking at 167700 and it has a couple of # NOTE lines and no (name) not a big deal I just thought there was a check on that16:14
JayFJoshNang: https://review.openstack.org/#/c/162790 IPA patch to update some oslo stuff we should probably land, I just reviewed and +2'd16:15
JayFhttps://review.openstack.org/#/c/162791/5 as well16:16
lucasagomesshould we block patches now (-2) due the string freeze?16:17
lucasagomesif they are only updating logs etc16:18
openstackgerritShivanand Tendulker proposed openstack/ironic: boot_mode is overwritten in node properties  https://review.openstack.org/16795216:18
JoshNangJayF: +a'ed the second one, looking at the first16:18
* JoshNang loves seeing this oslo library-ifaction16:19
JayFJoshNang: the only thing that I hrm about with the first is that there's no good way to ensure logs were printed by IPA16:19
JayFJoshNang: without checking that one out and running it locally16:19
NobodyCamlucasagomes: some log changer are allowed on SF16:19
lucasagomesNobodyCam, https://review.openstack.org/#/c/16712216:19
NobodyCamchanges even16:19
NobodyCamlol nope you are correct!16:20
NobodyCamthat on is not allowed16:20
JoshNangJayF: hmm i'll try it16:20
NobodyCam:-p16:20
lucasagomeshttps://wiki.openstack.org/wiki/StringFreeze#What_changes_are_affected_by_the_string_freeze.3F16:21
lucasagomesas a ref :)16:22
lucasagomesdo we have a list somewhere of patches we need to unblock after L is open?16:22
* lucasagomes will start one if not16:23
lucasagomesadded it at the bottom of the whiteboard etherpad16:24
NobodyCamlucasagomes: awesome16:24
NobodyCamTY16:24
* NobodyCam understands the StringFreeze but does not agree with it :-p16:25
JayFAs a note: IPA is not translated or released, and *Freeze does not apply16:26
*** ChuckC_ has quit IRC16:26
*** ekarlso has joined #openstack-ironic16:27
*** Marga_ has quit IRC16:30
JoshNangJayF: looks like it's actually logging :) tried to prepare an image on my laptop and got appropriate output16:31
JayFwoo16:31
*** chlong has quit IRC16:31
*** Marga_ has joined #openstack-ironic16:31
* BadCub goes to eat16:31
openstackgerritChris Krelle proposed openstack/ironic: Support the equivalent of the Juno API  https://review.openstack.org/16436916:32
*** jmccrory has quit IRC16:33
*** Marga_ has quit IRC16:41
lucasagomessounds like a good idea http://theuserisdrunk.com/16:46
* lucasagomes will start a "OpenStack should be so simple, a drunk person could use it"16:46
lucasagomesand I can test ur OS cloud16:46
NobodyCamlucasagomes: lol16:53
openstackgerritMerged openstack/ironic-python-agent: Sync from oslo.incubator  https://review.openstack.org/16279116:54
jrolllucasagomes: I feel like that won't go well16:54
jroll:P16:54
lucasagomesjroll, yeah, we should make an experiment. I can test the rax cloud while drunk16:55
lucasagomesfor free as a test drive16:55
jrollogod16:55
lucasagomesheh jk :)16:55
jrolllucasagomes: how about you test the rdo installer drunk :P16:55
lucasagomeslol16:55
jrolland I'll test the result as a user16:56
lucasagomesfair enuff16:56
lucasagomesI will test neutron while drunk16:56
lucasagomesjk I can't even sober16:56
*** achanda has joined #openstack-ironic16:56
jrolllol16:56
NobodyCamlucasagomes: that may make testing neutron easier16:56
NobodyCamlol16:56
JayFI barely run openstack when I'm sober16:57
BadCublol16:57
lucasagomesNobodyCam, more optimistic for sure16:57
NobodyCam:)16:57
lucasagomesactually, we should drink while we do the bug squash16:57
dtantsur++16:57
lucasagomesthat would make the bugs more optimistic16:57
jrolllol16:57
NobodyCamand more of them16:57
*** omnibus7 is now known as davidlenwell16:57
jrolla previous company of mine did that16:57
lucasagomesjroll, did it work?16:57
jrollfriday was bug day, each bugfix commit earns a drink16:57
dtantsuractually I should have gotten seriously drunk before digging into devstack16:57
* BadCub likes the concept16:57
jrollit was fun at least :D16:57
dtantsurwait, I have some beers!16:58
lucasagomesdtantsur, go for it16:58
* BadCub has cherry noir gray goose, and other fun bottles16:58
lucasagomesjroll, seems you guys commited a lot bugs at that day :)16:58
dtantsuractually I'm close to figuring out how networking in devstack works, so it might be too late...16:58
jrolllucasagomes: lol, nah, we actually got stuff done16:59
lucasagomes:D16:59
jroll... until lunch at least16:59
*** alexpilotti has joined #openstack-ironic16:59
*** jistr has quit IRC17:01
*** davide_ has joined #openstack-ironic17:02
openstackgerritMerged openstack/ironic: Agent driver: no-op heartbeat for maintenanced node  https://review.openstack.org/16780117:03
*** saripurigopi has quit IRC17:03
NobodyCamJayF: would you like me to remove the Enabling section of the readme? I ask because there is no ref to the agent drivers on the page lucasagomes refferenced.. thoughts?17:03
JayFI say remove it and paste it into the place lucas linked17:03
lucasagomes^ yeah I think it's easier to keep updated in Ironic tree, since that section is about enabling it in Ironic17:04
lucasagomeshaving it on IPA is a bit odd, and likely to get outdated17:04
NobodyCamack it's gone17:04
lucasagomes(not saying in Ironic our doc doesn't get outdated, it does a lot)17:05
openstackgerritChris Krelle proposed openstack/ironic-python-agent: Update README file  https://review.openstack.org/16784117:06
NobodyCamtake a look and let me know what ya think17:06
NobodyCamcd ../ironic17:06
NobodyCamdoh ww17:06
JayF+2'd17:06
BadCub+1d17:07
*** krtaylor has quit IRC17:08
openstackgerritMerged openstack/ironic-python-agent: Use oslo_log lib  https://review.openstack.org/16279017:08
*** korekhov has joined #openstack-ironic17:10
*** derekh has quit IRC17:13
*** penick has joined #openstack-ironic17:14
*** korekhov has quit IRC17:14
openstackgerritMerged stackforge/ironic-discoverd: Clean up firewall rules on exit  https://review.openstack.org/16802517:15
dtantsurwill go for now, see you tomorrow17:19
*** dtantsur is now known as dtantsur|afk17:19
*** korekhov has joined #openstack-ironic17:19
*** Marga_ has joined #openstack-ironic17:22
rloohi NobodyCam. Do you want to be co-authored on https://review.openstack.org/#/c/164369/?17:24
rlooalso NobodyCam; should there be a bug associated with it? (given it is FF now)?17:25
NobodyCamyes on bug. :-p i didn't do enough for co-aurthor :-p17:27
NobodyCamjust cut and paste17:27
NobodyCamand hit delete a few times17:27
NobodyCamhehehehe17:27
rlooNobodyCam: whatever you say! I am never sure when people use co-authored (or even why it matters except for not +A) ;)17:28
NobodyCamrloo: for tacking maybe we should be adding our selfs as co-auth17:29
NobodyCam?17:29
rlooNobodyCam: I'm good to +2 after the bug. although jenkins isn't happy.17:29
rlooNobodyCam: huh? I didn't touch that patch17:29
NobodyCamlol I'll look in a minute17:29
NobodyCamrloo: was speaking in queens english when I used our selfs17:30
NobodyCam:-p17:30
rlooNobodyCam: oh, the Royal selves ;)17:30
NobodyCamhehehehe ;)17:30
*** jmccrory has joined #openstack-ironic17:35
*** korekhov has quit IRC17:37
*** korekhov has joined #openstack-ironic17:38
*** jmankov has joined #openstack-ironic17:39
*** jmanko has quit IRC17:43
*** ijw has joined #openstack-ironic17:43
*** kkoski has quit IRC17:44
NobodyCamrloo: btw... good morning :-p17:45
NobodyCamheheh17:45
rlooNobodyCam: good morning to you!17:46
jlvillalGood morning Ironic.  From the PSU/PDX Hackathon :)17:46
BadCubg'morning rloo :-)17:46
BadCubg'mornin to you as well jlvillal :-017:46
NobodyCammornign jlvillal17:46
BadCub:-)17:46
jlvillal:D17:46
*** krtaylor has joined #openstack-ironic17:47
rloomorning BadCub, jlvillal, and everyone else! although I've already had my lunch so it seems odd ;)17:47
BadCublol17:47
NobodyCamlol Ironiclly its always morning here17:48
NobodyCam:-p17:48
*** jerryz has quit IRC17:49
rlooNobodyCam: that's the ironic thing about ironic :)17:49
lucasagomesmorning rloo jlvillal17:49
*** krotscheck is now known as netierh17:50
*** netierh is now known as krotscheck17:50
NobodyCamhehehe :)17:50
lucasagomesok folks I will call it a day17:52
lucasagomescame to the office today, so gotta get the train back home17:52
lucasagomeshave a good night everyone17:52
rloog'night lucasagomes17:53
lucasagomessee y'all17:53
*** lucasagomes has quit IRC17:53
*** jmccrory has quit IRC17:53
NobodyCamnight lucas17:55
*** ndipanov has quit IRC17:58
*** igordcard_ has joined #openstack-ironic17:59
openstackgerritChris Krelle proposed openstack/ironic: Add IPA to enable drivers wiki page  https://review.openstack.org/16809517:59
NobodyCamjroll: JayF: BadCub: ^^^^18:00
*** igordcard_ has quit IRC18:01
*** igordcard_ has joined #openstack-ironic18:02
* BadCub pokes at whitespace18:04
*** korekhov has quit IRC18:04
openstackgerritChris Krelle proposed openstack/ironic: Add IPA to enable drivers wiki page  https://review.openstack.org/16809518:04
NobodyCamdoh18:04
BadCub+1'd18:05
*** andreykurilin_ has joined #openstack-ironic18:05
*** alexpilotti has quit IRC18:09
*** korekhov has joined #openstack-ironic18:10
NobodyCambrb18:11
*** korekhov has quit IRC18:11
rloowhohoo: https://review.openstack.org/#/c/164313/. We may get cleaning by default after all...18:11
*** korekhov has joined #openstack-ironic18:12
JoshNang\o/18:12
rlooJoshNang: what I can't figure out is if they are really nit'ing, or there are valid reasons to -1 (was it cuz of a blank line, I've already forgotten)18:13
*** alexpilotti has joined #openstack-ironic18:13
*** alexpilotti has quit IRC18:13
*** atuvenie has joined #openstack-ironic18:13
JoshNangrloo: we made some whitespace changes in our states.py in ironic, i added them to nova (figured keeping them in sync was good), someone disagreed with that18:14
rlooJoshNang: yeah. that's what I mean. white space seems like a nit.18:15
*** jmccrory has joined #openstack-ironic18:20
*** jmccrory has quit IRC18:20
*** andreykurilin_ has quit IRC18:20
* BadCub needs to step away for a few18:21
*** Sukhdev has joined #openstack-ironic18:21
*** sdake_ has quit IRC18:24
*** korekhov has quit IRC18:25
*** igordcard_ has quit IRC18:25
openstackgerritShivanand Tendulker proposed openstack/ironic: boot_mode is overwritten in node properties  https://review.openstack.org/16795218:25
*** korekhov_ has joined #openstack-ironic18:27
NobodyCamlol I'm add nit's to my own patches lol18:29
* NobodyCam needs more coffee18:29
*** stendulker has quit IRC18:31
*** andreykurilin_ has joined #openstack-ironic18:33
*** igordcard_ has joined #openstack-ironic18:37
*** atuvenie has quit IRC18:37
*** priteau has quit IRC18:40
*** pradipta has quit IRC18:40
NobodyCamhumm we really should link this page somewhere: https://wiki.openstack.org/wiki/Ironic/Drivers18:42
*** pshchelo has quit IRC18:42
NobodyCamgah I didn't know this page existed: https://wiki.openstack.org/wiki/Ironic/Drivers/iLODrivers18:42
NobodyCam:(18:42
*** andreykurilin_ has quit IRC18:43
openstackgerritNisha Agarwal proposed stackforge/proliantutils: Adding functionality required by ilo-inspection  https://review.openstack.org/16414218:45
rloohi NobodyCam, I see we have our whiteboard back.18:46
NobodyCamrloo: ya, but not hte history that was lost18:46
rlooNobodyCam: does anyone know what it only keeps the history for the past 24 or so hours?18:46
rlooNobodyCam: s/what/why/18:46
NobodyCamthats the latest  version fungi could dig up18:46
NobodyCamrloo: fungi said something about a bug but i'm not 100% sure on that18:47
rlooNobodyCam: better than nothing, and it isn't that old. but going forward, I wonder if it is configurable to set the history to be more than a day.18:47
NobodyCam??? I dont know.. would love a weekly snap shot type option!18:48
*** atuvenie has joined #openstack-ironic18:48
NobodyCamrloo: don't think that error on 164369 was real: http://logs.openstack.org/69/164369/8/check/check-tempest-dsvm-ironic-pxe_ssh-postgres/3f3fb1f/logs/devstacklog.txt.gz#_2015-03-26_17_36_45_64418:53
*** atuvenie has quit IRC18:53
rlooNobodyCam: that is an odd error18:53
NobodyCamya18:54
NobodyCamtrying just a recheck didn't see a bug with anything like that so we'll see18:57
*** kozhukalov has quit IRC19:00
NobodyCambrb19:02
rlooNobodyCam: just in case, this got merged in devstack a couple of hours ago: https://github.com/openstack-dev/devstack/commit/de77c471f3df400c4c7df724c78dbd7dc771c618#diff-3d410372a53c7608535796d758ee9d5b19:05
*** atuvenie has joined #openstack-ironic19:06
*** atuvenie has quit IRC19:14
*** hemna has quit IRC19:15
*** kkoski has joined #openstack-ironic19:15
*** igordcard_ has quit IRC19:17
*** atuvenie has joined #openstack-ironic19:17
rlooadam_g: if you're around, this might be a dumb question but should check-tempest-dsvm-ironic-pxe_ssh-postgres be running mysql? I'd think only postgres?19:19
rlooadam_g: wrt https://review.openstack.org/#/c/164369/19:19
* BadCub cleans up review-day pad19:23
*** igordcard_ has joined #openstack-ironic19:23
*** lifeless1 is now known as lifeless19:25
*** igordcard_ has quit IRC19:28
*** aswadr has quit IRC19:32
*** atuvenie has quit IRC19:32
* NobodyCam is back19:34
openstackgerritJulia Kreger proposed openstack/ironic: Add retry logic to _exec_ipmitool  https://review.openstack.org/16812019:35
NobodyCamrloo: wow19:37
NobodyCamare other patches making thru ?19:37
TheJuliadevananda: ^^^19:37
rlooNobodyCam: i was wondering the same question (if other patches are succeeding)19:38
* NobodyCam will watch TheJulia's patch19:38
TheJuliamy brain hurts :)19:39
NobodyCamTheJulia: :(19:39
NobodyCamrloo: I see other patch failing in the gate :(19:40
*** atuvenie has joined #openstack-ironic19:40
NobodyCams/patch/patches/19:40
rlooNobodyCam: probably need to revert that patch.19:40
rlooNobodyCam: the patch that adam_g added to devstack19:40
rlooNobodyCam: or update the patch to look for mysql OR postgres19:41
NobodyCamadam_g: should be on shortly19:41
NobodyCambut ya19:41
* NobodyCam updates the WhiteBaord19:41
*** achanda has quit IRC19:41
rloothx NobodyCam19:41
*** achanda has joined #openstack-ironic19:44
*** atuvenie has quit IRC19:45
* JayF does a cleaning-landed-in-nova dance19:46
JoshNang\o/19:46
NobodyCamJayF: JoshNang: w00t19:47
rlooJoshNang: yay!19:51
rlooJoshNang: so now we can set it to be enabled by default!19:51
*** hemna has joined #openstack-ironic19:51
JayFJoshNang: ^ did you fix devstack for cleaning?19:52
JoshNangJayF: yup19:52
*** openstackgerrit has quit IRC19:52
*** openstackgerrit has joined #openstack-ironic19:53
JoshNangrloo: once tests pass...i was fighting with tempest yesterday19:53
rlooJoshNang: ahh. cool. soon then ;)19:53
openstackgerritMerged openstack/ironic-python-agent: Update README file  https://review.openstack.org/16784119:53
adam_goh no19:56
adam_gsorry, just got back from lunch19:56
NobodyCamahh hey adam_g19:56
NobodyCamknew you'd be back :)19:56
NobodyCami think https://github.com/openstack-dev/devstack/commit/de77c471f3df400c4c7df724c78dbd7dc771c618#diff-3d410372a53c7608535796d758ee9d5b broke our gate19:57
adam_gya19:57
adam_gsorry :(19:57
NobodyCam:-p19:57
NobodyCamhhehehehehe19:57
adam_gis someone reverting or ?19:57
* NobodyCam recalls something about tossing a wet cat, but think the elk he was hit with has affected his memory19:58
NobodyCamadam_g: I have not started on that19:58
* BadCub picks up pieces of broken gate and buries them in a hole somewhere19:58
NobodyCamlol19:58
TheJulialol19:58
* NobodyCam hopes not our back yard19:59
* TheJulia wonders if BadCub tries to look innocent19:59
*** achanda has quit IRC19:59
jlvillalJoshNang: Congrats on both of your cleaning patches getting into Nova :)19:59
JoshNangjlvillal: thanks :)19:59
BadCubTheJulia: try to look innocent? That is easy cuz I am always innocent! *bats eyelashes sweetly*19:59
TheJuliaBadCub: your forgetting, I can see through the innocent look now19:59
* NobodyCam *chokes*19:59
* BadCub turns and glares at NobodyCam20:00
NobodyCam:-p20:00
BadCubTheJulia: but...but.... *smiles sweetly*20:00
TheJuliaoh alright :)20:01
BadCubhehehehe..20:01
*** achanda has joined #openstack-ironic20:02
TheJuliaBadCub: but I'm watching you....20:03
BadCubTheJulia: Yes, you do have special "tracking privies" in place.20:04
* TheJulia laughs evilly20:06
openstackgerritChris Krelle proposed openstack/ironic: Add IPA to enable drivers wiki page  https://review.openstack.org/16809520:06
* BadCub raises brow and starts plotting evil, er... innocent things20:06
adam_ghttps://review.openstack.org/#/c/168127/20:07
NobodyCamadam_g: +1'd20:10
BadCubbrb20:10
NobodyCamthank you for updating the WhiteBoard20:10
rlooadam_g: what is 'key' service?20:11
adam_grloo, keystone20:11
rlooadam_g: oh! thx.20:11
rlooadam_g: we're assuming no one wil want to test with no keystone.20:12
adam_grloo, ya, not in devstack20:12
rlooadam_g: I suppose you don't even need to test for that stuff. One would think a test would fail if it needs a service and it isn't avail. but the fail would be later.20:13
adam_grloo, ya, was hoping to keep it obvious to the user20:15
*** Haomeng|2 has joined #openstack-ironic20:16
*** Haomeng has quit IRC20:16
*** korekhov_ has quit IRC20:18
rlooadam_g: no worries. As long as devstack is configured to work with our existing tests, it can get changed later if need be. And yeah, if it fails later, someone will say 'but why didn't you check for this stuff before getting so far into the test'20:18
NobodyCamadam_g: ++ for pre flight checks :-p20:20
*** Sukhdev has quit IRC20:21
*** hblixt has joined #openstack-ironic20:25
openstackgerritMerged openstack/python-ironicclient: Clean openstack-common module list  https://review.openstack.org/16279920:28
*** Marga_ has quit IRC20:29
*** dprince has quit IRC20:29
NobodyCam168127 +a'd :)20:35
*** igordcard_ has joined #openstack-ironic20:41
NobodyCamanyone put any though in to Bug: #140782020:47
openstackbug 1407820 in Ironic "server will not boot from pxe if it does not support command 'chassis bootdev pxe ' with 'persistent' option" [Medium,Triaged] https://launchpad.net/bugs/140782020:47
TheJuliaThat is kind of a nasty issue thinking about it.20:49
NobodyCamTheJulia: we had this : https://review.openstack.org/#/c/151268 but....20:50
NobodyCam:(20:50
TheJulia:\20:54
openstackgerritAdam Gandelman proposed openstack/python-ironicclient: Testing functional tests (DO NOT MERGE)  https://review.openstack.org/16320321:00
*** kkoski has quit IRC21:00
*** kkoski has joined #openstack-ironic21:05
*** kkoski has quit IRC21:08
*** devlaps has joined #openstack-ironic21:11
*** mtanino has quit IRC21:14
*** mtanino has joined #openstack-ironic21:16
mrdaMorning Ironic21:16
NobodyCammorning mrda21:17
*** jcoufal has quit IRC21:19
NobodyCamHaomeng|2: when your around I wanted to ask what if you got any kind of detectable error on https://bugs.launchpad.net/ironic/+bug/140782021:19
openstackLaunchpad bug 1407820 in Ironic "server will not boot from pxe if it does not support command 'chassis bootdev pxe ' with 'persistent' option" [Medium,Triaged]21:19
*** kkoski has joined #openstack-ironic21:19
*** jgrimm is now known as zz_jgrimm21:20
*** jamielennox|away is now known as jamielennox21:21
*** Marga_ has joined #openstack-ironic21:22
*** kkoski has quit IRC21:22
*** kkoski has joined #openstack-ironic21:22
*** athomas has quit IRC21:25
*** kkoski has quit IRC21:27
NobodyCaminstrsting blog post if you haven't read.. luv the last two lines :-p http://vmiss.net/openstack/the-irony-of-openstack-ironic-bare-metal-for-the-cloud/21:28
*** jmccrory has joined #openstack-ironic21:28
NobodyCamwow this list is supper cool. only took me like 6 months to find it.. lol http://www.eweek.com/cloud/openstack-execs-discuss-kilo-release-ironic-project-and-more.html21:33
*** kkoski has joined #openstack-ironic21:34
NobodyCam"The big new project that is set to debut with the OpenStack Kilo release is the Ironic project"21:34
NobodyCamlol21:34
*** oomichi_ has joined #openstack-ironic21:36
* TheJulia blinks21:40
* BadCub gives mock look of shock21:42
* NobodyCam thanks each and every one who was invlolved!!!21:42
NobodyCamlol involved even21:42
*** jjohnson2 has quit IRC21:43
*** andreykurilin_ has joined #openstack-ironic21:45
*** jmccrory has quit IRC21:45
openstackgerritJosh Gachnang proposed openstack/ironic: Enable cleaning by default  https://review.openstack.org/16549621:46
*** pelix has quit IRC21:50
*** kkoski has quit IRC21:52
*** kkoski has joined #openstack-ironic21:53
*** andreykurilin_ has quit IRC21:53
*** andreykurilin_ has joined #openstack-ironic21:54
JayFSo IDK how much we care about this, but ...21:54
JayFif you look here: http://logs.openstack.org/96/165496/2/check/check-tempest-dsvm-ironic-agent_ssh/4d11e22/console.html#_2015-03-24_19_33_03_04621:54
JayFyou can see that even with the API microversioning, we still expose target_provision_state: AVAILABLE21:54
JayFidk if we care about that or not, but it's def an API change21:54
*** jcoufal has joined #openstack-ironic21:55
NobodyCamJayF: looks like a bug to me21:55
NobodyCam1.1 should be nostate21:56
NobodyCam1.2 should be AVAILABLE21:56
NobodyCamif my understanding is correct :-p21:58
JayFI'm fixing $other_bugs at the moment21:58
* JayF and JoshNang working on cleaning-in-tempest21:58
*** foexle has quit IRC22:00
mrdahttps://review.openstack.org/#/c/165666 (increasing the node name length field and fixing the name regex) is waiting for me to fix the database migration tests - they currently aren't running (and we haven't noticed) due to some strange test class mixing weirdness (I think).  What do we want to do? Merge this as is without the test?  I won't be able to get back to this for 3 days, and we have a22:01
mrdadeadline looming.22:01
mrdawe really should increase the node name length for K22:01
mrdathoughts?22:01
*** kkoski has quit IRC22:04
*** jmccrory has joined #openstack-ironic22:05
NobodyCamhummm22:07
BadCubwithout the test, what is the potential for DB corruption exposure?22:07
rloomrda: do we know the migration works?22:07
NobodyCammrda: 3 days is still < the rc cutoff (4/9 i believe) would you be able to have the test working with enough time to land it?22:08
mrdarloo: the migration works, definitiely22:09
mrdaNobodyCam: So I can't work out why the _check_<migrationtag> methods in test_migrations.py aren't running22:09
mrdaThere's the strangeness due to the use of mixins in the test code22:10
rlooI'm fine if it goes in w/o the test if there truly is some bug in our tests.22:10
mrdaright now, migrations aren't running22:10
mrdamigration tests22:10
mrdaI should say22:10
mrdaI don't feel comfortable adding a test, when the the existing ones aren't running22:10
mrda(and I can't work out why)22:11
*** andreykurilin_ has quit IRC22:15
*** andreykurilin_ has joined #openstack-ironic22:16
rloomrda: I have to run. Have you looked to see who modified/added those migration tests? if they're around, maybe you can ping them. see if they know. there's a mirantis guy that seemed to know a lot but i don't know his name right now.22:17
mrdaI'll try, I'm about to jump on a plane...22:18
*** penick has quit IRC22:18
JayFhttps://review.openstack.org/#/c/168175/ a devstack review for folks if you have time22:19
NobodyCammrda: i'll also chat with lucas tomorrow morning22:19
mrdaNobodyCam: thanx22:19
NobodyCamJayF: turning off cleaning?22:19
JayFno, disabling erase_devices step22:20
JayFwhich in a VM == "shred /dev/sda" == "lol your disks are dead and I'll be done with this in April"22:20
JayFthat's a config option Josh added almost explicitly to set it in devstack :)22:21
JayFJoshNang: ^ you should probably +1 that too, even though I know you over-shoulder reviewed it22:21
NobodyCam+1'd22:21
NobodyCamJayF: lol22:21
*** jmanko has joined #openstack-ironic22:24
JayFI just updated that patch ^22:26
JayFto also adjust the tempest setting22:26
openstackgerritJosh Gachnang proposed openstack/ironic: Enable cleaning by default  https://review.openstack.org/16549622:27
*** Marga_ has quit IRC22:27
*** jmankov has quit IRC22:27
NobodyCamJayF: thats not covered by the BUILD_TIMEOUT?22:28
*** anderbubble has quit IRC22:29
JayFThis isn't during build22:29
JayFthis is where tempest is polling to see the Ironic node get back to NOSTATE22:29
JayFafter the nova instance has gone away22:29
JayFI /think/ that's the right option. JoshNang just lined up all the deps so we'll find out Real Soon Now(tm)22:29
NobodyCamhehe22:30
JayFhonestly my concern is more if we have the right number for that ... if it's long enough to catch even on slow VMs22:30
NobodyCamJayF: this is what caused me to ask: https://github.com/openstack-dev/devstack/blob/master/lib/tempest#L60-L6322:30
JoshNangcleaning for agent jobs requires booting the agent and querying the api, which takes more than 60 seconds (sadly). we're skipping the actual disk erase part, so it still should be reasonably fast22:30
JoshNangNobodyCam: that's an oddly specific number...196 secs22:31
NobodyCamlol I didn't put it there22:31
JayFI mean, 180 was just as arbitrary22:31
JayF"uh, how about 3 times as much?"22:31
NobodyCambut they do call out state type changes there22:32
JayFThose are /instance/ state changes22:32
JayFnot node state changes22:32
JayFnote the listed states are nova task_states not Ironic provision_states22:32
NobodyCamah .. little dim light goes on22:32
JoshNangNobodyCam: here's the line in question: https://github.com/openstack/tempest/blob/master/tempest/scenario/manager.py#L128122:33
NobodyCambut should 180 be 196?22:33
NobodyCamlol22:33
JayFhttp://logs.openstack.org/95/168095/3/check/check-tempest-dsvm-ironic-pxe_ssh-postgres/43357ba/ this just failed on a docs change :X22:33
JayFthat's not a good sign22:33
* JayF digs22:33
JayFhttp://logs.openstack.org/95/168095/3/check/check-tempest-dsvm-ironic-pxe_ssh-postgres/43357ba/logs/devstacklog.txt.gz#_2015-03-26_21_59_32_89622:33
NobodyCamhas https://review.openstack.org/#/c/168127/ landed22:34
JayFadam_g: ^ JoshNang said you're working on this?22:34
* JayF just making sure22:34
NobodyCamnope22:34
JayFah wonderful, I see that patch22:34
NobodyCamgate will still be down22:34
JayFwhy?22:34
NobodyCamthe error you just linked22:35
*** andreykurilin_ has quit IRC22:35
*** hemna has quit IRC22:37
*** Marga_ has joined #openstack-ironic22:42
*** derekh has joined #openstack-ironic22:46
*** jmankov has joined #openstack-ironic22:46
*** jmanko has quit IRC22:50
*** jcoufal has quit IRC22:50
*** hemna has joined #openstack-ironic22:55
*** hblixt has quit IRC22:56
* devananda waves from the wee hours of the morning in France23:21
NobodyCammorning devananda23:21
NobodyCam:)23:21
NobodyCamthey do have good coffee over there23:22
devanandaNobodyCam: seattle has better. but the average coffee here is still better than bad coffee23:24
NobodyCam:) true and much more $$$$ over there23:24
* NobodyCam drinks starbucks and likes it so ....23:25
TheJuliadevananda: good morning! :)23:26
*** anderbubble has joined #openstack-ironic23:27
NobodyCamdevananda: got a new rev of 164369 up ... waiting onthe gate to come back so I can recheck it23:27
TheJuliawin 1423:27
NobodyCamlol23:27
* TheJulia apparently can't type23:27
BadCubdevananda: g'morning!23:27
*** chlong has joined #openstack-ironic23:33
*** penick has joined #openstack-ironic23:33
*** jmank has joined #openstack-ironic23:37
*** r-daneel has quit IRC23:38
*** jmankov has quit IRC23:39
*** dmellado has quit IRC23:40
*** dmellado has joined #openstack-ironic23:43
*** hemna has quit IRC23:49
*** jmccrory has quit IRC23:50
*** achanda has quit IRC23:52
NobodyCam168127 has landed... gate should be back... recheck if you got 'um23:54
devanandahow's the overall state of things looking?23:55
devanandaI should be asleep, but seem to have some spare brain power left, so, you know ...23:55
BadCubdevananda: most of the bugs tagged for RC1 have merged. The two Nova patches have merged23:56
NobodyCamdevananda: gates been doen most of the day :(23:56
NobodyCambut should be back now23:56
devanandaBadCub: ooh! nova patches in? that's awesome23:56
BadCubyes!23:56
NobodyCamseveral devstack patches to support cleanning got up, not sure if they landed23:56
NobodyCamJayF: you know if they landed ^^^^23:57
*** achanda has joined #openstack-ironic23:57
BadCubI added a little RC1 To-Do List on the white board to highlight the stuffs we need to pay attention to before RC1 hits23:57
* NobodyCam updates gate status on WhiteBoard23:59
NobodyCamand with that calls it a day23:59
NobodyCam:)23:59
* BadCub has a platter of Fish-N-Chips in his near future23:59
NobodyCammmmmm23:59

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