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

chore(CI): Add envs in build_debug_on_centos7 to ensure job can be run on centOS 7 #2065

Merged
merged 1 commit into from
Jul 11, 2024

Conversation

acelyc111
Copy link
Member

@acelyc111 acelyc111 commented Jul 8, 2024

@github-actions github-actions bot added the github label Jul 8, 2024
@acelyc111 acelyc111 marked this pull request as ready for review July 8, 2024 10:15
@acelyc111 acelyc111 merged commit 42bd5bb into apache:master Jul 11, 2024
20 of 21 checks passed
acelyc111 added a commit that referenced this pull request Jul 12, 2024
…ckout from v3 to v4 (#2062)

After All GitHub Actions run on Node20 instead of Node16 by default [1],
this patch bumps the actions/checkout version from v3 to v4.

When the related yaml files changed, the clients CIs are triggered, then
they expose that the new introduced structure `host_port` in IDL as unknow
because it is not generated by thrift automatically, we have to implement
it manually. So this patch also simplily implements the `host_port`
structure in python-client and nodejs-client.

It should be mentioned that only the `build_debug_on_centos7` is still using
`actions/checkout@v3`, see [2] for details.

1. https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
2. #2065
ruojieranyishen pushed a commit to ruojieranyishen/incubator-pegasus that referenced this pull request Jul 17, 2024
ruojieranyishen pushed a commit to ruojieranyishen/incubator-pegasus that referenced this pull request Jul 17, 2024
…ckout from v3 to v4 (apache#2062)

After All GitHub Actions run on Node20 instead of Node16 by default [1],
this patch bumps the actions/checkout version from v3 to v4.

When the related yaml files changed, the clients CIs are triggered, then
they expose that the new introduced structure `host_port` in IDL as unknow
because it is not generated by thrift automatically, we have to implement
it manually. So this patch also simplily implements the `host_port`
structure in python-client and nodejs-client.

It should be mentioned that only the `build_debug_on_centos7` is still using
`actions/checkout@v3`, see [2] for details.

1. https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
2. apache#2065
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