Jira (FACT-3073) Inconsistencies to set Facter::Core::Execution.execute timeout

14 views
Skip to first unread message

Romain Tartière

unread,
Sep 9, 2021, 1:22:03 AM9/9/21
to puppe...@googlegroups.com
Romain Tartière created an issue
 
Facter / Bug FACT-3073
Inconsistencies to set Facter::Core::Execution.execute timeout
Issue Type: Bug Bug
Assignee: Unassigned
Components: Facter 4
Created: 2021/09/08 10:21 PM
Priority: Normal Normal
Reporter: Romain Tartière

The Facter::Core::Execution.execute method accept an option hash to control execution.

The documentation tell to use "time_limit" to set a max duration for the command to complete, but the code actually use "limit" to set this.

I happen that the previous version of Facter used "timeout" for this duration.

Add Comment Add Comment
 
This message was sent by Atlassian Jira (v8.13.2#813002-sha1:c495a97)
Atlassian logo

Gabriel Nagy (Jira)

unread,
Sep 14, 2021, 5:35:03 AM9/14/21
to puppe...@googlegroups.com

Gabriel Nagy (Jira)

unread,
Sep 14, 2021, 5:40:02 AM9/14/21
to puppe...@googlegroups.com
Gabriel Nagy updated an issue
 
Change By: Gabriel Nagy
Release Notes: Bug Fix
Release Notes Summary: Fix the mismatch of `Facter::Core::Execution.execute` arguments between Facter 3 and Facter 4. The `.execute` method now accepts a `timeout` option like Facter 3 does, and warns when unsupported options are passed in. (Fix contributed by community members https://github.com/smortex and https://github.com/johanfleury)

Ciprian Badescu (Jira)

unread,
Sep 15, 2021, 3:45:03 AM9/15/21
to puppe...@googlegroups.com

Ciprian Badescu (Jira)

unread,
Sep 15, 2021, 3:45:03 AM9/15/21
to puppe...@googlegroups.com

Ciprian Badescu (Jira)

unread,
Sep 15, 2021, 3:46:03 AM9/15/21
to puppe...@googlegroups.com

Luchian Nemes (Jira)

unread,
Sep 16, 2021, 7:20:02 AM9/16/21
to puppe...@googlegroups.com

Dorin Pleava (Jira)

unread,
Sep 28, 2021, 4:06:03 AM9/28/21
to puppe...@googlegroups.com

Claire Cadman (Jira)

unread,
Oct 5, 2021, 8:24:02 AM10/5/21
to puppe...@googlegroups.com
Reply all
Reply to author
Forward
0 new messages