-
Notifications
You must be signed in to change notification settings - Fork 2
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
disable terminals #243
disable terminals #243
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this PR linked to the wrong Jira ticket.
Check the branch name
@sigmarkarl This will be superseded by your PR in the jupyter-server repo right ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
image tag
and appVersion
?
@@ -10,6 +10,7 @@ server: | |||
token: '' | |||
ip: '0.0.0.0' | |||
allowOrigin: '*' | |||
terminals_enabled: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what will happen if set to true
in case the terminal is totally removed in the Jupiter server?
we will remove the terminado and jupyter_server_terminals in our custom jupyterlab Dockerfile (bigdata-infrastructure) instead as other components might install terminado |
Jira ticket
https://spotinst.atlassian.net/browse/BGD-5956
Description
Hiding terminal in jupyterlab. This is a cosmetic change and will be further handled in https://spotinst.atlassian.net/browse/BGD-5956
Demo
The launcher now looks like
No terminals launcher visible
Checklist
How to test
See demo screenshot. The terminal launcher should no longer be visible on the jupyterlab launcher page.