andrewdmcleod / magpie-layer

testing
Apache License 2.0
0 stars 12 forks source link

Sort output of 'icmp failed' status message #14

Closed phausman closed 5 years ago

phausman commented 5 years ago

This commit updates the output of 'icmp failed' status message:

Example: Before: "icmp failed: ['0', '9', '3', '6', '7', '5', '11', '12']" After: "icmp failed: [0, 3, 5, 6, 7, 9, 11, 12]"

Sorting the list makes it easier to troubleshoot and spot the patterns. Removing apostropes makes the status line shorter and easier to read.

andrewdmcleod commented 5 years ago

This does look good, but can you please propose it against

https://github.com/openstack-charmers/magpie-layer/