Skip to content

Commit

Permalink
fix: remove duplicate securityContext (radius-project#6996)
Browse files Browse the repository at this point in the history
# Description

When installing the 0.29.0 helm chart, I received an error message:

```
Helm install failed for release radius-system/radius with chart radius@0.29.0: error while running post render on files: map[string]interface {}(nil): yaml: unmarshal errors:
        line 61: mapping key "securityContext" already defined at line 54
```

This PR removed the duplicate key :)

## Type of change

<!--

Please select **one** of the following options that describes your
change and delete the others. Clearly identifying the type of change you
are making will help us review your PR faster, and is used in authoring
release notes.

If you are making a bug fix or functionality change to Radius and do not
have an associated issue link please create one now.

-->

- This pull request fixes a bug in Radius and has an approved issue
(issue link required).

<!--

Please update the following to link the associated issue. This is
required for some kinds of changes (see above).

-->

Issue: radius-project#6997

Signed-off-by: Ivan Josipovic <9521987+IvanJosipovic@users.noreply.github.com>
Signed-off-by: willdavsmith <willdavsmith@gmail.com>
  • Loading branch information
IvanJosipovic authored and willdavsmith committed Jan 17, 2024
1 parent e1218cf commit 0da92cb
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions deploy/Chart/templates/rp/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -73,8 +73,6 @@ spec:
mountPath: {{ .Values.global.rootCA.mountPath }}
readOnly: true
{{- end }}
securityContext:
allowPrivilegeEscalation: false
{{- if .Values.rp.resources }}
resources:{{ toYaml .Values.rp.resources | nindent 10 }}
{{- end }}
Expand Down

0 comments on commit 0da92cb

Please sign in to comment.