From eacbd213ce675e21b8c0c31da25066c011a3ed2c Mon Sep 17 00:00:00 2001 From: Yaron Blum Date: Tue, 25 Aug 2026 14:22:51 +0300 Subject: [PATCH 1/4] FR-26617: pass the venv application switches through to the builder The mcp-gateway and custom-code applications ship disabled in the venv chart. The venv-builder overrides that per environment, but only if the switch reaches it, so both entrypoints that compose venv files now forward it. Both default to false: a caller that says nothing gets the same venv it gets today. Co-Authored-By: Claude Opus 5 --- .github/shared-actions/start-venv/action.yaml | 10 ++++++++++ .github/workflows/build-docker-start-venv.yaml | 12 ++++++++++++ .github/workflows/start-venv.yaml | 12 ++++++++++++ 3 files changed, 34 insertions(+) diff --git a/.github/shared-actions/start-venv/action.yaml b/.github/shared-actions/start-venv/action.yaml index 6aa4533..0ebbdce 100644 --- a/.github/shared-actions/start-venv/action.yaml +++ b/.github/shared-actions/start-venv/action.yaml @@ -31,6 +31,14 @@ inputs: description: "Should the venv disable dashboard" required: false default: "false" + enableMcpGateway: + description: "Should the venv deploy the mcp-gateway application" + required: false + default: "false" + enableCustomCode: + description: "Should the venv deploy the custom-code application" + required: false + default: "false" ref: description: "The branch name to start the venv from" required: false @@ -82,6 +90,8 @@ runs: venvRepository: ${{ inputs.venvRepository }} venvRepositoryMainBranchName: ${{ inputs.venvRepositoryMainBranchName }} disableDashboard: ${{ inputs.disableDashboard }} + enableMcpGateway: ${{ inputs.enableMcpGateway }} + enableCustomCode: ${{ inputs.enableCustomCode }} branchName: ${{ inputs.ref }} - name: Commit venv file to repo shell: bash diff --git a/.github/workflows/build-docker-start-venv.yaml b/.github/workflows/build-docker-start-venv.yaml index 92d9e8c..f67da10 100644 --- a/.github/workflows/build-docker-start-venv.yaml +++ b/.github/workflows/build-docker-start-venv.yaml @@ -36,6 +36,16 @@ on: required: false description: 'Set to true in order to disable starting dashboard application' default: false + enableMcpGateway: + type: boolean + required: false + description: 'Set to true in order to deploy the mcp-gateway application into the venv' + default: false + enableCustomCode: + type: boolean + required: false + description: 'Set to true in order to deploy the custom-code application into the venv' + default: false ref: type: string required: false @@ -102,6 +112,8 @@ jobs: leaseTime: ${{ inputs.leaseTime }} venvRepoDirectory: ${{ github.workspace }}/venv/environments disableDashboard: ${{ inputs.disableDashboard }} + enableMcpGateway: ${{ inputs.enableMcpGateway }} + enableCustomCode: ${{ inputs.enableCustomCode }} ref: ${{ inputs.ref }} - name: Generate matrix for docker build uses: actions/github-script@v7 diff --git a/.github/workflows/start-venv.yaml b/.github/workflows/start-venv.yaml index 15c1be0..e426efa 100644 --- a/.github/workflows/start-venv.yaml +++ b/.github/workflows/start-venv.yaml @@ -36,6 +36,16 @@ on: required: false description: 'Set to true in order to disable starting dashboard application' default: false + enableMcpGateway: + type: boolean + required: false + description: 'Set to true in order to deploy the mcp-gateway application into the venv' + default: false + enableCustomCode: + type: boolean + required: false + description: 'Set to true in order to deploy the custom-code application into the venv' + default: false ref: type: string required: false @@ -112,6 +122,8 @@ jobs: leaseTime: ${{ inputs.leaseTime }} venvRepoDirectory: ${{ github.workspace }}/venv/environments disableDashboard: ${{ inputs.disableDashboard }} + enableMcpGateway: ${{ inputs.enableMcpGateway }} + enableCustomCode: ${{ inputs.enableCustomCode }} ref: ${{ inputs.ref }} - name: Generate matrix for docker build uses: actions/github-script@v7 From 798c39a0f2f09fcf45b05bfa9bf17f2b00e9f756 Mon Sep 17 00:00:00 2001 From: Yaron Blum Date: Tue, 25 Aug 2026 14:39:20 +0300 Subject: [PATCH 2/4] FR-26617: run the AI agent suite against the mcp gateway The suite is the one caller that needs the gateway, so it is the one caller that asks for it. Every other caller keeps the venv it has today. Co-Authored-By: Claude Opus 5 --- .github/workflows/ai-agent-test-suite.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/ai-agent-test-suite.yaml b/.github/workflows/ai-agent-test-suite.yaml index bf364f0..f6d4ad2 100644 --- a/.github/workflows/ai-agent-test-suite.yaml +++ b/.github/workflows/ai-agent-test-suite.yaml @@ -75,6 +75,7 @@ jobs: with: volatileEnvironment: true triggerSlack: false + enableMcpGateway: true ref: ${{ inputs.ref }} secrets: inherit From 2a8ac483c31b9c01e87c54b57274993ac91b9d70 Mon Sep 17 00:00:00 2001 From: Yaron Blum Date: Tue, 25 Aug 2026 16:02:48 +0300 Subject: [PATCH 3/4] FR-26617: take one extended list instead of a flag per application Matches the builder input: callers name the applications they want raised rather than carrying a boolean per service. The AI agent suite asks for mcp-gateway; every other caller passes nothing and gets the venv it has today. Co-Authored-By: Claude Opus 5 --- .github/shared-actions/start-venv/action.yaml | 13 ++++--------- .github/workflows/ai-agent-test-suite.yaml | 2 +- .github/workflows/build-docker-start-venv.yaml | 16 +++++----------- .github/workflows/start-venv.yaml | 16 +++++----------- 4 files changed, 15 insertions(+), 32 deletions(-) diff --git a/.github/shared-actions/start-venv/action.yaml b/.github/shared-actions/start-venv/action.yaml index 0ebbdce..e308249 100644 --- a/.github/shared-actions/start-venv/action.yaml +++ b/.github/shared-actions/start-venv/action.yaml @@ -31,14 +31,10 @@ inputs: description: "Should the venv disable dashboard" required: false default: "false" - enableMcpGateway: - description: "Should the venv deploy the mcp-gateway application" + extended: + description: "Comma separated list of optional applications to deploy into the venv" required: false - default: "false" - enableCustomCode: - description: "Should the venv deploy the custom-code application" - required: false - default: "false" + default: "" ref: description: "The branch name to start the venv from" required: false @@ -90,8 +86,7 @@ runs: venvRepository: ${{ inputs.venvRepository }} venvRepositoryMainBranchName: ${{ inputs.venvRepositoryMainBranchName }} disableDashboard: ${{ inputs.disableDashboard }} - enableMcpGateway: ${{ inputs.enableMcpGateway }} - enableCustomCode: ${{ inputs.enableCustomCode }} + extended: ${{ inputs.extended }} branchName: ${{ inputs.ref }} - name: Commit venv file to repo shell: bash diff --git a/.github/workflows/ai-agent-test-suite.yaml b/.github/workflows/ai-agent-test-suite.yaml index f6d4ad2..c17c21c 100644 --- a/.github/workflows/ai-agent-test-suite.yaml +++ b/.github/workflows/ai-agent-test-suite.yaml @@ -75,7 +75,7 @@ jobs: with: volatileEnvironment: true triggerSlack: false - enableMcpGateway: true + extended: mcp-gateway ref: ${{ inputs.ref }} secrets: inherit diff --git a/.github/workflows/build-docker-start-venv.yaml b/.github/workflows/build-docker-start-venv.yaml index f67da10..d328b95 100644 --- a/.github/workflows/build-docker-start-venv.yaml +++ b/.github/workflows/build-docker-start-venv.yaml @@ -36,16 +36,11 @@ on: required: false description: 'Set to true in order to disable starting dashboard application' default: false - enableMcpGateway: - type: boolean - required: false - description: 'Set to true in order to deploy the mcp-gateway application into the venv' - default: false - enableCustomCode: - type: boolean + extended: + type: string required: false - description: 'Set to true in order to deploy the custom-code application into the venv' - default: false + description: 'Comma separated list of optional applications to deploy into the venv, for example mcp-gateway,custom-code' + default: '' ref: type: string required: false @@ -112,8 +107,7 @@ jobs: leaseTime: ${{ inputs.leaseTime }} venvRepoDirectory: ${{ github.workspace }}/venv/environments disableDashboard: ${{ inputs.disableDashboard }} - enableMcpGateway: ${{ inputs.enableMcpGateway }} - enableCustomCode: ${{ inputs.enableCustomCode }} + extended: ${{ inputs.extended }} ref: ${{ inputs.ref }} - name: Generate matrix for docker build uses: actions/github-script@v7 diff --git a/.github/workflows/start-venv.yaml b/.github/workflows/start-venv.yaml index e426efa..d9693b8 100644 --- a/.github/workflows/start-venv.yaml +++ b/.github/workflows/start-venv.yaml @@ -36,16 +36,11 @@ on: required: false description: 'Set to true in order to disable starting dashboard application' default: false - enableMcpGateway: - type: boolean - required: false - description: 'Set to true in order to deploy the mcp-gateway application into the venv' - default: false - enableCustomCode: - type: boolean + extended: + type: string required: false - description: 'Set to true in order to deploy the custom-code application into the venv' - default: false + description: 'Comma separated list of optional applications to deploy into the venv, for example mcp-gateway,custom-code' + default: '' ref: type: string required: false @@ -122,8 +117,7 @@ jobs: leaseTime: ${{ inputs.leaseTime }} venvRepoDirectory: ${{ github.workspace }}/venv/environments disableDashboard: ${{ inputs.disableDashboard }} - enableMcpGateway: ${{ inputs.enableMcpGateway }} - enableCustomCode: ${{ inputs.enableCustomCode }} + extended: ${{ inputs.extended }} ref: ${{ inputs.ref }} - name: Generate matrix for docker build uses: actions/github-script@v7 From f9d53b40e6747248927b058b3ab20c731145f757 Mon Sep 17 00:00:00 2001 From: Yaron Blum Date: Tue, 25 Aug 2026 16:06:22 +0300 Subject: [PATCH 4/4] FR-26617: make extended a switch rather than a list of names Callers ask for the extended venv, not for named services: the builder holds the list of applications the switch covers. The AI agent suite asks for it, every other caller passes nothing and gets the venv it has today. Co-Authored-By: Claude Opus 5 --- .github/shared-actions/start-venv/action.yaml | 4 ++-- .github/workflows/ai-agent-test-suite.yaml | 2 +- .github/workflows/build-docker-start-venv.yaml | 6 +++--- .github/workflows/start-venv.yaml | 6 +++--- 4 files changed, 9 insertions(+), 9 deletions(-) diff --git a/.github/shared-actions/start-venv/action.yaml b/.github/shared-actions/start-venv/action.yaml index e308249..9a4dd10 100644 --- a/.github/shared-actions/start-venv/action.yaml +++ b/.github/shared-actions/start-venv/action.yaml @@ -32,9 +32,9 @@ inputs: required: false default: "false" extended: - description: "Comma separated list of optional applications to deploy into the venv" + description: "Should the venv deploy the extended applications" required: false - default: "" + default: "false" ref: description: "The branch name to start the venv from" required: false diff --git a/.github/workflows/ai-agent-test-suite.yaml b/.github/workflows/ai-agent-test-suite.yaml index c17c21c..d27ca32 100644 --- a/.github/workflows/ai-agent-test-suite.yaml +++ b/.github/workflows/ai-agent-test-suite.yaml @@ -75,7 +75,7 @@ jobs: with: volatileEnvironment: true triggerSlack: false - extended: mcp-gateway + extended: true ref: ${{ inputs.ref }} secrets: inherit diff --git a/.github/workflows/build-docker-start-venv.yaml b/.github/workflows/build-docker-start-venv.yaml index d328b95..4530130 100644 --- a/.github/workflows/build-docker-start-venv.yaml +++ b/.github/workflows/build-docker-start-venv.yaml @@ -37,10 +37,10 @@ on: description: 'Set to true in order to disable starting dashboard application' default: false extended: - type: string + type: boolean required: false - description: 'Comma separated list of optional applications to deploy into the venv, for example mcp-gateway,custom-code' - default: '' + description: 'Set to true in order to deploy the extended applications into the venv, currently mcp-gateway and custom-code' + default: false ref: type: string required: false diff --git a/.github/workflows/start-venv.yaml b/.github/workflows/start-venv.yaml index d9693b8..b651ecd 100644 --- a/.github/workflows/start-venv.yaml +++ b/.github/workflows/start-venv.yaml @@ -37,10 +37,10 @@ on: description: 'Set to true in order to disable starting dashboard application' default: false extended: - type: string + type: boolean required: false - description: 'Comma separated list of optional applications to deploy into the venv, for example mcp-gateway,custom-code' - default: '' + description: 'Set to true in order to deploy the extended applications into the venv, currently mcp-gateway and custom-code' + default: false ref: type: string required: false