Project

General

Profile

Actions

Bug #6925

closed

Improve error handling for nodes without network

Added by Lukas Zapletal over 9 years ago. Updated over 5 years ago.

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

Description

When discovered host is booted without network, ugly error is issued and the client is returned with 500 without further description:

address family must be specified (IPAddr::AddressFamilyError)
/home/lzap/.rbenv/versions/2.0.0-p247/lib/ruby/2.0.0/ipaddr.rb:475:in `initialize'
/home/lzap/work/foreman/app/models/subnet.rb:71:in `new'
/home/lzap/work/foreman/app/models/subnet.rb:71:in `block in subnet_for'
/home/lzap/work/foreman/app/models/subnet.rb:71:in `each'
/home/lzap/work/foreman/app/models/subnet.rb:71:in `subnet_for'
/home/lzap/work/foreman_discovery/app/models/host/discovered.rb:76:in `populate_fields_from_facts'
/home/lzap/work/foreman/app/models/host/base.rb:52:in `import_facts'
/home/lzap/work/foreman_discovery/app/models/host/discovered.rb:67:in `import_facts'
/home/lzap/work/foreman_discovery/app/models/host/discovered.rb:59:in `import_host_and_facts'
/home/lzap/work/foreman_discovery/app/controllers/api/v2/discovered_hosts_controller.rb:97:in `facts'

We should test for IP fact presence.

Actions #1

Updated by Lukas Zapletal over 9 years ago

  • Status changed from New to Ready For Testing
Actions #2

Updated by The Foreman Bot over 9 years ago

  • Pull request https://github.com/theforeman/foreman_discovery/pull/80 added
  • Pull request deleted ()
Actions #3

Updated by Anonymous over 9 years ago

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

Also available in: Atom PDF