ui: Rendering of node names is inconsistent in the code

[dhartunian]

Currently a lot of different code is responsible for rendering node names in the Admin UI. It would make sense to standardize on a single format and ensure that one place in the code is responsible. We're not too far from this since the number of code spots are few for now.

Ideally we can either change the types in the code to make it easier to use the getDisplayName function that already exists or modify the types we're using and the function to make them work well together.

See #45551 for examples.

[girishramnani]

My initial search in the code shows these 2 places which need the change

1 -

cockroach/pkg/ui/src/views/reports/containers/network/legend/index.tsx

Line 101 in 7860cf9

<span className="noConnections__table--item__bold">{`N${value.from.nodeID}: `}</span>

2 -

cockroach/pkg/ui/src/views/reports/containers/network/legend/index.tsx

Line 106 in 7860cf9

<span className="noConnections__table--item__bold">{`N${value.to.nodeID}: `}</span>

[girishramnani]

opened a PR which changes both of these places #45665

If there are any other places then I can change those too.

[girishramnani]

@dhartunian if you could assign this issue to me then that would be amazing

[girishramnani]

Thanks for the assign @nathanstilwell