FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update Code to 1.90.0 by benz0li · Pull Request #6824 · coder/code-server · GitHub

Update Code to 1.90.0 - #6824

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

Update Code to 1.90.0#6824
code-asher merged 4 commits into
coder:mainfrom
benz0li:code-1.90.0

Conversation

benz0li commented Jun 6, 2024

Copy link
Copy Markdown
Contributor
  • Update Node to 20.11.1
  • Update documentation

Fixes #6822

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

benz0li commented Jun 6, 2024

Copy link
Copy Markdown
Contributor Author

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 commented Jun 6, 2024

Copy link
Copy Markdown
Contributor Author

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 commented Jun 6, 2024

Copy link
Copy Markdown
Contributor Author

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

@code-asher Please review.

Comment thread patches/base-path.diff
Comment thread patches/marketplace.diff
code-asher self-assigned this Jun 6, 2024

code-asher commented Jun 6, 2024
edited
Loading

Copy link
Copy Markdown
Member

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
Copy Markdown
Member

Ah yeah here is the issue eclipse-openvsx/openvsx#922

code-asher left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Thank you for putting this together! I will put out an RC soon.

code-asher marked this pull request as ready for review June 6, 2024 22:57
code-asher enabled auto-merge (squash) June 6, 2024 22:57
code-asher disabled auto-merge June 6, 2024 22:58
code-asher enabled auto-merge (squash) June 6, 2024 22:59
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 merged commit 6d9530a into coder:main Jun 6, 2024

benz0li commented Jun 7, 2024

Copy link
Copy Markdown
Contributor Author

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 commented Jun 7, 2024
edited
Loading

Copy link
Copy Markdown
Member

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 commented Jun 7, 2024

Copy link
Copy Markdown
Contributor Author

but it is timing out spawning at the moment.

Just try again. It will work eventually.

benz0li commented Jun 7, 2024

Copy link
Copy Markdown
Contributor Author

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 commented Jun 8, 2024
edited
Loading

Copy link
Copy Markdown
Member

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 commented Jun 13, 2024

Copy link
Copy Markdown
Contributor Author

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 commented Jun 13, 2024
edited
Loading

Copy link
Copy Markdown
Member

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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL