-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Not possible to configure projects sources for LS launched on standalone machine #10032
Comments
I'm not really sure what you're trying to achieve here: should a URI like file:///projects/project1/pom.xml be mapped to something like file:///groupdir/mylocation/project1/pom.xml in all requests to the language server? |
If there is no urgent requirement behind this change, I would ask that we postpone it until we have switched to jdt.ls |
I thought we did not have plans to introduce side-cars to Che 6 line? |
@gazarenkov could you share some more details on which language server you're trying to enable? Maybe there is an alternative way to work around the problem. |
@tsmaeder sure, I am working with Type Script language server (dogfooding Che for client side code) as a standalone docker container and have no ability to have the same projects folder mount path for it and ws-agent. |
@gazarenkov I still don't see why we can't point the typescript language server at /projects and mount that directory in the image. They need to be the same files anyway, because ws-agent is changing those files on disk. |
@tsmaeder thanks for being reactive :) In my case I can not physically, indeed, since I use local machine and source code is not on the root directory level (apparently coding under root user is not the choice). But even if not taking it into account, we already have ability to mount projects to any directory, it is documented feature and it is expected that any application using source code have to respect this. |
@gazarenkov mounting the projects is fine, but it has to be in the same directory on both machines. I have objections against this PR for two reasons:
|
@tsmaeder |
Seems like after jdt.ls branch is merged there are some inconsistencies in related pull request, I see some changes to text document service and language server configuration, probably in some other places too. So we will first need to resolve conflicts, revise the pull request and adapt the solution to newly introduced changes. |
LS always expects source code on the same path as it configured for "ws-agent" machine (machine where Project API located).
Inability to configure this path limit using it on in case when LS launched in standalone machine (a k a sidecar) especially for the cases when there are no permissions to use the same dir or LS image is provided by third party.
The text was updated successfully, but these errors were encountered: