On Thursday 18 Feb 2010 16:51:38 Andrew Schulman wrote:
> > Hello Andrew,
> >
> > I'm getting a syntax error when trying to use the ssh::auth module
> > ("Syntax error at '{'; expected '}' at .../auth.pp:111").  It looks like
> > it's something to do with parameters after a collection, i.e. line 111
> > is:
> >
> > if $ensure { Ssh_auth_key_client <| title == $title |> { ensure =>
> > $ensure } }
> >
> > I've had a quick look in puppet documentation and can't find reference
> > (e.g. in language tutorial, etc.) to that syntax.  I also searched the
> > mailing lists and found something in unrelated discussion on dev list
> > from which I gathered that it is a valid syntax.  Is it something new
> > (i.e. newer than 0.24.8)?
> >
> > Should ssh::auth work on 0.24.8?
> 
> Hi Michael.  The reference for this syntax is at
> http://reductivelabs.com/trac/puppet/wiki/VirtualResources (linked from
> http://reductivelabs.com/trac/puppet/wiki/LanguageTutorial#virtual-resource
> s). The docs say that virtual resources are supported since 0.20.0, but
>  they don't say whether the "collection" syntax ( <|...|> ) is supported
>  since then-- maybe it's more recent.  I'm running 0.25.4, so it's possible
>  that you need a more recent version of Puppet than 0.24.8 to use
>  ssh::auth.

OK

> 
> If someone can clarify what Puppet version is required for the collection
> syntax, I'll add it to the list of requirements of ssh::auth.
> 
> > BTW, I gather the above should be equivalent to:
> >
> > if $ensure { Ssh_auth_key_client[$title] { ensure => $ensure } }
> >
> > yes?
> 
> No, that's not correct.  To realize a virtual resource, you have to use
> either the collection syntax, as in ssh::auth; or the realize function:
> 
>   realize Ssh_auth_key_client[$title]
> 
> which, according to the docs, doesn't seem to support adding parameters,
> e.g.
> 
>   realize Ssh_auth_key_client[$title]{ ensure => $ensure }
> 
> However, in my testing I found that the above does in fact work, but only
> if the parameters inside the brackets haven't been set yet.  That makes it
> not useful for ssh::auth, where the purpose is to override defaults.  It's
> also undocumented.

Even if collection syntax with parameter override also realizes the resource I 
don't think it matters because in e.g. ssh::auth::client you have:

if $ensure { Ssh_auth_key_client <| title = $title |> { ensure => $ensure } }
...
realize Ssh_auth_key_client[$title]

So even if parameters are overridden first and the resource is realized 
separately it still works.

> 
> If the collection syntax makes ssh::auth unusable in Puppet 0.24, I'd
> gladly change it to something more compatible.  But because of the
> limitation of the realize function, I can't see a way to do that right now.

I did manage to make necessary adjustments to make it work in 0.24.8 but the 
syntax above turned out to be a smaller issue :)  replacing it with the one I 
suggested above worked.

The bigger issue was the =~ operator introduced in 0.25.  I did manage to 
replace it with regsubst but it's ugly (usually one line in 0.25 translates to 
~4 in 0.24.8 using regsubst).  I can attach the file with my changes if you 
want to have a look at it, but it's your call if you want to keep 
compatibility at a price of ... well a bit more convoluted code ;)

Did 0.25 ever land in EPEL?  I really need to have a look at upgrading 
sometime.

> 
> Andrew.
> 

-- 
Michael Gliwinski
Henderson Group Information Services
9-11 Hightown Avenue, Newtownabby, BT36 4RT
Phone: 028 9034 3319

**********************************************************************************************
The information in this email is confidential and may be legally privileged.  
It is intended solely for the addressee and access to the email by anyone else 
is unauthorised.
If you are not the intended recipient, any disclosure, copying, distribution or 
any action taken or omitted to be taken in reliance on it, is prohibited and 
may be unlawful.
When addressed to our clients, any opinions or advice contained in this e-mail 
are subject to the terms and conditions expressed  in the governing client 
engagement leter or contract.
If you have received this email in error please notify 
[email protected]

John Henderson (Holdings) Ltd
Registered office: 9 Hightown Avenue, Mallusk, County Antrim, Northern Ireland, 
BT36 4RT.
Registered in Northern Ireland
Registration Number NI010588
Vat No.: 814 6399 12
*********************************************************************************

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Users" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-users?hl=en.

Reply via email to