small bug in the network plugin

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 0.6.10
  • Component/s: None
  • Labels:
    None

Description

network["interfaces"][network[:default_interface]]["addresses"] is wrongly assumed to exist and not to be nil in lib/ohai/plugins/network.rb

if network[:default_interface]
  Ohai::Log.debug("Using default interface for default ip and mac address")
  im = find_ip_and_mac(network["interfaces"][network[:default_interface]]["addresses"])
  ipaddress im.shift
  macaddress im.shift
else

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Laurent Désarmes added a comment -

https://github.com/opscode/ohai/pull/43

Show
Laurent Désarmes added a comment - https://github.com/opscode/ohai/pull/43
Laurent Désarmes made changes -
Field Original Value New Value
Status Open [ 1 ] Closed [ 6 ]
Resolution Fixed [ 1 ]
Bryan McLellan [Opscode] made changes -
Resolution Fixed [ 1 ]
Status Closed [ 6 ] Reopened [ 4 ]
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Merged to opscode/master. Thanks!

Show
Bryan McLellan [Opscode] added a comment - Merged to opscode/master. Thanks!
Bryan McLellan [Opscode] made changes -
Status Reopened [ 4 ] Resolved [ 5 ]
Fix Version/s 0.6.10 [ 10602 ]
Resolution Fixed [ 1 ]
Bryan McLellan [Opscode] made changes -
Status Resolved [ 5 ] Closed [ 6 ]
Bryan McLellan [Opscode] made changes -
Workflow jira [ 14559 ] New OSS [ 23118 ]
Bryan McLellan [Opscode] made changes -
Workflow New OSS [ 23118 ] OSS: Product Management [ 29068 ]
Transition Time In Source Status Execution Times Last Executer Last Execution Date
Open Open Closed Closed
10m 35s 1 Laurent Désarmes 21/Oct/11 10:22 AM
Closed Closed Reopened Reopened
4h 34m 1 Bryan McLellan [Opscode] 21/Oct/11 2:57 PM
Reopened Reopened Resolved Resolved
13s 1 Bryan McLellan [Opscode] 21/Oct/11 2:57 PM
Resolved Resolved Closed Closed
10s 1 Bryan McLellan [Opscode] 21/Oct/11 2:57 PM

People

  • Assignee:
    Unassigned
    Reporter:
    Laurent Désarmes
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: