Update to Theia 1.66.x and switch to vscode-trace-extension#103
Update to Theia 1.66.x and switch to vscode-trace-extension#103jfaltermeier merged 75 commits intomasterfrom
Conversation
Whenever a new master commit, scheduled or triggered build run is executed, we now upload the packaged app to the artifacts. We already did that for the mac build, because we need it for the publishing, but now we do it for the other systems as well. But only on non PRs to not increase the PR build time. Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
* update dockerfile to node 20 * update license check workflow to node 20
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Removed ai-editor for now. Added issue to readd it later. Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Add `@theia/ai-editor` as it was recently introduced. Added `@theia/preview` and `@theia/scanoss` as they were indirectly installed previously. This should make it more clear, that we ship them. Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
…sv 2017 Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
- Update release announcement templates for Preview testing discussion topic
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Signed-off-by: Simon Graband <sgraband@eclipsesource.com>
Contributed on behalf of STMicroelectronics
Contributed on behalf of STMicroelectronics
* Update Blueprint version to `1.66.200`, align all packages to Theia `1.66.2` * Upgrade Electron to `38.4.0`, electron-builder to `26.0.12`, electron-updater to `6.6.2` * Require Node.js `>=20`, update CI to Node.js `22.x` and Python `3.13` * Replace legacy Jenkins release pipeline with split preview build, sign, upload jobs * Refresh bundled VS Code extensions and native dependencies
42ffa6f to
1e8b6d1
Compare
1e8b6d1 to
12e4891
Compare
3b59f09 to
1c6a1a6
Compare
* Drop `theia-traceviewer` dependency from browser, docker and electron apps * Add `@eclipse-cdt-cloud/trace-server-manager` Theia extension package * Include VS Code trace extension plugin for trace viewing support Contributed on behalf of STMicroelectronics
1c6a1a6 to
cd21453
Compare
* Add `@eclipse-cdt-cloud/trace-server-proxy` Theia extension to proxy trace server traffic via `/trace-server-proxy` * Configure `trace-compass.traceserver.*` preferences on startup based on proxy config when `THEIACLOUD_SESSION_URL` is set * include it in browser and docker application dependencies Contributed on behalf of STMicroelectronics
cd21453 to
2bb1879
Compare
|
@jfaltermeier Thanks for your work. Regarding the proxy feature, do you have a guide on how to test it locally? |
|
I think the easiest way would be via docker. Only expose port 3000. This should reproduce the problem: This should work, because the proxy gets enabled: |
sdirix
left a comment
There was a problem hiding this comment.
Thank you! All features work (as they did before) and I like the Trace Server integration. With your manual I was also able to run it within docker.
There is one regression I noticed: When I start the Electron application I am stuck in the splash screen for 30 seconds until the main window shows up. My guess is that the window should already be ready but somehow the messaging/events are broken to show it. So we wait for the 30 seconds timeout for it to be shown. Can you reproduce the issue?
|
From what I can see, when I use On X11 it also works. This fix was added with 1.67, eclipse-theia/theia#16658, so this is not included here. Maybe this missing fix could cause a slowdown instead of a crash as well? |
|
Hmm not sure, I will approve for now but we should create an issue and revisit with the next release |
|
I've opened #104 |
|
I've retriggered the verification jobs, because they produce inputs for the mac builds I still need to download. I will merge the PR once they complete. |
New build jobs:
What it does
1.66.200, align all packages to Theia1.66.238.4.0, electron-builder to26.0.12, electron-updater to6.6.2>=20, update CI to Node.js22.xand Python3.13theia-traceviewer replacement
theia-traceviewerwith theeclipse-cdt.vscode-trace-extensionVS Code plugin. This requires adding some features of the theia extension to this workspace:@eclipse-cdt-cloud/trace-server-managerextension, providing commands to start, stop and restart the bundled Trace Compass server@eclipse-cdt-cloud/trace-server-proxyextension, which proxies trace server traffic through the Theia backend at/trace-server-proxyTHEIACLOUD_SESSION_URLenv variable is set, as this is our typical cloud use caseContributed on behalf of STMicroelectronics