Project

General

Profile

Actions

Bug #24795

closed

service restart does not work on Proxy

Added by Martin Bacovsky over 5 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Category:
Procedure
Target version:
-
Difficulty:
Triaged:
No
Fixed in Releases:
Found in Releases:

Description

On a Proxy 'service restart' is trying to setup hammer which is probably wrong.
We can not use hammer to query Proxy status and most likely this step does not bring us anything.

[root@capsule-6-4-compose-rhel7 ~]# foreman-maintain service restart
Running preparation steps required to run the next scenarios
================================================================================
Setup hammer:                                                         [FAIL]
undefined method `[]' for nil:NilClass
--------------------------------------------------------------------------------
Scenario [preparation steps required to run the next scenarios] failed.

The following steps ended up in failing state:

  [hammer-setup]

Resolve the failed steps and rerun
the command. In case the failures are false positives,
use --whitelist="hammer-setup" 

We should consider if it makes sense to allow the hammer feture on the Proxy and investigate what consequences it has on various tasks.

Expected behavior:
'foreman-maintain service restart' runs with no error on a proxy


Related issues 3 (0 open3 closed)

Related to Foreman Maintain - Feature #24793: Add check for started services that do not rely on hammer cli (and work on proxy)ClosedMartin BacovskyActions
Related to Foreman Maintain - Bug #24796: service restart has no --whitelist optionRejectedActions
Has duplicate Foreman Maintain - Bug #24715: `foreman-maintain service restart` is broken in develDuplicateActions
Actions #1

Updated by Martin Bacovsky over 5 years ago

  • Related to Feature #24793: Add check for started services that do not rely on hammer cli (and work on proxy) added
Actions #2

Updated by Martin Bacovsky over 5 years ago

  • Related to Bug #24796: service restart has no --whitelist option added
Actions #3

Updated by Martin Bacovsky over 5 years ago

  • Has duplicate Bug #24715: `foreman-maintain service restart` is broken in devel added
Actions #4

Updated by Ivan Necas over 5 years ago

  • Bugzilla link set to 1626651
Actions #5

Updated by The Foreman Bot over 5 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_maintain/pull/217 added
Actions #6

Updated by Martin Bacovsky over 5 years ago

  • Status changed from Ready For Testing to Closed
  • Pull request https://github.com/theforeman/foreman_maintain/pull/210 added
  • Pull request deleted (https://github.com/theforeman/foreman_maintain/pull/217)
Actions #7

Updated by Martin Bacovsky over 5 years ago

  • Bugzilla link deleted (1626651)
Actions #8

Updated by Martin Bacovsky over 5 years ago

  • Bugzilla link set to 1625169
Actions

Also available in: Atom PDF