## Summary of changes

Updates `/analyze-error` to not update all `Log.Error` calls in the to
`Log.ErrorSkipTelemetry` as it did in
#8133

Added some more generic logging considerations to `AGENTS.md`

## Reason for change

The `/analyze-error` command was instructing to update all instances of
`Log.Error` instead of just the correct / expected one.

AGENTS.md had some logging information but I think it lacked some
specificity, so updating very basically with some "best practices"

## Implementation details

## Test coverage

## Other details
<!-- Fixes #{issue} -->

I reran the `/analyze-error` command with these changes again on the
same error I did with
#8133 and with both of
these it changed to this following diff. It isn't _perfect_ but at least
it isn't super wrong, it also changed the `ToString` used to a generic
call. Note that the initial change was
5ab5b10


  ```diff
diff --git
a/tracer/src/Datadog.Trace/OpenTelemetry/Metrics/OtlpExporter.cs
  b/tracer/src/Datadog.Trace/OpenTelemetry/Metrics/OtlpExporter.cs
  index e90bee8..8c2acd3c2 100644
  --- a/tracer/src/Datadog.Trace/OpenTelemetry/Metrics/OtlpExporter.cs
  +++ b/tracer/src/Datadog.Trace/OpenTelemetry/Metrics/OtlpExporter.cs
  @@ -353,21 +353,27 @@ namespace Datadog.Trace.OpenTelemetry.Metrics
                   }
catch (TaskCanceledException) when (attempt < maxRetries)
                   {
+ Log.Debug<int>("OTLP request timed out (attempt {Attempt}),
retrying...", attempt + 1);
+ retryDelay =
TimeSpan.FromMilliseconds((long)(retryDelay.TotalMilliseconds * 2));
+ await Task.Delay(retryDelay).ConfigureAwait(false);
  +                }
  +                catch (TaskCanceledException ex)
  +                {
  +                    // Final timeout after all retries exhausted
+ Log.ErrorSkipTelemetry<Uri, int>(ex, "OTLP request to {Endpoint} timed
out after {Attempt}
attempts. See https://docs.datadoghq.com/tracing/troubleshooting/ for
troubleshooting.", _endpoint, attempt + 1);
  +                    return false;
  +                }
  +                catch (Exception ex) when (attempt < maxRetries)
  +                {
+ Log.Debug<int>(ex, "Error sending OTLP request (attempt {Attempt}),
retrying...", attempt + 1);
retryDelay =
TimeSpan.FromMilliseconds((long)(retryDelay.TotalMilliseconds * 2));
await Task.Delay(retryDelay).ConfigureAwait(false);
                   }
                   catch (Exception ex)
                   {
- Log.Error(ex, "Error sending OTLP request (attempt {Attempt})",
(attempt + 1).ToString());
  -                    if (attempt < maxRetries)
  -                    {
- retryDelay =
TimeSpan.FromMilliseconds((long)(retryDelay.TotalMilliseconds * 2));
- await Task.Delay(retryDelay).ConfigureAwait(false);
  -                    }
  -                    else
  -                    {
  -                        return false;
  -                    }
  +                    // Final failure after all retries exhausted
+ Log.ErrorSkipTelemetry<Uri, int>(ex, "Failed to send OTLP request to
{Endpoint} after {Attempt}
attempts. See https://docs.datadoghq.com/tracing/troubleshooting/ for
troubleshooting.", _endpoint, attempt + 1);
  +                    return false;
                   }
               }
  ```

<!--  ⚠️ Note:

Where possible, please obtain 2 approvals prior to merging. Unless
CODEOWNERS specifies otherwise, for external teams it is typically best
to have one review from a team member, and one review from apm-dotnet.
Trivial changes do not require 2 reviews.

MergeQueue is NOT enabled in this repository. If you have write access
to the repo, the PR has 1-2 approvals (see above), and all of the
required checks have passed, you can use the Squash and Merge button to
merge the PR. If you don't have write access, or you need help, reach
out in the #apm-dotnet channel in Slack.
-->

---------

Co-authored-by: Andrew Lock <[email protected]>