Skip to content

Conversation

@ostrain
Copy link
Collaborator

@ostrain ostrain commented May 2, 2025

No description provided.

@ostrain ostrain force-pushed the mco-overview-docs branch from 1a7e796 to 3be44fb Compare May 2, 2025 21:12
@ostrain ostrain marked this pull request as ready for review May 2, 2025 21:14
**Release Target:** The project is starting a public preview in mid-late April
2025. Partners interested in collaborting can email [email protected] for
more information.
**Release Target:** The project is currently available as an early public
Copy link
Collaborator

Choose a reason for hiding this comment

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

let's remove mention of release and all given we don't provide a date. Let's just state what is out there available.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think we should make it clear that the plan is for it to be open source. How about "The project is currently available as an early public preview. An initial open source beta release is planned for later in 2025."?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we should be specific about what is available to manage expectations as there's no formal definition for public previews.

Same for what is planned to be open source (e.g. placement API, mco controller). The MCO overview mentions other components that will not be oss (e.g. cp syncer).

# Overview of Multi-Cluster Orchestrator

Multi-Cluster Orchestrator (MCO) dynamically schedules an application’s
multi-cluster and multi-region deployment, allowing the application to
Copy link
Collaborator

Choose a reason for hiding this comment

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

multi-region is technically cluster dependent.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Right. I think people will understand what we mean, but if you think it's unclear I'm open to suggestions

run the workload and which clusters/regions are more desirable to use
* Information about region capacity, such as which regions currently have
certain machine types available
* Current load on the application
Copy link
Collaborator

Choose a reason for hiding this comment

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

technically this is only if they have an HPA.... which we quite require. but mco itself doesn't look at load?

Copy link
Collaborator Author

@ostrain ostrain May 2, 2025

Choose a reason for hiding this comment

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

Right, this is a simplification but I think it's worth including as part of the conceptual framework. Technically the previous one is a simplification too since we don't actually know when machines are available, only when they're unavailable.

@ostrain ostrain force-pushed the mco-overview-docs branch from 356a96c to 0b21778 Compare May 2, 2025 21:44
@ostrain ostrain force-pushed the mco-overview-docs branch from 0b21778 to 60a470e Compare May 2, 2025 21:54
Copy link
Collaborator

@apeabody apeabody left a comment

Choose a reason for hiding this comment

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

A few suggestions, LGTM :)

ostrain and others added 5 commits May 2, 2025 15:55
Co-authored-by: Andrew Peabody <[email protected]>
Signed-off-by: Owen Strain <[email protected]>
Co-authored-by: Andrew Peabody <[email protected]>
Signed-off-by: Owen Strain <[email protected]>
Co-authored-by: Andrew Peabody <[email protected]>
Signed-off-by: Owen Strain <[email protected]>
Co-authored-by: Andrew Peabody <[email protected]>
Signed-off-by: Owen Strain <[email protected]>
Signed-off-by: Owen Strain <[email protected]>
Signed-off-by: Owen Strain <[email protected]>
@apeabody apeabody merged commit cc80030 into GoogleCloudPlatform:main May 5, 2025
1 check passed
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.

4 participants