Skip to content

Commit 2485c15

Browse files
authored
removes unused cli flags from proxyrunner (#2515)
* removes unused cli flags from proxyrunner Signed-off-by: ChrisJBurns <29541485+ChrisJBurns@users.noreply.github.com> * removes redundant foreground flag Signed-off-by: ChrisJBurns <29541485+ChrisJBurns@users.noreply.github.com> * dont need to check for foreground anymore Signed-off-by: ChrisJBurns <29541485+ChrisJBurns@users.noreply.github.com> * removes redundant cmd test file Signed-off-by: ChrisJBurns <29541485+ChrisJBurns@users.noreply.github.com> --------- Signed-off-by: ChrisJBurns <29541485+ChrisJBurns@users.noreply.github.com>
1 parent 5156575 commit 2485c15

File tree

5 files changed

+62
-679
lines changed

5 files changed

+62
-679
lines changed

cmd/thv-operator/controllers/mcpserver_controller.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -898,7 +898,7 @@ func (r *MCPServerReconciler) deploymentForMCPServer(
898898
replicas := int32(1)
899899

900900
// Prepare container args
901-
args := []string{"run", "--foreground=true"}
901+
args := []string{"run"}
902902

903903
// Prepare container volume mounts
904904
volumeMounts := []corev1.VolumeMount{}

cmd/thv-operator/test-integration/mcp-server/mcpserver_controller_integration_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,6 @@ var _ = Describe("MCPServer Controller Integration Tests", func() {
189189

190190
// Verify container args contain the required parameters
191191
Expect(container.Args).To(ContainElement("run"))
192-
Expect(container.Args).To(ContainElement("--foreground=true"))
193192
Expect(container.Args).To(ContainElement(mcpServer.Spec.Image))
194193

195194
// Verify container ports

cmd/thv-proxyrunner/app/execution.go

Lines changed: 0 additions & 149 deletions
This file was deleted.

0 commit comments

Comments
 (0)