Project

General

Profile

Actions

Feature #30284

closed

Provide clearer Smart Proxy registration errors with as much context as possible based on the error

Added by Eric Helms over 3 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Foreman modules
Target version:
-
Fixed in Releases:
Found in Releases:

Description

Handle the following:

* Apache can be down
  * Connection refused or connection timeout
* Foreman can be down
  * HTTP 502 or 503
* Foreman can throw an error
  * Does not always return a proper error message
  * Need to check/fix Foreman to return a proper JSON structured error message
  * Typically a 500
* Authorization error
  * Show a proper invalid credentials error
* Instruct users where to look for error messages
  * Include in error message to look in production.log when appropriate

Related issues 1 (1 open0 closed)

Related to Foreman - Bug #30285: Smart proxy registration API does not always return a structured JSON error responseNewActions
Actions #1

Updated by Eric Helms over 3 years ago

  • Description updated (diff)
Actions #2

Updated by Eric Helms over 3 years ago

  • Description updated (diff)
Actions #3

Updated by Eric Helms over 3 years ago

  • Related to Bug #30285: Smart proxy registration API does not always return a structured JSON error response added
Actions #4

Updated by Eric Helms over 3 years ago

  • Pull request https://github.com/theforeman/puppet-foreman/pull/865 added
Actions #5

Updated by Eric Helms over 3 years ago

  • Target version set to 2.2.0
Actions #6

Updated by Ewoud Kohl van Wijngaarden over 3 years ago

Noting down so I don't forget: we can also retry on some HTTP errors, like 5xx errors on listing.

Actions #7

Updated by Ewoud Kohl van Wijngaarden over 3 years ago

Perhaps we can also use the #28384 to provide better debug instructions. I've asked @ https://github.com/theforeman/foreman/pull/7220#issuecomment-665141626 if it does.

Actions #8

Updated by Ewoud Kohl van Wijngaarden over 3 years ago

  • Target version deleted (2.2.0)
  • Category set to Foreman modules
  • Triaged changed from No to Yes

Removing the milestone since I don't think we'll get to this before 2.2.0 and it shouldn't block.

Actions #9

Updated by The Foreman Bot about 3 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to William Clark
  • Pull request https://github.com/theforeman/puppet-foreman/pull/912 added
Actions #10

Updated by The Foreman Bot about 3 years ago

  • Fixed in Releases 2.4.0 added
Actions #11

Updated by William Clark about 3 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF