-
Notifications
You must be signed in to change notification settings - Fork 18
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.
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
Merge Telemetry TCK into a single archive #257
base: main
Are you sure you want to change the base?
Conversation
Move sources and resources Update POM file Modify artifactID to match
7e85139
to
80201ad
Compare
This PR reduces the wiring triplication on the TCK. |
Right. This doesn't prevent someone from running tests for just one signal type (I'm not sure why one would, as that's not how TCKs work, think :) , but it does make it easier to set up the environment to run the suite in general. |
I think having separate archives allows the runtime to just focus on one set of tcks if they only support subset of this spec. Besides, the three pillars have different progress on making changes etc. |
My guess is that the general case is that someone is implementing "the spec" and we should optimize for that. Someone can still test only one pillar if so desired. The process would be same (configure with a single jar), but the tests would be restricted to a given package. Playing the "what if" game too much (e.g., "what if someone only wants to support JVM metrics and not application?") can lead (as is the case here, I think) of optimizing for random, probably rare (ish) scenarios, which increases complexity for more common cases with little added value. I'm not sure the pace of development per pillar is relevant here, as the MP Telemetry spec pins otel to a given version, and the TCK covering all three pillars (whether in 1 jar or 3) is released with a single version (that of the spec) covering that version of otel. We are not producing pillar-specific TCK artifacts with varying versions, so this seems moot. |
I don't feel there is a business need for merging the archives. I prefer to keep as it is for now unless there are business reasons arsing for the merge. |
I don't see a business NOT to merge it, as it both maintains current flexibility while lowering the barrier to entry by reducing complexity. The only "valid" argument I can see is maintaining the status quo, but I find that pretty uncompelling. |
Move sources and resources
Update POM file
Modify artifactID to match
Merge READMEs
Resolves #244