Pin system tests for release branch#10189
Closed
dd-octo-sts[bot] wants to merge 154 commits into
Closed
Conversation
* set benchmarking jobs to be interruptible outside master * remove interruptible: true from benchmarks.yml * set macrobenchmarking jobs to be interuptible outside master * test interruptibility
* chore: Drop org.gradle.java.installations.fromEnv toolchain restriction * chore: Drop org.gradle.java.installations toolchain restriction for local env, but keep them in CI * chore: Drop org.gradle.java.installations toolchain restriction for local env, but keep them in CI
…raint (#9968) * chore: Drop org.gradle.java.installations.fromEnv toolchain restriction * chore: Drop org.gradle.java.installations toolchain restriction for local env, but keep them in CI * chore: Drop org.gradle.java.installations toolchain restriction for local env, but keep them in CI * feat: Allow discovery of testJvm without environment variables * feat: Only rely on Gradle's toolchain to run tests * fix: Auto discover JAVA_xx_HOME variables from the build image * typo: Use `-P` Co-authored-by: Sarah Chen <sarah.chen@datadoghq.com> --------- Co-authored-by: Sarah Chen <sarah.chen@datadoghq.com>
* Attempting to reduce contention for virtual threads * spotless
* chore: Move CSI plugin to it's own package * chore: Tweak some API to use kotlin friendly functions * fix: Restrict default CSI classpath to main and test, compile and runtime configurations. * chore: arg name rename * fix: Include the main source set output * fix: Always make sure java plugin is applied * chore: Make csi plugin not running in project.afterEvaluate * fix: Incorrect URLClassLoader initialization when url is a directory * fix: Make depends on instrument task The 'instrument' plugin changes destination around for now, it needs to be depended on for the time being. * chore: Refactor additional configurations to a `ConfigurableFileCollection` * fix: extension property should not be queried at configuration time * style: Reformat * fix: call site plugin still has to add entries to classpath after jvm test suite ran * chore: code tweaks * fix: generateCallSite was missing a few input properties for up-date checks * fix: csi plugin was patching test compile and test task classpath This forced to apply patching in afterEvaluate, instead, it's easier and cleaner to add the csi output as dependency in the appropriate configuration. * fix: unnecessary targetFolder directory creation and unnecessary transformation to string * chore: Readability of CallSiteUtils::toURL * typo: inout property type * fix: Exclude Kotlin compiler daemon files
…ager is present (#10009) * Safely handle Files.exist on discovery adn config * Add filesystem-utils only to shared
* Extracting static helper * Extract static version of buildSpanContext * Switch to singleSpanBuilder * Extracting static version of startSpan * spotless * Reorganizing link & parent resolution Previous static extraction caused a test to fail because links logic relied on having a side effect on the builder instance's List<AgentSpanLink> * spotless * Apply suggestion from @mcculls Co-authored-by: Stuart McCulloch <stuart.mcculloch@datadoghq.com> * Addressing review feedback - Change CoreSpanBuilder constants to be more intelligible * Addressing review feedback - using constants throughout --------- Co-authored-by: Stuart McCulloch <stuart.mcculloch@datadoghq.com>
* Replace IsPublicFilter with ClassNameFilter * Replace NoMatchFilter with ClassNameFilter * Remove now unused ClassCodeFilter * Document that NoMatchFilter is now a utility class * Catch potential security exception when checking file / installing shutdown hook
need to wait for having added the fingerprint into the Concurrent Map otherwise it will try to reinstrument the exception on the second call Generalize the usage of NonRetryable annotation because all debugger smoke tests rely on the fact that they are run only once (log file naming)
it seems to lead to crashes on JDK8
…9907) * Fix * New approach * remove tests * sef review * sef review * review
* move to syscall * update after review * update after review
* Fixed CallSiteInstrumentationPluginTest by fixing test `build.gradle` and copy jar file to test build path. * Fixed InstrumentPluginTest by addin missing `instrumentPluginClasspath` config to the test `build.gradle` * Introduced `check_build_src` job to run test in `buildSrc` folder on CI.
Co-authored-by: Matthew Li <matthew.li@datadoghq.com>
… 2 updates (#10023) Bumps the gh-actions-packages group with 2 updates in the / directory: [actions/checkout](https://github.com/actions/checkout) and [github/codeql-action](https://github.com/github/codeql-action). Updates `actions/checkout` from 5.0.0 to 6.0.0 - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@08c6903...1af3b93) Updates `github/codeql-action` from 4.31.0 to 4.31.5 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@4e94bd1...fdbfb4d) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: 6.0.0 dependency-type: direct:production update-type: version-update:semver-major dependency-group: gh-actions-packages - dependency-name: github/codeql-action dependency-version: 4.31.5 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: gh-actions-packages ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Crashtracker: dual ship to error tracking * Fix error tracking payload * Handle OOM and clean exit * leftover * adjust smoke test * Make error intake upload optin
Contributor
|
Testing #10174 ... Sorry again for the noise! |
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.
This PR pins the system-tests reference for the release branch.