Project

General

Profile

Actions

Bug #12559

closed

Nicer error when refresh facts generates 500

Added by Ohad Levy over 8 years ago. Updated about 8 years ago.

Status:
Closed
Priority:
Normal
Category:
Discovery plugin
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

Since timeout on discovered host is not an uncommon event, I would prefer that timeout would not generate 500 on the server side, but rather provide a nicer error:

Warning!
Could not get facts from proxy https://10.35.27.150:8443: Connection timed out - connect(2) for "10.35.27.150" port 8443

If you feel this is an error with Foreman itself, please open a new issue with Foreman ticketing system, You would probably need to attach the Full trace , relevant log entries, and it is highly recommended to also attach the foreman-debug output.

Back

Actions #1

Updated by Lukas Zapletal over 8 years ago

  • Subject changed from refresh facts generates 500 to Nicer error when refresh facts generates 500
  • Category set to Discovery plugin

Sure, on my radar for quite some time :-)

Actions #2

Updated by Ohad Levy over 8 years ago

note: this is also the same when the proxy is off (connection refused).

Actions #3

Updated by The Foreman Bot about 8 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Lukas Zapletal
  • Pull request https://github.com/theforeman/foreman_discovery/pull/251 added
Actions #4

Updated by Anonymous about 8 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #5

Updated by Lukas Zapletal about 8 years ago

  • Bugzilla link set to 1313667
Actions

Also available in: Atom PDF