for external service': fail => $fail }
service {'local_service':
require => Runtime_fail['quorum not reached for external service']
}
}
>
> I was thinking it would be a catalog compile service as opposed to a
> client side operation to minimize dis
This is tracked in jira:
> https://tickets.puppetlabs.com/browse/MODULES-1982
>
> This pattern has poked up a few different places. As the PR says, it has
> shown up in the monogodb module and the puppetdb module. I know that
> Michael Chapman added something like this to his O
On Fri, Nov 28, 2014 at 3:08 PM, Kylo Ginsberg wrote:
> On Fri, Nov 28, 2014 at 8:30 AM, Dan Bode wrote:
>
>> On Tue, Nov 25, 2014 at 5:25 PM, Trevor Vaughan
>> wrote:
>>
>>> Hi All,
>>>
>>> I've got a couple of issues running rspec-puppe
On Tue, Nov 25, 2014 at 5:25 PM, Trevor Vaughan
wrote:
> Hi All,
>
> I've got a couple of issues running rspec-puppet tests under Ruby 2 that I
> was hoping someone could shed some light on.
>
> First, under Ruby 2, any validation functions in the code appear to be
> getting parsed *after* the in
On Thu, Dec 12, 2013 at 2:08 AM, Markus Burger
wrote:
> Hi,
>
> On 11-12-2013 10:51:08, Dan Bode wrote:
> > Hi all,
> >
> > I had a bit of time to research the existing device code to see if I can
> > use it for an integration with two specific use cases:
> &g
On Thu, Dec 12, 2013 at 1:46 AM, Nan Liu wrote:
> On Wed, Dec 11, 2013 at 8:51 AM, Dan Bode wrote:
>>
>> I had a bit of time to research the existing device code to see if I can
>> use it for an integration with two specific use cases:
>>
>
> I'm not sure
Hi Wolf,
see if this works for you
https://github.com/ripienaar/puppet-catalog-diff
On Mon, Oct 21, 2013 at 12:25 PM, Wolf Noble wrote:
> Hi there,
>
> I realize this isn't quite a dev question, but I'm going to ask anyways :)
>
> my situation is that I'm trying to refactor a large module bas
On Fri, Oct 11, 2013 at 11:09 AM, Eric Sorenson <
eric.soren...@puppetlabs.com> wrote:
>
> Thanks to everyone who kicked the tires on the experimental data in
> modules feature included in Puppet 3.3.0. We got a lot of feedback, some
> cool proof-of-concept modules, and a definitive conclusion to
We took this conversation to #puppet-openstack yesterday to discuss further.
*The problem:*
To summarize, with a Modulefile, we are able to specify release ranges:
1.2.x
We are looking for a similar capability for .fixtures to use for unit
testing.
Not being able to express the same in .fixt
I've done it before and didn't have any problems.
On Wed, Aug 21, 2013 at 11:15 AM, Trevor Vaughan wrote:
> This probably won't do what you want in all cases as it will probably be
> manifest parse order dependent.
>
I'm not sure what part you are saying would be parse order dependent.
Referenc
HI Ikka,
thanks for the note.
This project has its own mailing list (puppet-openst...@puppetlabs.com) for
future reference.
On Tue, Mar 5, 2013 at 7:28 AM, Ilkka Tengvall wrote:
> I noticed puppetlabs/openstack and puppetlabs/cinder sets the cinder
> volume group inconsistently. It allows sett
On Mon, Feb 25, 2013 at 10:13 AM, David Stewart
wrote:
> Hi,
>
> I am new to puppet and have hit a problem in developing a module.
> I have defined a Type and Provider, but as the Provider is implemented in
> Ruby (i.e. a '.rb' file) I cannot leverage Puppet style constructs, e.g. if
> 'create' is
On Mon, Oct 8, 2012 at 7:56 AM, Axel B wrote:
> Hello readers,
>
> I have already asked this question in Puppet Users, but so far without
> working solution. Maybe I have more luck in here :) . I have developed a
> custom SVN type (based on http://is.gd/W0cIMS among others), and I guess
> it's fa
On Wed, Oct 3, 2012 at 4:23 PM, Luke Kanies wrote:
> On Oct 3, 2012, at 3:53 PM, Dan Bode wrote:
>
> > Hi Puppet devs,
> >
> > I am trying to use a parent provider from another module.
> >
> > I put the following code in the provider that wants to inher
Hi Puppet devs,
I am trying to use a parent provider from another module.
I put the following code in the provider that wants to inherit from the
parent:
Puppet::Type.type(:ini_setting)#.providers
Puppet::Type.type(:keystone_config).provide(
:ini_setting,
:parent => Puppet::Type::Ini_settin
Hi Happy Puppet Developers,
I have a few questions about the device management code.
1. What is the recommended way to retrieve the device
connection information from a provider? Is it:
Puppet::Util::NetworkDevice.current
2. Is it possible to have more settings in device.conf than just url?
Fo
On Fri, May 4, 2012 at 2:03 PM, Philip Brown wrote:
> Hmm, i guess I slightly misunderstood: my ensure routine should return
> false or true, i guess.
>
> But I'm still missing info :(
> In my trivial example below, the "ensure" routine gets called and prints
> out its debug line. But the other r
On Fri, May 4, 2012 at 1:46 PM, Philip Brown wrote:
>
>
> On Friday, May 4, 2012 1:29:19 PM UTC-7, Luke Kanies wrote:
>
>> On May 4, 2012, at 11:08 AM, Philip Brown wrote:
>>
>>
>>
>> I dont see how to send some kind of message to the logs, "hey, something
>> changed, but I'm fixing it".
>>
into account the data that is going to be present at compile time.
>
> On May 1, 2012, at 9:16 PM, Dan Bode wrote:
>
> So, I dopped into the debugger with:
>
> $x = inline_template("<%= require 'ruby-debug';debugger;puts 'foo' %>"
>
>
ion of my question is rather or not nil can be treated the
same as undefined in Puppet.
On Tue, May 1, 2012 at 9:06 PM, Dan Bode wrote:
> Hi all,
>
> I have run into an example where Puppet behaves exactly like I hoped it
> would. I am a little concerned that it may just be a coincidence
Hi all,
I have run into an example where Puppet behaves exactly like I hoped it
would. I am a little concerned that it may just be a coincidence that it
happens to work like this. My question is rather or not the following is
specified behavior that I can take advantage of (I may take advantage of
+1
On Mon, Apr 9, 2012 at 2:09 PM, Michael Stahnke wrote:
> Since our move to github for pull requests and patches, the usefulness
> of puppet-dev has declined significantly. puppet-dev used to be a
> great list for development discussion of puppet and the ecosystem
> around it. With the informa
hosting our internal
> diablo deployment. But we plan to move to essex asap. I'm happy to
> help, so if someone has some efforts in progress, I'd rather
> contribute to that than do work off in my little corner.
>
> -Isaac
>
> On Mar 11, 10:02 pm, Dan Bode wrote:
Hi Lee,
Glad to here that you are interested in collaborating.
On Fri, Mar 9, 2012 at 10:06 AM, Lee Thompson wrote:
> We've been developing puppet manifests for the Essex release of
> OpenStack including nova, keystone, glance and swift.
Did you fork the exiting modules?
https://github.com/pu
You could use the resource_type indirection to query all of the resource
types and select the ones of type node.
This will only query the nodes that were specified in manifests (which is
quite different from all of the nodes that have checked in)
Below is some example code:
require "puppet/face"
This is kind of in a broken state at the moment, but its the same concept as
what you are describing (actually if you revert to a previous commit, it
should work)
https://github.com/bodepd/puppet-ec2
On Mon, Sep 12, 2011 at 12:58 PM, Nan Liu wrote:
> On Sun, Sep 11, 2011 at 8:09 PM, Rob Terhaar
In Puppet, it is not possible to reassign hash
values.
This function allows a reasonable way to perform
hash munging in Puppet.
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
lib/puppet/parser/functions/merge.rb| 28
spec/unit/puppet/parser/functions
It is difficult to use existance of keys in a hash
as a boolean condition in Puppet (see #8705)
This function provides a working solution until
the underlying issue in Puppet can be resolved.
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
lib/puppet/parser/functions/has_key.rb
Changed the type name from append_line to
whole_line.
After feedback that having a type with a verb in
the name was confusing.
Signed-off-by: Dan Bode
---
.../provider/{append_line => whole_line}/ruby.rb |2 +-
lib/puppet/type/{append_line.rb => whole_line.rb}
Changed the type name from append_line to
whole_line.
After feedback that having a type with a verb in
the name was confusing.
---
lib/puppet/provider/append_line/ruby.rb| 15 ---
lib/puppet/provider/whole_line/ruby.rb | 15 +++
lib/puppet/type/append_line.rb
On Wed, Jul 27, 2011 at 10:00 AM, Ken Barber wrote:
> So I managed to make some of this work in core. I've got a very basic
> patch ... its not what one would call complete but I want to make sure
> I'm going in the right direction:
>
>
> https://github.com/kbarber/puppet/commit/74d32f3d8c565567f
methods.
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
lib/puppet/cloudpack.rb |2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/lib/puppet/cloudpack.rb b/lib/puppet/cloudpack.rb
index b2d27b8..38c1796 100644
--- a/lib/puppet/cloudpack.rb
+++ b/lib/puppet
Renamed the master_source script to
indicate that it is only intended to
be used for development/testing.
Added more logic for installing the
dashboard for testing.
Signed-off-by: Dan Bode
---
.../{master_source.erb => master_source_dev.erb} | 35 +++-
1 files changed,
There were some timing issues where net/ssh could
get stuck in a loop when connecting to ubuntu.
This has been resolved by specifying a 10 second
timeout on the method call that tests the initial
connection.
Signed-off-by: Dan Bode
---
lib/puppet/cloudpack.rb | 14 +-
1 files
Previously success was indicated with PE specific
messages.
This could be confusing for users installing
FOSS with cloud-provisoner.
Signed-off-by: Dan Bode
---
lib/puppet/cloudpack.rb |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/lib/puppet/cloudpack.rb b/lib
been removed.
Signed-off-by: Dan Bode
---
lib/puppet/cloudpack.rb |9 -
spec/unit/puppet/face/node/bootstrap_spec.rb |6 --
spec/unit/puppet/face/node/classify_spec.rb | 13 +++--
spec/unit/puppet/face/node/init_spec.rb |7 ---
4
please disregard the first email in the thread. I decided to clean up the
wording of the unit tests and resubmitted.
On Tue, Jul 19, 2011 at 11:40 AM, Dan Bode wrote:
> The certificate generate action was previously
> attepting to save the csr twice.
>
> This patch resolves t
The certificate generate action was previously
attepting to save the csr twice.
This patch resolves the issue and eliminates the
resulting error message.
Signed-off-by: Dan Bode
---
Local-branch: issue/2.7.x/7637_creates_csr_twice
lib/puppet/face/certificate.rb |1 -
spec/unit/face
The certificate generate action was previously
attepting to save the csr twice.
This patch resolves the issue and eliminates the
resulting error message.
Signed-off-by: Dan Bode
---
Local-branch: issue/2.7.x/7637_creates_csr_twice
lib/puppet/face/certificate.rb |1 -
spec/unit/face
To allow for easier testing.
This script installs a puppet master from source
and authorizes the localhost (based on hostname
where puppet node install is executed) to be
autosigned and to be authorized to sign
certificates.
Signed-off-by: Dan Bode
---
lib/puppet/cloudpack/scripts
Install script execution should support authorized
super users.
This patch pre-pends sudo to the install_command
when the login option is not root.
This commit also adds some basic unit testing of
the install command
Signed-off-by: Dan Bode
---
lib/puppet/cloudpack.rb|1
This allows install_scripts access to the
public dns name of the server.
Used in the master installation script
to supply public_dns_name as a certdnsname
Signed-off-by: Dan Bode
---
lib/puppet/cloudpack.rb|1 +
spec/unit/puppet/cloudpack_spec.rb |7 +++
2 files changed
It makes sense for puppet-version to refer to
revisions when installing puppet from source.
I have modified the regex to allow it to take
revisions of the form \d+\.\d+\.x and master
I also added unit tests for validation of the
puppet-version option.
Signed-off-by: Dan Bode
---
lib/puppet
On Sun, Jul 17, 2011 at 10:23 PM, Luke Kanies wrote:
> On Jul 17, 2011, at 4:53 PM, Tim Sharpe wrote:
>
> Hi Dan,
>
> Yeah, I'm focusing mainly on getting a clean RSpec API at the moment. What
> it looks like currently is
>
> require 'rspec-puppet'
>
> describe 'sysctl', :type => :defin
> (probably using rspec) to test that node with facts applying
> define with params creates a catalogue with the following
> resources...
>
> Basically, I want to generate the catalogue for the following and then
> inspect it's contents.
>
> node "foo"
- initial tests for upload_payloads function
- test that files are uploaded when specified
- verify validation of combination of params.
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
spec/unit/puppet/cloudpack_spec.rb | 41
1 files changed, 41
To make testing easier.
Moved option assignments out of method for options
that are used by other functions.
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
lib/puppet/cloudpack.rb | 13 +++--
1 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/lib/puppet
Verifying that given a valid script, the compile
template method can create a compiled temp file.
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
spec/unit/puppet/cloudpack_spec.rb | 22 ++
1 files changed, 22 insertions(+), 0 deletions(-)
diff --git a/spec/unit
To make testing easier.
clarified the variables required for running the
installation script.
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
lib/puppet/cloudpack.rb |7 +--
1 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/lib/puppet/cloudpack.rb b/lib/puppet
Add before section to build out a tempfile and
mock connections object that can be used
in the tests.
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
spec/unit/puppet/cloudpack_spec.rb | 10 +-
1 files changed, 9 insertions(+), 1 deletions(-)
diff --git a/spec/unit/puppet
Moved code related to files that are scp'ed to its
own method to make testing easier.
Moved validation of related options to this method
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
lib/puppet/cloudpack.rb | 17 ++---
1 files changed, 10 insertions(+), 7 dele
Install changed to upload for clarity.
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
spec/unit/puppet/cloudpack_spec.rb |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/spec/unit/puppet/cloudpack_spec.rb
b/spec/unit/puppet/cloudpack_spec.rb
index 4f88e87
Previously it was not so clear that
Signed-off-by: Dan Bode
Reviewed-by: Jeff McCune
---
spec/unit/puppet/cloudpack_spec.rb |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/spec/unit/puppet/cloudpack_spec.rb
b/spec/unit/puppet/cloudpack_spec.rb
index 107dd07..f33dc06
This is part of a refactor to simplify testing.
Splitting the ssh connection code into its own
method.
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
lib/puppet/cloudpack.rb | 10 ++
1 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/lib/puppet/cloudpack.rb b/lib
Required in order to identify the ssh
exceptions.
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
lib/puppet/cloudpack.rb |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/lib/puppet/cloudpack.rb b/lib/puppet/cloudpack.rb
index 3fce683..0ad3f1e 100644
--- a/lib
Adding initial tests of ssh connection code.
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
spec/unit/puppet/cloudpack_spec.rb | 19 +++
1 files changed, 19 insertions(+), 0 deletions(-)
diff --git a/spec/unit/puppet/cloudpack_spec.rb
b/spec/unit/puppet
Was incorrectly referring to keyfile as a symbol
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
lib/puppet/cloudpack.rb |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/lib/puppet/cloudpack.rb b/lib/puppet/cloudpack.rb
index 1d94f68..3fce683 100644
--- a/lib
Stubbed sleep to make it retry and fail faster
raised an AuthenticationFailed (to emulate a more
likely failure)
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
spec/unit/puppet/cloudpack_spec.rb |5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/spec/unit
Template compilation expects this variable to be
set.
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
lib/puppet/cloudpack.rb |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/lib/puppet/cloudpack.rb b/lib/puppet/cloudpack.rb
index 63f3815..1d94f68 100644
--- a/lib
Signed-off-by: Dan Bode
---
spec/unit/puppet/cloudpack_spec.rb | 27 +++
1 files changed, 27 insertions(+), 0 deletions(-)
diff --git a/spec/unit/puppet/cloudpack_spec.rb
b/spec/unit/puppet/cloudpack_spec.rb
index 27aa94d..160cb9a 100644
--- a/spec/unit/puppet
Initial test for run_install_script.
tests that the expected commands are called with
and without sudo.
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
spec/unit/puppet/cloudpack_spec.rb | 15 +++
1 files changed, 15 insertions(+), 0 deletions(-)
diff --git a/spec/unit
As part of ths refactor, the code for creating
a tmp_dir has changed.
- ssh is now connections[:ssh]
- the tmp dir should be determined in the install
method, because the path is required by multiple
other methods.
Reviewed-by: Jeff McCune
Signed-off-by: Dan Bode
---
lib/puppet/cloudpack.rb
behavior.
regards,
Dan
On Wed, Jul 13, 2011 at 3:43 PM, Dan Bode wrote:
> The problem was caused by the fact that the
> options method returns a list of options that
> treated the aliases as seperate options.
>
> The fix is to only maintain a list of options
> and not add
The problem was caused by the fact that the
options method returns a list of options that
treated the aliases as seperate options.
The fix is to only maintain a list of options
and not add all aliases to the options list.
Signed-off-by: Dan Bode
---
Local-branch: issue/2.7.x/7699
lib/puppet
passed after this
patch)
If anything else is expecting a list of all of the face aliases, it should
probably use
@options_hash.keys
On Wed, Jul 6, 2011 at 10:04 PM, Dan Bode wrote:
> puppet help was reprinting every option once
> for every alias that is had.
>
> This fix involves only
puppet help was reprinting every option once
for every alias that is had.
This fix involves only storing the option.name
in the @options instance var for both face and
actions options. The @options_hash still
maintains the list of options and aliases as its
keys.
Signed-off-by: Dan Bode
This allows the puppet man command
to succeed if an option exists without
documentation.
It was previously failing with 'err: private
method `gsub' called for nil:NilClass;
Signed-off-by: Dan Bode
---
Local-branch: issue/master/8273
lib/puppet/face/help/man.erb |2 +-
1 files
On Thu, Jun 23, 2011 at 3:33 PM, Carl Caum wrote:
> I'm working on a provider and noticed my getter method for parameters can't
> pass back a hash or embedded array. I need the user to pass in a hash for a
> parameter. If the getter method can't return a hash, I don't see how I can
> see if the
Hi,
I just wanted to clarify something from this email. Although the best way to
get traction for a ticket in the future will be voting in the ticketing
system, people are always welcome to highlight particular tickets in order
to solicit votes.
With that in mind, a quick list of things that I ha
Hi all,
I am trying to understand how self.instances is supposed to work.
I have implemented it as below (which I tried to construct based on examples
I found in puppet's source) :
def self.instances
rabbitmqctl('list_users').split(/\n/)[1..-2].collect do |line|
if line =~ /^(\S+)\s+
On Thu, Jun 9, 2011 at 9:41 AM, markus wrote:
>
> > >
> > > Huge +1
>
unmeasurable +1i
> > >
> >
> > Because you've reviewed it, or because you think it's a good idea?
> >
>
> One convention that works (for me at least) is to use complex numbers
> where the real part is for the actual code an
On Wed, Jun 8, 2011 at 1:43 PM, Adrien Thebo wrote:
> Added osfamily fact to determine if a given operating system is a
> derivative of a common operating system.
>
Huge +1
> Signed-off-by: Adrien Thebo
> ---
> Local-branch: ticket/master/6792
> lib/facter/osfamily.rb | 24
(namely
puppet resouce) to break, I am still looking into those.
https://github.com/bodepd/puppet/tree/mastercompositenamevar
On Mon, May 23, 2011 at 2:25 PM, Stefan Schulte <
stefan.schu...@taunusstein.net> wrote:
> On Mon, May 23, 2011 at 11:45:45AM -0700, Dan Bode wrote:
> > This is
On Mon, May 23, 2011 at 10:45 PM, Luke Kanies wrote:
> On May 23, 2011, at 11:45 AM, Dan Bode wrote:
>
> > This is an experimental set of patches that I wrote for composite
> namevars
> > to work with parsedfile and to support purging.
> >
> > This is not ready t
can you point me to a branch I can pull from?
On Mon, May 23, 2011 at 3:12 PM, Stefan Schulte <
stefan.schu...@taunusstein.net> wrote:
> On Mon, May 23, 2011 at 02:39:40PM -0700, Dan Bode wrote:
> > :name is special (and hardcoded) in several parts of the Puppet code.
> Could
:name is special (and hardcoded) in several parts of the Puppet code. Could
you try using some other name that :name and see if it resolves those
issues?
On Mon, May 23, 2011 at 2:25 PM, Stefan Schulte <
stefan.schu...@taunusstein.net> wrote:
> On Mon, May 23, 2011 at 11:45:45AM -0700,
On Mon, May 23, 2011 at 11:45 AM, Dan Bode wrote:
> For types with composite namevars, hash2resource
> should use namevar_join to populate tilte with
> namevar.
>
> Signed-off-by: Dan Bode
> ---
> Local-branch: issue/master/7629
> lib/puppet/type.rb |7 +
On Mon, May 23, 2011 at 11:45 AM, Dan Bode wrote:
> - for composite namevars to support purging
> and 'puppet resource' quiery all, it needs to
> be possible to map from the composite namevars
> to a single id.
>
> - the basic idea is that authors of types need to
On Mon, May 23, 2011 at 11:45 AM, Dan Bode wrote:
> - this ensures that resources are unique based on
> composite namevars.
> - I need to review why this is required..
>
> Signed-off-by: Dan Bode
> ---
> Local-branch: issue/master/7629
> lib/puppet/transaction.r
its been like that for at least a year:
http://projects.puppetlabs.com/issues/4049
On Mon, May 23, 2011 at 11:53 AM, Jacob Helwig wrote:
> So, I'm working on #7188 and thought "hey, it would be great if we had
> built-in checks that these required parameters were actually specified".
> Especial
- Introduction of composite namevars places
extra restrictions on resource title format
(in order to guarantee uniqueness
it should be based on all namevars)
- Puppet was not explicitly handling the case
where the title is invalid
Signed-off-by: Dan Bode
---
Local-branch: issue/master
.
Signed-off-by: Dan Bode
---
Local-branch: issue/master/7629
lib/puppet/provider/parsedfile.rb |7 ++-
1 files changed, 6 insertions(+), 1 deletions(-)
diff --git a/lib/puppet/provider/parsedfile.rb
b/lib/puppet/provider/parsedfile.rb
index 75a215f..522930e 100755
--- a/lib/puppet
- it was previously hardcoded to use :name param
which does not work for composite namevars.
Signed-off-by: Dan Bode
---
Local-branch: issue/master/7629
lib/puppet/provider.rb |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/lib/puppet/provider.rb b/lib/puppet
For types with composite namevars, hash2resource
should use namevar_join to populate tilte with
namevar.
Signed-off-by: Dan Bode
---
Local-branch: issue/master/7629
lib/puppet/type.rb |7 +--
1 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/lib/puppet/type.rb b/lib/puppet
This is an experimental set of patches that I wrote for composite namevars
to work with parsedfile and to support purging.
This is not ready to be merged, I would just like to get some input on the
following:
- does this work (for people who have pending composite namevar work)
- will this b
- this ensures that resources are unique based on
composite namevars.
- I need to review why this is required..
Signed-off-by: Dan Bode
---
Local-branch: issue/master/7629
lib/puppet/transaction.rb |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/lib/puppet
ned-off-by: Dan Bode
---
Local-branch: issue/master/7629
lib/puppet/type.rb |8
1 files changed, 8 insertions(+), 0 deletions(-)
diff --git a/lib/puppet/type.rb b/lib/puppet/type.rb
index 1dbf124..dc4764e 100644
--- a/lib/puppet/type.rb
+++ b/lib/puppet/type.rb
@@ -219,6 +219,14 @@
Signed-off-by: Dan Bode
---
Local-branch: issue/master/7629
lib/puppet/type.rb |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/lib/puppet/type.rb b/lib/puppet/type.rb
index dc4764e..e62a764 100644
--- a/lib/puppet/type.rb
+++ b/lib/puppet/type.rb
@@ -893,8 +893,7
I just finished something very similar:
def self.title_join(hash)
"#{hash[:domain]}/#{hash[:type]}/#{hash[:item]}"
end
where the default case was:
+ def self.title_join(hash)
+case key_attributes.length
+when 0,1; hash[:name]
+else
+ Puppet.warning('you should specify
-- Forwarded message --
From: Dan Bode
Date: Sun, May 22, 2011 at 3:02 PM
Subject: questions regarding #5605
To: stefan.schu...@taunusstein.net
Hi Stefan,
I recently was trying to used parsedfile with composite namevars, so I, of
course ran across all of your tickets/patches
On Sun, May 22, 2011 at 4:42 PM, markus wrote:
>
> > It actually seems to get a little sticky for purging, below I am
> > outlining how it works, I was just curious if anyone has spent time
> > thinking about this use case before and how it should work:
> >
> > As far as I can tell, the title_pat
Hi all,
I have been working on getting composite namevars to work with parsedfile.
It actually seems to get a little sticky for purging, below I am outlining
how it works, I was just curious if anyone has spent time thinking about
this use case before and how it should work:
As far as I can tell
does this resolve:
http://projects.puppetlabs.com/issues/6650
?
On Mon, Mar 28, 2011 at 10:49 AM, Luke Kanies wrote:
> This commit series significantly improves interacting with the list
> of Resource Types over the REST api (or locally).
>
> The first patch simplifies the JSON form of the typ
On Thu, Mar 17, 2011 at 10:53 AM, Daniel Pittman wrote:
> On Thu, Mar 17, 2011 at 02:22, Dan Bode wrote:
>
> > This function allows you to dynamically generate resources,
> > passing them as a hash to the create_resources function.
> >
> > This was originally writt
On Thu, Mar 17, 2011 at 10:08 AM, Luke Kanies wrote:
> On Mar 17, 2011, at 8:35 AM, Dan Bode wrote:
>
>
>
> On Thu, Mar 17, 2011 at 8:26 AM, Luke Kanies wrote:
>
>> On Mar 17, 2011, at 6:53 AM, Nigel Kersten wrote:
>>
>> > On Thu, Mar 17, 2011 at 2:22 A
On Thu, Mar 17, 2011 at 8:26 AM, Luke Kanies wrote:
> On Mar 17, 2011, at 6:53 AM, Nigel Kersten wrote:
>
> > On Thu, Mar 17, 2011 at 2:22 AM, Dan Bode wrote:
> >> This function allows you to dynamically generate resources,
> >> passing them as a hash to
On Thu, Mar 17, 2011 at 6:53 AM, Nigel Kersten wrote:
> On Thu, Mar 17, 2011 at 2:22 AM, Dan Bode wrote:
> > This function allows you to dynamically generate resources,
> > passing them as a hash to the create_resources function.
>
> One of the bits of feedback we're
ermine how webserver instances are deployed to nodes
by updating the YAML files.
Signed-off-by: Dan Bode
---
lib/puppet/parser/functions/create_resources.rb| 47 +++
.../unit/parser/functions/create_resources_spec.rb | 135
2 files changed, 182 insertions(+), 0
Hi Nick (long time :) )
On Tue, Mar 1, 2011 at 3:14 AM, Nick Moffitt wrote:
> I mean to write a lint checker for my manifests, but I am not a ruby
> developer to any degree of proficiency. What I think I would like is
> some kind of serialization of the AST for a given .pp file, which I can
> t
On Thu, Feb 24, 2011 at 6:57 PM, Jesse Wolfe wrote:
> >
> > 1. when files updating due to pluginsync
> > 2. any new files where source is managed
>
> Actually, I suspect that these are all manifestations of the same bug.
> If you have time, try out 2.6.next and see if you still see either of
> th
1 - 100 of 161 matches
Mail list logo
