Skip to content
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

Add initial Spring Native smoke tests #4034

Merged
merged 15 commits into from
Dec 19, 2022
Merged

Conversation

PerfectSlayer
Copy link
Contributor

@PerfectSlayer PerfectSlayer commented Oct 17, 2022

What Does This Do

This PR introduces smoke tests for Java tracer running on GraalVM Native Image project.

It is based on a Spring Native project with Spring Boot MVC.
Like the quarkus smoke test, it contains a sub-project in an application folder seperated from our main build script.
I could not find a way to make work our buildscripts with the AOT plugin, the native-image plugin, and Spring all together.

Motivation

In order to assess our progress on GraalVM support, smoke tests are the minimal tests we can run.

Additional Notes

It requires: DataDog/dd-trace-java-docker-build#42

@PerfectSlayer PerfectSlayer changed the title Add initial spring native smoke tests Add initial Spring Native smoke tests Oct 17, 2022
@PerfectSlayer PerfectSlayer added the comp: native-image GraalVM native-image label Oct 18, 2022
@PerfectSlayer PerfectSlayer force-pushed the bbujon/native-smoke-tests branch from 0e6c887 to 5322a65 Compare November 28, 2022 15:47
@mcculls mcculls force-pushed the bbujon/native-smoke-tests branch from a05342f to 375b99a Compare December 19, 2022 10:12
@mcculls mcculls marked this pull request as ready for review December 19, 2022 11:18
@mcculls mcculls requested a review from a team as a code owner December 19, 2022 11:18
@PerfectSlayer PerfectSlayer merged commit 5dc0eae into master Dec 19, 2022
@PerfectSlayer PerfectSlayer deleted the bbujon/native-smoke-tests branch December 19, 2022 12:52
@github-actions github-actions bot added this to the 1.3.0 milestone Dec 19, 2022
@mcculls mcculls added comp: testing Testing tag: no release notes Changes to exclude from release notes labels Dec 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp: native-image GraalVM native-image comp: testing Testing tag: no release notes Changes to exclude from release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants