Use DCP injected values for OTEL service id and name

[JamesNK]

This PR makes the OTEL names consistent with DCP names. It fixes the problem of replicas, e.g. catalogservice, being duplicated. Now the OTEL name matches what is displayed on the projects page.

I'd like guidance on how this pattern will work when deployed to other environments. Do they ignore/overwrite the env vars set for service id and name and apply their own? Does this PR need to change to reflect that?

Once this approach is verified as good then port the changes to the template.

Note: this change relies on a preview build of DCP that isn't in the workload yet. The right version of DCP should be available in a few days.

[davidfowl]

Fix the dapr impl as well

aspire/src/Aspire.Hosting.Dapr/DaprDistributedApplicationLifecycleHook.cs

Lines 149 to 151 in 58ae089

	env["OTEL_EXPORTER_OTLP_ENDPOINT"] = this._configuration[DashboardOtlpUrlVariableName] ?? DashboardOtlpUrlDefaultValue;
	env["OTEL_EXPORTER_OTLP_INSECURE"] = "true";
	env["OTEL_EXPORTER_OTLP_PROTOCOL"] = "grpc";

[JamesNK]

Fix it in what way? The environment variables are automatically set on it. I ran the dapr samples and the expected env vars are present with good values:

I'm not sure why this code was added:

aspire/src/Aspire.Hosting.Dapr/DaprDistributedApplicationLifecycleHook.cs

Lines 149 to 151 in 58ae089

	env["OTEL_EXPORTER_OTLP_ENDPOINT"] = this._configuration[DashboardOtlpUrlVariableName] ?? DashboardOtlpUrlDefaultValue;
	env["OTEL_EXPORTER_OTLP_INSECURE"] = "true";
	env["OTEL_EXPORTER_OTLP_PROTOCOL"] = "grpc";

The endpoint is already set, and OTLP is insecure and grpc by default.

[JamesNK]

Fix the dapr impl as well

Done:

Bonus: Centralized OTEL configuration between projects and dapr sidecar.

[davidfowl]

Is this DCP version in VS?

[JamesNK]

Worked when I tested it.