chore: revisit manipulation of status field in helm charts
qdzlug opened this issue · comments
Jason Schmidt commented
Is your feature request related to a problem? Please describe.
This fix was put into place at the start of the project because Pulumi's chart implementation had issues with the status
field.
It simply removes the field. With the various changes of the Helm implementation in Pulumi this may no longer be needed.
# Removes the status field from the Nginx Ingress Helm Chart, so that i#t is
# compatible with the Pulumi Chart implementation.
def remove_status_field(obj):
if obj['kind'] == 'CustomResourceDefinition' and 'status' in obj:
del obj['status']
Describe the solution you'd like
- Validate that we no longer need this.
- Remove it from our code that uses helm charts.
Describe alternatives you've considered
None.
Additional context
None.