Tuesday, 2014-12-30

*** Masahiro has quit IRC00:01
*** david-lyle is now known as david-lyle_afk00:06
*** andreykurilin has quit IRC00:26
*** Masahiro has joined #openstack-ironic00:57
jiangfei|2good morning mrda, jroll00:59
jrollhiya jiangfei|2 :)01:00
jiangfei|2Envy you have holidays :)01:00
*** Masahiro has quit IRC01:02
mrdahi jiangfei|201:04
jiangfei|2:)01:05
*** chenglch has joined #openstack-ironic01:23
*** ryanpetrello has quit IRC01:27
*** Haomeng has joined #openstack-ironic01:31
*** Haomeng|2 has quit IRC01:32
*** jerryz has joined #openstack-ironic01:36
*** eghobo has quit IRC02:38
*** Masahiro has joined #openstack-ironic02:46
*** Masahiro has quit IRC02:50
*** ryanpetrello has joined #openstack-ironic03:11
*** ryanpetrello has quit IRC03:16
*** dlaube has quit IRC03:19
*** dlaube1 has joined #openstack-ironic03:19
*** dlaube1 has quit IRC03:20
*** aweeks has quit IRC03:20
*** bigjools has quit IRC03:21
*** aweeks has joined #openstack-ironic03:22
*** bigjools has joined #openstack-ironic03:25
*** Masahiro has joined #openstack-ironic03:47
*** Masahiro has quit IRC03:51
*** dlaube has joined #openstack-ironic03:52
*** dlaube has quit IRC04:22
*** pcrews has quit IRC04:43
openstackgerritHaomeng,Wang proposed openstack/ironic: call set_failed_state if can not get root_uuid  https://review.openstack.org/14391905:07
*** Marga_ has joined #openstack-ironic05:27
*** Marga_ has quit IRC05:30
*** Marga_ has joined #openstack-ironic05:31
*** Masahiro has joined #openstack-ironic05:35
*** Masahiro has quit IRC05:39
*** killer_prince has quit IRC05:44
*** pcrews has joined #openstack-ironic05:48
*** Marga_ has quit IRC05:50
*** Marga_ has joined #openstack-ironic05:54
*** killer_prince has joined #openstack-ironic06:02
*** killer_prince is now known as lazy_prince06:02
*** pensu has joined #openstack-ironic06:05
*** Marga_ has quit IRC06:06
*** Marga_ has joined #openstack-ironic06:09
*** Marga_ has quit IRC06:09
*** Marga_ has joined #openstack-ironic06:10
*** Haomeng|2 has joined #openstack-ironic06:17
*** Haomeng has quit IRC06:18
*** Marga_ has quit IRC06:24
*** Marga_ has joined #openstack-ironic06:25
*** enikanorov has quit IRC06:31
*** pcrews has quit IRC06:32
*** Masahiro has joined #openstack-ironic06:36
*** achanda has joined #openstack-ironic06:39
*** Masahiro has quit IRC06:40
*** mrda is now known as mrda-away06:58
*** achanda has quit IRC07:06
*** achanda has joined #openstack-ironic07:44
*** achanda has quit IRC08:01
*** wanghua has joined #openstack-ironic08:22
*** Masahiro has joined #openstack-ironic08:25
*** jerryz has quit IRC08:27
*** Masahiro has quit IRC08:29
*** achanda has joined #openstack-ironic08:31
*** Marga_ has quit IRC08:36
*** achanda has quit IRC08:36
*** Marga_ has joined #openstack-ironic08:44
*** achanda has joined #openstack-ironic08:47
*** Marga_ has quit IRC08:50
*** Marga_ has joined #openstack-ironic08:53
*** Marga_ has quit IRC08:54
*** andreykurilin has joined #openstack-ironic09:00
*** Marga_ has joined #openstack-ironic09:04
*** andreykurilin has quit IRC09:10
*** andreykurilin has joined #openstack-ironic09:11
*** achanda has quit IRC09:13
*** Marga_ has quit IRC09:16
*** Marga_ has joined #openstack-ironic09:16
*** romcheg has joined #openstack-ironic09:22
*** foexle has joined #openstack-ironic09:22
*** Haomeng has joined #openstack-ironic09:24
*** pcaruana has joined #openstack-ironic09:24
*** Haomeng|2 has quit IRC09:24
*** krtaylor has quit IRC09:24
*** krtaylor has joined #openstack-ironic09:26
*** andreykurilin has quit IRC09:30
*** andreykurilin has joined #openstack-ironic09:31
*** andreykurilin has quit IRC09:41
*** Marga_ has quit IRC09:44
*** bradjones has quit IRC10:12
*** bradjones has joined #openstack-ironic10:13
*** Masahiro has joined #openstack-ironic10:14
*** wanghua has quit IRC10:17
*** Masahiro has quit IRC10:18
*** chenglch has quit IRC10:48
*** achanda has joined #openstack-ironic10:53
*** Haomeng has quit IRC11:06
*** Haomeng has joined #openstack-ironic11:07
*** achanda has quit IRC11:19
*** achanda has joined #openstack-ironic11:33
*** Marga_ has joined #openstack-ironic11:36
*** Masahiro has joined #openstack-ironic12:02
*** Marga_ has quit IRC12:03
*** Marga_ has joined #openstack-ironic12:03
*** Masahiro has quit IRC12:07
*** Marga_ has quit IRC12:18
*** romcheg has quit IRC12:25
*** pcaruana has quit IRC12:36
*** romcheg has joined #openstack-ironic12:41
*** achanda has quit IRC12:45
*** pcaruana has joined #openstack-ironic12:47
*** openstack has joined #openstack-ironic13:07
*** achanda has joined #openstack-ironic13:09
*** pensu has quit IRC13:31
*** pensu has joined #openstack-ironic13:32
*** pensu has quit IRC13:40
*** jerryz has joined #openstack-ironic13:41
*** achanda has quit IRC13:46
*** achanda has joined #openstack-ironic13:47
*** Masahiro has joined #openstack-ironic13:51
*** achanda has quit IRC13:52
*** Masahiro has quit IRC13:55
*** ryanpetrello has joined #openstack-ironic13:59
*** EmilienM is now known as EmilienM|afk14:03
*** trown has quit IRC14:08
*** trown has joined #openstack-ironic14:09
*** EmilienM|afk is now known as EmilienM14:15
*** achanda has joined #openstack-ironic15:14
*** sambetts has joined #openstack-ironic15:24
*** Masahiro has joined #openstack-ironic15:40
NobodyCammorning Ironic15:44
*** Masahiro has quit IRC15:44
*** jerryz has quit IRC15:51
*** vdrok has quit IRC15:51
jrollmorning NobodyCam :)16:05
jrollhttp://www.slideshare.net/codeblue_jp/igor-skochinsky-enpub16:05
jrollawesome slide deck about intel ME16:05
victor_lowtherYeah, that was pretty neat.16:12
victor_lowtherAnd a testament on not giving untrusted actors access to your systems.16:12
jrollheh16:12
jroll>.>16:13
victor_lowthergetting a rootkit in ME would be nearly impossible to detect or clean without reflashing the hard way16:13
*** vdrok has joined #openstack-ironic16:18
*** pcrews has joined #openstack-ironic16:23
NobodyCamya I tend to not like things that allow Poison Pill type attacks17:03
*** sirushti has quit IRC17:11
*** romcheg has quit IRC17:13
*** romcheg has joined #openstack-ironic17:13
*** sirushti has joined #openstack-ironic17:13
*** romcheg has quit IRC17:17
*** achanda_ has joined #openstack-ironic17:21
*** achanda_ has quit IRC17:22
*** achanda_ has joined #openstack-ironic17:22
*** achanda has quit IRC17:24
*** Masahiro has joined #openstack-ironic17:29
*** foexle has quit IRC17:29
*** achanda_ has quit IRC17:32
*** achanda has joined #openstack-ironic17:33
*** Masahiro has quit IRC17:33
*** achanda_ has joined #openstack-ironic17:35
*** achanda has quit IRC17:36
*** ryanpetrello_ has joined #openstack-ironic17:38
*** ryanpetrello has quit IRC17:38
*** ryanpetrello_ is now known as ryanpetrello17:38
*** achanda_ has quit IRC17:41
*** achanda has joined #openstack-ironic17:42
*** dlaube has joined #openstack-ironic17:45
*** achanda has quit IRC17:46
*** harlowja has joined #openstack-ironic18:13
*** romcheg has joined #openstack-ironic18:13
devanandamorning, folks18:19
jrollheya devananda18:21
*** EmilienM is now known as EmilienM|food18:24
*** EmilienM|food is now known as EmilienM|afk18:25
*** achanda has joined #openstack-ironic18:35
*** coolsvap has quit IRC18:36
*** coolsvap has joined #openstack-ironic18:39
*** ijw has joined #openstack-ironic18:51
* jroll plugs some reviews18:53
jrollhttps://review.openstack.org/#/c/143167/18:53
jrollhttps://review.openstack.org/#/c/143168/18:53
jrollhttps://review.openstack.org/#/c/143480/18:53
JayFthat spec is in my queue for review today18:56
jrollwoot18:58
*** pcaruana has quit IRC19:00
*** pensu has joined #openstack-ironic19:06
*** datajerk has quit IRC19:09
*** afaranha has joined #openstack-ironic19:13
*** romcheg has quit IRC19:13
*** Masahiro has joined #openstack-ironic19:17
openstackgerritJosh Gachnang proposed openstack/ironic-specs: Implement Cleaning States  https://review.openstack.org/10268519:19
*** Masahiro has quit IRC19:21
JayFI just put the first +2 on https://review.openstack.org/#/c/102685 if someone else wants to take a look.19:23
JayFJoshNang: ^ great job19:23
JoshNangJayF: thanks!19:23
*** Haomeng|2 has joined #openstack-ironic19:31
*** Haomeng has quit IRC19:32
*** afaranha has quit IRC20:18
*** ryanpetrello has quit IRC20:45
*** ryanpetrello has joined #openstack-ironic20:50
*** david-lyle_afk is now known as david-lyle20:58
*** pensu has quit IRC21:00
*** Masahiro has joined #openstack-ironic21:06
*** Masahiro has quit IRC21:10
*** achanda has quit IRC21:13
*** mrda-away is now known as mrda21:17
mrdaMorning Ironic21:17
*** EmilienM|afk is now known as EmilienM21:19
*** ijw has quit IRC21:28
*** Marga_ has joined #openstack-ironic21:32
*** Marga_ has quit IRC21:35
*** andreykurilin has joined #openstack-ironic21:43
devanandag'morning, mrda22:07
mrdahey devananda22:11
mrdadevananda: have you had a nice holiday break?22:12
*** harlowja is now known as harlowja_away22:17
*** ryanpetrello has quit IRC22:25
*** ryanpetrello has joined #openstack-ironic22:26
*** romcheg has joined #openstack-ironic22:35
JayFdevananda: I'm going to get tests passing + a new test case for https://review.openstack.org/#/c/143193/ --- but WDYT about "internal" API breakage for IPA? I'm thinking I should toss something out to the mailing list, but in reality, the existing behavior isn't very useful and the new behavior is pretty much how we've been telling people it worked all along22:39
* devananda slogs into the third hour of filing expense reports, and continues to ignore IRC22:44
*** andreykurilin has quit IRC22:52
*** andreykurilin has joined #openstack-ironic22:52
*** Masahiro has joined #openstack-ironic22:55
*** pcm has joined #openstack-ironic22:58
*** ryanpetrello has quit IRC22:58
*** Masahiro has quit IRC22:59
clif_hCan someone check my thinking here? It seems like the RPCService object in ironic/common/service.py tears down things in the wrong order. If I initialize things in an order-dependent way, as in I init A, then B, then C, should I tear down in this order: C, then B, then A, right?23:01
clif_hRPCService tears down the service before stopping the server23:01
clif_hShouldn't it tear down the server before stopping the service?23:02
clif_hwe should stop accepting new requests before trying to stop the service23:02
clif_hotherwise a request could come in but the service is already down23:03
JayFThat makes sense to me, but I haven't looked at the code for that yet23:05
jroll^23:06
clif_hat least it makes sense23:06
pcmAnyone here interested in pyghmi?  I pushed a very small patch up for review and so I can figure out the general process before I try anything bigger, and I'm eager for feedback on anything I did wrong or should have done, etc23:14
jrollpcm: link to review? there's only a couple folks that work on pyghmi here, they may be on holiday23:15
pcmhttps://review.openstack.org/#/c/144181/23:16
JayFHey pcm, are you Peter?23:16
*** romcheg has quit IRC23:17
pcmHehe yep.  pcm is Peter C Martini23:17
jrollohai23:17
jroll:D23:17
JayFWelcome :) me + jroll  + clif_h + JoshNang + russell_h + aweeks + morgabra are all your partners in OCP-crime at Rackspace :)23:17
JoshNango/23:18
* aweeks waves23:18
clif_hOCP?23:18
jrollopen compute23:18
pcmHi!  jroll, JayF, and russell_h I guessed, glad to see a few names I didn't recall or didn't yet know too.  Nice to meet y'all23:19
jrollcool :)23:20
jrollpcm: so one thing to note, you don't need to / shouldn't vote on your own patches23:20
clif_hOpen ComPute?23:20
JayFclif_h: Open Compute Project23:20
clif_hoh yeah23:20
jrollTIL :P23:20
pcmheh, whoops, sorry23:20
pcmCan I undo that?23:21
JayFReply -> select 0 as the vote23:21
JayFor just don't worry about it :)23:21
clif_hOCP is from robocop23:21
clif_hthat's what I thought of when I saw that acronym23:22
pcmI hadn't remembered that the acronym was in Robocop until I saw the remake.  Then OpenCompute got a whole lot funnier23:22
jrollpcm: threw a review on there23:22
jrolland yes, you can hit "add comment" again and change your vote to 023:22
clif_hjroll: the pep8 check on that review seems to have passed23:23
clif_hunless it just didn't do anything23:23
jrollclif_h: yeah :|23:24
jrollI think it's not using the hacking package23:24
jrollwhich is some addons to flake23:24
pcmSo the -1 basically means split out that change to a separate thing to handle the pep8 cleanliness?23:24
jrolloh, it is23:24
jrollwtf23:24
jrollpcm: no, just don't delete those lines23:25
jrollpcm: to push a fixed patch, just amend your previous commit (git commit --amend) and run git review again23:25
pcmoh, my bad.  Can you point me to why that makes a difference?23:25
jrollwell23:25
jroll1) it's unrelated to your change23:26
jroll2) it's a common style rule in most openstack projects, not sure why pep8 isn't picking it up here though23:26
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Allow use of multiple simultaneous HW managers  https://review.openstack.org/14319323:29
pcmk, 1 makes perfect sense, though I'm not clear on where the style rule comes into play.  Can you explain what that rule is or point me to something?  Or better yet, let me know where I can find the test that should have failed so I can dig deeper :-)23:29
JayFI suspect the pep8 on that repo is configured less aggressively for reasons that are completely irrelevent to your change :)23:30
jrollyeah, that23:30
pcmI'm happy to go through and clean up style stuff too (if that's something worthwhile), and obviously as a separate bit of work23:30
jrollthere's a rule where imports go: stdlib, third-party, project modules, in alphabetical order, with a blank line between each23:31
jrollthere's another where something something two blank lines between things at the outmost indent23:31
JayFand three lines between classes, right?23:32
jrollCI should catch these for you, you shouldn't need to worry about them with your own brain23:32
jrollI just... tend to do that23:32
jrollJayF: two blanks23:32
jrollso three returns23:32
JayFthree-ent... yeah23:32
JoshNangfrom the docs: https://www.python.org/dev/peps/pep-0008/#blank-lines23:32
jroll:P23:32
*** andreykurilin has quit IRC23:32
jrollah, yeah23:32
jrollOH23:33
jrollthat file doesn't have a .py23:33
JayFpcm: most likely the change you'd make if you wanted CI to be happier would be to look at the tox.ini, see how the hacking configuration differs from Ironic (maybe even the version in requirements), change it to match and then fix things until it passes :)23:33
jrollI bet that's why pep8 doesn't see it23:33
*** romcheg has joined #openstack-ironic23:33
JayFaha23:33
JayFgood call jroll23:33
JayFthat should be fixed in the tox.ini as well :)23:33
jrollso like... that's a way bigger change23:33
jrollI mean23:33
pcmThanks for the link!23:33
JayFno it's not23:33
jrollthat's just how pep8 works, I think23:33
JayFaha, https://github.com/openstack/ironic-python-agent/blob/master/tox.ini#L2023:33
JayFI thought we specified a filename there, but only a dir :(23:34
jrollyeah23:34
JayFthat file should probably have a .py added and the non-.py version could be symlinked back23:34
JayFbut that's a different change and possibly picking a fight :)23:34
jrollyeah23:34
jrollI mean, I would go all the way to using setup.cfg things23:34
jrollanyway, we digress23:34
JayFbtw, welcome to irc pcm :)23:35
JayFyou'll find us here most business hours PST, and usually the channel is hopping with more than just us even now23:35
JayFOnly Rackers have the, uh, dedication to work the last two days of the year :P23:36
jrollpcm: I think you should fix those... as far as getting it through, devananda and jjohnso (who is on holiday) have core reviewer status on that project23:36
pcmI'm at work now, and its EST (EDT?  always forget that ...)23:36
jrolllol23:36
jrolls/business hours/all hours/23:36
pcmYep, will do later tonight.  Going to poke at this a bit more at home23:38
victor_lowtherheh. I was shocked to see so much activity in the channel.23:38
jroll:P23:38
victor_lowtherThis morning was D E D dead.23:38
jrollsame yesterday23:38
JayFWhat'd you get for christmas victor_lowther? Did your $close_family_or_friend tell you "Dude, you're getting a DELL" perhaps?23:38
* jroll cringes23:39
JayFyeah, had to work too hard for that one :P23:39
victor_lowtherI got a lollipop made of hairbands from my hairband-stealing wife, a wallet from same, a cut-down tree, and a new fence.23:39
* JayF got mostly gift cards which he used to buy nice seats to tonight's hockey game23:40
victor_lowtherMy wife got a new Alienware desktop to replace her dead XPS laptop and my aged D830 she was using in the interim.23:40
JayFI deployed a bunch of D{6,8}{2,3}0 laptops around 8 years ago23:40
JayFyou aren't kidding it was aged23:40
victor_lowtherMussing several keys on the keyboard due to random act of toddler23:41
victor_lowtherbut it still ran.23:41
victor_lowtherThe kids got all the awesome presents, of course.23:41
JayFSo I got my Bro-in-law a chromebook for Christmas. First one we got, pulled it out of the box to check it (family is on the east coast so wanted to make sure it worked before flying) and it had a broken key on it23:41
JayFIDK how you break a keyboard on a laptop before it gets into a sealed box23:42
victor_lowtherChiclet keys are real easy to break.  I have gotten good at fixing them. :)23:42
JayFthis one the plastic hook on the chicklet was busted out23:43
JayFso no repair without the replacement key23:43
*** pcrews has quit IRC23:45
victor_lowtherThere is that.23:49
*** bradjones_ has joined #openstack-ironic23:49
*** bradjones has quit IRC23:51
*** bradjones_ is now known as bradjones23:51
*** bradjones has joined #openstack-ironic23:51
*** bradjones has quit IRC23:59

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