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

JetBrains IDEs 2024.3 stable release #20370

Merged
merged 6 commits into from
Nov 16, 2024
Merged

JetBrains IDEs 2024.3 stable release #20370

merged 6 commits into from
Nov 16, 2024

Conversation

mustard-mh
Copy link
Contributor

@mustard-mh mustard-mh commented Nov 15, 2024

Description

Related Issue(s)

Fixes CLC-941

How to test

✅ Smoke test

  • Create workspaces with stable JB editors, their versions should be correct.
  • IntelliJ IDEA 2024.3
  • GoLand 2024.3
  • Clion 2024.2
IDEA WebStorm CLion Version List
image image image SCR-20241115-pstg

✅ Integration tests should pass
https://github.com/gitpod-io/gitpod/actions/runs/11859552844/job/33053613264?pr=20370

Documentation

Preview status

Gitpod was successfully deployed to your preview environment.

Build Options

Build
  • /werft with-werft
    Run the build with werft instead of GHA
  • leeway-no-cache
  • /werft no-test
    Run Leeway with --dont-test
Publish
  • /werft publish-to-npm
  • /werft publish-to-jb-marketplace
Installer
  • analytics=segment
  • with-dedicated-emulation
  • workspace-feature-flags
    Add desired feature flags to the end of the line above, space separated
Preview Environment / Integration Tests
  • /werft with-local-preview
    If enabled this will build install/preview
  • /werft with-preview
  • /werft with-large-vm
  • /werft with-gce-vm
    If enabled this will create the environment on GCE infra
  • /werft preemptible
    Saves cost. Untick this only if you're really sure you need a non-preemtible machine.
  • with-integration-tests=jetbrains
    Valid options are all, workspace, webapp, ide, jetbrains, vscode, ssh. If enabled, with-preview and with-large-vm will be enabled.
  • with-monitoring

/hold

@roboquat roboquat added size/L and removed size/M labels Nov 15, 2024
@@ -151,6 +151,7 @@ packages:
- NO_VERIFY_JB_PLUGIN=true
config:
commands:
- ["rm", "-rf", "src/main/kotlin/io/gitpod/jetbrains/remote/GitpodMetricControlProvider.kt"]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

To unblock rider plugin build

return image.replace("gitpod-dev-artifact", "gitpod-core-dev");
for (const imgFromMain of imagesBuildFromMain) {
if (image.includes(imgFromMain)) {
return image.replace("gitpod-dev-artifact", "gitpod-core-dev");
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Make sure we could use these images built from main in preview env, repository is set by installer, we correct it in this patch script

  • gitpod-dev-artifact - dev images
  • gitpod-core-dev - prod images from main

@@ -814,10 +862,10 @@
"logo": "{{.IdeLogoBase}}/clionLogo.svg",
"image": "{{.Repository}}/ide/clion:{{.WorkspaceVersions.Workspace.DesktopIdeImages.CLionImage.Version}}",
"latestImage": "{{.ResolvedJBImageLatest.CLion}}",
"pluginImage": "{{.JetBrainsPluginImage}}",
"pluginImage": "{{.JetBrainsPluginImagePrevious}}",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

CLion and RustRover 2024.3 are not release yet, we need to force them to use old backend plugin

Copy link
Contributor Author

@mustard-mh mustard-mh Nov 15, 2024

Choose a reason for hiding this comment

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

Rider is a special one, we don't take care of it for a long time already, (it had a bug before that we can't build, and the bug was fixed, but we don't have time to work on it)

@@ -67,6 +68,9 @@ func GenerateIDEConfigmap(ctx *common.RenderContext) (*ide_config.IDEConfig, err
WorkspaceVersions versions.Components
}

// {{.Repository}}/ide/jb-backend-plugin:commit-4c69ad0670cc4cfbf43910e1db700ad90acd5ac6
previousStableJetBrainsBackendPluginVersion := "commit-4c69ad0670cc4cfbf43910e1db700ad90acd5ac6"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The value is a copy current stable backend plugin image from main build

@mustard-mh
Copy link
Contributor Author

/unhold

@roboquat roboquat merged commit 21f546d into main Nov 16, 2024
73 of 74 checks passed
@roboquat roboquat deleted the hw/CLC-941 branch November 16, 2024 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants