Skip to content
This repository was archived by the owner on Feb 18, 2022. It is now read-only.

Example of using in module data to avoid PUP-6453 #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

adreyer
Copy link
Contributor

@adreyer adreyer commented Aug 10, 2016

PUP-6453 prevents us from using class params of classes included in a
component in that components produced resources. This is a workaround
showing how you can use in module data to accomplish the same thing.

Instead of referencing the class param in the produce statement you can
use lookup to lookup the value of that class param. This won't work if a
user overrides the bound value of that param with an ENC or resource
like declaration.

PUP-6453 prevents us from using class params of classes included in a
component in that components produced resources. This is a workaround
showing how you can use in module data to accomplish the same thing.

Instead of referencing the class param in the produce statement you can
use lookup to lookup the value of that class param. This won't work if a
user overrides the bound value of that param with an ENC or resource
like declaration.
@adreyer adreyer changed the title Example of using in module data to avoid ordering issues Example of using in module data to avoid PUP-6453 Aug 10, 2016
@@ -7,6 +7,7 @@
"source": "https://github.com/puppetlabs/puppetlabs-wordpress_app",
"project_page": "https://github.com/puppetlabs/puppetlabs-wordpress_app",
"issues_url": "https://tickets.puppetlabs.com/browse/MODULES",
"data_provider": "hiera",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this do? I'm a lookup newb.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants