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.

Sign up for GitHub

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

Jump to bottom

Fix checkout action failure #2626

Closed
wants to merge 1 commit into from
Closed

Fix checkout action failure #2626

wants to merge 1 commit into from

Conversation

cwperks
Copy link
Member

@cwperks cwperks commented Jul 9, 2024

Description

CI is failing with

/__e/node20/bin/node: /lib64/libm.so.6: version `GLIBC_2.27' not found (required by /__e/node20/bin/node)
/__e/node20/bin/node: /lib64/libc.so.6: version `GLIBC_2.28' not found (required by /__e/node20/bin/node)

Added ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION: true variable to use Node16

PR in SQL repo where this was also addressed: opensearch-project/sql#2807

Issues Resolved

actions/checkout#1809

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@cwperks
Fix checkout action failure
28d852d 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks cwperks temporarily deployed to ml-commons-cicd-env July 9, 2024 14:41 — with GitHub Actions Inactive
@cwperks cwperks had a problem deploying to ml-commons-cicd-env July 9, 2024 14:41 — with GitHub Actions Failure
@cwperks cwperks temporarily deployed to ml-commons-cicd-env July 9, 2024 15:13 — with GitHub Actions Inactive
@cwperks cwperks had a problem deploying to ml-commons-cicd-env July 9, 2024 15:13 — with GitHub Actions Failure
@cwperks cwperks closed this Jul 9, 2024
@peterzhuamazon
Copy link
Member

peterzhuamazon commented Jul 9, 2024 •
edited
Loading

Hi @cwperks ,

Fixed in #2627, due to pull_request_target the change will only take effect after merge.

Thanks.

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

@b4sjoo b4sjoo Awaiting requested review from b4sjoo b4sjoo is a code owner

@dhrubo-os dhrubo-os Awaiting requested review from dhrubo-os dhrubo-os is a code owner

@jngz-es jngz-es Awaiting requested review from jngz-es jngz-es is a code owner

@model-collapse model-collapse Awaiting requested review from model-collapse model-collapse is a code owner

@rbhavna rbhavna Awaiting requested review from rbhavna rbhavna is a code owner

@ylwu-amzn ylwu-amzn Awaiting requested review from ylwu-amzn ylwu-amzn is a code owner

@zane-neo zane-neo Awaiting requested review from zane-neo zane-neo is a code owner

@Zhangxunmt Zhangxunmt Awaiting requested review from Zhangxunmt Zhangxunmt is a code owner

@austintlee austintlee Awaiting requested review from austintlee austintlee is a code owner

@HenryL27 HenryL27 Awaiting requested review from HenryL27 HenryL27 is a code owner

@sam-herman sam-herman Awaiting requested review from sam-herman

@xinyual xinyual Awaiting requested review from xinyual xinyual is a code owner

Assignees
No one assigned
Labels
infra
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cwperks @peterzhuamazon