oscerd opened a new pull request, #25071:
URL: https://github.com/apache/camel/pull/25071

   ### Motivation
   
   [CAMEL-23460](https://issues.apache.org/jira/browse/CAMEL-23460), continuing 
from batch 1 (#24974, merged). This is the issue's **AI/ML & misc** batch of 
Google Cloud span decorators.
   
   ### Changes
   
   Six new decorators (all `AbstractSpanDecorator`):
   
   | Decorator | Tags emitted |
   |---|---|
   | `GoogleFunctionsSpanDecorator` | operation, entryPoint, runtime |
   | `GoogleSecretManagerSpanDecorator` | operation, secretId, versionId |
   | `GoogleVertexAISpanDecorator` | operation, modelId, location |
   | `GoogleVisionSpanDecorator` | operation |
   | `GoogleSpeechToTextSpanDecorator` | operation |
   | `GoogleTextToSpeechSpanDecorator` | operation |
   
   **Tag minimization** per the rules established in the AWS work — sensitive 
or bulky values are deliberately dropped:
   
   - **VertexAI**: the `prompt` and `chatMessages` headers carry user content 
and are **not** emitted (the unit test asserts they are never tagged); the 
tuning parameters (temperature/topP/…) are omitted as low-value.
   - **Secret Manager**: only the secret **identifier** and **version** are 
tagged — never a secret value.
   - **Vision / Speech-to-Text / Text-to-Speech**: only `operation` — their 
`RESPONSE_OBJECT` header can carry large audio/image payloads and is not 
emitted.
   
   Header constants are **mirrored** from each component's `*Constants` 
interface (with a Javadoc `@link` back to the source), so `camel-telemetry` 
gains no hard dependency on the Google component modules. Registered 
alphabetically in `META-INF/services/org.apache.camel.telemetry.SpanDecorator`, 
one unit test per decorator.
   
   The remaining batch — workspace/productivity (Calendar, Drive, Mail, Sheets) 
— needs a different treatment (Mail's headers are To/From/Cc/Subject, i.e. PII; 
Drive exposes no data headers) and will be a separate PR.
   
   ### Testing
   
   - `mvn clean install` in `components/camel-telemetry`: all tests pass (6 new 
+ existing).
   - Full reactor `mvn clean install -DskipTests` from root: success, no stale 
generated files.
   
   Metadata/observability addition — main/4.22.0 only, no backport.
   
   _Claude Code on behalf of Andrea Cosentino (@oscerd)._
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to