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 d5eb23a
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions hack/tools/tilt-prepare/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -775,6 +775,10 @@ 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
d.Spec.Template.Spec.SecurityContext = nil
container.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 d5eb23a

Please sign in to comment.