Skip to content

Conversation

@jsquyres
Copy link
Member

Signed-off-by: Jeff Squyres [email protected]

@jsquyres jsquyres added this to the v2.1.0 milestone Aug 24, 2016
@jsquyres
Copy link
Member Author

@bturrubiates Please review.

This functionality came up the other day when helping debug some cluster-setup issues; we wanted to trace the traffic from one MPI process to another, but MPI processes didn't provide diag output indicating their source IP address and UDP port. This commit adds verbose output (when running with --mca btl_base_verbose) that shows both the local UDP ports, and also, when "connecting" to a remote peer, show both the local and remote IP/UDP ports. It's solely for diagnostic / verbose output purposes.

@jsquyres
Copy link
Member Author

@bturrubiates actually recommends this kind of thing: https://gist.github.com/jsquyres/dd959a48b6d9d34a41261c7cf7998a5e

I don't have time to look at this at the moment, but wanted to stash it here so I don't lose it.

@jsquyres jsquyres force-pushed the pr/usnic-show-local-udp-ports branch from 61364d9 to 228a91e Compare August 26, 2016 17:30
@jsquyres
Copy link
Member Author

@bturrubiates Updated with your suggestion. Have a look.

@jsquyres jsquyres force-pushed the pr/usnic-show-local-udp-ports branch from 228a91e to 87a5ccc Compare August 26, 2016 19:25
@jsquyres
Copy link
Member Author

@bturrubiates Got your latest feedback in this update.

@bturrubiates
Copy link

Looks good 👍

@jsquyres
Copy link
Member Author

I'm not sure why the Mellanox Jenkins would timeout in its test; this PR only deals with code in the usnic BTL, which the Mellanox Jenkins does not test.

@jsquyres jsquyres merged commit 09ad7e8 into open-mpi:master Aug 26, 2016
@jsquyres jsquyres deleted the pr/usnic-show-local-udp-ports branch August 26, 2016 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants