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

Offline chart installs broken #20539

Closed
2 tasks done
jedcunningham opened this issue Dec 28, 2021 · 0 comments · Fixed by #20544
Closed
2 tasks done

Offline chart installs broken #20539

jedcunningham opened this issue Dec 28, 2021 · 0 comments · Fixed by #20544
Assignees
Labels
area:helm-chart Airflow Helm Chart kind:bug This is a clearly a bug

Comments

@jedcunningham
Copy link
Member

Official Helm Chart version

main (development)

Apache Airflow version

2.2.3 (latest released)

Kubernetes Version

1.21.5

Helm Chart configuration

No response

Docker Image customisations

No response

What happened

If you don't have outbound routing to the internet, you cannot install/template/lint the helm chart, e.g:

$ helm upgrade --install oss ~/github/airflow/chart 
Release "oss" does not exist. Installing it now.
Error: values don't meet the specifications of the schema(s) in the following chart(s):
airflow:
Get "https://raw.githubusercontent.com/yannh/kubernetes-json-schema/master/v1.22.0-standalone-strict/networkpolicypeer-networking-v1.json": dial tcp: lookup raw.githubusercontent.com: no such host

What you expected to happen

The chart should function without requiring outbound internet access.

How to reproduce

Disable networking and try installing the chart.

Anything else

We use external schema references in our chart, e.g:

"$ref": "https://raw.githubusercontent.com/yannh/kubernetes-json-schema/master/v1.22.0-standalone-strict/resourcerequirements-v1.json"

Unfortunately, naively using relative filesystem reference in values.schema.json doesn't work properly in helm (see helm/helm#10481).

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:helm-chart Airflow Helm Chart kind:bug This is a clearly a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant