nginxinc / kic-reference-architectures

MARA: Modern Application Reference Architecture

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

chore: revisit manipulation of status field in helm charts

qdzlug opened this issue · 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.