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

Few updates from reading through the docs #325

Merged
merged 3 commits into from Oct 22, 2021

Conversation

znicholls
Copy link
Contributor

@znicholls znicholls commented May 17, 2019

Closes #338

@znicholls
Copy link
Contributor Author

@yuvipanda and team thanks for all your efforts on this again! I'm just spinning up my own server and will add any typos I find in here, hopefully that's useful.

One other thing. I don't know if it's useful but I somehow managed to break my server yesterday, but it now works after re-building the machine - would it be worth adding a 'Have you tried re-booting from scratch?' reminder to the troubleshooting page of custom servers?

@yuvipanda
Copy link
Collaborator

This is awesome, @znicholls! Thank you for making the effort to make the PR. Your contributions are very useful, and hopefully we'll see more :) This PR is still marked as draft, so am happy for you to add more to this or mark it as ready - it's in a mergeable state now.

Re: re-booting from scratch, did you just reboot the machine or delete things and start over? Might be worth opening a separate issue to discuss that.

Welcome to the TLJH github!

@znicholls
Copy link
Contributor Author

This PR is still marked as draft, so am happy for you to add more to this or mark it as ready - it's in a mergeable state now.

I'll add more as I keep reading and then mark it ready later.

@znicholls
Copy link
Contributor Author

Re: re-booting from scratch, did you just reboot the machine or delete things and start over? Might be worth opening a separate issue to discuss that.

#338 :)

Copy link
Member

@consideRatio consideRatio left a comment

Choose a reason for hiding this comment

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

I think the comment about restarting/deleting the server was a big vague.

Does it mean something to restart jupyterhub and traefik, the entire machine. And respectively, does delete mean to reset the entire machine, or to do an uninstallation of TLJH - which I recall to not be documented in such a clear way.


The other changes look great to me without a doubt!

@znicholls
Copy link
Contributor Author

It means either a restart or simply deleting the server entirely and spinning up a new virtual machine. I don't think it's possible to easily uninstall tljh (or it wasn't when I made this PR)

@minrk minrk marked this pull request as ready for review October 22, 2021 11:44
@consideRatio consideRatio merged commit 56b6521 into jupyterhub:main Oct 22, 2021
@welcome
Copy link

welcome bot commented Oct 22, 2021

Congrats on your first merged pull request in this project! 🎉
congrats
Thank you for contributing, we are very proud of you! ❤️

@consideRatio
Copy link
Member

Thanks @znicholls and @minrk for the followup adjustment!

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.

Document 'delete everything and start over' as a troubleshooting tactic
4 participants