Jira (PUP-8480) Puppet::Util::Execution.execute should always default failonfail to true, even if other options are set

3 views
Skip to first unread message

Melissa Stone (JIRA)

unread,
Feb 21, 2018, 12:06:03 PM2/21/18
to puppe...@googlegroups.com
Melissa Stone created an issue
 
Puppet / Task PUP-8480
Puppet::Util::Execution.execute should always default failonfail to true, even if other options are set
Issue Type: Task Task
Assignee: Unassigned
Created: 2018/02/21 9:05 AM
Fix Versions: PUP 6.0.0
Priority: Normal Normal
Reporter: Melissa Stone

Currently, failonfail defaults to true UNLESS any other options are passed in to the execute command (even if they do not modify failonfail). In this case, failonfail will switch to false. This is unexpected and should not happen. Failonfail should always be true unless explicitely set to false.

Add Comment Add Comment
 
This message was sent by Atlassian JIRA (v7.5.1#75006-sha1:7df2574)
Atlassian logo

Josh Cooper (JIRA)

unread,
Aug 3, 2018, 1:38:03 PM8/3/18
to puppe...@googlegroups.com
Josh Cooper commented on Task PUP-8480
 
Re: Puppet::Util::Execution.execute should always default failonfail to true, even if other options are set

For this ticket, I think we want to create a new API with sane semantics. There's no telling how many modules/providers we'd break changing the current API.

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

Rob Braden (JIRA)

unread,
Aug 13, 2018, 5:06:02 PM8/13/18
to puppe...@googlegroups.com

Josh Cooper (JIRA)

unread,
Sep 26, 2018, 8:54:15 PM9/26/18
to puppe...@googlegroups.com

Josh Cooper (Jira)

unread,
Jun 11, 2021, 8:10:03 PM6/11/21
to puppe...@googlegroups.com
Josh Cooper commented on Task PUP-8480
 
Re: Puppet::Util::Execution.execute should always default failonfail to true, even if other options are set

I'm going to close this as we can't really change this behavior. Better to define a new API which doesn't have this insane behavior.

This message was sent by Atlassian Jira (v8.13.2#813002-sha1:c495a97)
Atlassian logo
Reply all
Reply to author
Forward
0 new messages