Jira (PUP-9699) Add pluggable API / enabler for recording all keys looked up during compilation

0 views
Skip to first unread message

Henrik Lindberg (JIRA)

unread,
May 13, 2019, 2:37:03 PM5/13/19
to puppe...@googlegroups.com
Henrik Lindberg assigned an issue to Henrik Lindberg
 
Puppet / New Feature PUP-9699
Add pluggable API / enabler for recording all keys looked up during compilation
Change By: Henrik Lindberg
Assignee: Henrik Lindberg
Add Comment Add Comment
 
This message was sent by Atlassian JIRA (v7.7.1#77002-sha1:e75ca93)
Atlassian logo

Henrik Lindberg (JIRA)

unread,
May 13, 2019, 2:37:03 PM5/13/19
to puppe...@googlegroups.com
Henrik Lindberg created an issue
Issue Type: New Feature New Feature
Assignee: Unassigned
Created: 2019/05/13 11:36 AM
Priority: Normal Normal
Reporter: Henrik Lindberg

In order to perform impact analysis of changes to hiera data Puppet should have the ability to record all keys being looked up during a compilation.

To do this, lookup should obtain the key :lookup_key_recorder from the context and if this key is bound, it must be to an object that responds to the message record(key).

Thus, the contract for someone that wants to process these callbacks is to give the puppet compiler a context where :lookup_key_recorder is bound and then do what is wanted in that specific bound recorder.

There may be opportunity to coalesce this feature with another ticket where it is requested that puppet should be able to give profiling per backend/key. If that is wanted, the API must include both a "begin" and an "end" call for every key. It is expected that a first implementation does not take that into account.

There will be no call to the record(key method for the meta key lookup_options. From an impact analysis standpoint; as if there is any lookup of any key, there will also be a lookup of the lookup_options - so an analyzer should handle that key implicitly. (It may also want to do special processing of the content of lookup_options since it is possible to compute that a change of lookup_options for a key that is never looked up has no impact).

Maggie Dreyer (JIRA)

unread,
Jul 8, 2019, 6:57:04 PM7/8/19
to puppe...@googlegroups.com

Alexandru Popa (JIRA)

unread,
Aug 14, 2019, 4:35:03 AM8/14/19
to puppe...@googlegroups.com

Alexandru Popa (JIRA)

unread,
Aug 14, 2019, 4:36:03 AM8/14/19
to puppe...@googlegroups.com

Henrik Lindberg (JIRA)

unread,
Aug 14, 2019, 4:46:03 AM8/14/19
to puppe...@googlegroups.com

Alexandru Popa I moved it to "Ready for CI" - don't know what the actual state is beyond that (it was merged 19 days ago). Ping Maggie Dreyer.

Alexandru Popa (JIRA)

unread,
Aug 14, 2019, 9:24:02 AM8/14/19
to puppe...@googlegroups.com

Hey Henrik Lindberg the commit was promoted to master and CI passed with this commit. This is ok to be closed

Reply all
Reply to author
Forward
0 new messages