Skip to content

Commit

Permalink
Forces HELM to use auto settings as output_type
Browse files Browse the repository at this point in the history
  • Loading branch information
Alessandro De Maria committed Dec 18, 2024
1 parent 99dabad commit cc2cebc
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 3 deletions.
4 changes: 2 additions & 2 deletions kapitan/inputs/helm.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@

from kapitan.errors import HelmTemplateError
from kapitan.helm_cli import helm_cli
from kapitan.inputs.base import CompiledFile, InputType
from kapitan.inputs.base import InputType
from kapitan.inputs.kadet import BaseModel, BaseObj
from kapitan.inventory.model.input_types import KapitanInputTypeHelmConfig
from kapitan.inventory.model.input_types import KapitanInputTypeHelmConfig, OutputType

logger = logging.getLogger(__name__)

Expand Down
12 changes: 11 additions & 1 deletion kapitan/inventory/model/input_types.py
Original file line number Diff line number Diff line change
@@ -1,9 +1,12 @@
import logging
from typing import Annotated, List, Literal, Optional, Union

from pydantic import BaseModel, ConfigDict, Field
from pydantic import BaseModel, ConfigDict, Field, field_validator

from kapitan.utils import StrEnum

logger = logging.getLogger(__name__)


class InputTypes(StrEnum):
JSONNET = "jsonnet"
Expand Down Expand Up @@ -72,6 +75,13 @@ class KapitanInputTypeHelmConfig(KapitanInputTypeBaseConfig):
helm_path: Optional[str] = None
kube_version: Optional[str] = None

@field_validator("output_type")
@classmethod
def type_must_be_auto(cls, _: OutputType) -> OutputType:
"""Helm output type must be auto."""
logger.debug("field `output_type` for helm input type must be set to 'auto': enforcing.")
return OutputType.AUTO


class KapitanInputTypeKadetConfig(KapitanInputTypeBaseConfig):
input_type: Literal[InputTypes.KADET] = InputTypes.KADET
Expand Down

0 comments on commit cc2cebc

Please sign in to comment.