Project

General

Profile

Bug #22663

Check for verifying syntax for ISP DHCP configurations failed with error undefined method `strip' for nil:NilClass

Added by Kavita Gaikwad almost 4 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Category:
-
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Description of problem: When katello-services are stopped and Check for verifying syntax for ISP DHCP configuration is performed, it fails with error showing undefined method `strip' for nil:NilClass.

Version-Release number of selected component (if applicable): foreman-maintain snap 3

  1. rpm -q rubygem-foreman_maintain
    rubygem-foreman_maintain-0.0.10-1.el7sat.noarch

How reproducible: always

Steps to Reproduce:
1. katello-service stop
2. foreman-maintain health check --label foreman-proxy-verify-dhcp-config-syntax
Running ForemanMaintain::Scenario::FilteredScenario ================================================================================
Check for verifying syntax for ISP DHCP configurations: [FAIL]
undefined method `strip' for nil:NilClass
--------------------------------------------------------------------------------
Scenario [ForemanMaintain::Scenario::FilteredScenario] failed.

The following steps ended up in failing state:

[foreman-proxy-verify-dhcp-config-syntax]

Resolve the failed steps and rerun
the command. In case the failures are false positives,
use --whitelist="foreman-proxy-verify-dhcp-config-syntax"

Actual results:
Displayed error undefined method `strip' for nil:NilClass

Expected results: Error should get handled.

Additional info: When katello-services are stopped and user performed check for verifying syntax for ISP DHCP configuration, can we add interaction for user like hammer-ping?

Continue with step [katello-service restart]?, [y(yes), n(no), q(quit)]

Associated revisions

Revision 51663ffa (diff)
Added by Kavita Gaikwad almost 4 years ago

Fixes #22663 - handled nil.strip for DHCP check

History

#1 Updated by The Foreman Bot almost 4 years ago

  • Pull request https://github.com/theforeman/foreman_maintain/pull/145 added

#2 Updated by Kavita Gaikwad almost 4 years ago

  • % Done changed from 0 to 100
  • Status changed from New to Closed

#3 Updated by Ivan Necas over 3 years ago

  • Legacy Backlogs Release (now unused) set to 348

Also available in: Atom PDF