Activity
From 05/27/2011 to 06/02/2011
06/03/2011
-
05:59 pm Bug #7764: Many small issues with Faces CLI short help
- Okay, first off, this is way too much for one ticket, and it will be unusable for the noise within another three post...
-
05:44 pm Bug #7624: resource purging causes audit messages
- Apply it and produce the next RC.
-
05:30 pm Bug #7770 (Needs More Information): warning: Found multiple default providers
- Using a noop provider to work around suitability checks for commands from yet-to-be installed packages, causes a warn...
-
05:25 pm Feature #2728 (Merged - Pending Release): augeas show print file changes as applicable via --show...
- Merged into 2.7.x as commit:ae8069c0afc981d27012d7e1cf6fd1f63e0b6d94
06/02/2011
-
04:30 pm Bug #7764: Many small issues with Faces CLI short help
- >> Good, useful examples make the documentation!
>
> Absolutely. Examples should live in the man page, not in the CLI... -
- > I’m not at all convinced that the plethora of possible features that people
> would use a generic ‘options’ field w... -
- Added one of the two blockers of this ticket (#7630). It is also blockedby #6992, but I can't add that because it wo...
-
- Daniel Pittman wrote:
> When I spoke to Luke about those options, his expectation would be
> that this was a single... -
03:48 pm Bug #7768 (Investigating): intermitent test failure for ticket 7117 acceptance test
- When testing for status of the puppet master server, the curl command fails with a ssl protocol error. The error is ...
-
03:43 pm Bug #7764: Many small issues with Faces CLI short help
- Daniel Pittman wrote:
> > Render format: what are the options? How do I find out?
>
> You can't. We can introsp... -
01:58 pm Bug #7759: when parsing, puppet should not output warning "You cannot collect without storeconfig...
- Dan Bode wrote:
> this was already rejected in #7078, I would be happy if it gets opened again for discussion :)
> ... -
01:55 pm Bug #7520 (Duplicate): timestamp on reports should be taken from server, not client
- This is a duplicate of #5164.
-
01:38 pm Puppet Forge Feature #7766 (Accepted): Puppet module site needs to display Forge T&Cs
-
- Looks like #7699 was the one that took.
-
- OK. I can't convince myself that this is either safe, or dangerous. After investigating it looks like this might re...
-
- You missed that we use 'unknown option' where we mean 'undocumented
option' in the templates. Probably because we ha... -
11:29 am Bug #7624: resource purging causes audit messages
- this ticket is really just a subset of #6443 which was uncovered later.
-
- puppet help
* We should be consistent about using "subcommand" versus "face". I've been thinking that we should u... -
- > Can you be a bit more specific about what’s happening that’s a problem?
I can, indeed.
> What’s the code in Comma... -
- When I spoke to Luke about those options, his expectation would be
that this was a single parameter on the package t... -
- this is a regression in 2.7.0.
Puppet spuriously logs some refresh notifications when running as noop
Given the... -
- Definitely related, and in my opinion this is the better way to go about adding the functionality requested in #2247.
-
09:36 am Feature #7762: Allow "unless" as replacement for "if !"
- I also tried a 'if not' which is slightly quicker to read than 'if !'.
-
- This ticket proposes to implement "unless" as an optional equivalent to "if !".
In the DSL we sometimes need to wr... -
- this was already rejected in #7078, I would be happy if it gets opened again for discussion :)
I also have the exa... -
- This is targeted at 2.7.0, but should be fixed in 2.6.x as well.
-
07:41 am Bug #7759 (Duplicate): when parsing, puppet should not output warning "You cannot collect without...
- puppet throws a warning if you include any files with code that exports or collects resources if you are not running ...
-
05:16 am Bug #2309: Defined resources are not evaluated immediately which can cause ordering problems
- Is this bug being actively worked on? I have a similar situation in which I need to add a condition to a template ba...
-
01:51 am Facter Bug #7038: Cannot use prtdiag in Solaris non-global zones
- Adrien Thebo wrote:
> These failures are really manifestations of #7753, which I'll also fix. But to verify - on the... -
01:23 am Puppet Dashboard Bug #7033: Failed to run rake reports:schematize
- I'm also trying to upgrade from 1.0.4 to 1.1.0. The db:migrate was successful, the schematize process halted at 93% t...
-
01:15 am Feature #2247 (In Topic Branch Pending Review): enablerepo and disablerepo for yum type
- Hello,
I have created a branch and pushed out a fix and tests to the dev mailing list.
https://github.com/nath... -
- When, using the pip provider, I get events that the same package is installed, everytime puppet runs...
-
09:20 pm Facter Bug #7038: Cannot use prtdiag in Solaris non-global zones
- These failures are really manifestations of #7753, which I'll also fix. But to verify - on the Sun T5220, prtdiag onl...
-
08:40 pm Refactor #7749: Bootstrapping Puppet in Ruby code has nasty scope cycles...
- Can you be a bit more specific about what's happening that's a problem? What's the code in CommandLine and Applicati...
-
- As evidenced by #7038, if an exception is thrown when adding a resolve to a fact, facter crashes. Instead of just fai...
-
08:02 pm Bug #7428: Puppet cert fails on 1.9.2 with wrong number of arguments
- I think that is completely acceptable. 1.9.x shouldn't be a blocker because there is a solution, use Ruby 1.8.x. Al...
-
- > Fix verified in: https://github.com/puppetlabs/puppet/commit/c295ae699ff78f3d7e4922d00dcf5d9e92c790b8
>
> Needs t... -
06:29 pm Bug #7752 (Closed): Unrecognised escape sequence
- This happend for cron as well for exec !
consider the followings
cron { "fake_test" :
command ... -
- Fix verified in: https://github.com/puppetlabs/puppet/commit/c295ae699ff78f3d7e4922d00dcf5d9e92c790b8
Needs tests? -
- While debugging other failures in bootstrapping Puppet, I discovered that we seem to create two instances of the code...
-
- So, turns out this is a result of the fix for #7717, which caused us to get run-mode handling in settings wrong in co...
-
- So, in debugging an issue using code during early startup of Puppet, I found out that we have some very strange run-s...
-
05:54 pm Puppet Dashboard Bug #7630: Add pending_count to resource_statuses in the report format
- Here comes the brain dump.
The more I work with the new terms of pending and compliant the more potential I see fo...
06/01/2011
-
- Daniel Pittman wrote:
> This is merged upstream in 796900b..80c1978 on the 2.7rc branch.
Ahem. Stage one is impl... -
- This is merged upstream in 796900b..80c1978 on the 2.7rc branch.
-
- Indeed. Thanks Peter!
Made this one the original because it has more detail. -
-
- > Being able to specify the node name used for classification on the master
> via arbitrary facts won’t address the d... -
- I think #3807 and that one are the same? Aren't they?
-
- Starting puppet master fails when installed with ruby 1.8.7
error output on puppet master node:... -
03:34 pm Feature #2128: Allow arbitrary fact as node_name identifier
- Being able to specify the node name used for classification on the master via arbitrary facts won't address the deepe...
-
- Refactor the aptitude provider to be part of the V2 packate type.
-
- Refactor the dpkg provider to be part of the V2 package type.
-
- Refactor the apt provider to be part of the V2 package type
-
- Refactor the package type & providers to clean up the code & inheritance structure, as well as to provide good exampl...
-
- Turns out that vendoring Ronn isn't within our reach: ronn depends on rdiscount to parse markdown, and that is a C ex...
-
- https://github.com/puppetlabs/puppet/commit/796900b04e234b61a77fe4b64c006f24e15a7360 merges this into 2.7rc to resolv...
-
- https://github.com/puppetlabs/puppet/compare/db25a02...0370011 merges this into the 2.7rc branch, ready for the next ...
-
11:05 am Bug #7729 (Duplicate): SSH auth keys not updated if user is changed
- Guilty as charged. Sorry.
-
- At the moment we have multiple paths in the code that lead to deprecation warnings being issued, in about equal measu...
-
10:45 am Bug #6026: Config files don't allow for comments at the end of a line anymore
- rebased the patch on 2.7.x
-
- There's also a 2.7.x rebased version at:
https://github.com/masterzen/puppet/tree/tickets/2.7.x/5777 -
- I rebased on 2.7.x
-
10:06 am Bug #7729: SSH auth keys not updated if user is changed
- Looks like #7114 to me.
-
- Sent email to Todd asking him to change the version to 2.6.9 and resubmit.
-
- Looks like it doesn't work on T5220s either:...
-
- When you pass --destdir into install.rb, it still installs the config files in /etc/puppet, no matter what destdir is...
-
- An ssh_authorized_key object does not apply correctly if the user is changed.
I had a key I was initially distribu... -
11:22 pm Puppet Dashboard Bug #7678: excessive versioned dependency on rack
- I think this is a bug in rails when using passenger.
Line 33 in vendor/rails/actionpack/lib/action_controller.rb:
... -
- If a simple file resource fails.
file { '/tmp/nonexistentdir/food':
content => 'zen'
}
you used to ... -
- I believe we have a deprecation function that would be more appropriate, but yes, that's the general idea.
-
- > I’d like us to have this done for 2.7.0, but if it’s going to block release
> for a significant time period, I’m co... -
- Given the problems with trying to test exit status on a process that can possibly be daemonized, it feels like the ri...
-
- So we seem in agreement on this. Can someone take it to the community for feedback?
What do we want to do if someo... -
- I'd like us to have this done for 2.7.0, but if it's going to block release for a significant time period, I'm comfor...
-
- Separate branch for removing the prtconf output can be found at #7726
-
-
- In a Solaris zone, facter will call prtconf, which is not available inside zones. Facter will fall back to using vmst...
05/31/2011
-
- Merged to the 2.7rc branch as 0d9eaea..db25a02 upstream.
-
03:57 pm Facter Bug #7723 (Needs Decision): virtual xenu incorrectly detected on newer kernels.
- Newer mainline linux kernels have xen included, so testing for the existence of /sys/bus/xen will now give a false po...
-
- Merged a fix for this to master in commit:97927e321d1e24554ad21b92ef4afb1bd64c0167.
Our implementation will fall b... -
- Right now we look at /etc/issue to determine operatingsystemrelease for Ubuntu, which should not be trusted. This is ...
-
02:21 pm Facter Bug #7670 (In Topic Branch Pending Review): operatingsystem fact incorrect after clear on Ubuntu
- Should fix this issue and hopefully I followed all the rules to get this accepted quickly.
https://github.com/mojo... -
01:57 pm Feature #7719 (Rejected): Modules should "just work" with apt, yum, pkgdmg and pkgadd
- # Overview #
It is difficult to write re-usable modules which install packages not available in the OS distributio... -
- Added a reference to #7718, in which we discuss the strange behaviour around default actions, and how that plays ... ...
-
- So, we have these default actions implemented on faces to support things like help. Specifically, we have the behavi...
-
- The behaviour reported by Matt is actually related to other strangeness in our command line handling, especially arou...
-
- We should totally deprecate this for Statler. Do you have any sense of how much work it is to throw deprecation warni...
-
- At the moment we have a bunch of ways to implicitly invoke `puppet apply` on the command line, mostly in the name of ...
-
- I'm ambivalent about this.
- **Hand one:** Leaving it be would serve as a sort of implicit documentation --- ever... -
- Adrien Thebo wrote:
> I was able to duplicate the prtconf output to stderr; for the other two I did not see. I belie... -
- R.I. Pienaar wrote:
> Your get() function has more or less done a clean room implementation of extlookup and come ... -
- Targeting for 2.7.1. We will get this fixed, but it's not going to block 2.7.0
-
- Can you give us a little more detail on the kinds of changes you're pushing?
-
-
- I merged this into 2.7.x as commit:9834dcdd831092464143700010ae73544eea42e8
-
- Heh. and I didn't raise these concerns in the original.... let's do it.
-
- That sounds sensible. Regardless of implementation, the feature could accept configuration information (e.g., a chec...
-
07:07 am Bug #7273: Add additional signals for restarting Puppet agent runs
- Hey,
I think the issue we have with that is we can't wait an hour for the next run in order for the changed settin... -
- Hi Nigel, remember advising me about setting key rights last week on the mailing list so nagios could use the certs? ...
-
04:17 am Bug #7714 (Accepted): RPM Provider can't determine the version of unsigned packages
- If the rpm-provider tries to get the version of a package, he uses the command "rpm -q -qf "%{NAME} %|EPOCH?{%{EPOCH}...
-
02:40 am Puppet Dashboard Bug #7068: performance issue
- the index from the issue 4524 was removed
http://projects.puppetlabs.com/issues/4524
there is no more succes fi... -
12:22 am Bug #4762: Speed of light interfering with normal puppet operation
- We have tried to evaluate puppet for use over a satellite link. This was our first try at using puppet at all, as suc...
-
11:15 pm Puppet Dashboard Feature #7687: Limit CSV export somehow to make it more consumable
- Actually, that's not really right, as I still get all the resources for a host - even if only a few resources on a ho...
-
- My stars got eaten. Servers me right for not previewing
egrep -v 'unchanged,.*,0,0,.*,.*,.*,.*,.*,.*,.*,0,0,fa... -
- One option would be a check box that said "out of sync" servers, which would be the equivalent of ignoring all the in...
05/30/2011
-
- Fixed as of the 3rd patch in the branch.
-
- See the second patch in this branch.
-
- Patch is pending review on puppet-dev. The branch contains also fixes for #5010 and #6026
-
- I had some service (iscsid) which hung for ever for some reason (missing config file), when stopping it via `service ...
-
- Matt Robinson wrote:
> In lib/puppet/network/authstore.rb line 242ish a case statement deals with certnames with @ s... -
09:09 am Bug #7709: expanded_relationship.dot should not have both containment and relationship edges
- I believe the problem description is a little off here; not factually incorrect, just framed in a way that is less ac...
05/29/2011
-
- This is fixed in commit:50756588
05/28/2011
-
- I confirm the bug, despite the code was made to support it :(
The good news is that you can expect a fix sooner than... -
- This patchset contains also the fix to #7710 which was necessary.
-
- Check the first patch
-
- +1.
-
- Since commit:06fcece75ef52168a73013eba2b8bfc50cf71c97 it isn't possible to specify a config file owner/group differen...
-
- I wrote the NIM provider and I specifically remembering that I needed the single quote there for some reason. Oh wel...
-
- Duplicate of #5521
05/27/2011
-
- # Image #
 -
- # Overview #
Running against master (2.7.0rc3-135-g520cbc0) I notice that using --graph has obvious cycles in the ... -
- Matt asked, and for now we're assuming that we are not counting or reporting on audit events outside of inspect reports.
-
- Randall Hansen wrote:
> "bootstrap, pre, main, post" makes a lot of sense to me. If we don't have a need for more s... -
- Huh.
Note you still can't stop classes being singletons by doing this :)
I'm not sure we should stop this, but ... -
- Whup, forgot we'd merged this. Pushed to 2.7rc in commit:4662f4d.
-
-
- Nigel - this might be a 2.7.x blocker...
-
-
- Thanks for the example code.
I suspect this is a case of how much power do we want to allow people to really shoot... -
- Thank you for the concise report.
-
- When I've gone to document auth.conf, fileserver.conf, and now autosign.conf, I've run into the same pattern: I inter...
-
-
- R.I. Pienaar wrote:
> Knew of that syntax but looked at the code and assumed it wouldnt work https://github.com/pupp... -
- I've definitely run into init scripts that won't "restart" when the service isn't running, and I can see that argumen...
-
11:51 am Feature #4433: Add hash support to extlookup
- site.pp:
$network_interfaces = ymllookup("network_interfaces")
define create_net_interfaces () { }
cr... -
- FYI, I found an interesting issue a few minutes ago when using the attached ymllookup program: when I imported an arr...
-
- > The second error message is obviously a bug (albeit comical)
>
> $ puppet help test
> err: Could not find Puppet... -
- > Branch changed from
> https://github.com/duritong/puppet/tree/ticket/2.6.x/1886 to
> https://github.com/duritong/... -
- I reviewed the code, and it is just fine.
-
- Nigel, clearly a logic bug, and I know why; I didn't filter duplicates out in the code that generates this text. Can...
-
10:46 am Feature #4433: Add hash support to extlookup
- I am stil in favor for the extension to CSV, but mostly I am in favor of making extlookup backend pluggable so we can...
-
- I don't think I actually agree with `puppet` defaulting to `puppet apply` "by convention". The convention we're movi...
-
- R.I. Pienaar wrote:
> Nigel Kersten wrote:
> > What are the actual decisions we need to make here? I'm in favor of ... -
10:43 am Bug #7690: Puppet::Indirector::Terminus.terminus_classes(indirection) blows up if ActiveRecord is...
- Additionally, merged into 2.7.x in commit:d0592fa
-
- The specific bug is now resolved on 2.6.x and 2.7rc, and should merge upwards during our normal cycle. Once that has...
-
- This was merged to 2.7rc in commit:2aa9f2f
-
- Nigel Kersten wrote:
> What are the actual decisions we need to make here? I'm in favor of accepting this, are there... -
-
- The root cause of #5318 is that we handle the set of known resource types for an environment very strangely in our co...
-
- The change to address this was added in commit:fb5f859cf4a89042a1768b6cbc2dbfc43da49c99 which is in the current 2.7.0...
-
- Knew of that syntax but looked at the code and assumed it wouldnt work https://github.com/puppetlabs/puppet/blob/mast...
-
- This entered 2.6.x at https://github.com/puppetlabs/puppet/commit/75e2764d15de6cf1dee923019f579f436d5b1587
This en... -
- Daniel, I'd like us to close off this bug and set up another one for the more complex solution. Is that possible?
-
- A minimal fix for this is now pushed to the 2.7rc branch, as well as the 2.6.x branch, for the next two releases.
... -
- You know you can override this in the config file?
`lastrunfile = /var/lib/puppet/state/last_run_summary.yaml { owne... -
- What are the actual decisions we need to make here? I'm in favor of accepting this, are there any design questions st...
-
- "bootstrap, pre, main, post" makes a lot of sense to me. If we don't have a need for more stages than that, let's no...
-
- The text in the bug description is the output at the time of ticket creation. The first comment is my proposed output.
-
- I'm not entirely clear from the ticket description if the the text is the proposed output when an action doesn't exis...
-
- I spoke to Randall about this and he also thinks the estimate at the time (he recalls was made with Daniel) was based...
-
- -1 to the numeric stages. Descriptive much better as an aide memoire - also stages don't support that syntax anyway :)
-
- Let me play devils advocate against myself for a moment :)
What about simply supporting numeric stages?
`Stage[... -
- Dan Bode wrote:
> bootstrap, pre, main, post ?
>
> +1
>
> I would really like to stop there though :)
+1 -
- Patch for 2.7.x up @ https://github.com/duritong/puppet/tree/feature/2.7.x/1886
-
- It also lists actions twice -> #7700
-
- Bleh, Dan was a bit faster. I think this is a duplicate of #7699 - sorry for the double.
-
- Using today's 2.7.x branch I cannot generate any certificates, even if I pass the --ca-location option (see #7701). P...
-
- On today's 2.7.x branch, when trying to generate a certificate with the certificate faces, I need to pass --ca-locati...
-
- On today's 2.7.x branch I get the following:...
-
01:15 am Bug #7165: If services hasn’t started yet, where possible update the file first and then start th...
- I'm also seeing this (2.6.6 in this case). It's causing problems as some init scripts (tomcat6 from jpackage, I'm lo...
-
- given the following face action:...
Also available in: Atom