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

Update Code to 1.90.0 #6824

Merged
merged 4 commits into from
Jun 6, 2024
Merged

Update Code to 1.90.0 #6824

merged 4 commits into from
Jun 6, 2024

Conversation

benz0li
Copy link
Contributor

@benz0li benz0li commented Jun 6, 2024

  • Update Node to 20.11.1
  • Update documentation

Fixes #6822

- Update Node to 20.11.1
- Update documentation
@benz0li benz0li requested a review from a team as a code owner June 6, 2024 09:00
@benz0li benz0li marked this pull request as draft June 6, 2024 09:00
@benz0li
Copy link
Contributor Author

benz0li commented Jun 6, 2024

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-06T08: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-06T08: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)

@benz0li
Copy link
Contributor Author

benz0li commented Jun 6, 2024

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@2024.5.0

ℹ️ Installing extensions from VSIX works.

@benz0li
Copy link
Contributor Author

benz0li commented Jun 6, 2024

I followed the same procedure as with all previous release candidates. No idea what I might have missed.

@code-asher Please review.

@code-asher code-asher self-assigned this Jun 6, 2024
@code-asher
Copy link
Member

code-asher commented Jun 6, 2024

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.

@code-asher
Copy link
Member

Ah yeah here is the issue eclipse/openvsx#922

Copy link
Member

@code-asher code-asher left a 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.

@code-asher code-asher marked this pull request as ready for review June 6, 2024 22:57
@code-asher code-asher enabled auto-merge (squash) June 6, 2024 22:57
@code-asher code-asher disabled auto-merge June 6, 2024 22:58
@code-asher code-asher enabled auto-merge (squash) June 6, 2024 22:59
@code-asher code-asher disabled auto-merge June 6, 2024 23:00
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.
@code-asher code-asher merged commit 6d9530a into coder:main Jun 6, 2024
11 of 13 checks passed
@benz0li
Copy link
Contributor Author

benz0li commented Jun 7, 2024

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

@code-asher
Copy link
Member

code-asher commented Jun 7, 2024

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.

@benz0li
Copy link
Contributor Author

benz0li commented Jun 7, 2024

but it is timing out spawning at the moment.

Just try again. It will work eventually.

@benz0li
Copy link
Contributor Author

benz0li commented Jun 7, 2024

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.

@code-asher
Copy link
Member

code-asher commented Jun 8, 2024

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.

@benz0li
Copy link
Contributor Author

benz0li commented Jun 13, 2024

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?

ℹ️ In b-data's/my docker images --disable-telemetry is used when code-server is started.

@code-asher
Copy link
Member

code-asher commented Jun 13, 2024

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.

yiliang114 pushed a commit to yiliang114/code-server that referenced this pull request Jan 23, 2025
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.
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.

Update Code to 1.90
2 participants