Skip to content

Spring starter declarative config #14062

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 42 commits into
base: main
Choose a base branch
from
Open

Spring starter declarative config #14062

wants to merge 42 commits into from

Conversation

zeitlinger
Copy link
Member

Fixes #14048

Based on #14014

@zeitlinger zeitlinger self-assigned this Jun 18, 2025
@github-actions github-actions bot added the test native This label can be applied to PRs to trigger them to run native tests label Jun 18, 2025
@otelbot-java-instrumentation
Copy link
Contributor

🔧 The result from spotlessApply was committed to the PR branch.

@zeitlinger zeitlinger marked this pull request as ready for review June 19, 2025 16:09
@zeitlinger zeitlinger requested a review from a team as a code owner June 19, 2025 16:09
@zeitlinger zeitlinger force-pushed the spring-starter-declarative-config branch from 0259c55 to 17d949a Compare June 19, 2025 16:16
zeitlinger added 24 commits June 19, 2025 18:20
@zeitlinger
add span kind support for method instrumentation
9bd77a5
@zeitlinger
add span kind support for method instrumentation
6997acc
@zeitlinger
use declarative config
fbad8bb
@zeitlinger
format
66e8101
@zeitlinger
fix
6e3802b
@zeitlinger
fix
9296cc3
@zeitlinger
add test brigde for exporters
37256d6
@zeitlinger
add test bridge for exporters
2754cd5
@zeitlinger
separate test bridge for agent
c8b2613
@zeitlinger
reduce logging (gets it the way of tests)
2544ddd
@zeitlinger
simplify
cf806be
@zeitlinger
remove redundant imports
7b7de34
@zeitlinger
pr feedback
7a819f7
@zeitlinger
move properties bridge to separate project
4c60e6a
@zeitlinger
create method
6e35009
@zeitlinger
declarative config
1c71cdb
@zeitlinger
isolate map converter
e91a8ea
@zeitlinger
embedded config file
d85337c
@zeitlinger
embedded config file
75631b2
@zeitlinger
config bridge
a30c9c5
@zeitlinger
config bridge
d0819c3
@zeitlinger
use InstrumentationConfig for encapsulation
1001357
@zeitlinger
use InstrumentationConfig for encapsulation
afc9468
@zeitlinger
use InstrumentationConfig for encapsulation
6ccc126
@zeitlinger zeitlinger force-pushed the spring-starter-declarative-config branch from 17d949a to f81444f Compare June 19, 2025 16:57
@zeitlinger
Copy link
Member Author

@laurit can you help me figure out what the problem is?

AgentInstrumentationTest > classPathSetUp() FAILED	
    java.lang.NoClassDefFoundError: io/opentelemetry/sdk/autoconfigure/spi/ConfigProperties	
        at java.lang.Class.forName0(Native Method)	
        at java.lang.Class.forName(Class.java:348)	
        at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:328)	
        at java.lang.ClassLoader.loadClass(ClassLoader.java:351)	
        at java.lang.Class.forName0(Native Method)	
        at java.lang.Class.forName(Class.java:264)	
        at instrumentation.AgentInstrumentationTest.classPathSetUp(AgentInstrumentationTest.java:53)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@zeitlinger zeitlinger

Labels
test native This label can be applied to PRs to trigger them to run native tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Support declarative config for spring starter
1 participant
@zeitlinger