Fix various issues with refactors

[aauren]

@mrueg @twz123 @rbrtbnfgl

Fixes a couple of things:

• During one of the recent refactors, we lost the fact that the router-id for BGP should default to the IPv4 address when it isn't otherwise specified
• Fixes a potential for the IP addresses on nodes to not obey the ordering of internal first then external. This mostly impacted tests, but would have eventually impacted users as well once a release was createdc
• Removes a potential race condition from LoadBalancerAllocator tests
• Reduces the total unit test run times by setting t.Parrallel() on ecmp_vip_test.go which was the longest test in the suite
  • It would be nice to set this on more tests in the routing suite, but unfortunately, we can't use it any time we actually start a gobgp server because it has several global vars that introduce race conditions and testing irregularity
• Minor fixes:
  • Adds a python script which parses Go unit tests, which is helpful for understanding which tests in the suite take the longest
  • Fixes import definition order in linux_networking_moq.go
  • Updates makefile to run goimports as well as gofmt when gofmt-fix target is executed