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

Increases CI Speed #406

Merged
merged 57 commits into from
Sep 9, 2024
Merged

Conversation

FriedJannik
Copy link
Member

@FriedJannik FriedJannik commented Aug 29, 2024

This commit completely changes the CI workflow:

  • Instead of building each component and Docker image sequentially, there are now two separate workflows, each running multiple jobs in parallel.

The new workflows are:

  • Build and launch Docker images
  • Build and test BaSyx

There are also some minor changes to some test classes where the dirties context was set to AFTER_EACH_TEST_METHOD, which was not actually required. This setting restarted the context after each test, now with the new AFTER_CLASS setting the context is not restarted, improving test time.

In addition changes were made to the Build Plugins in the parent pom.xml:

  • Eclipse Dash License Tool is now not run on every build as it takes some time to run.
  • JavDoc Generator Plugin is also not run on every build. NOTE: JavaDoc is still Verified, only the HTML Documentation is not generated
    The removed plugins are now only in those build processes where they are required

With these changes we can improve the build time by 250% (or from ~30 minutes to ~12 minutes).

@FriedJannik FriedJannik self-assigned this Aug 29, 2024
@FriedJannik FriedJannik marked this pull request as ready for review August 30, 2024 07:59
@aaronzi aaronzi merged commit 2e65055 into eclipse-basyx:main Sep 9, 2024
27 checks passed
FriedJannik added a commit to FriedJannik/basyx-java-server-sdk that referenced this pull request Sep 9, 2024
FriedJannik added a commit to FriedJannik/basyx-java-server-sdk that referenced this pull request Sep 9, 2024
FriedJannik added a commit to FriedJannik/basyx-java-server-sdk that referenced this pull request Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants