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.

Sign up for GitHub

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

Jump to bottom

Reorganized jenkinsfile #24320

Merged
merged 1 commit into from
Feb 27, 2023
Merged

Reorganized jenkinsfile #24320

merged 1 commit into from
Feb 27, 2023

Conversation

dmatej
Copy link
Contributor

@dmatej dmatej commented Feb 25, 2023 •
edited
Loading

  • verify will follow the compilation, ant tests will be executed as last in parallel jobs with lower resources
  • more resources for compilation and maven integration tests
  • if maven fails, ant will not run now
  • Step to future: my intention is to start migrating the Zombie Army (Ant tests) to Maven. At some point I will start splitting the maven part to parallel tests too, but they will be much faster than now with Ant.

Note - this change is not going back before 1baa948 :

  • We are using standard images now (maven+temurin+ubuntu docker image and basic Jenkins nodes)
  • I just merged recommendations of https://wiki.eclipse.org/Jenkins with recently abandoned k8s settings, because state before this commit was quite sensitive to the hardware chosen by Jenkins ... sometimes. I have noticed at least two different CPUs, one 2,1GHz, second much more modern 4GHz. Another factor were k8s settings. Especially tests running more instances of GlassFish then had issues with timeouts.

@dmatej dmatej force-pushed the jenkins branch 6 times, most recently from db53380 to 4d65c54 Compare February 25, 2023 20:40
@dmatej dmatej marked this pull request as ready for review February 25, 2023 20:53
@dmatej dmatej self-assigned this Feb 25, 2023
@dmatej dmatej added this to the 7.0.3 milestone Feb 25, 2023
@dmatej dmatej marked this pull request as draft February 26, 2023 07:17
@dmatej
Reorganized jenkinsfile
ce3d2a8 
- verify will follow the compilation, ant tests will be executed as last
- more resources for maven integration test
- maven latest leads to 3.8.5, so I used explicit version

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
@dmatej dmatej marked this pull request as ready for review February 26, 2023 07:32
@dmatej dmatej mentioned this pull request Feb 26, 2023
Merged
@dmatej dmatej merged commit 8ef5403 into eclipse-ee4j:master Feb 27, 2023
@dmatej dmatej deleted the jenkins branch February 27, 2023 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hs536 hs536 hs536 approved these changes

@pzygielo pzygielo Awaiting requested review from pzygielo

@arjantijms arjantijms Awaiting requested review from arjantijms

Assignees

@dmatej dmatej

Labels
build and test improvement
Projects
None yet
Milestone
7.0.3
Development

Successfully merging this pull request may close these issues.
2 participants
@dmatej @hs536