Add a cosml.build command to build a project on a cluster - #518
Merged
Merged
Conversation
sjoubert
force-pushed
the
modapi_build_PROD-16371
branch
from
August 14, 2026 08:35
e546bcc to
349acd1
Compare
sjoubert
force-pushed
the
modapi_build_PROD-16371
branch
from
August 14, 2026 12:47
349acd1 to
97a4aed
Compare
sjoubert
force-pushed
the
modapi_build_PROD-16371
branch
2 times, most recently
from
August 28, 2026 15:29
28210d2 to
5f24078
Compare
Contributor
Author
|
Code has been updated with latest design decisions:
|
MohcineTor
approved these changes
Aug 31, 2026
MohcineTor
left a comment
Collaborator
There was a problem hiding this comment.
It’s okay for me we can go with this approach for now using --exclude. Later, we can have a quick discussion with the Integration team to see what their preference is.
I also have a quick remark: we could change the order of the steps. We could start with building the project, then move on to organizing the solution, etc. in applay command
sjoubert
force-pushed
the
modapi_build_PROD-16371
branch
2 times, most recently
from
September 2, 2026 09:56
93c2ca3 to
788d9a2
Compare
sjoubert
marked this pull request as ready for review
September 2, 2026 09:58
sjoubert
force-pushed
the
modapi_build_PROD-16371
branch
2 times, most recently
from
September 2, 2026 12:06
e6927cf to
accb458
Compare
MohcineTor
self-requested a review
September 3, 2026 09:16
sellisd
reviewed
Sep 10, 2026
This makes use of temporary project in the modeling api to build a simulator and optionnally push the resulting docker image in the tenant registry Until the modeling api is publicly exposed, babylon creates a temporary port-forward via kubectl
sjoubert
force-pushed
the
modapi_build_PROD-16371
branch
from
September 14, 2026 08:33
accb458 to
d561249
Compare
sellisd
approved these changes
Sep 15, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This makes use of temporary project in the modeling api, unless we decide to have full support for a persistent
Projectresource.Until the modeling api is publicly exposed, babylon creates a temporary port-forward via kubectl.
Design discussion points left open:
initcommand, but the actual form of the file will depend of the other discussion pointscosml? for separate one-shot commands,modapi/mod-api/modeling-api? for the api command wrappersapply? It probably needs a toggle to avoid triggering a build each time the project data is applied.Projectresource or do we keep using a temporary project for now? If we only have aProjectBuildfor now, is the project archive url better placed in ametadatafield or as asidecar.This is part of a multi repo effort that also involves the terraform modules to have the modeling api deploy in a tenant (Cosmo-Tech/terraform-shared#31, Cosmo-Tech/terraform-tenant#41) and will require the next SDK version (12.3.0)