Skip to content

Allow empty MCI and make it reusable#2264

Merged
cb-github-robot merged 1 commit intocloud-barista:mainfrom
seokho-son:main
Jan 11, 2026
Merged

Allow empty MCI and make it reusable#2264
cb-github-robot merged 1 commit intocloud-barista:mainfrom
seokho-son:main

Conversation

@seokho-son
Copy link
Copy Markdown
Member

@seokho-son seokho-son commented Jan 11, 2026

Allow empty MCI and make it reusable.

  • allow empty MCI (MCI without any VM)
  • add new MCI lifecycle status (Empty)
  • manage those empty MCIs officially (we can add a new subgroup to MCI with the existing operations)
  • block operations that require actual VM control to empty MCIs
image

Copilot AI review requested due to automatic review settings January 11, 2026 13:09
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request enables empty Multi-Cloud Infrastructure (MCI) support by introducing a new lifecycle status and adding appropriate validation guards. The changes allow MCIs to exist without VMs and be reused later, while preventing operations that require VM instances from executing on empty MCIs.

Changes:

  • Added StatusEmpty constant for MCIs without VMs
  • Implemented status determination logic to distinguish between provisioning, failed, and truly empty MCIs
  • Added validation checks to block VM-requiring operations (remote commands, monitoring agent installation) on empty MCIs
  • Extended deletion logic to allow empty MCIs to be safely deleted

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/core/model/mci.go Added StatusEmpty constant to represent MCIs with no VMs
src/core/infra/remoteCommand.go Added validation to prevent executing remote commands on empty MCIs
src/core/infra/monitoring.go Added validation to prevent installing monitoring agents on empty MCIs
src/core/infra/manageInfo.go Updated status determination logic to set StatusEmpty for MCIs without VMs and allowed deletion of empty MCIs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: Seokho Son <shsongist@gmail.com>
@seokho-son
Copy link
Copy Markdown
Member Author

/approve

@github-actions github-actions bot added the approved This PR is approved and will be merged soon. label Jan 11, 2026
@cb-github-robot cb-github-robot merged commit af0472a into cloud-barista:main Jan 11, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved This PR is approved and will be merged soon.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants