Project

General

Profile

Actions

Bug #9966

closed

Add debug logging when host interfaces cannot be matched to VM interfaces

Added by Dominic Cleal almost 9 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Category:
Network
Target version:
Difficulty:
easy
Triaged:
Fixed in Releases:
Found in Releases:

Description

#7456 changes compute orchestration to match host interfaces to the VM interfaces, which is usually matching based on the hypervisor network.

The orchestration iterates over host interfaces and attempts to find a VM interface, but if this fails then there's no logging provided to debug the situation.

https://github.com/theforeman/foreman/blob/1.8.0-RC2/app/models/concerns/orchestration/compute.rb#L294-L295

Here's an example of adding some debug:
https://github.com/theforeman/foreman/compare/develop...tstrachota:nic_debug


Related issues 2 (0 open2 closed)

Related to Foreman - Refactor #7456: Extract primary interface from hostClosedGreg Sutcliffe09/16/2014Actions
Related to Foreman - Bug #9871: Provisioning a new vmware hosts fails with 'Invalid MAC 'Resolved03/23/2015Actions
Actions #1

Updated by Dominic Cleal almost 9 years ago

  • Related to Refactor #7456: Extract primary interface from host added
Actions #2

Updated by Dominic Cleal almost 9 years ago

  • Related to Bug #9871: Provisioning a new vmware hosts fails with 'Invalid MAC ' added
Actions #3

Updated by Dominic Cleal almost 9 years ago

Perhaps it should be more than debug level when matching fails? Should it even fail the entire process so users don't get "invalid MAC" errors etc.

Actions #4

Updated by Dominic Cleal almost 9 years ago

  • translation missing: en.field_release set to 28
  • Difficulty set to easy
Actions #5

Updated by Dominic Cleal almost 9 years ago

  • translation missing: en.field_release changed from 28 to 50
Actions #6

Updated by Tomáš Strachota almost 9 years ago

  • Status changed from New to Assigned
  • Assignee set to Tomáš Strachota
Actions #7

Updated by Marek Hulán almost 9 years ago

based on IRC discussion we should trigger failure if the matching fails

Actions #8

Updated by The Foreman Bot almost 9 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2360 added
  • Pull request deleted ()
Actions #9

Updated by Anonymous almost 9 years ago

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

Also available in: Atom PDF