=tes #15165 MetricsKit should not nullpointer on racy graphite usage
The report() call can come from multiple threads in our code,
yet it was opening/closing a shared graphite client instance in this
method, causing NPE when periodic reporting kicked in during a manual
report() call. We want to be able to do this - thus rewrite to safe impl

Removed metrics-graphite dependency, replaced by in house impl.
The previous was using discrete write() calls, which could be entangled
when called from multiple threads, and produce invalid metric lines.
Current impl is simpler and threadsafe, as the lock for write(String) to
aquired for the entire string lenght's write (unlike multiple write calls).

Resolves #15165

Conflicts:
	akka-testkit/src/test/scala/akka/testkit/metrics/reporter/AkkaGraphiteReporter.scala
	project/Dependencies.scala
+scr additional functions for tagging issues on github
Merge pull request #15263 from ktoso/scr-github-tagging-pr-validator

+scr functions for tagging issues on github, for pr-validator
Merge pull request #15247 from ktoso/port-graphite-client-rewrite-metricskit-ktoso

=tes #15228 backport - metricskit fix for racy graphite client (for validation)
=act #15165 should fix out-of-bounds in actor creation
Other than that, also improved failure logging when AIOOB happens in the histogram
Added a test which shows why the updateWithExpectedInterval method
should be preferred too.

Backport from master

Conflicts:
	akka-testkit/src/test/scala/akka/testkit/metrics/reporter/AkkaGraphiteReporter.scala