Puppet users - Jun 2012 - error after upgrading from 2.7.14-2puppetlabs1 to 2.7.17-1puppetlabs1

Hello all -

For the last few weeks I''ve been running puppet 2.7.14-2 from the
puppetlabs apt repo. This is on Ubuntu 12.04 LTS, x64. Things have
been going great, no issues (at least none related to puppet
itself...).

I upgraded to 2.7.17-1 this morning, again from the official
puppetlabs apt repository, and after doing this, started getting the
following syntax error during puppet agent runs:

err: Could not retrieve catalog from remote server: Error 400 on
SERVER: Could not parse for environment production: Syntax error at
''-''; expected ''}'' at
/etc/puppet/manifests/classes/server-classes.pp:36 on node
host.example.com

Initially, I thought that I had pushed a bad config, so I reverted to
a previous known-good commit from my puppet git repo, but saw the same
error. I then down-graded back to 2.7.14-2 and the problem went away.
I looked through the puppet change logs, and didn''t see anything that
looked like it should have caused this.

Here are the contents of the file referred to in the error message:

https://gist.github.com/3006908

Do you have any ideas what could be causing this?

Thank you,
Erik

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

Comments inline

On Wednesday, June 27, 2012 4:24:44 PM UTC-5, Erik Anderson
wrote:
>
> Hello all - 
>
> For the last few weeks I''ve been running puppet 2.7.14-2 from the 
> puppetlabs apt repo. This is on Ubuntu 12.04 LTS, x64. Things have 
> been going great, no issues (at least none related to puppet 
> itself...). 
>
> I upgraded to 2.7.17-1 this morning, again from the official 
> puppetlabs apt repository, and after doing this, started getting the 
> following syntax error during puppet agent runs: 
>
> err: Could not retrieve catalog from remote server: Error 400 on 
> SERVER: Could not parse for environment production: Syntax error at 
> ''-''; expected ''}'' at 
> /etc/puppet/manifests/classes/server-classes.pp:36 on node 
> host.example.com 
>
>
Dashes ''-'' are not allowed in variable and class names.

We were faced with this recently.

We wound up rolling back and then refactoring all  our code, using 
puppet-lint as a guide and to help find all the potential issues.

The dashes have been semi-supoported for a short while, and depreciated for 
most of 2.7.x

I had originally thought that 3.X would be the change where it went from 
warning to error, but it looks like they pushed this out with 2.7.17 for 
whatever reason.


Initially, I thought that I had pushed a bad config, so I reverted to
> a previous known-good commit from my puppet git repo, but saw the same 
> error. I then down-graded back to 2.7.14-2 and the problem went away. 
> I looked through the puppet change logs, and didn''t see anything
that
> looked like it should have caused this. 
>
> Here are the contents of the file referred to in the error message: 
>
> https://gist.github.com/3006908 
>
> Do you have any ideas what could be causing this? 
>
> Thank you, 
> Erik 
>
-- 
You received this message because you are subscribed to the Google Groups
"Puppet Users" group.
To view this discussion on the web visit
https://groups.google.com/d/msg/puppet-users/-/reVBikTSfEkJ.
To post to this group, send email to puppet-users@googlegroups.com.
To unsubscribe from this group, send email to
puppet-users+unsubscribe@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/puppet-users?hl=en.

On Wed, Jun 27, 2012 at 3:36 PM, llowder@oreillyauto.com
<llowder@oreillyauto.com> wrote:
> The dashes have been semi-supoported for a short while, and depreciated for
> most of 2.7.x
Dashes in variable names started working again in 2.7.3.  According to
Daniel Pittman, they shouldn''t have started working again.  We looked
at the documented behavior and decided to change Puppet to match the
documentation.  This change went into the 2.7.x branch and was
released in 2.7.17.

This is what happened and why it happened.

Now...  Forcing our users to refactor all of their manifests is
something I''m not so keen on.  Particularly in a minor release.

Do you think we should continue allowing hyphens in variables in 2.7
and only "fix" this to error out in 3.0?
> I had originally thought that 3.X would be the change where it went from
> warning to error, but it looks like they pushed this out with 2.7.17 for
> whatever reason.
It was never a deprecation warning as far as I can tell.  2.6.16
errors out, as does 2.6.7 and 2.6.12.  2.7.0 and 2.7.1 did error out.
2.7.3 didn''t error out for whatever reason and this was the point we
accidentally regressed to allowing hyphens in variable names.  This
allowance remained all the way through 2.7.14.  In 2.7.16 we "fixed"
it again to error out with hyphens in variable names.  This remained
in 2.7.17.

So, I think the story is...  Don''t use hyphens in variable names (and
class names).  We accidentally allowed them from 2.7.3 through 2.7.14
but we''ve gone back to it being an error.

-Jeff

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

On Wed, Jun 27, 2012 at 5:36 PM, llowder@oreillyauto.com
<llowder@oreillyauto.com> wrote:
> Dashes ''-'' are not allowed in variable and class names.
>
> We were faced with this recently.
>
> We wound up rolling back and then refactoring all  our code, using
> puppet-lint as a guide and to help find all the potential issues.
>
> The dashes have been semi-supoported for a short while, and depreciated for
> most of 2.7.x
>
> I had originally thought that 3.X would be the change where it went from
> warning to error, but it looks like they pushed this out with 2.7.17 for
> whatever reason.
Ahh, excellent. Thanks for the heads up.

It''s odd that this change happened in a minor version upgrade, but at
least I know now that I''m not going crazy.

Luckily I''m very early on in our puppet deployment, so re-factoring
variable names won''t be all that big of a deal.

-Erik

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

On Wed, Jun 27, 2012 at 7:00 PM, Jeff McCune <jeff@puppetlabs.com>
wrote:
> So, I think the story is...  Don''t use hyphens in variable names
(and
> class names).  We accidentally allowed them from 2.7.3 through 2.7.14
> but we''ve gone back to it being an error.
Understood, Jeff. Thanks for the clarification.

-Erik

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

On 28 June 2012 02:00, Jeff McCune <jeff@puppetlabs.com>
wrote:
> On Wed, Jun 27, 2012 at 3:36 PM, llowder@oreillyauto.com
>> I had originally thought that 3.X would be the change where it went
from
>> warning to error, but it looks like they pushed this out with 2.7.17
for
>> whatever reason.
>
> It was never a deprecation warning as far as I can tell.  2.6.16
> errors out, as does 2.6.7 and 2.6.12.  2.7.0 and 2.7.1 did error out.
> 2.7.3 didn''t error out for whatever reason and this was the point
we
> accidentally regressed to allowing hyphens in variable names.  This
> allowance remained all the way through 2.7.14.  In 2.7.16 we
"fixed"
> it again to error out with hyphens in variable names.  This remained
> in 2.7.17.
>
> So, I think the story is...  Don''t use hyphens in variable names
(and
> class names).  We accidentally allowed them from 2.7.3 through 2.7.14
> but we''ve gone back to it being an error.
>
But it is only dashes in variables that produce errors in 2.7.16+, right?
I know they are deprecated in class names as well, but they have
worked in them for the most part since 0.24.x releases at least AFAIK.
Ofc if they aren''t allowed in variable names you shouldn''t be
able to
refer to variables in classes with dashes, like $class-name::variable.

We unfortunately still have some classes with dashes in the names but
no variables, so I want to know if it is safe to upgrade to 2.7.17?

-- 
Erik Dalén

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

----- Original Message -----
> From: "Erik Dalén" <erik.gustav.dalen@gmail.com>
> To: puppet-users@googlegroups.com
> Sent: Thursday, June 28, 2012 9:42:43 AM
> Subject: Re: [Puppet Users] Re: error after upgrading from
2.7.14-2puppetlabs1 to 2.7.17-1puppetlabs1
> 
> On 28 June 2012 02:00, Jeff McCune <jeff@puppetlabs.com> wrote:
> > On Wed, Jun 27, 2012 at 3:36 PM, llowder@oreillyauto.com
> >> I had originally thought that 3.X would be the change where it
> >> went from
> >> warning to error, but it looks like they pushed this out with
> >> 2.7.17 for
> >> whatever reason.
> >
> > It was never a deprecation warning as far as I can tell.  2.6.16
> > errors out, as does 2.6.7 and 2.6.12.  2.7.0 and 2.7.1 did error
> > out.
> > 2.7.3 didn''t error out for whatever reason and this was the
point
> > we
> > accidentally regressed to allowing hyphens in variable names.  This
> > allowance remained all the way through 2.7.14.  In 2.7.16 we
> > "fixed"
> > it again to error out with hyphens in variable names.  This
> > remained
> > in 2.7.17.
> >
> > So, I think the story is...  Don''t use hyphens in variable
names
> > (and
> > class names).  We accidentally allowed them from 2.7.3 through
> > 2.7.14
> > but we''ve gone back to it being an error.
> >
> 
> But it is only dashes in variables that produce errors in 2.7.16+,
> right? I know they are deprecated in class names as well, but they have
> worked in them for the most part since 0.24.x releases at least
> AFAIK. Ofc if they aren''t allowed in variable names you
shouldn''t be able to
> refer to variables in classes with dashes, like
> $class-name::variable.
> 
> We unfortunately still have some classes with dashes in the names but
> no variables, so I want to know if it is safe to upgrade to 2.7.17?
It does unfortunately still allow - in class names, I think the fix for
this was half baked as you couldn''t access variables in those classes
from outside of the class

docs says:

"Class names, module names, and the names of defined and custom resource
types should be restricted to lowercase alphanumeric characters and
underscores, and should begin with a lowercase letter; that is, they
should match the expression [a-z][a-z0-9_]*. Although some names that
violate these restrictions currently work, using them is not
recommended."

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

Hi, 

On Thu, 28 Jun 2012 10:42:43 +0200
Erik Dalén <erik.gustav.dalen@gmail.com> wrote:
> We unfortunately still have some classes with dashes in the names but
> no variables, so I want to know if it is safe to upgrade to 2.7.17?
For us at least classes like this still work with .17. I should say
though that these are all very simple classes (describing packages
typically), most aren''t even parameterized.

Cheers,

Steph

-- 
Steph Gosling <steph@chuci.org>

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