Skip to content

[SPARK-47325][INFRA] Use the latest buf-setup-action in github workflow#45433

Closed
panbingkun wants to merge 1 commit intoapache:masterfrom
panbingkun:test_buf-setup-action_1_30_0
Closed

[SPARK-47325][INFRA] Use the latest buf-setup-action in github workflow#45433
panbingkun wants to merge 1 commit intoapache:masterfrom
panbingkun:test_buf-setup-action_1_30_0

Conversation

@panbingkun
Copy link
Contributor

@panbingkun panbingkun commented Mar 8, 2024

What changes were proposed in this pull request?

The pr aims to unpin specific version buf-setup-action in github workflow building.

Why are the changes needed?

  • The last pin to a specific version was due to a bug in the version v1.29.0-1. The latest version has been upgraded to v1.30.0, and testing has found that this version is ok.

  • This latest version v1.30.0 has a change regarding the upgrade from node16 to node20.
    bufbuild/buf-setup-action@v1.29.0...v1.30.0
    image

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass GA.

Was this patch authored or co-authored using generative AI tooling?

No.

@github-actions github-actions bot added the INFRA label Mar 8, 2024
@zhengruifeng
Copy link
Contributor

LGTM pending CI

@panbingkun
Copy link
Contributor Author

LGTM pending CI

Well, let it run through GA first, and then I'll observe the log.
This version has a change regarding the upgrade from node16 to node20.
bufbuild/buf-setup-action@v1.29.0...v1.30.0
image

@panbingkun panbingkun changed the title [WIP][INFRA] Test buf-setup-action v1.30.0 [SPARK-47325][INFRA] Use the latest buf-setup-action in github workflow Mar 8, 2024
@panbingkun panbingkun marked this pull request as ready for review March 8, 2024 10:57
@panbingkun
Copy link
Contributor Author

panbingkun commented Mar 8, 2024

The related logs on GA are as follows:
image

@HyukjinKwon
Copy link
Member

Merged to master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments