[core] - tracingPolicy only invoked once on retries

[maorleger]

Tested this on both core V1 and core V2 - it looks like tracingPolicy is only run once regardless of number of retries.

To reproduce (using Key Vault as an example):

async function main(): Promise<void> {
  const provider = new NodeTracerProvider();
  provider.addSpanProcessor(new SimpleSpanProcessor(new ConsoleSpanExporter()));
  provider.register();

  const tracer = provider.getTracer("my-service");

  const keyClient = new KeyClient("<KEYVAULT_NAME>", new AzureCliCredential(), {
    retryOptions: {
      maxRetries: 2,
    },
  });
  await tracer.startActiveSpan("root", async (span) => {
    await keyClient.createEcKey("retry");
  });
}

Then modify exponentialRetryPolicy to always retry up to the retryCount (easier than setting up the retry conditions).

Expected:

• 3 spans should be created, one for original try and 2 for 2 retries

Actual:

• Only one span is created

Note: haven't had a chance to dig into the root cause yet - would we just need to reorder policies in coreHttp and coreClient?

[maorleger]

This might get fixed by #19078 thanks to @sadasant 😄

This is fine to parent against Azure Core Fundamentals for JS IMO - it won't block core-tracing GA