rpc: Slightly clarify the clock offset suicide error message

[a-robinson]

We've had a couple users not be sure what it meant, hopefully this
helps.

@onkar, would this have helped you? It would have printed as:

fewer than half the known nodes are within the maximum clock offset of 500ms (0 of 1); check your machines' clock synchronization

[cockroach-teamcity]

This change is 

[petermattis]

---

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved, some commit checks failed.

---

Comments from Reviewable

[bdarnell]

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

---

pkg/rpc/clock_offset.go, line 230 at r1 (raw file):

		if numClocks > 0 && healthyOffsetCount <= numClocks/2 {
			return errors.Errorf(
				"fewer than half the known nodes are within the maximum clock offset of %s (%d of %d); "+

How about something like "clock synchronization error: This node is more than %s away from at least half the known nodes (%d of %d)"

---

Comments from Reviewable

[a-robinson]

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

---

pkg/rpc/clock_offset.go, line 230 at r1 (raw file):

Previously, bdarnell (Ben Darnell) wrote…

How about something like "clock synchronization error: This node is more than %s away from at least half the known nodes (%d of %d)"

Sure, that's a bit clearer. Done.

---

Comments from Reviewable