This repository was archived by the owner on Dec 10, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 27
This repository was archived by the owner on Dec 10, 2024. It is now read-only.
Subcharts declared in subfolders under charts/ are ignored #49
Copy link
Copy link
Open
Labels
bugSomething isn't workingSomething isn't working
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.
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) |
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working