Jira (BOLT-1154) Evaluate guard conditions in bolt plan functions

8 views
Skip to first unread message

Cas Donoghue (JIRA)

unread,
Feb 27, 2019, 4:21:03 PM2/27/19
to puppe...@googlegroups.com
Cas Donoghue created an issue
 
Puppet Task Runner / Improvement BOLT-1154
Evaluate guard conditions in bolt plan functions
Issue Type: Improvement Improvement
Assignee: Unassigned
Created: 2019/02/27 1:20 PM
Priority: Normal Normal
Reporter: Cas Donoghue

BOLT-1131 involved defining explicitly which plan functions are available in apply blocks. The TASK_OPERATION_NOT_SUPPORTED_WHEN_COMPILING issue from the Puppet::Pops::Issues module (https://github.com/puppetlabs/puppet/blob/aaebc6cd3769c937f7da59320daadfa68ca8da23/lib/puppet/pops/issues.rb#L904-L914) had been raised when guarding incompatible plan functions in apply blocks. The work in https://github.com/puppetlabs/bolt/pull/876 introduced a pattern for defining puppet issues in bolt and replaced the TASK_OPERATION_NOT_SUPPORTED_WHEN_COMPILING.

There is another guard for plan functions that will raise a similar issue: TASK_MISSING_BOLT. It seems like there are places where the guard logic is unnecessary. We should evaluate each plan function that can raise TASK_MISSING_BOLT and determine if it is necessary and whether to replace it with an issue defined in bolt. We should also evaluate removing the issues from puppet that have been replaced in bolt.

Add Comment Add Comment
 
This message was sent by Atlassian JIRA (v7.7.1#77002-sha1:e75ca93)
Atlassian logo

David Kramer (JIRA)

unread,
Apr 16, 2019, 12:47:03 PM4/16/19
to puppe...@googlegroups.com
David Kramer updated an issue
Change By: David Kramer
Sprint: Bolt Ready for Grooming Kanban

Cas Donoghue (JIRA)

unread,
May 1, 2019, 7:35:01 PM5/1/19
to puppe...@googlegroups.com

Cas Donoghue (JIRA)

unread,
May 8, 2019, 3:13:04 PM5/8/19
to puppe...@googlegroups.com

Cas Donoghue (JIRA)

unread,
May 8, 2019, 3:14:02 PM5/8/19
to puppe...@googlegroups.com

Melissa Amos (JIRA)

unread,
May 13, 2019, 12:49:03 PM5/13/19
to puppe...@googlegroups.com
Reply all
Reply to author
Forward
0 new messages