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

Filezilla remote tunneling instructions #453

Merged
merged 5 commits into from
Jul 25, 2024

Conversation

cdkharris
Copy link
Contributor

Adds a site snippet to provide instructions for using Filezilla with tunnels. At UCL many users need to connect to the cluster remotely through our SSH bastion without using our VPN, so we provide instructions for using tunnels to do this.

@reid-a
Copy link
Contributor

reid-a commented Jul 25, 2024

Thanks for this, this looks useful. It's breaking the CI, it looks like some of the configs don't like it. I'll have a look, but it might be an easier fix at your end?

@cdkharris
Copy link
Contributor Author

cdkharris commented Jul 25, 2024

Looks like the problem is that I created a new snippet file for our site, but I haven't created the corresponding files for the other sites. I'll add some blank files for these snippets.

Before I add those, is this a useful addition to the lesson? It's useful for us, but I don't know whether it's useful for other sites.

@reid-a
Copy link
Contributor

reid-a commented Jul 25, 2024

Honestly not sure. There are two reasons not to do it, firstly it might be specific to your site, and secondly we have a long-standing TODO about moving our repo away from snippets when we migrate it to the Carpentries Workbench.

My own personal opinion is that we should err on the site of inclusiveness and enriched functionality, and that doing this is a small increase in migration work, so I favor it. But I am not the one doing the work, either. So if you're looking for encouragement, consider yourself encouraged, but not much more than that!

@cdkharris
Copy link
Contributor Author

cdkharris commented Jul 25, 2024

Understood; I will add this change because it's possible that someone else will be interested in using SSH tunnels with Filezilla, but I'll keep anything more site-specific than this in our fork.

Copy link
Contributor

@reid-a reid-a left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@reid-a reid-a merged commit 20aa87c into carpentries-incubator:gh-pages Jul 25, 2024
10 checks passed
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.

3 participants