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: Support ubi9-based containers for running VS Code editor #547

Merged
merged 2 commits into from
Feb 12, 2024

Conversation

RomanNikitenko
Copy link
Contributor

@RomanNikitenko RomanNikitenko commented Jan 29, 2024

What does this PR do?

  • At the moment root Dockerfile for the local build of the devspaces-code doesn't work, so it's not possible to build devspaces-code assembly. The current PR fixes that Dockerfile.
  • Also the PR adds ability to build ubi9-based assembly for both: local build and build on Brew.
  • The PR does not add musl/alpine-based containers support for running devspaces-code

Support ubi9-based containers for running VS Code editor:

  • Node.js provided by registry.access.redhat.com/ubi9/nodejs-18 is used for running VS Code in a ubi9-based container
  • libbrotli library from the same image is copied to a user's container as a dependency for Node.js
  • LD_LIBRARY_PATH env variable to define additional shared libs place (libbrotli)

What issues does this PR fix?

https://issues.redhat.com/browse/CRW-5629

How to test this PR?

Image for testing: quay.io/rnikitenko/devspaces-code:ubi8-ubi9-jenkins

Successful build on jenkins: DS_CI/job/get-sources-rhpkg-container-build_3.x/6138

I tested the following images as a container for running Devspaces Code:

Image Workspace starts Devfile registry link Link to start a workspace
registry.access.redhat.com/ubi8 +
Git not found
- click here
registry.access.redhat.com/ubi9 +
Git not found
- click here
registry.access.redhat.com/ubi9/ubi:9.3-1476 +
Git not found
- click here
registry.access.redhat.com/ubi9/ubi-minimal:9.3-1475 +
Git not found
- click here
registry.access.redhat.com/ubi9/podman:9.3-8 +
Git not found
- click here
registry.access.redhat.com/ubi9/perl-532:1-121 + - click here
registry.access.redhat.com/ubi9/php-81:1-43 + - click here
registry.access.redhat.com/ubi9/ruby-30:1-138 + - click here
registry.access.redhat.com/ubi9/openjdk-21-runtime +
Git not found
- click here
registry.access.redhat.com/ubi9/nodejs-16:1-143 + - click here
registry.access.redhat.com/ubi9/nodejs-18:1-84 + - click here
registry.access.redhat.com/ubi9/nodejs-20:1-20 + - click here
registry.access.redhat.com/ubi9/openjdk-11:1.17-1.1705602311 +
Git not found
- click here
registry.access.redhat.com/ubi9/openjdk-17:1.17-1 +
Git not found
Basic Spring Boot click here
registry.access.redhat.com/ubi9/go-toolset:1.19.13-4.1697647145 + Go Runtime click here
registry.access.redhat.com/ubi8/nodejs-16:latest + Basic Node.js click here
registry.access.redhat.com/ubi9/python-39:1-161 + Python-Django click here
registry.access.redhat.com/ubi9/python-39:1-161 + Basic Python click here
registry.access.redhat.com/ubi8/openjdk-17:1.18-2 +
Git not found
Quarkus Java click here
quay.io/eclipse/che-java11-maven:7.37.2 No
the cause of the problem
Vert.x Java click here
quay.io/jaegertracing/all-in-one:1.50 No
the cause of the problem
WildFly Bootable Jar click here

@RomanNikitenko
Copy link
Contributor Author

working on adaptation of brew.Dockerfile to build VS Code assembly using a ubi9-based image.

@RomanNikitenko RomanNikitenko force-pushed the support-ubi9-based-containers branch 2 times, most recently from 3190406 to ff29a83 Compare February 1, 2024 19:34
@RomanNikitenko RomanNikitenko force-pushed the support-ubi9-based-containers branch 2 times, most recently from f1849a8 to 1f88c3a Compare February 8, 2024 14:08
@RomanNikitenko RomanNikitenko force-pushed the support-ubi9-based-containers branch from 1f88c3a to b59d3c7 Compare February 9, 2024 08:47
@RomanNikitenko
Copy link
Contributor Author

WildFly Bootable Jar sample has musl/alpine-based container, it's not supported by devspaces-code.

@RomanNikitenko RomanNikitenko marked this pull request as ready for review February 12, 2024 08:42
Copy link
Member

@ibuziuk ibuziuk left a comment

Choose a reason for hiding this comment

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

👍 let's merge upstream as well che-incubator/che-code#324

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.

2 participants