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

Test: ESM exploration builds #226399

Closed
7 tasks done
bpasero opened this issue Aug 23, 2024 · 17 comments
Closed
7 tasks done

Test: ESM exploration builds #226399

bpasero opened this issue Aug 23, 2024 · 17 comments
Labels
ESM Issues caused by our AMD => ESM work testplan-item
Milestone

Comments

@bpasero
Copy link
Member

bpasero commented Aug 23, 2024

Refs: #160416

Complexity: 5

Authors: @jrieken @bpasero
Roles: Developer

Create Issue


Setup: Desktop / Remote

Image

Setup: Web

  • for testing on insiders.vscode.dev, two URL query parameters are needed:
    • vscode-version=<commit>: to point to the ESM enabled web bits
    • vscode-esm=true: to enable the server to serve ESM web bits
  • use this for testing: https://insiders.vscode.dev/?vscode-version=d903f161caa4a4aa9cc965d95d24cf6704940020&vscode-esm=true
    • you will see a modal dialog confirming that version that you then press "OK"
  • Note: if you see the workbench not opening, make sure you did not get redirected, with the 2 query params lost, just open the URL again with devtools opened and caches cleared

Testing
This is about exploratory testing all aspects of the build and checking for:

  • things that do not work anymore, regressions
  • suspicious error output in logs or dev tools that you do not see in insiders
  • extensions not working anymore

Areas to cover:

  • Editor colorization and workbench themes work
  • Text Editing works fine, including text diffing and multi-diff editors
  • Notebooks are functioning
  • Explorer operations on files and folders work
  • Fulltext search and quick open work
  • Git SCM operations work from the changes view
  • Extension development and debugging works (F5 on a sample extension, breakpoints are hit and tests can run)
  • Command line works (passing folders/files to open, extension management)
  • Telemetry is properly sent and received (please reach out for advice)
  • Extension installation and management works from the UI
  • Terminal works
  • Settings sync works
  • Profiles can be created, managed and switched to
  • GH Copilot Chat is functional
  • Selfhosting: keep using the exploration build for the endgame and see if you are limited in any way
@bpasero bpasero added ESM Issues caused by our AMD => ESM work testplan-item labels Aug 23, 2024
@bpasero bpasero added this to the August 2024 milestone Aug 23, 2024
@tats-u
Copy link
Contributor

tats-u commented Aug 24, 2024

https://builds.code.visualstudio.com/builds/exploration looks like staff only.
Is it newer than #160416 (comment)?
Which issue should we non-staffs report issues on these builds if we found a bug in them, #160416 or here?

image

You do not have permission to access this

You have signed in, but you do not have permission to access this resource.

Sign out and sign in with a different account.

More Details

@bpasero
Copy link
Member Author

bpasero commented Aug 24, 2024

Thanks for willing to test! Here are direct links:

VS Code Build: 79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b

cli-alpine-arm64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/vscode_cli_alpine_arm64_cli.tar.gz
cli-alpine-x64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/vscode_cli_alpine_x64_cli.tar.gz
cli-darwin-arm64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/vscode_cli_darwin_arm64_cli.zip
cli-darwin-x64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/vscode_cli_darwin_x64_cli.zip
cli-linux-arm64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/vscode_cli_linux_arm64_cli.tar.gz
cli-linux-armhf: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/vscode_cli_linux_armhf_cli.tar.gz
cli-linux-x64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/vscode_cli_linux_x64_cli.tar.gz
cli-win32-arm64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/vscode_cli_win32_arm64_cli.zip
cli-win32-x64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/vscode_cli_win32_x64_cli.zip
darwin: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/VSCode-darwin.zip
darwin-arm64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/VSCode-darwin-arm64.zip
linux-arm64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/code-exploration-arm64-1724413495.tar.gz
linux-armhf: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/code-exploration-armhf-1724413482.tar.gz
linux-deb-arm64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/code-exploration_1.93.0-1724413576_arm64.deb
linux-deb-armhf: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/code-exploration_1.93.0-1724413558_armhf.deb
linux-deb-x64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/code-exploration_1.93.0-1724414801_amd64.deb
linux-rpm-arm64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/code-exploration-1.93.0-1724413669.el8.aarch64.rpm
linux-rpm-armhf: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/code-exploration-1.93.0-1724413637.el8.armv7hl.rpm
linux-rpm-x64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/code-exploration-1.93.0-1724414884.el8.x86_64.rpm
linux-snap-x64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/code-exploration-x64-1724416998.snap
linux-x64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/code-exploration-x64-1724413480.tar.gz
win32-arm64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/VSCodeSetup-arm64-1.93.0-exploration.exe
win32-arm64-archive: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/VSCode-win32-arm64-1.93.0-exploration.zip
win32-arm64-user: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/VSCodeUserSetup-arm64-1.93.0-exploration.exe
win32-x64: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/VSCodeSetup-x64-1.93.0-exploration.exe
win32-x64-archive: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/VSCode-win32-x64-1.93.0-exploration.zip
win32-x64-user: https://vscode.download.prss.microsoft.com/dbazure/download/exploration/79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b/VSCodeUserSetup-x64-1.93.0-exploration.exe

@tats-u
Copy link
Contributor

tats-u commented Aug 25, 2024

Thank you for providing a newer build.
I hope it make it easier for extension developers using ESM-only packages to prepare this change in VS Code.

@bpasero
Copy link
Member Author

bpasero commented Aug 25, 2024

To clarify: we did not change anything for extensions, this is core only!

@tats-u
Copy link
Contributor

tats-u commented Aug 25, 2024

I know the impact on extensions is just a side effect.
If the core is changed to ESM, it will be (theoretically) possible for extensions to run ESM code, won't it?

@bpasero
Copy link
Member Author

bpasero commented Aug 25, 2024

Technically probably yes, but entirely out of the scope of this issue and not in the near future I think.

@tats-u
Copy link
Contributor

tats-u commented Aug 25, 2024

not in the near future

That's disappointing, but I hope the fix of this issue make it easier (or possible directly possible) to allow extensions consume ESM code.

I try this build for everyday use for a while.

@tats-u
Copy link
Contributor

tats-u commented Aug 25, 2024

The following error occurs sometimes when trying to install extensions:

2024-08-25 18:13:46.029 [error] TypeError: Failed to fetch
    at dDs (vscode-file://vscode-app/c:/Users/tatsu/AppData/Local/Programs/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:2923:39991)
    at vscode-file://vscode-app/c:/Users/tatsu/AppData/Local/Programs/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:2923:41171
    at vut.c (vscode-file://vscode-app/c:/Users/tatsu/AppData/Local/Programs/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:371:52668)
    at vut.request (vscode-file://vscode-app/c:/Users/tatsu/AppData/Local/Programs/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:2923:41153)
    at Aze.F (vscode-file://vscode-app/c:/Users/tatsu/AppData/Local/Programs/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:1118:167)
    at async Aze.getManifest (vscode-file://vscode-app/c:/Users/tatsu/AppData/Local/Programs/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:1116:40360)
    at async Wut.S (vscode-file://vscode-app/c:/Users/tatsu/AppData/Local/Programs/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:2928:9085)
    at async Wut.installFromGallery (vscode-file://vscode-app/c:/Users/tatsu/AppData/Local/Programs/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:2928:5935)
    at async vscode-file://vscode-app/c:/Users/tatsu/AppData/Local/Programs/Microsoft%20VS%20Code%20Exploration/resources/app/out/vs/workbench/workbench.desktop.main.js:2251:33378

However, I was able to install them at the second trial.

バージョン: 1.93.0-exploration (user setup)
コミット: 79f3a3ad60190a7f0cc5e222f7cc1a366b6c8d3b
日付: 2024-08-23T10:56:41.243Z
Electron: 30.4.0
ElectronBuildId: 10073054
Chromium: 124.0.6367.243
Node.js: 20.15.1
V8: 12.4.254.20-electron.0
OS: Windows_NT x64 10.0.22631

@bpasero
Copy link
Member Author

bpasero commented Aug 25, 2024

Please see if you can reproduce with insiders as well or not.

@tats-u
Copy link
Contributor

tats-u commented Aug 25, 2024

I installed the latest Insiders and tried installing some extensions, but no "fetch failed" has occurred.
The fetch failure occurred once when I visited the detail page of an extension.

@bpasero
Copy link
Member Author

bpasero commented Aug 27, 2024

If someone has trouble getting remote to work with exploration builds, https://builds.code.visualstudio.com/builds/insider?commit=4edd1ad59d7e15a3c08e7111f00a2cf3785bbe87&dev=true is an insiders build that is ESM enabled, but you need to disable auto update if you plan on using that.

@andreamah
Copy link
Contributor

Everything looks good from my side- just need to check if telemetry is ingested correctly. Since there is an ingestion delay, I'll wait a few hours and then check back.

@bhavyaus
Copy link
Collaborator

bhavyaus commented Aug 27, 2024

Telemetry is properly sent and received (please reach out for advice)

I verified by setting the log level to trace and that I saw telemetry events emitted in the output channel. How do I confirm that events were received on our end (besides looking for my events in our telemetry pipeline once they are ingested).

Otherwise, everything looks good on macOS on first pass. I'll continue to selfhost through the week as my primary.

@bhavyaus bhavyaus removed their assignment Aug 27, 2024
@meganrogge
Copy link
Contributor

@bhavyaus you check for a specific event using kusto

@meganrogge
Copy link
Contributor

Example: create a file with your name as the extension. Then:

RawEventsVSCode
| where EventName == "monacoworkbench/fileput"
| where Properties['ext'] == ".meganrogge"

I still get a network error when I try this, so will wait a little longer.

@meganrogge meganrogge removed their assignment Aug 27, 2024
@sandy081 sandy081 removed their assignment Aug 27, 2024
@roblourens
Copy link
Member

@meganrogge if you are getting a network error in the kusto window, you now are supposed to be on the azure vpn to access that data. I haven't set that up, I just use a devbox.

@karthiknadig karthiknadig removed their assignment Aug 27, 2024
@andreamah andreamah removed their assignment Aug 27, 2024
@roblourens roblourens removed their assignment Aug 28, 2024
@roblourens
Copy link
Member

I will keep selfhosting

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ESM Issues caused by our AMD => ESM work testplan-item
Projects
None yet
Development

No branches or pull requests

9 participants