Skip to content

feat(bootstrap-local): add support for managed services#310

Open
schrodit wants to merge 1 commit intomainfrom
local-mg-support
Open

feat(bootstrap-local): add support for managed services#310
schrodit wants to merge 1 commit intomainfrom
local-mg-support

Conversation

@schrodit
Copy link
Copy Markdown
Member

No description provided.

@schrodit schrodit requested a review from NautiluX April 10, 2026 16:03
@schrodit schrodit self-assigned this Apr 10, 2026
Signed-off-by: Tim Schrodi <tim@codesphere.com>
@@ -540,10 +540,19 @@ func (b *LocalBootstrapper) UpdateInstallConfig() (err error) {
b.Env.InstallConfig.Cluster.PgOperator = &files.PgOperatorConfig{
Enabled: false,
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

we still use the one operator we install in oms.
There is no need for a second one

}
serviceProfiles["deployment_service"].(map[string]any)["replicas"] = 2
serviceProfiles["public_api_service"].(map[string]any)["replicas"] = 2
serviceProfiles["deployment_service"].(map[string]any)["replicas"] = 1
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think we set these to 2 because it gets saturated quickly during integration tests

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is there an issue when running two instances locally given they don't request any CPU or memory?

corev1 "k8s.io/api/core/v1"
)

func TestParseMonitorEndpointHost(t *testing.T) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Y no ginkgo?

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.

2 participants