Skip to content

Update release/nightly uploads for new destination - #4837

Merged
rgaudin merged 1 commit into
mainfrom
upload_new_dest
May 6, 2026
Merged

Update release/nightly uploads for new destination#4837
rgaudin merged 1 commit into
mainfrom
upload_new_dest

Conversation

@rgaudin

@rgaudin rgaudin commented Apr 24, 2026

Copy link
Copy Markdown
Member

As part of a server/infra change, the upload of nightly and release builds is done to a new destination.

The receiving service now uses per-project (well per-repo) users with independent SSH Keys. Each users now only has access to the folders in which it is expected to upload files.

  • Host remains the same: master.download.kiwix.org
  • Username changes from ci to kiwix-android
  • Port changes from 30022 to 30322
  • Paths remains the same

⚠️ DO NOT MERGE ATM

@rgaudin rgaudin self-assigned this Apr 24, 2026
@kelson42 kelson42 added this to the 4.0.0 milestone Apr 29, 2026
As part of a server/infra change, the upload of nightly and release builds
is done to a new destination.

The receiving service now uses per-project (well per-repo) users with
independent SSH Keys. Each users now only has access to the folders
in which it is expected to upload files.

- Host remains the same: `master.download.kiwix.org`
- Username changes from `ci` to `kiwix-android`
- Port changes from `30022` to `30322`
- Paths remains the same
@kelson42
kelson42 force-pushed the upload_new_dest branch from 5a58de6 to cdb8381 Compare May 6, 2026 04:32
@kelson42

kelson42 commented May 6, 2026

Copy link
Copy Markdown
Collaborator

@rgaudin What is the status here? Are we ready to review and merge?

@codecov

codecov Bot commented May 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.17%. Comparing base (5d18dd1) to head (cdb8381).

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #4837      +/-   ##
============================================
- Coverage     59.36%   59.17%   -0.19%     
+ Complexity     1636     1631       -5     
============================================
  Files           334      334              
  Lines         16308    16308              
  Branches       2278     2278              
============================================
- Hits           9681     9651      -30     
- Misses         5147     5176      +29     
- Partials       1480     1481       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rgaudin
rgaudin marked this pull request as ready for review May 6, 2026 13:38
@rgaudin

rgaudin commented May 6, 2026

Copy link
Copy Markdown
Member Author

Not meant to be reviewed, it's only infra details.

@rgaudin
rgaudin merged commit a5618cb into main May 6, 2026
35 checks passed
@rgaudin
rgaudin deleted the upload_new_dest branch May 6, 2026 14:15
@MohitMaliFtechiz MohitMaliFtechiz mentioned this pull request May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants