-
Notifications
You must be signed in to change notification settings - Fork 95
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
duplicate securityContext in Helm Chart #6997
Comments
👋 @IvanJosipovic Thanks for filing this bug report. A project maintainer will review this report and get back to you soon. If you'd like immediate help troubleshooting, please visit our Discord server. For more information on our triage process please visit our triage overview |
@shalabhms @sylvainsf @AaronCrawfis @willtsai - this is probably worth a patch release since it's blocking install. Thoughts? |
Please note, this is only blocking installs through Flux, as it appears to do additional validation. Installing through the Helm CLI worked as expected
|
# 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: #6997 Signed-off-by: Ivan Josipovic <9521987+IvanJosipovic@users.noreply.github.com>
The fix for this has been merged into |
triage: decided not to issue the patch as it is blocking install through Flux only and not through Helm. |
👍 We've reviewed this issue and have agreed to add it to our backlog. Please subscribe to this issue for notifications, we'll provide updates when we pick it up. We also welcome community contributions! If you would like to pick this item up sooner and submit a pull request, please visit our contribution guidelines and assign this to yourself by commenting "/assign" on this issue. For more information on our triage process please visit our triage overview |
# 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>
Bug information
Steps to reproduce (required)
When installing the Helm Chart version 0.29.0 using Flux, the installation fails.
Duplicate key:
radius/deploy/Chart/templates/rp/deployment.yaml
Line 76 in 3b28a2f
Observed behavior (required)
Flux was not able to install the Helm Chart due to the following error:
Desired behavior (required)
Helm install should succeed
Workaround (optional)
Deploy the chart using the Helm CLI
System information
rad Version (required)
0.29.0
Operating system (required)
Windows 11, amd64
Additional context
AB#10827
The text was updated successfully, but these errors were encountered: