Fixes #38046 - Make sure IPv6 interface can be primary

[ShimShtein]

No description provided.

[ekohl]

Interesting you hit this code. Facter has networking.primary but I suspect you're using another fact provider.

[ShimShtein]

Interesting you hit this code. Facter has networking.primary but I suspect you're using another fact provider.

I didn't try a real-life scenario. This was one of the places that had a reference only to IPv4, but not to IPv6. I didn't try to change the logic more than is needed.

While I am at it, changed the implementation to resolve both IPv4 and IPv6 separately.

[ShimShtein]

Now it should look a bit more reasonable. Also fixed the tests to use the testing range.

[ekohl]

I think the separate try_resolve method is a good refactor.

[ekohl]

I'd prefer to be stricter and provide just a regular hash with the right types. Though you are right that it's presented as indifferent access

Suggested change

	'br0' => {'ipaddress6' => '2001:db8::10', 'macaddress' => '00:00:00:00:00:10'},
	'em1' => {'ipaddress6' => '2001:db8::20', 'macaddress' => '00:00:00:00:00:20'},
	'em2' => {'ipaddress6' => '2001:db8::30', 'macaddress' => '00:00:00:00:00:30'},
	'bond0' => {'ipaddress6' => '2001:db8::40', 'macaddress' => '00:00:00:00:00:40'},
	}.with_indifferent_access)
	'br0' => {ipaddress6: '2001:db8::10', macaddress: '00:00:00:00:00:10'},
	'em1' => {ipaddress6: '2001:db8::20', macaddress: '00:00:00:00:00:20'},
	'em2' => {ipaddress6: '2001:db8::30', macaddress: '00:00:00:00:00:30'},
	'bond0' => {ipaddress6: '2001:db8::40', macaddress: '00:00:00:00:00:40'},
	})

[ShimShtein]

I prefer having the style consistent with the rest of the file. So unless you feel it's really wrong, I think we should leave it with the indifferent access.