-
Notifications
You must be signed in to change notification settings - Fork 6.3k
Update Code to 1.90.0 #6824
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
Update Code to 1.90.0 #6824
Conversation
- Update Node to 20.11.1 - Update documentation
code-server-4.90.0-rc.1-linux-amd64.tar.gz
with Code 1.90.0 is deployed at https://coder.jupyter.b-data.ch.
Functionality [modified by patches] tested and fond not to work:
- marketplace
Neither working in GUI nor Terminal. Output when executing e.g. code-server --install-extension alefragnani.project-manager
#8 11.48 Error while installing extensions: End of central directory record signature not found. Either not a zip file, or file is truncated.
#8 11.48 End of central directory record signature not found. Either not a zip file, or file is truncated.
#8 11.48 [2024年06月06日T08:35:55.162Z] error parent:70 Uncaught exception: End of central directory record signature not found. Either not a zip file, or file is truncated.
#8 11.48 [2024年06月06日T08:35:55.162Z] error parent:70 Error: End of central directory record signature not found. Either not a zip file, or file is truncated.
#8 11.48 at y (/opt/code-server/lib/vscode/out/vs/server/node/server.main.js:105:2374)
#8 11.48 at /opt/code-server/lib/vscode/out/vs/server/node/server.main.js:105:3812
#8 11.48 at /opt/code-server/lib/vscode/node_modules/yauzl/index.js:40:7
#8 11.48 at /opt/code-server/lib/vscode/node_modules/yauzl/index.js:190:5
#8 11.48 at /opt/code-server/lib/vscode/node_modules/yauzl/index.js:705:5
#8 11.48 at /opt/code-server/lib/vscode/node_modules/yauzl/fd-slicer.js:33:7
#8 11.48 at callback (/opt/code-server/lib/vscode/node_modules/graceful-fs/polyfills.js:138:21)
#8 11.48 at FSReqCallback.wrapper [as oncomplete] (node:fs:682:5)
Functionality [modified by patches] tested and found to work:
- base-path
- cli-window-open
- local-storage
- proxy-url
- service-worker
- web view
Workspaces and Jupyter Notebooks also work fine:
ms-toolsai.jupyter@202450
i️ Installing extensions from VSIX works.
I followed the same procedure as with all previous release candidates. No idea what I might have missed.
@code-asher Please review.
I think the marketplace error is related to the Microsoft.VisualStudio.Services.VsixSignature
asset. From Microsoft's marketplace, this is a zip file, but from Open VSX it is not. I will do some more investigation, but maybe we will just have to disable signature verification.
Ah yeah here is the issue eclipse/openvsx#922
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.
Thank you for putting this together! I will put out an RC soon.
This works around an issue where the Open VSX is not returning the expected zip. I think verification does not work anyway because @vscode/vsce-sign is missing in the OSS version.
I would like to confirm, that the marketplace is working fine with code-server v4.90.0-rc.1
Python extension, too: ms-python.python@2024年4月1日
Awesome thank you for confirming.
I am seeing a weird thing where the first web socket connection keeps timing out (seeing the handshake timed out
in the browser console), making it appear like loading code-server takes a really long time. I am trying to figure out what it is, but I was curious to see if you were experiencing the same?
Edit: it works fine for me locally though. I tried to test on https://coder.jupyter.b-data.ch to see if maybe the problem was related to the way I was deploying it but it is timing out spawning at the moment.
but it is timing out spawning at the moment.
Just try again. It will work eventually.
but I was curious to see if you were experiencing the same?
No, I do not experience this on https://coder.jupyter.b-data.ch.
Very weird, I tried clearing the CI cache to force a rebuild of VS Code and for some reason CI keeps dumping core, so I am trying 20.12.1 instead. It looks like VS Code uses the .nvmrc (20.12.1) to build, but then they ship the web version with the .yarnrc (20.11.1).
I have no idea why they use two different versions...I am going to just try building a new RC 20.12.1 and hopefully that works fine.
Edit: did not work, trying some things out here: #6830
Edit: resolved a segfault in CI, but I still get the first timeout every time. Trying 20.12.1 again: #6833
Edit: finally figured it out, putting out a patch release.
but I was curious to see if you were experiencing the same?
No, I do not experience this on https://coder.jupyter.b-data.ch.
Did a735495 resolve the web socket connection time outs?
i️ In b-data's/my docker images --disable-telemetry
is used when code-server
is started.
Did a735495 resolve the web socket connection time outs?
It did, yup! In our environment for some reason os.cpus()
is very slow (1-2 seconds), maybe because there are 64 cores? On my laptop it takes around 0.04 seconds. I think probably what happened is VS Code added new telemetry calls on startup, and those new calls caused enough delay due to the slow os.cpus()
to trigger the timeout.
In b-data's/my docker images --disable-telemetry is used when code-server is started.
Ahhh that makes sense.
Additionally: - Update Node to 20.11.1 - Update documentation - Disable extension signature verification This works around an issue where the Open VSX is not returning the expected zip. Verification is skipped later anyway because @vscode/vsce-sign is missing in the OSS version.
Fixes #6822