There was a typo in a comment https://github.com/knative/pkg/issues/3224
This change will update the OTel prom exporter metric names to match
what will become the default in the future. If N is the current
SDK version the N+2 is when the new default will apply.
In N+1 a new TranslationStrategy option will be introduced and we
will want to set the strategy to 'UnderscoreEscapingWithSuffixes'.
Then in N+2 we can drop that option as it will become the default
(or leave it explicitly).
* Fix otel exported names
* based on feedback from OTel devs these settings will become the default
* gofumpt
---------
Co-authored-by: Dave Protasowski <dprotaso@gmail.com>
* apply TrimSpace from config map values
* Convert opaque URLs to hiearchical ones.
When testing prometheus endpoints I entered
some.domain.com:9090/some/path
And this took `some.domain.com` as the scheme. To avoid this landmine
if we end up with an opaque URL we prepend the 'https' scheme to
the endpoint
* sharedmain changes
* go mod
* vendor
* address PR feedback
* bump semconv to v1.34.0
* drop globalviews in favour of sharedmain option
Given Views in OTel are overrides for end-users this should really be a
sharedmain option instead of a dedicated package. In OpenCensus views
were required but in OTel they are not
* PR feedback
* panic if we drift semconv version from the SDK
* fix testing import
* bump runtime instrumentation
* gofumpt file
* don't set enabled=false on init since the profiling Handler already does it
Dave Protasowski