Skip to content

Conversation

@diamonwiggins
Copy link
Member

What this PR does / why we need it:

Adds support for the when property on ConfigGroup objects to achieve full feature parity with KOTS conditional group display functionality. This allows entire configuration tabs to be conditionally shown or hidden based on other config values, enabling dynamic configuration flows.

Which issue(s) this PR fixes:

https://app.shortcut.com/replicated/story/130043

Does this PR require a test?

NONE

Does this PR require a release note?

NONE

Does this PR require documentation?

NONE

@github-actions
Copy link

github-actions bot commented Oct 14, 2025

This PR has been released (on staging) and is available for download with a embedded-cluster-smoke-test-staging-app license ID.

Online Installer:

curl "https://staging.replicated.app/embedded/embedded-cluster-smoke-test-staging-app/ci/appver-dev-c01dc10" -H "Authorization: $EC_SMOKE_TEST_LICENSE_ID" -o embedded-cluster-smoke-test-staging-app-ci.tgz

Airgap Installer (may take a few minutes before the airgap bundle is built):

curl "https://staging.replicated.app/embedded/embedded-cluster-smoke-test-staging-app/ci-airgap/appver-dev-c01dc10?airgap=true" -H "Authorization: $EC_SMOKE_TEST_LICENSE_ID" -o embedded-cluster-smoke-test-staging-app-ci.tgz

Happy debugging!

@emosbaugh
Copy link
Member

I assume we already have tests for this in the backend?

@diamonwiggins
Copy link
Member Author

I assume we already have tests for this in the backend?

for the templating engine? yes - https://github.com/replicatedhq/embedded-cluster/blob/main/api/integration/app/install/config_test.go#L501.

not sure if there's other coverage to consider.

@diamonwiggins diamonwiggins merged commit 1f1669f into main Oct 15, 2025
271 of 290 checks passed
@diamonwiggins diamonwiggins deleted the diamonwiggins/sc-130043/support-when-config-group branch October 15, 2025 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants