Skip to content

Improve release jenkins build #512

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

sgraband
Copy link
Contributor

@sgraband sgraband commented Apr 7, 2025

What it does

Improvements include a multi step setup, consisting of:

  • Building
  • Sigining
  • Uploading

Where now each of the steps can be triggered individually and if one succeeds we do not need to run it again.
This is done by creating artifacts between the steps and a build number parameter, that can be reused to retrieve the successful runs.
Also tried to improve the error handling where we now should fails faster without having to wait for all steps before we can start a new build.
The build now needs to be manually triggered for releases. This avoids overhead for merging non release changes.
For PRs the building and signing is still executed. We could remove the signing from this, but for testing this PR is should be useful.

Updated the Jenkins pipeline configs for this.

How to test

This PR should trigger the build. I would like to test this with the Theia IDE release today.

Successful PR build: https://ci.eclipse.org/theia/job/theia-ide-pr-check/view/change-requests/job/PR-512/39/

Successful Dry_RUN (manually triggered):

Review checklist

Reminder for reviewers

@sgraband sgraband requested a review from jfaltermeier April 7, 2025 14:02
@sgraband sgraband force-pushed the sg/improvePublishing branch 4 times, most recently from faf4b2c to dac4fa4 Compare April 8, 2025 11:55
@sgraband sgraband force-pushed the sg/improvePublishing branch 12 times, most recently from 1bd9ac4 to 2c49087 Compare April 23, 2025 13:21
@sgraband sgraband force-pushed the sg/improvePublishing branch from 2c49087 to c417b1d Compare April 24, 2025 06:54
@sgraband sgraband force-pushed the sg/improvePublishing branch 3 times, most recently from 0f47deb to 6f0574e Compare May 8, 2025 07:24
@sgraband sgraband force-pushed the sg/improvePublishing branch from 6f0574e to e074132 Compare May 8, 2025 07:31
sgraband added 2 commits May 8, 2025 10:24
Signed-off-by: Simon Graband <[email protected]>
Signed-off-by: Simon Graband <[email protected]>
@sgraband sgraband force-pushed the sg/improvePublishing branch from b432302 to c0181e9 Compare May 8, 2025 08:57
Signed-off-by: Simon Graband <[email protected]>
@sgraband sgraband force-pushed the sg/improvePublishing branch from c0181e9 to a623e16 Compare May 8, 2025 09:03
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.

1 participant