Use weblog metadata to declare weblog's category, and connect to scenario matrix builder#7286
Draft
cbeauchesne wants to merge 8 commits into
Draft
Use weblog metadata to declare weblog's category, and connect to scenario matrix builder#7286cbeauchesne wants to merge 8 commits into
cbeauchesne wants to merge 8 commits into
Conversation
Contributor
|
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
Knowning which scenario runs on which weblog is quite complex. So far, it was done by one of the crappiest piece of code I've ever wrote : hardcoded names with organic rules that are added on top of each over threw times.
SSI and libinjection also has it own recipes, with a cleaner approach, where dat is stored in json, and logic in python.
As we'll create an official workflow in gitlab, we need a unified (and clean) way of doing so.
This PR propose an unified way of tacking this, and implement it to end-to-end scenario, SSI will come later
Changes
categoriesfield on weblog metadata data. This category describe what library is shipped inside the weblog :dd_trace: the basic dd-trace HTTP instrumentationdd_trace_graphql: a dd-trace GraphQL instrumentationdd_trace_lambda: a dd-trace lambda instrumentationdd_trace_frameworks: a dd-trace framework instrumentation (mostly AI)open_telemetry: an Open Telemetry instrumentationweblog_categoriesproperty onScenarioThe logic is quite simple : a scenario supports a given weblog if both of them has the same weblog category.
On top of that, weblog metadata has two extra field :
supported_scenariosandexcluded_scenarios, for any fine tune inclusion/exclusion.Few notes :
ymltojsonto store data, as it's easier to read for human (as long as we don't use crazy yml syntax)Workflow
🚀 Once your PR is reviewed and the CI green, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist
tests/ormanifests/is modified ? I have the approval from R&P teambuild-XXX-imagelabel is present