Skip to content

Chart: fix securityContext.containers/ingress.apiServer in values.schema.json #60575

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.

Sign up for GitHub

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

Merged
jedcunningham merged 1 commit into from
Jan 15, 2026
Merged

Chart: fix securityContext.containers/ingress.apiServer in values.schema.json #60575

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
114 changes: 113 additions & 1 deletion chart/values.schema.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@
}
]
},
"container": {
"containers": {
"description": "Default container security context definition. The values in this parameter will be used when `securityContexts` is not defined for specific containers",
"type": "object",
"$ref": "#/definitions/io.k8s.api.core.v1.SecurityContext",
Expand Down Expand Up @@ -256,6 +256,118 @@
],
"default": null
},
"apiServer": {
"description": "Configuration for the Ingress of the API server.",
"type": "object",
"additionalProperties": false,
"properties": {
"enabled": {
"description": "Enable API server ingress resource.",
"type": "boolean",
"default": false
},
"annotations": {
"description": "Annotations for the API server Ingress.",
"type": "object",
"default": {},
"additionalProperties": {
"type": "string"
}
},
"path": {
"description": "The path for the API server Ingress.",
"type": "string",
"default": "/"
},
"pathType": {
"description": "The pathType for the API server Ingress (required for Kubernetes 1.19 and above).",
"type": "string",
"default": "ImplementationSpecific"
},
"host": {
"description": "The hostname for the API server Ingress. (Deprecated - renamed to `ingress.apiServer.hosts`)",
"type": "string",
"default": ""
},
"hosts": {
"description": "The hostnames or hosts configuration for the API server Ingress.",
"type": "array",
"default": [],
"items": {
"oneOf": [
{
"type": "object",
"additionalProperties": false,
"properties": {
"name": {
"description": "The hostname for the API server Ingress.",
"type": "string",
"default": ""
},
"tls": {
"description": "Configuration for API server Ingress TLS.",
"type": "object",
"additionalProperties": false,
"properties": {
"enabled": {
"description": "Enable TLS termination for the API server Ingress.",
"type": "boolean",
"default": false
},
"secretName": {
"description": "The name of a pre-created Secret containing a TLS private key and certificate.",
"type": "string",
"default": ""
}
}
}
},
"required": [
"name"
]
},
{
"type": "string",
"default": "",
"$comment": "Deprecated by object above"
}
]
}
},
"ingressClassName": {
"description": "The Ingress Class for the API server Ingress.",
"type": "string",
"default": ""
},
"tls": {
"description": "Configuration for API server Ingress TLS. (Deprecated - renamed to `ingress.apiServer.hosts[*].tls`)",
"type": "object",
"additionalProperties": false,
"properties": {
"enabled": {
"description": "Enable TLS termination for the API server Ingress.",
"type": "boolean",
"default": false
},
"secretName": {
"description": "The name of a pre-created Secret containing a TLS private key and certificate.",
"type": "string",
"default": ""
}
}
},
"precedingPaths": {
"description": "HTTP paths to add to the API server Ingress before the default path.",
"type": "array",
"default": []
},
"succeedingPaths": {
"description": "HTTP paths to add to the API server Ingress after the default path.",
"type": "array",
"default": []
}
}
},
"web": {
"description": "Configuration for the Ingress of the web Service.",
"type": "object",
Expand Down
4 changes: 4 additions & 0 deletions scripts/ci/prek/chart_schema.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,10 @@

CHART_DIR = AIRFLOW_ROOT_PATH / "chart"
KNOWN_INVALID_TYPES = {
# I don't know the data structure for this type with 100 certainty. We have no tests.
"$['properties']['ingress']['properties']['apiServer']['properties']['precedingPaths']",
# I don't know the data structure for this type with 100 certainty. We have no tests.
"$['properties']['ingress']['properties']['apiServer']['properties']['succeedingPaths']",
# I don't know the data structure for this type with 100 certainty. We have no tests.
"$['properties']['ingress']['properties']['web']['properties']['precedingPaths']",
# I don't know the data structure for this type with 100 certainty. We have no tests.
Expand Down