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

chore: update Code to 1.73.1 #5769

Merged
merged 2 commits into from
Nov 10, 2022
Merged

chore: update Code to 1.73.1 #5769

merged 2 commits into from
Nov 10, 2022

Conversation

jsjoeio
Copy link
Contributor

@jsjoeio jsjoeio commented Nov 10, 2022

Fixes #5768

image

@jsjoeio jsjoeio self-assigned this Nov 10, 2022
@jsjoeio jsjoeio temporarily deployed to npm November 10, 2022 16:49 Inactive
@github-actions
Copy link

github-actions bot commented Nov 10, 2022

✨ code-server dev build published to npm for PR #5769!

  • Last publish status: success
  • Commit: 624779b

To install in a local project, run:

npm install @coder/code-server-pr@5769

To install globally, run:

npm install -g @coder/code-server-pr@5769

@codecov
Copy link

codecov bot commented Nov 10, 2022

Codecov Report

Merging #5769 (624779b) into main (e2f585f) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #5769   +/-   ##
=======================================
  Coverage   72.66%   72.66%           
=======================================
  Files          30       30           
  Lines        1683     1683           
  Branches      370      370           
=======================================
  Hits         1223     1223           
  Misses        397      397           
  Partials       63       63           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e2f585f...624779b. Read the comment docs.

@jsjoeio jsjoeio marked this pull request as ready for review November 10, 2022 18:53
@jsjoeio jsjoeio requested a review from a team as a code owner November 10, 2022 18:53
@jsjoeio jsjoeio temporarily deployed to npm November 10, 2022 18:55 Inactive
@jsjoeio jsjoeio merged commit 69d469b into main Nov 10, 2022
@jsjoeio jsjoeio deleted the jsjoeio/code-1.73.1 branch November 10, 2022 19:46
@benz0li
Copy link
Contributor

benz0li commented Nov 11, 2022

I have tested code-server v4.9.0-rc.1 (code-server-4.9.0-rc.1-linux-amd64.tar.gz):

JupyterHub log:

[...]
The factory function of "vs/platform/product/common/product" has thrown an exception
Error: EACCES: permission denied, open '/opt/code-server/lib/vscode/product.json'
    at Object.openSync (node:fs:590:3)
    at Object.readFileSync (node:fs:458:35)
    at Object.Module._extensions..json (node:internal/modules/cjs/loader:1215:22)
    at Module.load (node:internal/modules/cjs/loader:1033:32)
    at Function.Module._load (node:internal/modules/cjs/loader:868:12)
    at Function.require._load (/opt/code-server/out/node/plugin.js:54:74)
    at Module.require (node:internal/modules/cjs/loader:1057:19)
    at require (node:internal/modules/cjs/helpers:103:18)
    at Function.r [as __$__nodeRequire] (/opt/code-server/lib/vscode/out/vs/loader.js:5:101)
    at Object.<anonymous> (/opt/code-server/lib/vscode/out/vs/server/node/server.main.js:88:60032)
    at Function.s._safeInvokeFunction (/opt/code-server/lib/vscode/out/vs/loader.js:5:1337)
    at Function.s._invokeFactory (/opt/code-server/lib/vscode/out/vs/loader.js:5:1522)
    at s.complete (/opt/code-server/lib/vscode/out/vs/loader.js:5:1801)
    at s._onModuleComplete (/opt/code-server/lib/vscode/out/vs/loader.js:6:1112)
    at s._resolve (/opt/code-server/lib/vscode/out/vs/loader.js:6:510)
    at s.defineModule (/opt/code-server/lib/vscode/out/vs/loader.js:5:6066)
    at y (/opt/code-server/lib/vscode/out/vs/loader.js:6:1825)
    at h (/opt/code-server/lib/vscode/out/vs/loader.js:4:2554)
    at Object.<anonymous> (/opt/code-server/lib/vscode/out/vs/server/node/server.main.js:88:59568)
    at Object.<anonymous> (/opt/code-server/lib/vscode/out/vs/server/node/server.main.js:185:2106)
    at e._createAndEvalScript (/opt/code-server/lib/vscode/out/vs/loader.js:4:2598)
    at /opt/code-server/lib/vscode/out/vs/loader.js:4:2237
    at FSReqCallback.readFileAfterClose [as oncomplete] (node:internal/fs/read_file_context:68:3) {
  errno: -13,
  syscall: 'open',
  code: 'EACCES',
  path: '/opt/code-server/lib/vscode/product.json',
  phase: 'factory',
  moduleId: 'vs/platform/product/common/product',
  neededBy: []
}
[...]

Further information:

$ ls -al /opt/code-server/lib/vscode
total 168
drwxr-xr-x   7 root root  4096 Nov 10 21:52 .
drwxr-xr-x   3 root root  4096 Nov 10 21:52 ..
drwxr-xr-x   4 root root  4096 Nov 10 20:59 bin
drwxr-xr-x 104 root root  4096 Nov 11 07:48 extensions
drwxr-xr-x 107 root root  4096 Oct 25 19:13 node_modules
lrwxrwxrwx   1 root root    12 Nov 10 21:52 node_modules.asar -> node_modules
-rw-r--r--   1 root root 91042 Nov 10 21:00 npm-shrinkwrap.json
drwxr-xr-x   3 root root  4096 Nov 10 20:59 out
-rw-r--r--   1 root root  1222 Nov 10 21:00 package.json
-rw-------   1 root root  3813 Nov 10 21:49 product.json
drwxr-xr-x   3 root root  4096 Nov 10 20:59 resources
-rw-r--r--   1 root root 38322 Nov 10 21:00 yarn.lock

👉 product.json should have permissions 644 not 600.

@benz0li
Copy link
Contributor

benz0li commented Nov 11, 2022

So I fixed the above; but then, the Terminal was not working:

Browser console:

[Error] TypeError: undefined is not an object (evaluating 'l._renderService.dimensions?.css.cell')
	getFont (workbench.web.main.js:3249:11305)
	_getDimension (workbench.web.main.js:3141:42098)
	_evaluateColsAndRows (workbench.web.main.js:3141:41211)
	layout (workbench.web.main.js:3144:2598)
	layout (workbench.web.main.js:3141:10188)
	layout (workbench.web.main.js:163:20741)
	layoutViews (workbench.web.main.js:163:34590)
	layout (workbench.web.main.js:163:26726)
	layout (workbench.web.main.js:3141:9115)
	layout (workbench.web.main.js:3141:16920)
	forEach
	layout (workbench.web.main.js:163:20741)
	layoutViews (workbench.web.main.js:163:34590)
	layout (workbench.web.main.js:163:26726)
	layout (workbench.web.main.js:2304:13600)
	layoutBody (workbench.web.main.js:3172:14145)
	layout (workbench.web.main.js:226:18534)
	layout (workbench.web.main.js:163:20741)
	layoutViews (workbench.web.main.js:163:34590)
	layout (workbench.web.main.js:163:26726)
	layout (workbench.web.main.js:226:23905)
	layout (workbench.web.main.js:3045:46287)
	layout (workbench.web.main.js:3045:69293)
	layout (workbench.web.main.js:2148:15766)
	layout (workbench.web.main.js:3052:6954)
	layout (workbench.web.main.js:3052:13074)
	_layout (workbench.web.main.js:163:53899)
	layout (workbench.web.main.js:163:53660)
	layout (workbench.web.main.js:163:20741)
	layoutViews (workbench.web.main.js:163:34590)
	layout (workbench.web.main.js:163:26726)
	layout (workbench.web.main.js:163:45387)
	layout (workbench.web.main.js:163:20741)
	layoutViews (workbench.web.main.js:163:34590)
	layout (workbench.web.main.js:163:26726)
	layout (workbench.web.main.js:163:45387)
	layout (workbench.web.main.js:163:20741)
	layoutViews (workbench.web.main.js:163:34590)
	layout (workbench.web.main.js:163:26726)
	layout (workbench.web.main.js:163:45387)
	layout (workbench.web.main.js:163:56258)
	layout (workbench.web.main.js:163:65949)
	layout (workbench.web.main.js:163:70368)
	layout (workbench.web.main.js:3123:32080)
	(anonymous function) (workbench.web.main.js:1808:37462)

@benz0li
Copy link
Contributor

benz0li commented Nov 11, 2022

@jsjoeio code-server v4.9.0-rc.1 (code-server-4.9.0-rc.1-linux-amd64.tar.gz; fixed version with CS_DISABLE_GETTING_STARTED_OVERRIDE=1 set) is deployed at https://vscode-r.jupyter.b-data.ch; Image R (base:test) + code-server.

Functionality [modified by patches] tested and found to work:

  • base-path
  • local-storage
  • marketplace
  • proxy-url
  • service-worker
  • webview

👉 I cannot test cli-window-open as the Terminal is not working.

Jupyter Notebooks also work fine:

  • ms-toolsai.jupyter@2022.10.110
  • ms-python.python@2022.16.1

@jsjoeio
Copy link
Contributor Author

jsjoeio commented Nov 30, 2022

@benz0li thank you for testing. I see the Terminal error but I'm not sure what's causing it 🤔 @code-asher and I will have to dig in. Thanks for helping test!

image

@jsjoeio jsjoeio mentioned this pull request Nov 30, 2022
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.

[Feat]: upgrade to 1.73.1
3 participants