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

Support helm --skip-schema-validation for values.schema.json that has references to the internet #3118

Closed
jomeier opened this issue Oct 18, 2024 · 0 comments · Fixed by #3165
Labels
enhancement ✨ New feature or request

Comments

@jomeier
Copy link

jomeier commented Oct 18, 2024

Hi,
I fail on creating a Zarf package for the Dagster Pipeline Orchestrator.
The Helm Chart of Dagster uses a values.schema.json file, that has a reference to the internet.
Because of that, the deployment of my Dagster Zarf package fails in my air gapped environment.

Helm seems to have a new option "skip-schema-validation" that overcomes that problem as described here:
fluxcd/flux2#4992

Is it possible, that Zarf supports that flag?

Zarf v0.41.0 seems to have the required version of its dependency on Helm:
#3026

Use this zarf.yaml and create a package, try to deploy it in an air gapped cluster. You must also call "zarf package create --confirm" on the air gapped cluster.
If you use that command in an environment, where it has internet access, it will work. But that's not our intended use case.

kind: ZarfPackageConfig

metadata:
  name: dagster
  description: auto-generated using `zarf dev generate`
  version: 1.8.7

components:
  - name: dagster
    required: true
    charts:
      - name: dagster
        version: 1.8.7
        url: https://dagster-io.github.io/helm
        namespace: dagster
    images:
      - docker.io/bitnami/minideb:stretch
      - docker.io/busybox:1.28
      - docker.io/dagster/dagster-celery-k8s:1.8.7
      - docker.io/dagster/user-code-example:1.8.7
      - docker.io/library/postgres:14.6
      - library/postgres:14.6

    actions:
      onDeploy:
        after:
          - cmd: |
            zarf tools kubectl label service dagster-dagster-webserver -n dagster zarf.dev/connect-name=dagster
          - cmd: |
            zarf tools kubectl annotate service dagster-dagster-webserver -n dagster zarf.dev/connect-description="Connect to the Dagster User Interface"

Thanks and greetings,
Josef

P.S.: I'm using Zarf v0.40.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ✨ New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant