All committers have signed the CLA.

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

Hey, thanks for this PR! I think we should merge this but we should bring it inline with the other SDKs first.

1. We should only create tracing spans for StartActivity. list, count, terminate, describe, etc are plain request/response or list/count calls that do not have meaningful latency boundaries, don't propagate trace headers, and are a bit noisy. The other SDKs intentionally do not track these and we should stay consistent with that.
2. We can reuse StartActivity + RunActivity (i.e. don't introduce StartStandaloneActivity + RunStandaloneActivity). There are no "standalone" variants in the other SDKs. We distinguish by tags and not by span names. For reference, see https://github.com/temporalio/sdk-python/blob/main/temporalio/contrib/opentelemetry/_interceptor.py#L373
3. For the worker interceptor, we should keep a single RunActivity span and vary the tags instead of forking on isInWorkflow(). For reference, Python's execute_activity emits one RunActivity span and conditionally adds the workflow attributes if we are in a workflow.

so the span builder would change to be something like this:

case RUN_ACTIVITY: {
    ImmutableMap.Builder<String, String> tags = ImmutableMap.builder();
    if (context.getActivityId() != null) tags.put(StandardTagNames.ACTIVITY_ID, context.getActivityId());
    if (context.getWorkflowId() != null) tags.put(StandardTagNames.WORKFLOW_ID, context.getWorkflowId());
    if (context.getRunId() != null)      tags.put(StandardTagNames.RUN_ID, context.getRunId());
    return tags.build();
  }

4. Lastly, we should add a README snippet showing how to register this.

Does that all sound reasonable? Lmk if you have more feedback or if you aren't able to make the changes and I can get to it sometime in the next few days. Again, appreciate the PR!

Hi @cconstable , thanks for the comments, those make sense. I've updated my PR to address your comments.

For README, I also changed previous "Client configuration" to "Workflow Client configuration". LMK if that aligns with your convention.

Looks good! I'll run the PR checks and get this merged after they pass.

@cconstable any action I need to take on the failed UT? I can't seem to run the UT on my local env (This test is flaky on the Test Server) so not sure how to troubleshoot this 😔

@444am you're good, thanks. It looks like it was just a flake. I reran the check and I'll monitor it + set it to auto-merge once it passes.

Waiting on a fix to hit main for updating PR check permissions.