Jira (PUP-10615) Add facts and vars back to PAL ScriptCompiler scope

0 views
Skip to first unread message

Tom Beech (Jira)

unread,
Aug 10, 2020, 4:58:02 PM8/10/20
to puppe...@googlegroups.com
Tom Beech created an issue
 
Puppet / Bug PUP-10615
Add facts and vars back to PAL ScriptCompiler scope
Issue Type: Bug Bug
Assignee: Tom Beech
Created: 2020/08/10 1:57 PM
Priority: Normal Normal
Reporter: Tom Beech

Puppet Version: 6.15.0 - 6.17.0

As part of work on Bolt, PUP-10397 removed vars and facts for the current node from PAL's script compiler. This resulted in a breaking change to a public API, affecting downstream projects that rely on PAL. The script compiler should make the current node's vars and facts available again.

Desired Behavior:

**PAL's script compiler should have access to vars and facts for the node running the script.

Actual Behavior:

PAL's script compiler does not have access to the vars and facts for the node running the script, causing project's that rely on the PAL API to lose access to this data.

Add Comment Add Comment
 
This message was sent by Atlassian Jira (v8.5.2#805002-sha1:a66f935)
Atlassian logo

Dorin Pleava (Jira)

unread,
Aug 19, 2020, 8:51:04 AM8/19/20
to puppe...@googlegroups.com
Dorin Pleava updated an issue
Change By: Dorin Pleava
Fix Version/s: PUP 6.18.0

Tom Beech (Jira)

unread,
Aug 19, 2020, 12:20:04 PM8/19/20
to puppe...@googlegroups.com
Tom Beech updated an issue

PAL's script compiler now sets local facts for the compiler's scope. A previous change to PAL removed setting facts, which resulted in breaking changes to the script compiler API. Now, setting local facts in the script compiler's scope by setting the 'set_local_facts' option to 'false' for 'with_script_compiler'. This option is set to 'true' by default.

Change By: Tom Beech
Release Notes: Bug Fix
Release Notes Summary: Added local facts back to PAL's script compiler scope.

Tom Beech (Jira)

unread,
Aug 19, 2020, 12:21:03 PM8/19/20
to puppe...@googlegroups.com
Tom Beech updated an issue
Change By: Tom Beech
Comment:
PAL's script compiler now sets local facts for the compiler's scope. A previous change to PAL removed setting facts, which resulted in breaking changes to the script compiler API. Now, setting local facts in the script compiler's scope by setting the 'set_local_facts' option to 'false' for 'with_script_compiler'. This option is set to 'true' by default.

Claire Cadman (Jira)

unread,
Aug 20, 2020, 4:57:03 AM8/20/20
to puppe...@googlegroups.com
Reply all
Reply to author
Forward
0 new messages