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

[Feat]: Upgrade vscode to 1.70 #5417

Closed
bala opened this issue Aug 5, 2022 · 25 comments
Closed

[Feat]: Upgrade vscode to 1.70 #5417

bala opened this issue Aug 5, 2022 · 25 comments
Assignees
Labels
enhancement Some improvement that isn't a feature
Milestone

Comments

@bala
Copy link

bala commented Aug 5, 2022

What is your suggestion?

Upgrade vscode to 1.70

Why do you want this feature?

Are there any workarounds to get this functionality today?

Are you interested in submitting a PR for this?

@bala bala added the enhancement Some improvement that isn't a feature label Aug 5, 2022
@jsjoeio jsjoeio added this to the August 2022 milestone Aug 5, 2022
@fritterhoff
Copy link
Contributor

grafik

So some first steps (#5422) towards the 1.70 look fine. Jupyter is working, code-server in the cli looks also good to me.

@fritterhoff
Copy link
Contributor

@benz0li feel free to have a look on the latest CI artifacts if you can detect some odd bugs. (https://github.com/coder/code-server/actions/runs/2815650333) 😉

@benz0li
Copy link
Contributor

benz0li commented Aug 9, 2022

@benz0li feel free to have a look on the latest CI artifacts if you can detect some odd bugs.

Will do.

@benz0li
Copy link
Contributor

benz0li commented Aug 10, 2022

I have deployed the latest latest CI artifact for linux-amd64 at https://vscode-r.jupyter.b-data.ch, Image R (base:test) + code-server.

@benz0li
Copy link
Contributor

benz0li commented Aug 10, 2022

Jupyter is working

Confirmed.

  • Python v2022.12.0
  • Jupyter v2022.8.1002170547

@benz0li
Copy link
Contributor

benz0li commented Aug 10, 2022

code-server in the cli looks also good to me.

JupyterLab Terminal:

╰─ code-server --version
4.5.1 c5c16630adb086bf2cfed1ba6d678ef2ac9d06e1 with Code 1.70.1
╰─ echo $PATH
/home/benz0li/.local/bin:/opt/code-server/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
╰─ which code-server
/opt/code-server/bin/code-server

code-server Terminal:

╰─ code-server --version
1.70.1
2062a59ca1a586d8a6e7bf483841085a94c440a4
x64
╰─ echo $PATH
/opt/code-server/lib/vscode/bin/remote-cli:/home/benz0li/.local/bin:/opt/code-server/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
╰─ which code-server
/opt/code-server/lib/vscode/bin/remote-cli/code-server

I can live with this 😉. Related to #5335 (comment) ff.


I might do some further testing tonight.

@bala
Copy link
Author

bala commented Aug 10, 2022

@benz0li @jsjoeio @code-asher can we publish this build as beta and test the obviously resolved bugs like #5335 and close it?

@benz0li
Copy link
Contributor

benz0li commented Aug 10, 2022

ℹ️ I am using

# set PATH so it includes user's private bin if it exists
if [ -d "$HOME/bin" -a "$SHLVL" = 1 -a ! "$TERM_PROGRAM" = "vscode" ] ; then
    PATH="$HOME/bin:$PATH"
fi

# set PATH so it includes user's private bin if it exists
if [ -d "$HOME/.local/bin" -a "$SHLVL" = 1 -a ! "$TERM_PROGRAM" = "vscode" ] ; then
    PATH="$HOME/.local/bin:$PATH"
fi

in ~/.zshrc for my Docker images to get identical PATH handling in JupyterLab and code-server, both for login and non-login shells as well as 'nested zsh', screen and tmux sessions.


I haven't figured out a proper solution for VS Code on macOS yet, as PATH is set in multiple places.

@fritterhoff
Copy link
Contributor

@benz0li great. Thanks for testing. Two features/patches required some more adjustments. One is regarding the language packs, the other is about settings storage. From my point of view both patches/features should work correctly at the moment but I'm not sure.

@benz0li
Copy link
Contributor

benz0li commented Aug 10, 2022

@benz0li @jsjoeio @code-asher can we publish this build as beta

I can't say anything about that because I do not work @coder.

But if a two-month rhythm is followed, [at least] one release candidate would be a good idea – for in-depth testing before the actual release.

and test the obviously resolved bugs like #5335 and close it?

@bala You may test at https://vscode-r.jupyter.b-data.ch with Image R (base:test) + code-server.
ℹ️ I have whitelisted your GitHub account.

@jsjoeio
Copy link
Contributor

jsjoeio commented Aug 10, 2022

side note: we had an issue with publishing a beta version to npm. we just fixed that.

yarn global add code-server@beta

will get you the latest off main

https://www.npmjs.com/package/code-server/v/4.5.1-beta-3335d0a456da0ea639b1e010aeb9801a876a9323

@benz0li
Copy link
Contributor

benz0li commented Aug 11, 2022

Functionality modified by patches tested and found to work:

  • base-path
  • cli-window-open
  • local-storage
    • I assume this is about settings
  • marketplace
  • proxy-url
  • service-worker
  • webview

@benz0li
Copy link
Contributor

benz0li commented Aug 11, 2022

Menu entry Sign out of code-server is available although code-server was started with --auth none.

@benz0li
Copy link
Contributor

benz0li commented Aug 11, 2022

@fritterhoff Regarding #5417 (comment): What does echo $PATH and which code-server return in code-server Terminal?

@fritterhoff
Copy link
Contributor

@fritterhoff Regarding #5417 (comment): What does echo $PATH and which code-server return in code-server Terminal?

Since #5360 fixes the script it self and not the way it is called the $PATH modification and output are produced by the official script and not by the modified code-server script.

@benz0li
Copy link
Contributor

benz0li commented Aug 11, 2022

Since #5360 fixes the script it self and not the way it is called the $PATH modification and output are produced by the official script and not by the modified code-server script.

Yours is not the output of the official script. That is why I am asking.

@fritterhoff
Copy link
Contributor

Whooopss... that was to simple 😁
A which code-server produces a /usr/bin/code-server, a echo $PATH

echo $PATH
/usr/lib/code-server/lib/vscode/bin/remote-cli:/opt/conda/bin:/opt/conda/condabin:/opt/conda/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin

@fritterhoff
Copy link
Contributor

fritterhoff commented Aug 11, 2022

In addition:

(base) jovyan@jupyter-ritterho-40hm-2eedu:/usr/lib/code-server/lib/vscode/bin/remote-cli$ ls /usr/lib/code-server/lib/vscode/bin/remote-cli
code.cmd  code-darwin.sh  code-linux.sh
(base) jovyan@jupyter-ritterho-40hm-2eedu:/usr/lib/code-server/lib/vscode/bin/remote-cli$ 

There is no code-server script only a code ... I have no idea whats wrong here 🤔

The script it self works fine, so the patch seems to be applied.

(base) jovyan@jupyter-ritterho-40hm-2eedu:~$ code-linux.sh --version
1.70.1
2062a59ca1a586d8a6e7bf483841085a94c440a4
x64

@benz0li
Copy link
Contributor

benz0li commented Aug 11, 2022

There is no code-server script only a code ... I have no idea whats wrong here 🤔

Maybe #5335 (comment)? Do you still delete the code-server link/script?

In my case there is a link from code-server to code-linux.sh

@fritterhoff
Copy link
Contributor

Ah your right. My Dockerfile contains a rm /usr/lib/code-server/lib/vscode/bin/remote-cli/code-server

@code-asher
Copy link
Member

code-asher commented Aug 11, 2022

I wanted to wait until getting the 1.70 update in but in order to get #5442 released I think we should make a release candidate today and then a regular release pretty soon after. Working on that right now.

@code-asher
Copy link
Member

code-asher commented Aug 11, 2022

Menu entry Sign out of code-server is available although code-server was started with --auth none

Nice find! I opened an issue and will fix it real swift #5448

@xiaoyun94
Copy link

xiaoyun94 commented Aug 15, 2022

3 WAY MERGE VIA CODE SERVER!!!!!!!!!!

@jsjoeio jsjoeio modified the milestones: v4.5.2, August 2022 Aug 15, 2022
@jsjoeio
Copy link
Contributor

jsjoeio commented Aug 17, 2022

Completed via #5422

@jsjoeio jsjoeio closed this as completed Aug 17, 2022
@benz0li
Copy link
Contributor

benz0li commented Aug 18, 2022

Menu entry Sign out of code-server is available although code-server was started with --auth none

Nice find! I opened an issue and will fix it real swift #5448

I can confirm this is now fixed.

P.S.: I have deployed v4.6.0-rc.1 at https://vscode-r.jupyter.b-data.ch, Image R (base:test) + code-server.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Some improvement that isn't a feature
Projects
None yet
Development

No branches or pull requests

6 participants