Skip to content
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

Meet 'NoneType' issue when install helm chart with empty entry of manifest #109

Open
xho22 opened this issue Aug 29, 2024 · 1 comment
Open

Comments

@xho22
Copy link

xho22 commented Aug 29, 2024

When I'm installing istio chart by helm plugin, it returns 'NoneType' error. After debug, the cause if manifest of helm_status contains empty entry. And will run into issue from below:

api_version = resource.get('apiVersion')

Chart manifest:

#Result of command: helm status istio-base -n istio-system -o yaml
config:
  defaultRevision: default
info:
  deleted: ""
  description: Install complete
  first_deployed: "2024-08-29T03:18:10.688440447Z"
  last_deployed: "2024-08-29T03:18:10.688440447Z"
  notes: |
    Istio base successfully installed!

    To learn more about the release, try:
      $ helm status istio-base -n istio-system
      $ helm get all istio-base -n istio-system
  status: deployed
manifest: |
  ---
  # Source: base/templates/reader-serviceaccount.yaml
  # This service account aggregates reader permissions for the revisions in a given cluster
  # Should be used for remote secret creation.
  apiVersion: v1
  kind: ServiceAccount
  metadata:
    name: istio-reader-service-account
    namespace: istio-system
    labels:
      app: istio-reader
      release: istio-base
  ---
  # Source: base/templates/zzz_profile.yaml
  #  Flatten globals, if defined on a per-chart basis
  ---
  # Source: base/templates/default.yaml
  apiVersion: admissionregistration.k8s.io/v1
  kind: ValidatingWebhookConfiguration
  metadata:

Coverted
image

The zzz_profile.yaml is empty. From my side, we don't suppose each key of manifest is present as some special chart design.

@xho22 xho22 changed the title Meet 'NoneType' issue when install helm chart with empty manifest Meet 'NoneType' issue when install helm chart with empty entry of manifest Aug 29, 2024
@xho22
Copy link
Author

xho22 commented Aug 29, 2024

I found it's caused by a comment:
https://github.com/istio/istio/blob/1.22.4/manifests/zzz_profile.yaml#L39
I installed istio 1.22.3 and get the job done. However the issue is still should be fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant