feat(instrumentation-memcached): support net.* and db.* semconv migra…

[JamieDanielson]

This adds support for using OTEL_SEMCONV_STABILITY_OPT_IN for controlled migration to stable net.* and db.* semconv. The net.* attributes are controlled by the http[/dup] token in OTEL_SEMCONV_STABILITY_OPT_IN (as discussed here) and db.* with the database[/dup] token.

Refs: open-telemetry/opentelemetry-js#5663
Refs: #2953

[github-actions]

This package does not have an assigned component owner and is considered unmaintained. As such this package is in feature-freeze and this PR will be closed with 14 days unless a new owner or a sponsor (a member of @open-telemetry/javascript-approvers) for the feature is found. It is the responsibility of the author to find a sponsor for this feature.
Are you familiar with this package? Consider becoming a component owner.

[maryliag]

just one small nit on the test, otherwise lgtm

[maryliag]

nevermind, you got some errors that need fixing 😅

[trentm]

nit: use DB_SYSTEM_NAME_VALUE_MEMCACHED for this one

[JamieDanielson]

I started to do that but realized DB_SYSTEM_NAME_VALUE_MEMCACHED is still experimental 🤔

[trentm]

Yes, it would mean adding a src/semconv.ts wiht a local copy of that var.

[trentm]

Oh, I see what you mean. Meh, I think it still counts as the right thing to do for the "stable" semconv.

DB_SYSTEM_VALUE_MEMCACHED is also "Stability: Development" and arguably is deprecated because it is an enum for db.system which is deprecated: https://github.com/open-telemetry/semantic-conventions/blob/41df6e41371948bb924cec6fe4468f0a68431852/model/database/deprecated/registry-deprecated.yaml#L370-L523

[JamieDanielson]

@maryliag I made the changes if you can have another look when you get a chance 🙏

[maryliag]

🚀