Jira (PUP-9509) Puppet 3 API functions should be loaded from thepluginsync location as part of the cache system loader

20 views
Skip to first unread message

Glenn Sarti (JIRA)

unread,
Feb 22, 2019, 4:09:03 AM2/22/19
to puppe...@googlegroups.com
Glenn Sarti updated an issue
 
Puppet / Bug PUP-9509
Puppet 3 API functions should be loaded from the pluginsync location as part of the cache system loader
Change By: Glenn Sarti
Summary: {brief summary Puppet 3 API functions should be loaded from the pluginsync location as part of issue} the cache system loader
Add Comment Add Comment
 
This message was sent by Atlassian JIRA (v7.7.1#77002-sha1:e75ca93)
Atlassian logo

Glenn Sarti (JIRA)

unread,
Feb 22, 2019, 4:11:03 AM2/22/19
to puppe...@googlegroups.com
Glenn Sarti updated an issue
*Puppet Version:* 6.3.0
*Puppet Server Version:* N/A
*OS Name/Version:* Windows 10 - 1803 Ruby 2.5

Describe your issue in as much detail as possible...
Describe steps to reproduce...

*Desired Behavior:*
Puppet 3 API functions that are plugin sync'd should be loaded from the cache libdir

*Actual Behavior:*
Puppet 3 API functions are missing

Previously in PUP-6964 (https://github.com/puppetlabs/puppet/pull/7019) the system loader was modified to to be able to load Puppet 3 API functions;

of note - https://github.com/puppetlabs/puppet/pull/7019/files#diff-83a8daf215ff39828e0e3758c2e5a866R34

See how the system loader has loadables of {
{ [:func_4x, :func_3x, :datatype]} }

However when the system cache loader was introduced in PUP-9035 (https://github.com/puppetlabs/puppet/commit/7404ee14745c78bd4a3173c79cac198809faea74 ) the loadables specified did not include the Puppet 3 API functions.

of note - https://github.com/puppetlabs/puppet/pull/6963/files#diff-83a8daf215ff39828e0e3758c2e5a866R30

See how the cache loader has loadables of {
{ [:func_4x, :datatype]} }

The comment at https://github.com/puppetlabs/puppet/pull/6963/files#diff-83a8daf215ff39828e0e3758c2e5a866R22 says they should be the same but is clear they are not.

Also note that the acceptance tests do not test for this;
https://github.com/puppetlabs/puppet/pull/6963/files#diff-d3791ca879b2c6c0586dd8974a7bec0eR25

That is just a Puppet 4 API location

Jorie Tappa (JIRA)

unread,
Feb 25, 2019, 12:53:04 PM2/25/19
to puppe...@googlegroups.com

Jorie Tappa (JIRA)

unread,
Feb 25, 2019, 12:54:03 PM2/25/19
to puppe...@googlegroups.com

Henrik Lindberg (JIRA)

unread,
May 9, 2019, 10:24:03 AM5/9/19
to puppe...@googlegroups.com
Henrik Lindberg commented on Bug PUP-9509
 
Re: Puppet 3 API functions should be loaded from the pluginsync location as part of the cache system loader

Earlier it was questionable if 3.x functions should be loaded at all as there are many 3.x functions that has unsafe implementations (adding methods to Scope, or every Object). As we now have safety checks when loading 3.x functions this is less of a concern.

Adding 3.x loading as Glenn Sarti shows should be easy and then "just work".

Josh Cooper (JIRA)

unread,
Oct 29, 2019, 10:38:03 PM10/29/19
to puppe...@googlegroups.com
Josh Cooper commented on Bug PUP-9509

Merged to master in 49926022c9 and 8dd3bbe88e.

Passed CI in a0e7c84236

Josh Cooper (JIRA)

unread,
Oct 29, 2019, 10:38:03 PM10/29/19
to puppe...@googlegroups.com

Josh Cooper (JIRA)

unread,
Oct 29, 2019, 10:38:03 PM10/29/19
to puppe...@googlegroups.com

Josh Cooper (JIRA)

unread,
Oct 29, 2019, 10:38:04 PM10/29/19
to puppe...@googlegroups.com

Josh Cooper (JIRA)

unread,
Oct 29, 2019, 10:39:03 PM10/29/19
to puppe...@googlegroups.com

Josh Cooper (JIRA)

unread,
Oct 29, 2019, 10:39:03 PM10/29/19
to puppe...@googlegroups.com

Glenn Sarti (JIRA)

unread,
Oct 29, 2019, 11:10:02 PM10/29/19
to puppe...@googlegroups.com

Glenn Sarti (JIRA)

unread,
Oct 29, 2019, 11:10:03 PM10/29/19
to puppe...@googlegroups.com
Glenn Sarti updated an issue
 
Change By: Glenn Sarti
Release Notes Summary: Legacy Ruby Functions, also known as Version 3 Functions, will now be available from the cached lib directory.  Previously only Version functions were available.  Note that functions that exist in both Version 3 and Version 4, will still use the Version 4 implementation.
Release Notes: Bug Fix

Heston Hoffman (JIRA)

unread,
Nov 18, 2019, 12:46:04 PM11/18/19
to puppe...@googlegroups.com
Reply all
Reply to author
Forward
0 new messages