kic-reference-architectures icon indicating copy to clipboard operation
kic-reference-architectures copied to clipboard

chore: revisit manipulation of status field in helm charts

Open qdzlug opened this issue 1 year ago • 0 comments

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

  1. Validate that we no longer need this.
  2. Remove it from our code that uses helm charts.

Describe alternatives you've considered None.

Additional context None.

qdzlug avatar Dec 01 '22 16:12 qdzlug