[Puppet - Bug #3939] (Unreviewed) ALLCAPS variables arent usable in templates.

[red...@puppetlabs.com]

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

---

Bug #3939: ALLCAPS variables arent usable in templates.

• 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:

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

Output is:

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

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

[red...@puppetlabs.com]

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.

• 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:

[tic...@puppetlabs.com]

Issue #3939 has been updated by James Turnbull.

• Target version deleted (0.25.6)

---

Bug #3939: ALLCAPS variables arent usable in templates.

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

[tic...@puppetlabs.com]

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.

• Author: R.I. Pienaar

• Status: Accepted
• Priority: Normal
• Assignee:

• Category: parser
• Target version:
• Affected Puppet version: 0.25.5
• Keywords:
• Branch:

[tic...@puppetlabs.com]

Issue #3939 has been updated by Ben Hughes.

• Description updated
• Status changed from Accepted to Unreviewed

---

Bug #3939: ALLCAPS variables arent usable in templates.

• Author: R.I. Pienaar
• Status: Unreviewed

• Priority: Normal
• Assignee:
• Category: parser
• Target version:

• Affected Puppet version: 0.25.5
• Keywords:
• Branch:

[tic...@puppetlabs.com]

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.

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

[tic...@puppetlabs.com]

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.

• Author: R.I. Pienaar
• Status: Accepted

• Priority: Normal
• Assignee:
• Category: parser

• Target version: 2.7.x
• Affected Puppet version: 2.7.9
• Keywords:
• Branch:

[tic...@puppetlabs.com]

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.

• Author: R.I. Pienaar

• Status: Accepted
• Priority: Normal
• Assignee:
• Category: parser

• Target version: Telly
• Affected Puppet version: 2.7.9
• Keywords:
• Branch:

[tic...@puppetlabs.com]

Issue #3939 has been updated by Daniel Pittman.

• Status changed from Accepted to Closed

Michael Stahnke wrote:

Ok, so the course of action is to not allow ALLCAPS variables in the Puppet DSL.

Actually, I think they should be available in the new style in templates:

⚡ 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.

• Author: R.I. Pienaar
• Status: Closed

• Priority: Normal
• Assignee:
• Category: parser
• Target version: Telly
• Affected Puppet version: 2.7.9

• Keywords:
• Branch:

[tic...@puppetlabs.com]

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.

• Author: R.I. Pienaar

• Status: Closed
• Priority: Normal
• Assignee:
• Category: parser

• Target version: 3.0.0
• Affected Puppet version: 2.7.9
• Keywords:
• Branch: