-
Notifications
You must be signed in to change notification settings - Fork 106
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
Shell app should read cluster configs for default login #358
Comments
For this, its to have the Dashboard app provide an endpoint that just returns the JSON for the list of available login hosts. The reason is that there is already logic in place to handle loading the cluster configs, and filtering out those clusters that do not contain ssh login hosts and are not hidden (like quick) or not valid (like Ruby for those not in the ruby group). |
For the shell app, the default would be the first of this list UNLESS the default is specified in the environment variable that is currently used to configured the shell app. |
See ondemand/apps/dashboard/app/apps/ood_app.rb Lines 87 to 91 in 538b6c5
|
So... there is a problem with my proposal of using the dashboard endpoint: currently there is no way for an app to do a backend request to another app because of authentication. We could do experimentation with making a request directly to the PUN domain socket if we modified the permissions of the directory containing the socket. A simple way to test that would be to use curl to send requests to the socket on webtest02 or webdev02. Otherwise, the only way is to make the request client side - which is fine if you want to do this for the files app, but does not work for the shell app's use case, which is to provide a sensible default for login host for the Shell app. |
Closing as duplicate to #508 |
Right now the shell app's default login is localhost, and must be changed by explicitly setting an env var. Another approach is if the nginx_stage script read the cluster configs and set some initial env vars appropriately. The Files app should also be able to read cluster config to provide a dropdown list of hosts to open ssh to.
The text was updated successfully, but these errors were encountered: