Skip to content

Conversation

@dividedmind
Copy link
Contributor

This pull request includes significant changes to the CI/CD pipeline and test setup. The primary focus is on migrating from Travis CI to GitHub Actions and improving the test environment configuration.

Required secrets

Note for release to work, some secrets need to be configured in Github Actions (see CI-README.md for details).

CI/CD Pipeline Migration and Improvements:

  • .github/workflows/build-and-test.yml: Added a new GitHub Actions workflow to handle building, testing, and releasing the project. This workflow includes jobs for building and checking the code, running the test suite across multiple Java versions, and handling releases.
  • .github/workflows/build.yml: Removed the existing GitHub Actions workflow for building the project, which included a matrix strategy for testing across different operating systems and Java versions.
  • .travis.yml: Removed the Travis CI configuration file, which included setup steps and jobs for testing the project with different JDK versions and handling deployments.

Test Environment Configuration:

  • agent/test/helper.bash: Enhanced the helper script to export JAVA_HOME if not already set, load BATS libraries if available, and handle logging more gracefully. [1] [2] [3] [4]
  • agent/test/access/access.bats, agent/test/agent_cli/agent_cli.bats, agent/test/classloading/classloading.bats, agent/test/event/event.bats, agent/test/http_client/httpclient/httpclient.bats, agent/test/http_client/setup_suite.bash, agent/test/http_client/springboot/springboot.bats, agent/test/httpcore/httpcore.bats, agent/test/intellij/intellij.bats, agent/test/jdbc/jdbc.bats, agent/test/petclinic-fw/petclinic-fw.bats: Removed the loading of BATS support and assert libraries, as they are now handled by the helper script. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]

Build Configuration Updates:

  • agent/test/http_client/httpclient/build.gradle, agent/test/httpcore/build.gradle, agent/test/jdbc/build.gradle: Updated the build scripts to fetch the appmapJar and annotationJar from environment variables instead of using file trees. [1] [2] [3]

Miscellaneous Changes:

These changes streamline the CI/CD process, improve the test setup, and ensure consistency across different environments.

@dividedmind dividedmind self-assigned this Mar 18, 2025
@dustinbyrne dustinbyrne self-requested a review March 20, 2025 14:58
Copy link
Collaborator

@dustinbyrne dustinbyrne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything's been re-keyed and should be good to go. See the comments above before merging.

@dustinbyrne
Copy link
Collaborator

agent/bin/multiline_input_to_travis_env.py can also be deleted now

@dividedmind
Copy link
Contributor Author

@dustinbyrne I've made these changes — can you merge? I cannot because gh is still waiting for travis — I think you need to disable this in the project settings.

@dividedmind dividedmind merged commit addd903 into master Mar 24, 2025
5 checks passed
@dividedmind dividedmind deleted the chore/gh-actions branch March 24, 2025 17:28
@appland-release
Copy link

🎉 This PR is included in version 1.27.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants