feat(k8sd)!: introduce support for storing helm charts on microcluster db #1093
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Microcluster Schema Changes DetectedPlease ensure that the changes are intentional and have been reviewed by the appropriate team members. Adding a migration:You should not change the Microcluster migration files in interim Kubernetes releases to ensure a smooth Do not under any circumstances remove any migrations and never change the order of migrations in the microcluster schema files. |
…r db - Added a table to the microcluster database for storing helm charts. - Added queries and helper functions for inserting and selecting helm charts. - Added the ChartLoader interface and updated the helm client to enable loading helm charts in different ways. - Added the database based ChartLoader to fetch charts from microcluser. - Added the file based ChartLoader to fetch charts from the filesystem, used in testing. - Added a helm chart controller that synchronizes available charts in the snap to the microcluster database. - Moved helm charts from `k8s/manifests` into respective feature directories and embedded the charts using go embed package. - Packed helm charts that are in directory form to tarballs. - Updated feature controller to create the helm char with the database loader and updated features to consume a helm client. - Fixed the bug of ports still being in use, in the testing util `WithState` by adding a retry loop for microcluster bootstrapping. - Updated InstallableChart type to include the name and version of the chart.
berkayoz
force-pushed
the
KU-2730/helm-charts-microcluster
branch
from
61374c4 to
75ff4bc
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
k8s/manifestsinto respective feature directories and embedded the charts using go embed package.WithStateby adding a retry loop for microcluster bootstrapping.