cli: Implement cloud build functionality in BuildRuntime - #435
cli: Implement cloud build functionality in BuildRuntime#435singhshresth26 wants to merge 7 commits into
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Welcome @singhshresth26! It looks like this is your first PR to volcano-sh/agentcube 🎉 |
There was a problem hiding this comment.
Code Review
This pull request implements the cloud build functionality in _build_cloud for BuildRuntime and adds corresponding unit tests. The feedback suggests returning the fully qualified image name (including the tag) instead of just the registry URL in _build_cloud to maintain consistency with local builds, and updating the unit test assertions accordingly.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #435 +/- ##
==========================================
+ Coverage 58.41% 59.96% +1.54%
==========================================
Files 36 36
Lines 3463 3589 +126
==========================================
+ Hits 2023 2152 +129
+ Misses 1231 1218 -13
- Partials 209 219 +10
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR adds a simulated cloud build path to the AgentCube CLI BuildRuntime so users can run agentcube build with --build-mode cloud and have the CLI resolve a cloud registry destination and persist build information into agent_metadata.yaml.
Changes:
- Implemented
_build_cloud()inBuildRuntimeto simulate a remote build and update metadata with cloud build details. - Added unit tests covering local and cloud build flows.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
cmd/cli/agentcube/runtime/build_runtime.py |
Implements the cloud build strategy and metadata persistence for cloud builds. |
cmd/cli/tests/test_build_runtime.py |
Adds tests for both local and cloud build paths. |
|
The PR description says users can select |
|
I have now exposed |
|
Have you seen AI's suggestions? |
Signed-off-by: Shresth Singh <shresthengineer@gmail.com>
Signed-off-by: Shresth Singh <shresthengineer@gmail.com>
… checks Signed-off-by: Shresth Singh <shresthengineer@gmail.com>
90cade7 to
c54f6c8
Compare
… validation Signed-off-by: Shresth Singh <shresthengineer@gmail.com>
|
Sorry for not seeing the AI suggestions earlier and for missing the sign-offs! I've just updated the code to address the AI review comments (implemented the registry URL base parsing logic, normalized provider checks, and added corresponding unit tests). I also signed off the new commits and force-pushed the branch. Thanks! |
… and registry port validation Signed-off-by: Shresth Singh <shresthengineer@gmail.com>
|
With the current behavior, cloud mode without |
|
Looks like most of the review comments have been addressed. Could you resolve those threads so it's easier to see what's still open? |
…shing Signed-off-by: Shresth Singh <shresthengineer@gmail.com>
Signed-off-by: Shresth Singh <shresthengineer@gmail.com>
6d4aa12 to
e45837c
Compare
|
Sorry for the delay! I have resolved the remaining issues in the latest commit: Cloud Build Support (without --dry-run): Allowed simulated cloud builds to succeed without --dry-run, writing the build details to agent_metadata.yaml on disk. This enables the end-to-end cloud publishing workflow. |
What type of PR is this?
/kind enhancement
What this PR does / why we need it:
This PR implements the simulated cloud build strategy (
_build_cloudmethod) in the CLIBuildRuntime.When a user specifies
--build-mode cloud(or configures it inagent_metadata.yaml), the CLI will:build_mode: cloudtoagent_metadata.yaml.It also adds comprehensive unit tests in
cmd/cli/tests/test_build_runtime.pycovering both the local and cloud build paths.Which issue(s) this PR fixes:
Fixes #434
Special notes for your reviewer:
This resolves the first code issue/TODO placeholder (
# TODO: Implement cloud build functionality) inbuild_runtime.py.Does this PR introduce a user-facing change?: