fixes #9249 - associated_host fixed for all compute resources

[unorthodoxgeek]

No description provided.

[dLobatog]

Thanks @unorthodoxgeek , this seems to work well.

Could you take this one step further and remove the code duplication by adding a method in the parent class (ComputeResource)? Something along the lines of

def associate_by(name, attributes)
 Host.authorized(:view_hosts, Host).joins(:primary_interface).
  where(:nics => {:primary => true}).
  where("nics.#{name}" => attributes).
  first
end

then change the methods in the specific CRs to associate_by('mac', vm.mac) and associate_by('ip', [vm.public_ip_address, vm.private_ip_address]). That's just what I came up with in 1 minute, maybe it can be done even DRYer 😄

[unorthodoxgeek]

@elobato - done

[dLobatog]

👏 thanks @unorthodoxgeek , merged as 81a02cd , I opened this issue for a future refactoring of the tests.