feat(o11y): Introduce `rpc.system.name` and `rpc.method` in gRPC by diegomarquezp · Pull Request #4121 · googleapis/sdk-platform-java

diegomarquezp · 2026-02-25T21:57:51Z

This PR introduces the rpc.system.name and rpc.method span attributes in gRPC.

To achieve this, all Traced*Callable classes were modified with a constructor overload that accepts an ApiTracerContext instance. Such an instance will be preferred over SpanName when deciding which (also newly introduced) overload of ApiTracerFactory.newTracer() will be called.

The new method in ApiTracerFactory, newTracer(ApiTracer parent, ApiTracerContext tracerContext, OperationType operationType) will by default merge the existing factory's context with the new one and simply create a new SpanName.

However, the logic in SpanTracerFactory will use the context to create the span name directly.

The rpc.system.name and rpc.method properties are obtained from GrpcCallableFactory and stored in ApiTracerContext, which now holds the regexes for parsing client and method names (used when building SpanNames).

gemini-code-assist · 2026-02-25T21:58:14Z

Summary of Changes

Hello @diegomarquezp, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request introduces significant improvements to the observability framework by enriching tracing contexts with RPC system and method information. It refactors the tracing mechanism to be more flexible and less dependent on specific span naming conventions, allowing for better integration with standardized observability practices. The changes streamline how RPC details are captured and propagated through the tracing system, enhancing the clarity and utility of generated traces.

Highlights

Enhanced Tracing Context: The ApiTracerContext has been significantly enhanced to include rpc.system.name and rpc.method attributes, along with clientName and methodName. This allows for more granular and standardized observability data in tracing spans.
Decoupling Tracing from SpanName: The tracing mechanism no longer directly relies on the SpanName object. Instead, ApiTracerFactory and related callable wrappers now receive an ApiTracerFactory that is pre-configured with the necessary context, including RPC system and method details.
Automated RPC Context Extraction: New static factory methods (fromGrpcMethodName and fromHttpJsonMethodName) have been added to ApiTracerContext to automatically parse full method names (gRPC or HTTP/JSON) and extract the rpcSystem, rpcMethod, clientName, and methodName for tracing.
Refactored Callable Factories: Both GrpcCallableFactory and HttpJsonCallableFactory have been updated to utilize the new ApiTracerContext for creating tracers, simplifying the tracing setup within these factories and removing redundant logic for extracting method names.

Changelog

gax-java/gax-grpc/src/main/java/com/google/api/gax/grpc/GrpcCallableFactory.java
- Removed InternalApi and SpanName related imports.
- Removed the FULL_METHOD_NAME_REGEX pattern and its usage.
- Updated various create*Callable methods to use a new getTracerFactory helper method for obtaining ApiTracerFactory instances.
- Introduced a new private static method getTracerFactory to construct ApiTracerContext from gRPC method descriptors.
gax-java/gax-httpjson/src/main/java/com/google/api/gax/httpjson/HttpJsonCallableFactory.java
- Removed InternalApi and SpanName related imports.
- Removed the FULL_METHOD_NAME_REGEX pattern and its usage.
- Updated createUnaryCallable to use a new getTracerFactory helper method.
- Introduced a new private static method getTracerFactory to construct ApiTracerContext from HTTP/JSON method descriptors.
gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerContext.java
- Added new fields: rpcSystem, rpcMethod, clientName, and methodName.
- Updated getAttemptAttributes to include rpc.system.name and rpc.method.
- Added merge method for combining ApiTracerContext instances.
- Introduced static factory methods fromGrpcMethodName and fromHttpJsonMethodName for parsing method names.
- Added corresponding builder methods for the new fields.
gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerFactory.java
- Modified the newTracer method signature to remove the SpanName parameter.
gax-java/gax/src/main/java/com/google/api/gax/tracing/BaseApiTracerFactory.java
- Updated the newTracer method signature to align with the ApiTracerFactory interface.
gax-java/gax/src/main/java/com/google/api/gax/tracing/ObservabilityAttributes.java
- Added new constants RPC_SYSTEM_NAME_ATTRIBUTE and RPC_METHOD_ATTRIBUTE.
gax-java/gax/src/main/java/com/google/api/gax/tracing/SpanTracerFactory.java
- Modified the newTracer method signature to remove the SpanName parameter.
- Updated the attemptSpanName calculation to use apiTracerContext fields.
- Modified withContext to merge the provided context with the existing apiTracerContext.
gax-java/gax/src/main/java/com/google/api/gax/tracing/TracedUnaryCallable.java
- Removed the spanName field and its usage.
- Updated the constructor and futureCall method to no longer accept or use SpanName.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	`/gemini review`	Performs a code review for the current pull request in its current state.
Pull Request Summary	`/gemini summary`	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	`/gemini help`	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

gemini-code-assist

Code Review

This pull request introduces a significant and beneficial refactoring to the tracing infrastructure. By replacing the explicit SpanName with a more structured ApiTracerContext carried by ApiTracerFactory, the code becomes more maintainable and extensible. The introduction of rpc.system.name and rpc.method attributes aligns with modern observability practices.

However, there are a couple of points that need attention:

Missing Tests (Critical): The PR adds new logic (e.g., in ApiTracerContext) and refactors existing components, but it lacks corresponding unit tests. The repository's contribution guidelines (line 148: Testing: All new logic should be accompanied by tests.) are not being met. Please add tests for the new functionality, especially for the parsing logic in ApiTracerContext.fromGrpcMethodName and ApiTracerContext.fromHttpJsonMethodName, and ensure existing tests are updated to reflect the API changes.
Robustness: There's a potential for a NullPointerException in SpanTracerFactory which can be addressed with a defensive check.

Overall, this is a good direction, but the lack of tests is a critical issue that must be addressed before merging.

gax-java/gax/src/main/java/com/google/api/gax/tracing/SpanTracerFactory.java

gax-java/gax/src/main/java/com/google/api/gax/tracing/TracedUnaryCallable.java

gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerFactory.java

gax-java/gax-grpc/src/main/java/com/google/api/gax/grpc/GrpcCallableFactory.java

blakeli0 · 2026-03-03T01:48:29Z

gax-java/gax-grpc/src/main/java/com/google/api/gax/grpc/GrpcCallableFactory.java

            baseCallable,
            clientContext.getTracerFactory(),
-            getSpanName(grpcCallSettings.getMethodDescriptor()),
+            getApiTracerContext(grpcCallSettings.getMethodDescriptor()),


Batching is technically OOS also, but since this is a very low risk change, I think we can take it.

gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerContext.java

blakeli0 · 2026-03-03T02:05:08Z

gax-java/gax/src/main/java/com/google/api/gax/tracing/TracedUnaryCallable.java

    this.innerCallable = innerCallable;
    this.tracerFactory = tracerFactory;
    this.spanName = spanName;
+    this.apiTracerContext = null;


Can we create an apiTracerContext from the SpanName so we don't have to do null check below?

We would need something like:

this.apiTracerContext = ApiTracerContext.newBuilder() .setLibraryMetadata(LibraryMetadata.empty()) .setClientName(spanName.getClientName()) .setMethodName(spanName.getMethodName()) .build();

However clientName and methodName are computed from fullMethodName (I"m moving the parsing logic to ApiTracerContext.

sdk-platform-java/gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerContext.java

Lines 107 to 112 in d9c4d80

/**

* @return the client name part of the span name

*/

public String getClientName() {

return getParsedFullMethodNameParts()[0];

}

I think we can achieve this by having a private methodName property that will either return the set value or compute it as it's being done now. WDYT?

Let's keep a setter as an override of the default regex parsing logic.

gax-java/gax/src/main/java/com/google/api/gax/tracing/MetricsTracerFactory.java

gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerFactory.java

blakeli0 · 2026-03-03T02:28:34Z

gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerContext.java

+  /**
+   * @return the client name part of the span name
+   */
+  public String getClientName() {


Is clientName just serviceName which we should already have it in StubSettings?

This is meant for computing span names, which may differ depending on the transport.

gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerContext.java

blakeli0 · 2026-03-03T02:30:58Z

gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerContext.java

+  }
+
+  /**
+   * @return the method name part of the span name


method name could be used for more than Span, can we use more generic terms instead of span name in the Java docs?

In addition, can we provide an example for each of the public attributes so its easier to understand?

Improved the javadocs, which now just "RPC" instead of "span names".

I added examples to the public attributes.

javadocs kept but I moved the SpanName related logic to SpanName instead of ApiTracerContext.

gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerContext.java

diegomarquezp · 2026-03-03T20:01:06Z

gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerFactory.java

+   */
+  default ApiTracer newTracer(
+      ApiTracer parent, ApiTracerContext tracerContext, OperationType operationType) {
+    SpanName spanName = SpanName.of(tracerContext.getClientName(), tracerContext.getMethodName());


Pass full method name and transport; let span name do the parsing (keep the regexes in there).

Making getters such as fullMethodName() package-private makes GrpcCallableFactory and HttpJsonCallableFactory fail due to lack of access.

Passing the whole context solves the problem.

…-attr/rpc.method-and-rpc.system

diegomarquezp · 2026-03-04T17:11:58Z

/gemini review

gemini-code-assist

Code Review

This pull request introduces rpc.system.name and rpc.method span attributes for gRPC tracing, which is a great enhancement for observability. The implementation is well-structured, refactoring tracing context creation into a new ApiTracerContext class and adding new overloads to the Traced*Callable classes and ApiTracerFactory. The test coverage is excellent, with new parameterized tests to handle both old and new code paths, and an integration test to verify the new attributes are correctly added to spans. I have a couple of minor suggestions to improve documentation and code consistency.

gax-java/gax/src/main/java/com/google/api/gax/tracing/ObservabilityAttributes.java

gax-java/gax/src/main/java/com/google/api/gax/tracing/TracedUnaryCallable.java

…ilityAttributes.java Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

…https://github.com/googleapis/sdk-platform-java into observability/tracing-attr/rpc.method-and-rpc.system

blakeli0 · 2026-03-04T18:20:31Z

gax-java/gax/src/main/java/com/google/api/gax/tracing/SpanName.java

+   * @return a new {@link SpanName} with the parsed client name and method name
+   */
+  public static SpanName of(ApiTracerContext apiTracerContext) {
+    Preconditions.checkState(apiTracerContext.fullMethodName() != null, "rpcMethod must be set");


I don't think fullMethodName() is nullable? Also can we only pass a String to this method?

I don't think fullMethodName() is nullable?

I made it @Nullable considering the client-level vs method-def-level contexts we can have. ClientContext will not populate fullMethodName().

Also can we only pass a String to this method?

The problem with SpanName.of(apiTracerContext.getFullMethodName()) is that the callable factories cannot access the package-private method.

sdk-platform-java/gax-java/gax-httpjson/src/main/java/com/google/api/gax/httpjson/HttpJsonCallableFactory.java

Lines 224 to 234 in cee771f

@InternalApi("Visible for testing")

static SpanName getSpanName(@Nonnull ApiMethodDescriptor<?, ?> methodDescriptor) {

ApiTracerContext apiTracerContext =

ApiTracerContext.newBuilder()

.setFullMethodName(methodDescriptor.getFullMethodName())

.setTransport(ApiTracerContext.Transport.HTTP)

.setLibraryMetadata(LibraryMetadata.empty())

.build();

return SpanName.of(apiTracerContext);

}

}

I made it @nullable considering the client-level vs method-def-level contexts we can have. ClientContext will not populate fullMethodName()

I see, makes sense. This might be another data point that it's time to create a ClientTracerContext that only contains client level tracer info. And we can merge it with method/request level before we create a request level tracer. I feel we have a lot of nullable fields in ApiTracerContext only because we don't have the info initially. But we can do it in another PR.

Sounds good, I'll follow up with separating the contexts.

blakeli0 · 2026-03-04T18:21:06Z

gax-java/gax/src/main/java/com/google/api/gax/tracing/TracedUnaryCallable.java

+    this.innerCallable = innerCallable;
+    this.tracerFactory = tracerFactory;
+    this.apiTracerContext = apiTracerContext;
+    this.spanName = null;


We can create a spanName from ApiTracerContext now.

But we wouldn't use it if we have the context, right?

AFAIU we are trying to provide logic that only handles a context object as the SoT for SpanName. That's why we made this change in ApiTracer:

sdk-platform-java/gax-java/gax/src/main/java/com/google/api/gax/tracing/TracedUnaryCallable.java

Lines 84 to 88 in cee771f

if (apiTracerContext != null) {

tracer = tracerFactory.newTracer(context.getTracer(), apiTracerContext, OperationType.Unary);

} else {

tracer = tracerFactory.newTracer(context.getTracer(), spanName, OperationType.Unary);

}

sdk-platform-java/gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerFactory.java

Lines 72 to 76 in cee771f

default ApiTracer newTracer(

ApiTracer parent, ApiTracerContext tracerContext, OperationType operationType) {

SpanName spanName = SpanName.of(tracerContext);

return newTracer(parent, spanName, operationType);

}

Agreed. Even if we don't use it, I think it's safer to populate it instead of making it nullable.

Sounds good, done.

blakeli0 · 2026-03-04T18:21:31Z

gax-java/gax/src/main/java/com/google/api/gax/tracing/TracedUnaryCallable.java

  private final UnaryCallable<RequestT, ResponseT> innerCallable;
  private final ApiTracerFactory tracerFactory;
-  private final SpanName spanName;
+  @Nullable private final SpanName spanName;


I think we can keep this spanName not nullable.

I'll do it once we confirm my question in #4121 (comment)

See answers below. Ideally we probably want to remove SpanName as a field, and only use ApiTracerContext as the source of truth. But it can not be done until we migrated all Callables to use ApiTracerContext.

until we migrated all Callables to use ApiTracerContext

I thought the problem was with other classes that create the callables, such as GrpcCallableFactory. Are there usages of Traced*Callable in downstream libraries?

blakeli0 · 2026-03-05T18:34:11Z

gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerFactory.java

+   * @param tracerContext the method-definition-specific tracer context
+   * @param operationType the type of operation that the tracer will trace
+   */
+  default ApiTracer newTracer(


The downstream failure in bigtable is legit. One possible solution is to move OperationType to ApiTracerContext

sonarqubecloud · 2026-03-05T18:51:37Z

Quality Gate passed for 'gapic-generator-java-root'

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
95.5% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

sonarqubecloud · 2026-03-05T18:55:40Z

Quality Gate failed for 'java_showcase_integration_tests'

Failed conditions
78.1% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

product-auto-label bot added the size: m Pull request size is medium. label Feb 25, 2026

gemini-code-assist bot reviewed Feb 25, 2026

View reviewed changes

gax-java/gax/src/main/java/com/google/api/gax/tracing/SpanTracerFactory.java Outdated Show resolved Hide resolved

diegomarquezp added do not merge Indicates a pull request not ready for merge, due to either quality or timing. needs work This is a pull request that needs a little love. wip Work in Progress labels Feb 25, 2026

Base automatically changed from observability/tracing-attr/repo-artifact to main February 26, 2026 19:03

feat: pass whole context to traced callables

e5b927a

diegomarquezp force-pushed the observability/tracing-attr/rpc.method-and-rpc.system branch from b32f09a to e5b927a Compare February 26, 2026 20:13

diegomarquezp added 3 commits February 26, 2026 15:41

fix: provide getter of SpanName with suffix

956ae29

fix: concise logic in http callable factory

dfe6216

chore: expose newTracer method without SpanName

0ee9f4a

product-auto-label bot added size: l Pull request size is large. and removed size: m Pull request size is medium. labels Feb 27, 2026

diegomarquezp added 4 commits February 27, 2026 14:02

test: adapt tests

3cddc1b

test: move span name tests

2880286

fix: remove SpanName out of ApiTracerContext

74f97f6

fix: keep suffix in the context class

418b954

diegomarquezp commented Feb 27, 2026

View reviewed changes

gax-java/gax/src/main/java/com/google/api/gax/tracing/TracedUnaryCallable.java Outdated Show resolved Hide resolved

diegomarquezp commented Feb 27, 2026

View reviewed changes

gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerFactory.java Outdated Show resolved Hide resolved

diegomarquezp added 7 commits March 2, 2026 11:30

fix: revert changes in httpjson

7189841

feat: merge context in factory method

e648a63

test: revert traced callable tests

6b679d5

feat: SpanTracerFactory to implement newTracer with context

6fbcc08

chore: rename rpcMethod to fullMethodName

8b0a403

test: adapt traced callable tests

b45401e

test: adapt showcase it

a49828f

product-auto-label bot added size: xl Pull request size is extra large. and removed size: l Pull request size is large. labels Mar 2, 2026

test: expand tests of SpanTracerFactory

5ecf1b0

blakeli0 reviewed Mar 3, 2026

View reviewed changes

diegomarquezp added 7 commits March 3, 2026 11:41

fix: revert changes in operation callables

401c7e8

fix: use label for transport enum

ed58c7d

docs: add javadocs to ApiTracerContext and LibraryMetadata

342f4e3

fix: remove old exception message

8a7dacd

fix: remove unnecessary merge

5a845da

fix: revert unnecesary changes

00060d8

fix: remove suffix property

fc679c9

diegomarquezp commented Mar 3, 2026

View reviewed changes

gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerContext.java Show resolved Hide resolved

diegomarquezp commented Mar 3, 2026

View reviewed changes

gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerContext.java Outdated Show resolved Hide resolved

diegomarquezp commented Mar 3, 2026

View reviewed changes

gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerContext.java Outdated Show resolved Hide resolved

diegomarquezp commented Mar 3, 2026

View reviewed changes

diegomarquezp added 5 commits March 4, 2026 10:46

Merge remote-tracking branch 'origin/main' into observability/tracing…

945366a

…-attr/rpc.method-and-rpc.system

fix: move span name logic to SpanName

726fbf1

fix: remove unnecessary transport check in SpanName

6fb4ed6

chore: remove unnecessary comment

f5cf4bf

test: expand tests in ApiTracerContextTest

6711aa2

diegomarquezp requested a review from blakeli0 March 4, 2026 16:29

gemini-code-assist bot reviewed Mar 4, 2026

View reviewed changes

gax-java/gax/src/main/java/com/google/api/gax/tracing/ObservabilityAttributes.java Outdated Show resolved Hide resolved

gax-java/gax/src/main/java/com/google/api/gax/tracing/TracedUnaryCallable.java Outdated Show resolved Hide resolved

diegomarquezp and others added 3 commits March 4, 2026 12:26

Update gax-java/gax/src/main/java/com/google/api/gax/tracing/Observab…

aa2c2af

…ilityAttributes.java Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

chore: mark TracedUnaryCallable nullable properties

efd2841

Merge branch 'observability/tracing-attr/rpc.method-and-rpc.system' of …

cee771f

…https://github.com/googleapis/sdk-platform-java into observability/tracing-attr/rpc.method-and-rpc.system

blakeli0 reviewed Mar 4, 2026

View reviewed changes

fix: make spanName non-nullable in traced callables

f5c2381

blakeli0 approved these changes Mar 5, 2026

View reviewed changes

blakeli0 reviewed Mar 5, 2026

View reviewed changes

fix: rpc.method only in grpc

97d1aac

	/**
	* @return the client name part of the span name
	*/
	public String getClientName() {
	return getParsedFullMethodNameParts()[0];
	}

	@InternalApi("Visible for testing")
	static SpanName getSpanName(@Nonnull ApiMethodDescriptor<?, ?> methodDescriptor) {
	ApiTracerContext apiTracerContext =
	ApiTracerContext.newBuilder()
	.setFullMethodName(methodDescriptor.getFullMethodName())
	.setTransport(ApiTracerContext.Transport.HTTP)
	.setLibraryMetadata(LibraryMetadata.empty())
	.build();
	return SpanName.of(apiTracerContext);
	}
	}

	if (apiTracerContext != null) {
	tracer = tracerFactory.newTracer(context.getTracer(), apiTracerContext, OperationType.Unary);
	} else {
	tracer = tracerFactory.newTracer(context.getTracer(), spanName, OperationType.Unary);
	}

	default ApiTracer newTracer(
	ApiTracer parent, ApiTracerContext tracerContext, OperationType operationType) {
	SpanName spanName = SpanName.of(tracerContext);
	return newTracer(parent, spanName, operationType);
	}

Conversation

diegomarquezp commented Feb 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

gemini-code-assist bot commented Feb 25, 2026

Summary of Changes

Highlights

Footnotes

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

diegomarquezp commented Mar 4, 2026

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

diegomarquezp Mar 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

diegomarquezp commented Feb 25, 2026 •

edited

Loading

diegomarquezp Mar 4, 2026 •

edited

Loading