Project

General

Profile

Bug #30496

hammer ping returns 0 even when services are in a failed state

Added by Jonathon Turel 4 months ago. Updated about 2 months ago.

Status:
Closed
Priority:
Normal
Category:
-
Target version:
-
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:

Description

I'm not sure if this is a regression but it does seem like a bug:

[vagrant@centos7-katello-devel katello (master)]$ hammer ping ; echo $?
database:         
    Status:          ok
    Server Response: Duration: 0ms
candlepin:        
    Status:          FAIL
    Server Response: Message: Failed to open TCP connection to localhost:8443 (Connection refused - connect(2) for "localhost" port 8443)
candlepin_events: 
    Status:          FAIL
    message:         Not running
    Server Response: Duration: 0ms
candlepin_auth:   
    Status:          FAIL
    Server Response: Message: A backend service [ Candlepin ] is unreachable
katello_events:   
    Status:          ok
    message:         7 Processed, 0 Failed
    Server Response: Duration: 0ms
pulp:             
    Status:          ok
    Server Response: Duration: 34ms
pulp_auth:        
    Status:          ok
    Server Response: Duration: 13ms
foreman_tasks:    
    Status:          ok
    Server Response: Duration: 3ms

0

Associated revisions

Revision 66d886de (diff)
Added by Oleh Fedorenko about 2 months ago

Fixes #30496 - Return non-zero exit code if services failed

History

#1 Updated by Oleh Fedorenko 2 months ago

  • Triaged changed from No to Yes
  • Assignee set to Oleh Fedorenko
  • Status changed from New to Assigned

#2 Updated by The Foreman Bot 2 months ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/545 added

#3 Updated by The Foreman Bot about 2 months ago

  • Fixed in Releases hammer-cli-foreman-2.3.0 added

#4 Updated by Oleh Fedorenko about 2 months ago

  • Status changed from Ready For Testing to Closed

#5 Updated by The Foreman Bot about 2 months ago

  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/548 added

Also available in: Atom PDF