Bug #11990

Updated by Julien Pivotto over 8 years ago

This looks like a bug: 

 While taking a look into the foreman code, I saw that snippet in app/services/fact_parser.rb 



 <pre> 
   def find_physical_interface(interfaces) 
     interfaces.detect { |int, _| int.to_s !~ FactParser::VIRTUAL_NAMES } 
   end 

   def find_virtual_interface(interfaces) 
     interfaces.detect { |int, _| int.to_s =~ /#{FactParser::BONDS}/ } 
   end 
 </pre> 

 I think this should be 

 <pre> 

 <pre> 

 </pre> 

 </pre> 
   def find_physical_interface(interfaces) 
     interfaces.detect { |int, _| int.to_s !~ FactParser::VIRTUAL_NAMES } 
   end 

   def find_virtual_interface(interfaces) 
     interfaces.detect { |int, _| int.to_s =~ FactParser::VIRTUAL_NAMES } 
   end 
 </pre>

Back