Jira (PUP-6026) Audit Windows::Error.new for additional eaten exceptions

2 views
Skip to first unread message

Ethan Brown (JIRA)

unread,
Mar 8, 2016, 11:53:02 PM3/8/16
to puppe...@googlegroups.com
Ethan Brown created an issue
 
Puppet / Task PUP-6026
Audit Windows::Error.new for additional eaten exceptions
Issue Type: Task Task
Affects Versions: PUP 4.3.2
Assignee: Unassigned
Created: 2016/03/08 8:52 PM
Fix Versions: PUP 4.4.1
Priority: Normal Normal
Reporter: Ethan Brown

In PUP-6025 it was discovered that we've been silently and expensively eating instances of Windows::Error being thrown, that were failing because a Windows API is being used incorrectly. Since this was discovered accidentally, I think it makes sense to do a quick audit, by instrumenting the code with some rudimentary logging, to see if there are other cases like this.

The instrumentation doesn't need to be much more complex than adding something like this to the initializer of the class at https://github.com/puppetlabs/puppet/blob/master/lib/puppet/util/windows/error.rb#L16:

open('c:/win-error.txt', 'a') { |f| f << "#{@code}: #{message}\n" }

Further exploration could add the callers details so we have a better idea of where the errors are originating to see if there are any patterns.

The net result of this ticket should be new tickets.

Add Comment Add Comment
 
This message was sent by Atlassian JIRA (v6.4.12#64027-sha1:e3691cc)
Atlassian logo

Steve Barlow (JIRA)

unread,
Mar 9, 2016, 1:53:05 PM3/9/16
to puppe...@googlegroups.com
Steve Barlow updated an issue
Change By: Steve Barlow
Sprint: Windows 2016- 03 04 - 23 06

Kylo Ginsberg (JIRA)

unread,
Mar 18, 2016, 6:04:03 PM3/18/16
to puppe...@googlegroups.com
Kylo Ginsberg updated an issue
Change By: Kylo Ginsberg
Fix Version/s: PUP 4.4.1
Fix Version/s: PUP 4.4.2

Craig Gomes (JIRA)

unread,
Mar 21, 2016, 4:46:04 PM3/21/16
to puppe...@googlegroups.com
Craig Gomes updated an issue
Change By: Craig Gomes
Sprint: Windows 2016- 04 05 - 06 04

Steve Barlow (JIRA)

unread,
Mar 28, 2016, 4:54:05 PM3/28/16
to puppe...@googlegroups.com
Steve Barlow updated an issue
Change By: Steve Barlow
Fix Version/s: PUP 4.4.2
Fix Version/s: PUP 4.x
This message was sent by Atlassian JIRA (v6.4.13#64028-sha1:b7939e9)
Atlassian logo

Steve Barlow (JIRA)

unread,
Apr 20, 2016, 1:41:07 PM4/20/16
to puppe...@googlegroups.com
Steve Barlow updated an issue
Change By: Steve Barlow
Sprint: Windows 2016-05- 04 18

Steve Barlow (JIRA)

unread,
May 4, 2016, 1:35:10 PM5/4/16
to puppe...@googlegroups.com
Steve Barlow updated an issue
Change By: Steve Barlow
Sprint: Windows 2016- 05 06 - 18 01

Steve Barlow (JIRA)

unread,
May 9, 2016, 4:57:07 PM5/9/16
to puppe...@googlegroups.com
Steve Barlow updated an issue
Change By: Steve Barlow
Sprint: Windows 2016-06- 01 15

Kenaz Kwa (JIRA)

unread,
May 23, 2016, 4:47:10 PM5/23/16
to puppe...@googlegroups.com
Kenaz Kwa updated an issue
Change By: Kenaz Kwa
Sprint: Windows 2016-06- 15 29

Craig Gomes (JIRA)

unread,
Jun 6, 2016, 5:16:05 PM6/6/16
to puppe...@googlegroups.com
Craig Gomes updated an issue
Change By: Craig Gomes
Sprint: Windows 2016-06-29

Kenaz Kwa (JIRA)

unread,
Aug 29, 2016, 7:22:26 PM8/29/16
to puppe...@googlegroups.com
Kenaz Kwa updated an issue
Change By: Kenaz Kwa
Team: Agent & Platform Support

Ethan Brown (JIRA)

unread,
May 16, 2017, 2:35:03 PM5/16/17
to puppe...@googlegroups.com
Ethan Brown updated an issue
Change By: Ethan Brown
Fix Version/s: PUP 4.y
Fix Version/s: PUP 5.7
This message was sent by Atlassian JIRA (v6.4.14#64029-sha1:ae256fe)
Atlassian logo

Ethan Brown (JIRA)

unread,
May 16, 2017, 2:35:04 PM5/16/17
to puppe...@googlegroups.com
Ethan Brown updated an issue
Change By: Ethan Brown
Fix Version/s: PUP 5.7
Fix Version/s: PUP 5.y

Sean McDonald (JIRA)

unread,
May 16, 2017, 2:35:05 PM5/16/17
to puppe...@googlegroups.com

Josh Cooper (Jira)

unread,
Jun 4, 2021, 1:29:02 AM6/4/21
to puppe...@googlegroups.com
Josh Cooper updated an issue
Change By: Josh Cooper
Epic Link: PUP-5853
This message was sent by Atlassian Jira (v8.13.2#813002-sha1:c495a97)
Atlassian logo
Reply all
Reply to author
Forward
0 new messages