Skip to content

Commit

Permalink
tilt: remove securityContext for live_update
Browse files Browse the repository at this point in the history
Signed-off-by: Benjamin Gentil <[email protected]>
  • Loading branch information
bengentil committed Jan 4, 2023
1 parent 192517b commit 850794a
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions hack/tools/tilt-prepare/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -775,6 +775,12 @@ func prepareWorkload(name, prefix, binaryName, containerName string, objs []unst
cmd := []string{"sh", "/start.sh", "/" + binaryName}
args := append(container.Args, []string(ts.ExtraArgs[name])...)

// remove securityContext for tilt live_update, see https://github.com/tilt-dev/tilt/issues/3060
container.SecurityContext = nil
// ensure it's also removed from the pod template matching this container
// setting this outside the loop would means altering every deployments
d.Spec.Template.Spec.SecurityContext = nil

// alter deployment for working nicely with delve debugger;
// most specifically, configuring delve, starting the manager with profiling enabled, dropping liveness and
// readiness probes and disabling leader election.
Expand Down

0 comments on commit 850794a

Please sign in to comment.