Skip to content

[DO NOT MERGE] Build CheCode editor image to test PR-376 - #779

Draft
vrubezhny wants to merge 3 commits into
che-incubator:mainfrom
vrubezhny:test/che-machine-exec-PR-376
Draft

[DO NOT MERGE] Build CheCode editor image to test PR-376#779
vrubezhny wants to merge 3 commits into
che-incubator:mainfrom
vrubezhny:test/che-machine-exec-PR-376

Conversation

@vrubezhny

@vrubezhny vrubezhny commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

What does this PR do?

Opened to create a CheCode editor image to test:

What issues does this PR fix?

How to test this PR?

Does this PR contain changes that override default upstream Code-OSS behavior?

  • the PR contains changes in the code folder (you can skip it if your changes are placed in a che extension )
  • the corresponding items were added to the CHANGELOG.md file
  • rules for automatic git rebase were added to the .rebase folder

Summary by CodeRabbit

  • Chores
    • Updated machine-exec component source and streamlined initialization process during system build and deployment.

RomanNikitenko and others added 3 commits July 8, 2026 14:25
…ainers

Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
Opened to create a CheCode editor image to test:

- eclipse-che/che-machine-exec#363

<!-- Please include any related issue from the Eclipse Che repository (or from another issue tracker). -->

- [ ] the PR contains changes in the [code](https://github.com/che-incubator/che-code/tree/main/code) folder (you can skip it if your changes are placed in a che extension )
- [ ] the corresponding items were added to the [CHANGELOG.md](https://github.com/che-incubator/che-code/blob/main/.rebase/CHANGELOG.md) file
- [ ] rules for automatic `git rebase` were added to the [.rebase](https://github.com/che-incubator/che-code/tree/main/.rebase) folder

<!-- This is an auto-generated comment: release notes by coderabbit.ai -->

* **Chores**
  * Updated the container image used during the build assembly process to a newer version, which may improve build reliability and consistency.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Signed-off-by: Victor Rubezhny <vrubezhny@redhat.com>
Opened to create a CheCode editor image to test:

- eclipse-che/che-machine-exec#376

Signed-off-by: Victor Rubezhny <vrubezhny@redhat.com>
@tolusha

tolusha commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Hi! I'm che-ai-assistant — I help with your pull requests.

I check for new comments every 10m0s, so there may be a short delay before I respond.

Available commands:

  • /che-ai-assistant generate-che-doc — Generate a documentation PR based on this PR's changes
  • /che-ai-assistant ok-pr-review — Run a comprehensive PR review (summary, code review, deep review, impact analysis)
  • /che-ai-assistant ok-pr-readiness — Ensure PR has validation steps
  • /che-ai-assistant check-pr-test-failures — Analyze failing CI checks, identify root causes, and suggest fixes
  • /che-ai-assistant update-che-e2e-tests — Update Eclipse Che e2e tests
  • /che-ai-assistant claude — Run a free-form instruction on this PR
  • /che-ai-assistant help — Show this help message

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Click here to review and test in web IDE: Contribute

@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 52684400-81a5-4584-8fe6-6a6fe5b1cb0f

📥 Commits

Reviewing files that changed from the base of the PR and between d2d7be6 and 3fa5948.

📒 Files selected for processing (3)
  • build/dockerfiles/assembly.Dockerfile
  • build/scripts/entrypoint-init-container.sh
  • build/scripts/entrypoint-volume.sh
💤 Files with no reviewable changes (1)
  • build/scripts/entrypoint-init-container.sh

📝 Walkthrough

Walkthrough

The Docker assembly now uses an updated machine-exec image and copies architecture-specific binaries into the UBI8, UBI9, and musl Checode runtime directories. Shared-volume initialization no longer creates /checode/bin. The volume entrypoint selects the runtime directory before starting ./machine-exec on port 3333.

Estimated code review effort: 2 (Simple) | ~10 minutes

Sequence Diagram(s)

sequenceDiagram
  participant DockerAssembly as assembly.Dockerfile
  participant InitContainer as entrypoint-init-container.sh
  participant VolumeEntrypoint as entrypoint-volume.sh
  participant MachineExec as machine-exec

  DockerAssembly->>DockerAssembly: Copy architecture-specific binaries into runtime directories
  InitContainer->>InitContainer: Skip shared-volume machine-exec copy
  VolumeEntrypoint->>VolumeEntrypoint: Select Checode runtime directory
  VolumeEntrypoint->>MachineExec: Start ./machine-exec on port 3333
Loading

Possibly related PRs

  • che-incubator/che-code#769: Both changes relocate and launch architecture-specific machine-exec binaries within CheCode runtime directories.

Suggested reviewers: azatsarynnyy, rgrunber, romannikitenko, vitaliy-guliy

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title is concise, uses imperative mood, and clearly describes building a CheCode editor image to test PR-376.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Rebase Rules For Upstream Changes ✅ Passed The full PR range changes only three files under build/; no non-excluded files under code/ are changed, so rebase rules and changelog updates are not required.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

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.

3 participants