[DO NOT MERGE] Build CheCode editor image to test PR-376 - #779
Conversation
…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>
|
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:
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
💤 Files with no reviewable changes (1)
📝 WalkthroughWalkthroughThe 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 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
Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
Pull Request images published ✨ Editor amd64: quay.io/che-incubator-pull-requests/che-code:pr-779-amd64 |
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?
git rebasewere added to the .rebase folderSummary by CodeRabbit