Skip to content

Conversation

@sirtimid
Copy link
Contributor

@sirtimid sirtimid commented Jan 31, 2025

Closes #400

Adds a template selector dropdown to the ConfigEditor component that allows users to quickly switch between different cluster configurations (Default with Alice/Bob/Carol vats, or Minimal with single MainVat).

Screenshot 2025-01-31 at 15 47 25

@sirtimid sirtimid requested a review from a team as a code owner January 31, 2025 15:05
"forceReset": true,
"vats": {
"main": {
"bundleSpec": "http://localhost:3000/sample-vat.bundle",
Copy link
Contributor

Choose a reason for hiding this comment

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

The current sample vat assumes the three vat subcluster configuration and will most likely behave badly when run by itself. This is probably a good opportunity to set things up with more than one vat definition in the vats directory.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, I've added an empty one for now and we will revisit this config when we have another meaningful example vat to add.

@sirtimid sirtimid force-pushed the sirtimid/multiple-sub-clusters branch from e670e99 to cf08539 Compare February 3, 2025 15:55
rekmarks
rekmarks previously approved these changes Feb 3, 2025
Copy link
Member

@rekmarks rekmarks left a comment

Choose a reason for hiding this comment

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

Looks good, just needs conflict resolution.

@sirtimid sirtimid force-pushed the sirtimid/multiple-sub-clusters branch from 0100819 to 046500a Compare February 3, 2025 22:35
@sirtimid sirtimid enabled auto-merge (squash) February 3, 2025 22:35
@sirtimid sirtimid requested review from FUDCo and rekmarks February 3, 2025 23:53
@sirtimid sirtimid merged commit 9a2fd60 into main Feb 4, 2025
17 checks passed
@sirtimid sirtimid deleted the sirtimid/multiple-sub-clusters branch February 4, 2025 00:03
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.

Create a simple UI with a sub-cluster selector dropdown above the config editor.

4 participants