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

added rsync to list of upload protocols #3580

Closed

Conversation

MichelMoriniaux
Copy link

was having a lot of issues with scp slowness to upload from my remote observatory to the other side of the world. rsync solved all my problems.

updated uploads.sh and the documentation

was having a lot of issues with scp slowness to upload from my remote observatory to the other side of the world. rsync solved all my problems.

updated uploads.sh and the documentation
@EricClaeys
Copy link
Collaborator

@MichelMoriniaux, Michel, nice job, very complete.
Would you be able to modify this PR (or create a new one) against the v2024.xx.xx branch? That's the release we're almost done with, so we're not making any more changes to the current release. It should be fairly easy.

In the new release we merged ftp-settings.sh and config.sh into config/settings.json, which means ALL settings (except Website ones) are now modified via the WebUI. We also added ~/env.json that contains all the private settings like passwords, etc. This is so a web page can't see them, and so people can send us their settings.json file without having to hide private data.

  • The config_repo/options.json.repo determines what settings appear in the WebUI, what their options are etc. You'll need to add "rsync" to it. Maybe put it after "scp" since they are somewhat related, but anywhere in the list is fine.
  • Your changes to config_repo/ftp-settings.sh.repo aren't needed.
  • The allskyWebsite.html changes will need minor modifications - you won't have any problems.
  • check_allsky.sh changes: these will need to go in check_remote_server() in scripts/installUpgradeFunctions.sh. You can probably just add "rsync" to the ftp | ftps | sftp | scp) line and add in the *) section.
  • functions.sh changes aren't needed.
  • upload.sh: I suggest adding "rsync" to the elif [[ "${PROTOCOL}" == "scp" ]] ; then line, then add an "if" after the existing "if" to perform the correct commands. I think "--port=PORT" is needed instead of "-p PORT" for the REMOTE_PORT. According to the rsync manual page "-p" is to preserver permissions.

I suspect you'll be able to make the changes in about 30 minutes. Feel free to download the v2024.xx.xx branch and play around with it (but keep your current code). Note that v2024.xx.xx doesn't currently allow you to install a remote website and there are some other bugs, but I'm running it on a few machines.

Thanks - Eric

@MichelMoriniaux
Copy link
Author

MichelMoriniaux commented Apr 20, 2024 via email

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