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

build: use rclone instead of aws CLI #55617

Merged
merged 1 commit into from
Nov 2, 2024
Merged

Conversation

targos
Copy link
Member

@targos targos commented Oct 31, 2024

Use rclone to upload assets to Cloudflare as it is more reliable.

Refs: nodejs/build#3508

Use rclone to upload assets to Cloudflare as it is more reliable.

Refs: nodejs/build#3508
@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. windows Issues and PRs related to the Windows platform. labels Oct 31, 2024
@targos
Copy link
Member Author

targos commented Oct 31, 2024

Draft because the server is not fully configured for it to work.

/cc @nodejs/build-infra @flakey5 @ovflowd

I'll undraft when nodejs/build#3945 is merged.

@targos targos removed the windows Issues and PRs related to the Windows platform. label Oct 31, 2024
@richardlau richardlau added lts-watch-v18.x PRs that may need to be released in v18.x. lts-watch-v20.x PRs that may need to be released in v20.x lts-watch-v22.x PRs that may need to be released in v22.x labels Oct 31, 2024
@targos targos marked this pull request as ready for review November 1, 2024 08:07
Copy link

codecov bot commented Nov 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.43%. Comparing base (9967080) to head (cf8c6cd).
Report is 14 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #55617      +/-   ##
==========================================
- Coverage   88.43%   88.43%   -0.01%     
==========================================
  Files         654      654              
  Lines      187698   187729      +31     
  Branches    36126    36142      +16     
==========================================
+ Hits       165986   166011      +25     
+ Misses      14953    14947       -6     
- Partials     6759     6771      +12     

see 45 files with indirect coverage changes

@targos targos added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 1, 2024
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 1, 2024
@nodejs-github-bot
Copy link
Collaborator

@targos targos added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 2, 2024
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 2, 2024
@nodejs-github-bot nodejs-github-bot merged commit d37214b into nodejs:main Nov 2, 2024
108 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in d37214b

louwers pushed a commit to louwers/node that referenced this pull request Nov 2, 2024
Use rclone to upload assets to Cloudflare as it is more reliable.

Refs: nodejs/build#3508
PR-URL: nodejs#55617
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
aduh95 pushed a commit that referenced this pull request Nov 5, 2024
Use rclone to upload assets to Cloudflare as it is more reliable.

Refs: nodejs/build#3508
PR-URL: #55617
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Issues and PRs related to build files or the CI. lts-watch-v18.x PRs that may need to be released in v18.x. lts-watch-v20.x PRs that may need to be released in v20.x lts-watch-v22.x PRs that may need to be released in v22.x needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants