Skip to content

add Spring declarative config example#911

Merged
trask merged 16 commits intoopen-telemetry:mainfrom
zeitlinger:spring-declarative-config
Dec 4, 2025
Merged

add Spring declarative config example#911
trask merged 16 commits intoopen-telemetry:mainfrom
zeitlinger:spring-declarative-config

Conversation

@zeitlinger
Copy link
Copy Markdown
Member

No description provided.

@zeitlinger zeitlinger self-assigned this Nov 21, 2025
@zeitlinger zeitlinger requested review from a team as code owners November 21, 2025 18:24
@zeitlinger zeitlinger marked this pull request as draft November 21, 2025 18:24
Comment thread spring-declarative-configuration/oats/docker-compose.yml Outdated
Comment thread spring-declarative-configuration/README.md Outdated
@zeitlinger
Copy link
Copy Markdown
Member Author

@trask it's good to go 😄

@zeitlinger zeitlinger force-pushed the spring-declarative-config branch from 8fcaa90 to 4c8443e Compare December 2, 2025 07:04
Comment thread spring-declarative-configuration/README.md Outdated
In many cases, you can start with declarative configuration (as in this module) and only fall back
to programmatic customization for highly dynamic or application-specific logic.

## Troubleshooting and Tips
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this readme is already pretty long, i wonder if this section could be removed

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

copilot going loose... I'll revisit

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I'd leave it in - I do plan to move some of this content to the general docs after the next release, which contains some bug related fixes.

http.route: "/api/example"
- traceql: '{ span.http.route = "/actuator/health" }'
count:
max: 0
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice 🎉

Co-authored-by: Jay DeLuca <jaydeluca4@gmail.com>
@zeitlinger
Copy link
Copy Markdown
Member Author

@trask can you take a look?

@trask trask merged commit 166dc3f into open-telemetry:main Dec 4, 2025
14 checks passed
@zeitlinger zeitlinger deleted the spring-declarative-config branch December 4, 2025 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants