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

CI: Release workflow #237

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ljubon
Copy link

@ljubon ljubon commented Apr 3, 2024

Previous PR for the same topic was #200, but closed as it required major changes and approach which is shown in this PR

What type of PR is this?

This is an improvement PR with the goal of enabling spark-extension release using GitHub workflows

What this PR does / why we need it:

The goal of this task is to move the release process to GitHub workflow from current scripts that are run manually.
The current release process is documented in [RELEASE.md](https://github.com/G-Research/spark-extension/blob/master/RELEASE.md/.
Due to its complexity, we reached out to one of the maintainers Enrico who approved this idea of moving release.sh to a github workflow for it's transparent and better usage.

Acceptance criteria / Notes for the reviewer:

  • Comment on the issues from the maintainer
  • Keep the current process as is (not introducing breaking changes), so the maintainers should always be able to do it the old way - but this PR should introduce a workflow that will copy that behavior

Most of the comments and testing were done forked repo
Part of maturity matrix: https://github.com/G-Research/oss-portfolio-maturity/issues/32


- name: Import GPG key
id: sign
uses: crazy-max/ghaction-import-gpg@01dd5d3ca463c7f10f7f4f7b4f177225ac661ee4
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we need to set the GPG_PRIVATE_KEY secret. Which key do you propose we use?

Copy link
Author

Choose a reason for hiding this comment

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

Hi @EnricoMi , in latest commit i added a environment release for this step - could you please create this environment and secret associated to it?
It would be best to have enabled Deployment branches and tags with Selected branches and tags as this job would be triggered only from v* tags pattern

Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't we use a private key owned by GR here, not my personal one? Same with the OSSRH login, that should be a general GR login used by all packages that deploy to maven, not a personal access.

.github/workflows/publish-release.yml Show resolved Hide resolved
pom.xml Outdated Show resolved Hide resolved
@ljubon
Copy link
Author

ljubon commented May 15, 2024

FYI rebase & resolving config is done

Copy link

github-actions bot commented May 15, 2024

Test Results

   631 files  ±0     631 suites  ±0   1h 45m 19s ⏱️ + 2m 11s
   499 tests ±0     499 ✅ ±0   0 💤 ±0  0 ❌ ±0 
19 419 runs  ±0  19 355 ✅ ±0  64 💤 ±0  0 ❌ ±0 

Results for commit 3cde51b. ± Comparison against base commit ce8737b.

♻️ This comment has been updated with latest results.

@ljubon
Copy link
Author

ljubon commented May 16, 2024

i just realised during resolving conflict i lost few changes that should be here, will move this PR as draft until i get them back
sorry for this issue which i caused 🙏

@ljubon ljubon marked this pull request as draft May 16, 2024 09:40
@ljubon ljubon marked this pull request as ready for review May 20, 2024 10:43
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.

None yet

2 participants