Skip to content

Migrate PR Builder to GitHub Actions#560

Merged
JackPGreen merged 1 commit into
masterfrom
make-pr-builder
Oct 23, 2025
Merged

Migrate PR Builder to GitHub Actions#560
JackPGreen merged 1 commit into
masterfrom
make-pr-builder

Conversation

@JackPGreen
Copy link
Copy Markdown
Contributor

@JackPGreen JackPGreen commented Oct 21, 2025

#558 failed the PR builder because it runs on Jenkins, inside a custom Python-3.7 Docker image.

Python 3.7 has been end-of-life for 2 years, and is too old to support the proposed changes.

We could upgrade to a newer Docker image (or even make a new one), but I think it would be easier to migrate to GitHub Actions instead - the runner already includes a recent Python installation.

Changes:

  • create new GitHub action
  • dynamically determine languages to execute/test - previously markdown was missed

Post-merge actions:

@JackPGreen JackPGreen force-pushed the make-pr-builder branch 2 times, most recently from 8e1c82b to 9bc6068 Compare October 21, 2025 16:05
@JackPGreen JackPGreen changed the title make pr builder Make GitHub-Actions PR builder Oct 21, 2025
@hazelcast hazelcast deleted a comment from hz-devops-test Oct 21, 2025
@hazelcast hazelcast deleted a comment from hz-devops-test Oct 21, 2025
@hazelcast hazelcast deleted a comment from hz-devops-test Oct 21, 2025
@hazelcast hazelcast deleted a comment from hz-devops-test Oct 21, 2025
@hazelcast hazelcast deleted a comment from hz-devops-test Oct 21, 2025
@JackPGreen JackPGreen changed the title Make GitHub-Actions PR builder Migrate PR Builder to GitHub Actions Oct 21, 2025
@JackPGreen JackPGreen requested a review from ihsandemir October 21, 2025 17:18
@JackPGreen JackPGreen marked this pull request as ready for review October 21, 2025 17:18
@JackPGreen
Copy link
Copy Markdown
Contributor Author

Tested against #558, passed.

@JackPGreen JackPGreen enabled auto-merge (squash) October 21, 2025 17:19
@JackPGreen JackPGreen requested a review from emreyigit October 22, 2025 07:53
@JackPGreen JackPGreen merged commit 0189e80 into master Oct 23, 2025
9 checks passed
gareth-johnston pushed a commit to gareth-johnston/hazelcast-client-protocol that referenced this pull request Jan 14, 2026
hazelcast#558 failed
the PR builder because it runs on Jenkins, inside a custom Python-3.7
Docker image.

[Python 3.7 has been end-of-life for 2
years](https://devguide.python.org/versions/), and is too old to support
the proposed changes.

We _could_ upgrade to a [newer Docker
image](https://hub.docker.com/r/hazelcast/jenkins-python-client-3.9) (or
even [make a new
one](https://github.com/hazelcast/DevOps/tree/master/dockerfiles/clients/python)),
but _I think_ it would be easier to migrate to GitHub Actions instead -
the runner _already includes_ a recent Python installation.

Changes:
- create new GitHub action
- dynamically determine languages to execute/test - previously markdown
was missed

Post-merge actions:
- [ ] disable [Jenkins
job](https://jenkins.hazelcast.com/job/Client-Protocol-pr-builder/)
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.

3 participants