-
Notifications
You must be signed in to change notification settings - Fork 11
Add Test Optimization #303
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
base: main
Are you sure you want to change the base?
Conversation
1b31a06 to
ef7d3a3
Compare
c2fd653 to
e3bcd2f
Compare
.gitlab-ci.yml
Outdated
| # Retrieve common go deps | ||
| - mkdir -p $GOPATH/pkg/mod && tar xJf modcache_e2e.tar.xz -C $GOPATH/pkg/mod && rm -f modcache_e2e.tar.xz | ||
| # Install orchestrion | ||
| - go install github.com/DataDog/orchestrion@v0.9.4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💭 thought
We could install orchestrion in our e2e runner image, defined in `test-infra-definitions. Can be a follow up and we can handle it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes we can do that, just keep in mind that v0.9.4 is an non GA version. Currently the stable version requires a recent Go version (>= 1.22.9) so it must be in sync with the Go version you are using.
Co-authored-by: pducolin <45568537+pducolin@users.noreply.github.com>
This PR replaces the usage of the junitxml files with the Go native instrumentation using orchestrion for the Test Optimization product
Results: https://app.datadoghq.com/ci/test-branch/gitlab.ddbuild.io%2FDataDog%2Fagent-linux-install-script/agent-linux-install-script/tony%2Ftest-visibility?query=test_level%3Atest%20env%3Aci%20%40git.repository.id%3Agitlab.ddbuild.io%2FDataDog%2Fagent-linux-install-script%20%40test.service%3Aagent-linux-install-script%20%40git.branch%3Atony%2Ftest-visibility&env=ci&fromUser=false&index=citest&start=1730295156612&end=1732887156612&paused=false