Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

Merged
code-asher merged 4 commits into coder:main from benz0li:code-1.90.0
Jun 6, 2024
Merged

Update Code to 1.90.0 #6824

code-asher merged 4 commits into coder:main from benz0li:code-1.90.0
Jun 6, 2024

Conversation

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
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月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)

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

i️ Installing extensions from VSIX works.

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 reacted with thumbs up emoji

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

code-asher commented Jun 6, 2024
edited
Loading

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.

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.

benz0li reacted with thumbs up emoji
@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 enabled auto-merge (squash) June 6, 2024 22:59
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
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日

Copy link
Member

code-asher commented Jun 7, 2024
edited
Loading

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.

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.

code-asher reacted with thumbs up emoji

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 reacted with thumbs up emoji

Copy link
Member

code-asher commented Jun 8, 2024
edited
Loading

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.

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?

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

Copy link
Member

code-asher commented Jun 13, 2024
edited
Loading

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
Reviewers

@code-asher code-asher code-asher approved these changes

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update Code to 1.90

AltStyle によって変換されたページ (->オリジナル) /