Issue #3939 has been reported by R.I. Pienaar aka Volcane.

----------------------------------------
Bug #3939: ALLCAPS variables arent usable in templates.
http://projects.puppetlabs.com/issues/3939

Author: R.I. Pienaar aka Volcane
Status: Unreviewed
Priority: Normal
Assigned to:
Category:
Target version:
Affected version: 0.25.5
Keywords:
Branch:

Puppet lets you set all caps variables in the manifest that you couldnt use in a template:

<pre>
$FOO = "bar"
notice "from puppet: ${FOO}"
notice inline_template("<%= FOO %>")
</pre>

Output is:

<pre>
notice: Scope(Class[main]): from puppet: bar
Failed to parse inline template: uninitialized constant Puppet::Parser::TemplateWrapper::FOO at /home/rip/test.pp:4 on node nephilim.ml.org
</pre>

Not sure if you can make the template not handle those as constants, but if not then the language shouldn't let us make all caps variables.

--
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account

Issue #3939 has been updated by James Turnbull.

Category set to parser
Status changed from Unreviewed to Accepted
Assigned to set to Jesse Wolfe
Target version set to 0.25.6

----------------------------------------
Bug #3939: ALLCAPS variables arent usable in templates.
http://projects.puppetlabs.com/issues/3939

Author: R.I. Pienaar aka Volcane
Status: Accepted
Priority: Normal
Assigned to: Jesse Wolfe
Category: parser
Target version: 0.25.6
Affected version: 0.25.5
Keywords:
Branch:

Puppet lets you set all caps variables in the manifest that you couldnt use in a template:

<pre>
$FOO = "bar"
notice "from puppet: ${FOO}"
notice inline_template("<%= FOO %>")
</pre>

Output is:

<pre>
notice: Scope(Class[main]): from puppet: bar
Failed to parse inline template: uninitialized constant Puppet::Parser::TemplateWrapper::FOO at /home/rip/test.pp:4 on node nephilim.ml.org
</pre>

Not sure if you can make the template not handle those as constants, but if not then the language shouldn't let us make all caps variables.

--
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account

Issue #3939 has been updated by James Turnbull.

Target version deleted (0.25.6)

----------------------------------------
Bug #3939: ALLCAPS variables arent usable in templates.
https://projects.puppetlabs.com/issues/3939

Author: R.I. Pienaar
Status: Accepted
Priority: Normal
Assignee: Jesse Wolfe
Category: parser
Target version:
Affected Puppet version: 0.25.5
Keywords:
Branch:

Puppet lets you set all caps variables in the manifest that you couldnt use in a template:

<pre>
$FOO = "bar"
notice "from puppet: ${FOO}"
notice inline_template("<%= FOO %>")
</pre>

Output is:

<pre>
notice: Scope(Class[main]): from puppet: bar
Failed to parse inline template: uninitialized constant Puppet::Parser::TemplateWrapper::FOO at /home/rip/test.pp:4 on node nephilim.ml.org
</pre>

Not sure if you can make the template not handle those as constants, but if not then the language shouldn't let us make all caps variables.

--
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account

Issue #3939 has been updated by Joshua Lifton.

Assignee deleted (Jesse Wolfe)

This issue was assigned to a former Puppet Labs employee. Adding back to the pool of unreviewed issues.
----------------------------------------
Bug #3939: ALLCAPS variables arent usable in templates.
https://projects.puppetlabs.com/issues/3939

Author: R.I. Pienaar
Status: Accepted
Priority: Normal
Assignee:
Category: parser
Target version:
Affected Puppet version: 0.25.5
Keywords:
Branch:

Puppet lets you set all caps variables in the manifest that you couldnt use in a template:

<pre>
$FOO = "bar"
notice "from puppet: ${FOO}"
notice inline_template("<%= FOO %>")
</pre>

Output is:

<pre>
notice: Scope(Class[main]): from puppet: bar
Failed to parse inline template: uninitialized constant Puppet::Parser::TemplateWrapper::FOO at /home/rip/test.pp:4 on node nephilim.ml.org
</pre>

Not sure if you can make the template not handle those as constants, but if not then the language shouldn't let us make all caps variables.

--
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account

Issue #3939 has been updated by Ben Hughes.

Description updated
Status changed from Accepted to Unreviewed

----------------------------------------
Bug #3939: ALLCAPS variables arent usable in templates.
https://projects.puppetlabs.com/issues/3939

Author: R.I. Pienaar
Status: Unreviewed
Priority: Normal
Assignee:
Category: parser
Target version:
Affected Puppet version: 0.25.5
Keywords:
Branch:

Puppet lets you set all caps variables in the manifest that you couldnt use in a template:

<pre>
$FOO = "bar"
notice "from puppet: ${FOO}"
notice inline_template("<%= FOO %>")
</pre>

Output is:

<pre>
notice: Scope(Class[main]): from puppet: bar
Failed to parse inline template: uninitialized constant Puppet::Parser::TemplateWrapper::FOO at /home/rip/test.pp:4 on node nephilim.ml.org
</pre>

Not sure if you can make the template not handle those as constants, but if not then the language shouldn't let us make all caps variables.

--
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account

Issue #3939 has been updated by Michael Stahnke.

Description updated
Status changed from Unreviewed to Needs Decision
Assignee set to Nigel Kersten
Affected Puppet version changed from 0.25.5 to 2.7.9

This is absolutely still a bug in 2.7.9.  

    [root@centos5-64 foo]# cat manifests/init.pp
    class foo {
    $FOO = "Hello World"
    file { "/tmp/output":
    ensure => present,
    mode => 0600,
    content => template('foo/test.erb'),
    }
    }
    [root@centos5-64 foo]# cat templates/test.erb
    <%= FOO %>
    [root@centos5-64 foo]# puppet apply --verbose --modulepath=.. -e 'include foo'
    Failed to parse template foo/test.erb: uninitialized constant Puppet::Parser::TemplateWrapper::FOO at /root/module/foo/manifests/init.pp:9 on node centos5-64.localdomain
    [root@centos5-64 foo]#

I'm not sure we should support this.  my recommended fix would be to not allow all cap variables in manifests.

----------------------------------------
Bug #3939: ALLCAPS variables arent usable in templates.
https://projects.puppetlabs.com/issues/3939

Author: R.I. Pienaar
Status: Needs Decision
Priority: Normal
Assignee: Nigel Kersten
Category: parser
Target version:
Affected Puppet version: 2.7.9
Keywords:
Branch:

Puppet lets you set all caps variables in the manifest that you couldnt use in a template:

<pre>
$FOO = "bar"
notice "from puppet: ${FOO}"
notice inline_template("<%= FOO %>")
</pre>

Output is:

<pre>
notice: Scope(Class[main]): from puppet: bar
Failed to parse inline template: uninitialized constant Puppet::Parser::TemplateWrapper::FOO at /home/rip/test.pp:4 on node nephilim.ml.org
</pre>

Not sure if you can make the template not handle those as constants, but if not then the language shouldn't let us make all caps variables.

--
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account

Issue #3939 has been updated by Michael Stahnke.

Status changed from Needs Decision to Accepted
Assignee deleted (Nigel Kersten)
Target version set to 2.7.x

Ok, so the course of action is to not allow ALLCAPS variables in the Puppet DSL.
----------------------------------------
Bug #3939: ALLCAPS variables arent usable in templates.
https://projects.puppetlabs.com/issues/3939

Author: R.I. Pienaar
Status: Accepted
Priority: Normal
Assignee:
Category: parser
Target version: 2.7.x
Affected Puppet version: 2.7.9
Keywords:
Branch:

Puppet lets you set all caps variables in the manifest that you couldnt use in a template:

<pre>
$FOO = "bar"
notice "from puppet: ${FOO}"
notice inline_template("<%= FOO %>")
</pre>

Output is:

<pre>
notice: Scope(Class[main]): from puppet: bar
Failed to parse inline template: uninitialized constant Puppet::Parser::TemplateWrapper::FOO at /home/rip/test.pp:4 on node nephilim.ml.org
</pre>

Not sure if you can make the template not handle those as constants, but if not then the language shouldn't let us make all caps variables.

--
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account

Issue #3939 has been updated by Michael Stahnke.

Target version changed from 2.7.x to Telly

----------------------------------------
Bug #3939: ALLCAPS variables arent usable in templates.
https://projects.puppetlabs.com/issues/3939

Author: R.I. Pienaar
Status: Accepted
Priority: Normal
Assignee:
Category: parser
Target version: Telly
Affected Puppet version: 2.7.9
Keywords:
Branch:

Puppet lets you set all caps variables in the manifest that you couldnt use in a template:

<pre>
$FOO = "bar"
notice "from puppet: ${FOO}"
notice inline_template("<%= FOO %>")
</pre>

Output is:

<pre>
notice: Scope(Class[main]): from puppet: bar
Failed to parse inline template: uninitialized constant Puppet::Parser::TemplateWrapper::FOO at /home/rip/test.pp:4 on node nephilim.ml.org
</pre>

Not sure if you can make the template not handle those as constants, but if not then the language shouldn't let us make all caps variables.

--
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account

Issue #3939 has been updated by Daniel Pittman.

Status changed from Accepted to Closed

`⚡ puppet apply -e '$FOO = "bar" notice(inline_template("[<%= @FOO %>]"))'`

The member variable style avoids the collision with Ruby constants.
----------------------------------------
Bug #3939: ALLCAPS variables arent usable in templates.
https://projects.puppetlabs.com/issues/3939

Author: R.I. Pienaar
Status: Closed
Priority: Normal
Assignee:
Category: parser
Target version: Telly
Affected Puppet version: 2.7.9
Keywords:
Branch:

Puppet lets you set all caps variables in the manifest that you couldnt use in a template:

<pre>
$FOO = "bar"
notice "from puppet: ${FOO}"
notice inline_template("<%= FOO %>")
</pre>

Output is:

<pre>
notice: Scope(Class[main]): from puppet: bar
Failed to parse inline template: uninitialized constant Puppet::Parser::TemplateWrapper::FOO at /home/rip/test.pp:4 on node nephilim.ml.org
</pre>

Not sure if you can make the template not handle those as constants, but if not then the language shouldn't let us make all caps variables.

--
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account

Issue #3939 has been updated by Daniel Pittman.

Target version changed from 3.X to 3.0.0

----------------------------------------
Bug #3939: ALLCAPS variables arent usable in templates.
https://projects.puppetlabs.com/issues/3939#change-62292

Author: R.I. Pienaar
Status: Closed
Priority: Normal
Assignee:
Category: parser
Target version: 3.0.0
Affected Puppet version: 2.7.9
Keywords:
Branch:

Puppet lets you set all caps variables in the manifest that you couldnt use in a template:

<pre>
$FOO = "bar"
notice "from puppet: ${FOO}"
notice inline_template("<%= FOO %>")
</pre>

Output is:

<pre>
notice: Scope(Class[main]): from puppet: bar
Failed to parse inline template: uninitialized constant Puppet::Parser::TemplateWrapper::FOO at /home/rip/test.pp:4 on node nephilim.ml.org
</pre>

Not sure if you can make the template not handle those as constants, but if not then the language shouldn't let us make all caps variables.

--
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account