Skip to content

Migrate deploys from SFTP to rsync (#146) — rsync-only - #169

Open
ijiraq wants to merge 1 commit into
ivoa:mainfrom
ijiraq:cursor/rsync-upload-only-ec7c
Open

ijiraq wants to merge 1 commit into
ivoa:mainfrom
ijiraq:cursor/rsync-upload-only-ec7c

Conversation

@ijiraq

@ijiraq ijiraq commented Nov 7, 2026 •

Copy link
Copy Markdown

Summary

Rsync-only change for #146: switch GitHub Actions production and branch (webtest) uploads from SFTP to a first-party rsync-upload action.

Colleagues asked to split this out of the mixed #162 PR. This PR is only the deploy transport swap. Docker local preview, mock deploy smoke, and secret-free CI preview tooling are intentionally not here — those come in a later PR.

Included

  • New .github/actions/rsync-upload/ composite (rsync over SSH)
  • deploy_main.yaml and deploy_version.yaml wired to rsync-upload
  • Same secret names as today (SFTP_PROD_*, SFTP_STAGE_*)
  • sftp-upload kept but marked deprecated until staging confirms rsync
  • Brief CONTRIBUTING note that deploys use rsync

Test plan

  • Review rsync-upload for SSH key handling and --delete mirror behavior
  • Confirm deploy workflows still pass the same SFTP_* secrets / inputs
  • Staging dry-run on webtest after merge (maintainer)
  • After staging OK, remove deprecated sftp-upload in a follow-up

Sandbox (fork CI): ijiraq#2

Add a first-party rsync-upload composite action and wire deploy_main /
deploy_version to it, keeping the existing SFTP_* secret names. Mark
sftp-upload deprecated until staging confirms rsync. Document the swap
briefly in CONTRIBUTING (rsync-only; Docker preview follows separately).

Co-authored-by: jj.kavelaars <jj.kavelaars@nrc-cnrc.gc.ca>
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