Review: Config.roles data storage is map
Jeremy Selan <jeremy...@...>
The list of defined roles within a config previously maintained order.
Conceptually, this is actually a mapping type where order does not
matter (and the yaml serialization dropped ordering). This switches
the underlying storage to use a map instead.