Skip to content
This repository has been archived by the owner on Jul 19, 2022. It is now read-only.

Suspicious stack trace in an otherwise passing test #13

Open
wphillipmoore opened this issue Aug 27, 2015 · 1 comment
Open

Suspicious stack trace in an otherwise passing test #13

wphillipmoore opened this issue Aug 27, 2015 · 1 comment

Comments

@wphillipmoore
Copy link
Contributor

This one's either an untrapped error, or just noise, and if it's the latter it should really be trapped and not logged, as the normal reaction to seeing this will be a false positive.

[2015-08-27 13:45:13] 2015/08/27 09:41:35 [500] Internal Server Error: An error occurred: Not a resource: foobar at /var/tmp/venn-core-1.0.0-b
uild011-build/x86-64.rhel.7/5.18/source/core-1.0.0/blib/lib/Venn/PlacementEngine/Request.pm line 254, <$fh> line 1.
[2015-08-27 13:45:13] 2015/08/27 09:41:35 Performing placement: of type hosting with strategy biggest_outlier
[2015-08-27 13:45:13] 2015/08/27 09:41:35 [500] Internal Server Error: An error occurred: Invalid request resource
[2015-08-27 13:45:13]
[2015-08-27 13:45:13] Trace begun at /var/tmp/venn-core-1.0.0-build011-build/x86-64.rhel.7/5.18/source/core-1.0.0/blib/lib/Venn/PlacementEngin
e/Strategy.pm line 237
[2015-08-27 13:45:13] Venn::PlacementEngine::Strategy::validate_request_resources('Venn::PlacementEngine::Strategy::BiggestOutlier=HASH(0xc6f3
f70)', 'Venn::PlacementEngine::Request=HASH(0xc45f630)') called at /var/tmp/venn-core-1.0.0-build011-build/x86-64.rhel.7/5.18/source/core-1.0.0
/blib/lib/Venn/PlacementEngine.pm line 201
...
...
[2015-08-27 13:45:13] Catalyst::Test::_local_request('Venn', 'HTTP::Request=HASH(0xc5778e0)') called at /efs/dist/perl5/Catalyst-Runtime/5.901
00/common/lib/perl5/Catalyst/Test.pm line 32
[2015-08-27 13:45:13] Catalyst::Test::ANON('HTTP::Request=HASH(0xc5778e0)') called at t/catalyst/placement.t line 475
[2015-08-27 13:45:13] t/catalyst/placement.t ............................................................................ ok

The full trace is several pages long, so if anyone wants it I'll attach the full logfile.

@bert42
Copy link
Contributor

bert42 commented Sep 4, 2015

Yes, it's intentional, bug I agree it's ugly.
I'll fix it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants