Tuesday, 2010-08-17

[2010/08/17 00:23:55] @ Log started by gepetto
[2010/08/17 00:23:55] @ ohookins joined channel #puppet-dev
[2010/08/17 04:25:25] <ohookins> hi, i'm a bit confused over this scoping behaviour: http://pastie.org/1097467
[2010/08/17 04:25:49] <ohookins> it's not a clean-cut example as found in the documentation, and i would have thought it would work
[2010/08/17 04:26:35] <duritong> ohookins: node inheritance is mess
[2010/08/17 04:26:59] <ohookins> duritong: i agree, but for the next month or so we can't immediately refactor
[2010/08/17 04:27:33] <duritong> ohookins: my advice: only set variables in a node inheritance tree, do only one include at the bottom of a node statement that is not inherited and do all the include inheritance in classes
[2010/08/17 04:28:11] <ohookins> sounds like a refactor is in order whether i like it or not...
[2010/08/17 04:28:37] <duritong> ohookins: the issue is that in the node bar setting midvariable doesn't affect the include in the toblevel node
[2010/08/17 04:29:19] <ohookins> duritong: i understand that, but why does the fact that the toplevel node (or class for that matter) is being inherited affect variables lower down?
[2010/08/17 04:29:34] <duritong> ohookins: if you follow these rules, then one can do a lot with inheritance without a pain. the only issue is that you have to maintain two inheritance trees
[2010/08/17 04:29:56] <ohookins> if we don't include the midlevel class, everything works fine, so it appears to be caused by the two classes inheriting from the same top-level class
[2010/08/17 04:31:22] <duritong> ohookins: yes and that's because the variables are evaluated at parsetime (imho)
[2010/08/17 04:33:42] <ohookins> ok, sounds like one way or the other we need to change some things
[2010/08/17 04:34:45] <duritong> I'm living with my solutions now for over 2 years and I'm very happy with it. I just needed to learn that if you need a variable in an included class you need to define that variable before you write the include
[2010/08/17 04:37:24] <ohookins> i know the right way of doing things, it's just that most of the manifests have been written by others and i haven't had enough time to come up with a decent architecture for what we want to do
[2010/08/17 04:37:36] <ohookins> but now that we've really hit a wall with scoping, it has to change
[2010/08/17 04:37:48] <duritong> yeah but it's you that have to fix it ;)
[2010/08/17 04:37:53] <ohookins> i know :(
[2010/08/17 04:38:05] <ohookins> one more thing to do, in addition to everything else
[2010/08/17 04:38:33] * duritong got > 100 open tickets :/
[2010/08/17 05:01:36] <nasrat> ReinH: pong
[2010/08/17 05:35:40] @ beata_ joined channel #puppet-dev
[2010/08/17 07:07:35] @ Quit: jbooth: Ping timeout: 258 seconds
[2010/08/17 07:07:37] @ ohookins left channel #puppet-dev ("Leaving.")
[2010/08/17 07:09:18] @ jbooth joined channel #puppet-dev
[2010/08/17 07:17:11] @ Quit: jbooth: Ping timeout: 265 seconds
[2010/08/17 07:18:57] @ jbooth joined channel #puppet-dev
[2010/08/17 07:25:08] @ notbrien joined channel #puppet-dev
[2010/08/17 08:52:40] @ jmccune joined channel #puppet-dev
[2010/08/17 09:40:57] @ lak joined channel #puppet-dev
[2010/08/17 09:43:38] <ReinH> nasrat: 'ello
[2010/08/17 09:49:02] <ReinH> nasrat: emailed jamesturnbull about 1.5.8
[2010/08/17 09:51:10] <nasrat> cool
[2010/08/17 09:51:24] <nasrat> shouldn't you be able to throw things across the room at him
[2010/08/17 09:51:52] @ github joined channel #puppet-dev
[2010/08/17 09:51:52] <github> facter: master Nick Lewis * b2c2114 (1 files in 1 dirs): Properly wrapped the windows ipaddress fact in a setcode block.
[2010/08/17 09:51:52] <github> facter: master Jiri Kubicek * 82286e4 (4 files in 4 dirs): Fix #4352 - Support for detecting virtuals (jails) on FreeBSD ...
[2010/08/17 09:51:52] <gepetto> github: facter: #4352 is http://projects.puppetlabs.com/issues/show/4352 "Facter - Bug #4352: Facter doesn't detect FreeBSD jail as a virtual - Puppet Labs"
[2010/08/17 09:51:52] <github> facter: master Jiri Kubicek * faaa169 (3 files in 3 dirs): Fix #4352 - Support for detecting KVM virtuals on FreeBSD ...
[2010/08/17 09:51:53] <gepetto> github: facter: #4352 is http://projects.puppetlabs.com/issues/show/4352 "Facter - Bug #4352: Facter doesn't detect FreeBSD jail as a virtual - Puppet Labs"
[2010/08/17 09:51:53] <github> facter: master Rein Henrichs * ce7bd9f (1 files in 1 dirs): Refactor rakefile to use spec.ops, separate rcov task
[2010/08/17 09:51:53] <github> facter: master Rein Henrichs * a2bcacd (3 files in 3 dirs): [#2330] Uptime should not make redundant system calls ...
[2010/08/17 09:51:53] <github> facter: master Rein Henrichs * 2b13972 (3 files in 3 dirs): Merge branch '2330-uptime-gets-computed-twice-on-systems-with-/proc/uptime' into next
[2010/08/17 09:51:53] <github> facter: master Rein Henrichs * 67f6604 (2 files in 1 dirs): [#4062] Implement operating system facts for MeeGo ...
[2010/08/17 09:51:54] <github> facter: master Marc Fournier * be411c0 (1 files in 1 dirs): Facter::Manufacturer - test for SunOS and FreeBSD ...
[2010/08/17 09:51:54] <github> facter: master Marc Fournier * e9a60bc (2 files in 2 dirs): Facter::Manufacturer - sunos test + simplified regex ...
[2010/08/17 09:51:55] <github> facter: master Rein Henrichs * b7fe989 (1 files in 1 dirs): [#2330] Update uptime calculation to use /bin/cat ...
[2010/08/17 09:51:55] <github> facter: master Rein Henrichs * 881249f (6 files in 4 dirs): Merge branches 'ticket/master/2330-uptime', 'ticket/master/4062-meego' and 'ticket/master/4453-rakefile'
[2010/08/17 09:51:56] <github> facter: master Rein Henrichs * bd3af7e (4 files in 4 dirs): Merge remote branch 'kubicek/ticket/master/4352'
[2010/08/17 09:51:56] <github> facter: master Rein Henrichs * a2d1a54 (1 files in 1 dirs): Merge branch 'ticket/master/4441-windows-ipaddress'
[2010/08/17 09:51:57] <github> facter: master Rein Henrichs * bff84c2 (1 files in 1 dirs): [#4156] Applying patch by Kai ...
[2010/08/17 09:51:59] @ github left channel #puppet-dev ()
[2010/08/17 09:53:26] <nasrat> ReinH: I like the way you've done the fixtures stuff btw, I need to change the ip.rb tests to do that
[2010/08/17 09:54:00] <ReinH> nasrat: thanks
[2010/08/17 09:54:19] <ReinH> nasrat: I also need to unify the default interface stuff with ip
[2010/08/17 09:54:21] <nasrat> dumb question, the context naming doesn't do anything magic right?
[2010/08/17 09:54:22] <ReinH> no need to do it twice
[2010/08/17 09:54:48] <ReinH> context is an alias for describe
[2010/08/17 09:54:53] <nasrat> ah ok
[2010/08/17 09:55:16] <ReinH> apparently it's recommend to use describe for methods (describe ".strip") and context for, er, contexts ("when the user is logged in")
[2010/08/17 09:55:21] <ReinH> *recommended
[2010/08/17 09:55:42] <ReinH> judging by rspec.info, some presentations, and the new book
[2010/08/17 09:55:50] <ReinH> ymmv but it works for me
[2010/08/17 09:56:23] <nasrat> yeah I've the bdd book on pre-order - is that the one you mean?
[2010/08/17 10:12:41] @ Quit: lak: Quit: lak
[2010/08/17 10:22:41] @ Quit: jbooth: Ping timeout: 265 seconds
[2010/08/17 10:24:31] @ jbooth joined channel #puppet-dev
[2010/08/17 10:28:19] @ plathrop-away is now known as plathrop
[2010/08/17 10:42:30] @ Quit: jbooth: Read error: Operation timed out
[2010/08/17 10:45:22] @ jbooth joined channel #puppet-dev
[2010/08/17 11:24:40] @ Quit: jmccune: Ping timeout: 246 seconds
[2010/08/17 11:24:59] @ jmccune joined channel #puppet-dev
[2010/08/17 11:25:07] @ lak joined channel #puppet-dev
[2010/08/17 11:29:51] @ Quit: beata-: Ping timeout: 240 seconds
[2010/08/17 11:34:26] @ beata- joined channel #puppet-dev
[2010/08/17 11:45:59] @ Quit: beata-: Ping timeout: 260 seconds
[2010/08/17 11:47:11] @ beata- joined channel #puppet-dev
[2010/08/17 11:52:11] <ReinH> nasrat: there's an Rspec book from pragprog: http://www.pragprog.com/titles/achbd/the-rspec-book
[2010/08/17 11:52:16] <ReinH> If that's the bdd book, then yes :)
[2010/08/17 11:55:10] @ Quit: lak: Quit: lak
[2010/08/17 12:11:45] @ github joined channel #puppet-dev
[2010/08/17 12:11:45] <github> facter: master James Turnbull * 98ef5e8 (1 files in 1 dirs): Updated CHANGELOG for 1.5.8rc1
[2010/08/17 12:11:45] <github> facter: master James Turnbull * f280703 (1 files in 1 dirs): Incremented version to 1.5.8
[2010/08/17 12:11:45] <github> facter: master James Turnbull * 81ccb48 (1 files in 1 dirs): Removed references to Reductive Labs in the Rakefile
[2010/08/17 12:11:45] <github> facter: master James Turnbull * 51bcebe (1 files in 1 dirs): Fixed Rakefile package task version detection
[2010/08/17 12:11:45] <github> facter: master commits 4398b36...51bcebe - http://bit.ly/akdscN
[2010/08/17 12:11:46] @ github left channel #puppet-dev ()
[2010/08/17 12:30:04] <ReinH> nasrat: 1.5.8 rc is out, yay :)
[2010/08/17 12:32:02] @ lak joined channel #puppet-dev
[2010/08/17 13:21:06] @ nlew joined channel #puppet-dev
[2010/08/17 14:23:57] @ Log started by gepetto
[2010/08/17 14:23:57] @ Joined channel #puppet-dev
[2010/08/17 14:23:58] @ Topic is "http://projects.reductivelabs.com"
[2010/08/17 14:23:58] @ Topic set by ReinH!~reinh@li14-106.members.linode.com on Wed Mar 17 16:29:02 -0700 2010
[2010/08/17 14:24:00] @ Mode +cnt by bartol.freenode.net
[2010/08/17 14:41:53] @ lak joined channel #puppet-dev
[2010/08/17 14:48:14] <duritong> jamesturnbull: about template documentation: yeah I can do if this feature will work ;)
[2010/08/17 14:48:38] <duritong> jamesturnbull: I'm also a bit running out of time, as other holidays are coming soon. But I though I document it at least
[2010/08/17 14:59:01] <jamesturnbull> duritong: I added a bit of note to the wiki
[2010/08/17 14:59:04] <nasrat> cool
[2010/08/17 14:59:19] <nasrat> jamesturnbull: thanks
[2010/08/17 15:00:02] <jamesturnbull> nasrat: I hope you liked the Kool-Aid reference
[2010/08/17 15:00:18] <nasrat> ReinH: can you close all the Ready for Checkin bugs that have been merged please
[2010/08/17 15:00:51] <jamesturnbull> nasrat: hopefully that works for you - I asked him this and ReinH totally ignored me ... that's the respect I inspire in the office :)
[2010/08/17 15:01:07] <nasrat> jamesturnbull: it's fine
[2010/08/17 15:01:25] <ReinH> nasrat: yes sir, will do so
[2010/08/17 15:01:29] <ReinH> jamesturnbull: what did I ignore? :(
[2010/08/17 15:01:43] <duritong> jamesturnbull: yeah saw it, unfortunately that example doesn't work due to #4549
[2010/08/17 15:01:44] <gepetto> duritong: jamesturnbull: #4549 is http://projects.puppetlabs.com/issues/show/4549 "Puppet - Bug #4549: not all puppet functions are available in the template scope - Puppet Labs"
[2010/08/17 15:01:52] <jamesturnbull> ReinH: what nasrat wanted :) Don't worry you probably assumed I was coughing
[2010/08/17 15:02:05] <jamesturnbull> duritong: that's a weird one
[2010/08/17 15:02:16] <duritong> jamesturnbull: yeah, I agree
[2010/08/17 15:02:16] <ReinH> jamesturnbull: Well, right now I assume you're joking :p
[2010/08/17 15:02:26] <ReinH> jamesturnbull: am happy to update tickets
[2010/08/17 15:02:36] <jamesturnbull> ReinH: I was joking :)
[2010/08/17 15:02:48] <ReinH> then we have an understanding :)
[2010/08/17 15:02:57] <jamesturnbull> duritong: weird because I don't see how it could be happening
[2010/08/17 15:03:47] <duritong> jamesturnbull: me too, I looked at the code and function loading should add it, but it looks like they get lost. but why doesn't inline_template
[2010/08/17 15:04:43] <duritong> jamesturnbull: and the function really also don't work, they're not catched by method missing or so also there wouldn't even be code that would do something like that
[2010/08/17 15:07:02] <nasrat> so I see 11 open issues against 1.5.8, I'll go over tomorrow
[2010/08/17 15:09:05] <nasrat> ReinH: so I was speaking with dean/unixdaemon and I suggest we file a bug to replace the getoptlong mess and debbuging vomit inducing handling with optparse and something a little cleaner than is there
[2010/08/17 15:09:40] <jamesturnbull> duritong: I don't see anything that would stop that function loading
[2010/08/17 15:09:59] <duritong> jamesturnbull: me too
[2010/08/17 15:11:13] @ Quit: lak: Quit: lak
[2010/08/17 15:11:34] @ Quit: notbrien: Quit: notbrien
[2010/08/17 15:13:50] <ReinH> nasrat: I support this
[2010/08/17 15:14:07] <ReinH> and have some relevant experience
[2010/08/17 15:15:39] <nasrat> filed
[2010/08/17 15:16:02] <nasrat> nothing like 5 year old ruby to clean up
[2010/08/17 15:16:07] <ReinH> nasrat: irt open issues on 1.5.8, some were deferred and some may have been reassigned. Last I checked there were only 6 open 1.5.8 tickets.
[2010/08/17 15:16:24] <ReinH> nasrat: :) more common than you might think. Luckily option parsing is full of known art
[2010/08/17 15:16:52] <ReinH> nasrat: as part of the cleanup process post 1.5.8, we should look at them and figure out how to reassign
[2010/08/17 15:17:04] <nasrat> yeah
[2010/08/17 15:17:12] <nasrat> spread the pain^Wlove
[2010/08/17 15:17:20] * ReinH just realized he's holding two conversations synchronously
[2010/08/17 15:17:25] <ReinH> *syncronously
[2010/08/17 15:17:39] <ReinH> (reassign target version, I mean)
[2010/08/17 15:17:48] <ReinH> I'm going to close tickets now...
[2010/08/17 15:20:08] <ReinH> done
[2010/08/17 15:20:28] <ReinH> nasrat: ok, 5 currently open tickets marked 1.5.8
[2010/08/17 15:20:43] <nasrat> your query must not be my query
[2010/08/17 15:20:53] <ReinH> http://projects.reductivelabs.com/projects/facter/issues?fixed_version_id=40&set_filter=1&status_id=o
[2010/08/17 15:20:55] <nasrat> I click on tickets for 1.5.8 and filter status is open
[2010/08/17 15:21:18] <ReinH> status -> open; target version is Facter - 1.5.8 is my query
[2010/08/17 15:21:31] <ReinH> I got there by clicking on Roadmap and then on the "open" link for 1.5.8
[2010/08/17 15:21:55] <ReinH> nasrat: I also JUST closed the Ready for Checkin tickets, so you may need to refresh
[2010/08/17 15:24:04] <ReinH> nasrat: I'm taking the CLI cleanup ticket if that's ok.
[2010/08/17 15:24:11] <nasrat> sure
[2010/08/17 15:24:18] <nasrat> I put against 1.6.0
[2010/08/17 15:24:23] <ReinH> oh :)
[2010/08/17 15:24:27] <nasrat> if you happen to get to it then fine
[2010/08/17 15:24:38] <nasrat> just it's not essential for the release
[2010/08/17 15:24:46] <ReinH> yeah, saw, should be fine
[2010/08/17 15:24:56] <ReinH> what's our ETA for 1.6.0?
[2010/08/17 15:25:31] <nasrat> we should think about what is realistic
[2010/08/17 15:25:45] <ReinH> 28 open tickets + 5 (or possibly more) deferred from 1.5.8
[2010/08/17 15:25:53] <nasrat> dunno is a 6 month scheduled release cycle fast enough
[2010/08/17 15:26:05] <ReinH> I want to make sure I'm seeing the right number of open 1.5.8 tickets btw, can you confirm?
[2010/08/17 15:26:21] <nasrat> ack (no ticks)
[2010/08/17 15:26:56] <nasrat> ok getting late here
[2010/08/17 15:27:01] <nasrat> will head shortly
[2010/08/17 15:27:06] <ReinH> ok
[2010/08/17 15:27:10] <ReinH> this is what I see http://skitch.com/reinh/d1qe6/facter-issues-puppet-labs
[2010/08/17 15:27:43] <ReinH> irt 1.6.0, 6 months seems like a long time, but I don't have a strong estimate of our velocity
[2010/08/17 15:28:21] <ReinH> if it will really take 6 months for 1.6.0, can we release a 1.5.9 more quickly with some sensible subset of the 1.6.0 tickets?
[2010/08/17 15:28:42] <ReinH> nasrat: we can continue this later as well
[2010/08/17 15:33:43] <nasrat> yeah bed calls will chat anon.
[2010/08/17 15:57:21] @ lak joined channel #puppet-dev
[2010/08/17 16:46:16] @ Quit: jmccune: Ping timeout: 276 seconds
[2010/08/17 17:28:48] @ Quit: lak: Quit: lak
[2010/08/17 18:18:12] @ Quit: nlew: Quit: Page closed
[2010/08/17 18:24:25] @ plathrop is now known as plathrop-away
[2010/08/17 20:17:40] @ Quit: rcrowley: Quit: rcrowley
[2010/08/17 20:34:42] @ lak joined channel #puppet-dev
[2010/08/17 22:01:55] @ quit (seeya)
[2010/08/17 22:02:17] @ Joined channel #puppet-dev
[2010/08/17 22:02:17] @ Topic is "http://projects.reductivelabs.com"
[2010/08/17 22:02:17] @ Topic set by ReinH!~reinh@li14-106.members.linode.com on Wed Mar 17 16:29:02 -0700 2010
[2010/08/17 22:02:20] @ Mode +cnt by gibson.freenode.net
[2010/08/17 23:02:03] @ jmccune joined channel #puppet-dev
[2010/08/17 23:52:37] @ Quit: jmccune: Ping timeout: 246 seconds
[2010/08/17 23:53:04] <nasrat> moni

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