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

Update GHA pants-init action to fix build #5782

Merged
merged 1 commit into from
Oct 24, 2022
Merged

Update GHA pants-init action to fix build #5782

merged 1 commit into from
Oct 24, 2022

Conversation

cognifloyd
Copy link
Member

Github updated the default minor version of python 3.9.x The cache key did not include the minor version which broke things for us.

Luckily, this was already fixed in pantsbuild/actions#6. The fix requires an update to the ./pants entrypoint/bootstrap script which was updated in pantsbuild/setup#128.

Based on pants docs, I updated ./pants with:

curl -L -O https://static.pantsbuild.org/setup/pants && chmod +x ./pants

Github updated the default minor version of python 3.9.x
The cache key did not include the minor version which broke things for us.

Luckily, this was already fixed in pantsbuild/actions#6.
The fix requires an update to the `./pants` entrypoint/bootstrap script
which was updated in pantsbuild/setup#128.

updated `./pants` with:
curl -L -O https://static.pantsbuild.org/setup/pants && chmod +x ./pants
@cognifloyd cognifloyd added this to the pants milestone Oct 22, 2022
@cognifloyd cognifloyd self-assigned this Oct 22, 2022
@pull-request-size pull-request-size bot added the size/M PR that changes 30-99 lines. Good size to review. label Oct 22, 2022
@cognifloyd cognifloyd enabled auto-merge October 22, 2022 13:29
@cognifloyd cognifloyd disabled auto-merge October 22, 2022 13:29
@cognifloyd cognifloyd enabled auto-merge (squash) October 22, 2022 13:35
Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

I think it's also fine pinning python=3.8

@cognifloyd
Copy link
Member Author

The version of python installed for pants (inside this pants-init action) is unrelated to the versions we use for our code.

@cognifloyd
Copy link
Member Author

If we do want to change the version of python used by pants, there is an action input parameter that handles that.

@arm4b arm4b requested a review from a team October 23, 2022 16:20
@cognifloyd cognifloyd merged commit 6aa6198 into master Oct 24, 2022
@cognifloyd cognifloyd deleted the pants-init branch October 24, 2022 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure: ci/cd maintenance pantsbuild size/M PR that changes 30-99 lines. Good size to review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants