Skip to content
This repository has been archived by the owner on Dec 10, 2024. It is now read-only.
This repository has been archived by the owner on Dec 10, 2024. It is now read-only.

Subcharts declared in subfolders under charts/ are ignored #49

Open
@consideRatio

Description

Frigate currently only considers Chart.yaml declared dependencies, but ignores subcharts directly located under the charts/ folder. I don't run into issues about this, but it is a broken expectation on how frigate works if I would use frigate on a helm chart wish subcharts and not just chart dependencies.

frigate/frigate/gen.py

Lines 58 to 74 in 4153bf5

root = [] if root is None else root
chart, values = load_chart(chartdir, root=root)
if "dependencies" in chart:
for dependency in chart["dependencies"]:
dependency_name = dependency["name"]
dependency_path = os.path.join(
chartdir, "charts", f"{dependency_name}-{dependency['version']}.tgz",
)
with tempfile.TemporaryDirectory() as tmpdirname:
shutil.unpack_archive(dependency_path, tmpdirname)
dependency_dir = os.path.join(tmpdirname, dependency_name)
update_chart_dependencies(tmpdirname, dependency_name)
_, dependency_values = load_chart_with_dependencies(
dependency_dir, root + [dependency_name]
)
values = squash_duplicate_values(values + dependency_values)

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions