From 282f0c05c53b9a00321227a1a91e42247074cf6e Mon Sep 17 00:00:00 2001 From: Adam Grare Date: Thu, 29 Mar 2018 12:07:29 -0400 Subject: [PATCH] Sort unitNumber as an integer not a string The get_network_adapters method was sorting NICs by their unitNumber as an integer. This was causing '10' to be sorted before '7', '8', and '9'. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1562089 --- .../vmware/infra_manager/provision/configuration/network.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/models/manageiq/providers/vmware/infra_manager/provision/configuration/network.rb b/app/models/manageiq/providers/vmware/infra_manager/provision/configuration/network.rb index 40707fe68..4322f255c 100644 --- a/app/models/manageiq/providers/vmware/infra_manager/provision/configuration/network.rb +++ b/app/models/manageiq/providers/vmware/infra_manager/provision/configuration/network.rb @@ -140,7 +140,7 @@ def get_network_adapters end devs = inventory_hash.fetch_path("config", "hardware", "device") || [] - devs.select { |d| d.key?('macAddress') }.sort_by { |d| d['unitNumber'] } + devs.select { |d| d.key?('macAddress') }.sort_by { |d| d['unitNumber'].to_i } end def get_network_device(vimVm, _vmcs, _vim = nil, vlan = nil)