You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Description:
While working with helm-s3, I encountered an issue where large metadata in Chart.yaml caused failures during reindexing. The issue arises because helm-s3 marshals the chart's metadata into JSON and stores it as x-amz-meta-chart-metadata in an S3 object, but S3 limits metadata to 2KB. If the metadata exceeds this limit, it gets truncated, leading to issues when reindexing charts.
Add a long string (e.g., in the description field) in Chart.yaml.
Push the chart using helm-s3.
The chart gets uploaded, but the metadata is marshaled into x-amz-meta-chart-metadata and truncated due to the 2KB limit.
When running helm s3 reindex, the metadata fails to unmarshal, causing the reindex operation to fail.
Proposal:
To avoid this issue and prevent future operations (such as reindexing) from failing, I propose that the Push command should throw an error if the metadata exceeds 2KB. This will inform users at the point of pushing the chart, allowing them to adjust the metadata size before uploading.
Benefit:
By adding this validation to the Push operation, users will be able to resolve the issue earlier, avoiding troubleshooting time and preventing failures in subsequent operations such as reindex.
The text was updated successfully, but these errors were encountered:
Description:
While working with
helm-s3
, I encountered an issue where large metadata inChart.yaml
caused failures during reindexing. The issue arises becausehelm-s3
marshals the chart's metadata into JSON and stores it asx-amz-meta-chart-metadata
in an S3 object, but S3 limits metadata to 2KB. If the metadata exceeds this limit, it gets truncated, leading to issues when reindexing charts.S3 docs
Steps to Reproduce:
description
field) inChart.yaml
.helm-s3
.x-amz-meta-chart-metadata
and truncated due to the 2KB limit.helm s3 reindex
, the metadata fails to unmarshal, causing the reindex operation to fail.Proposal:
To avoid this issue and prevent future operations (such as reindexing) from failing, I propose that the
Push
command should throw an error if the metadata exceeds 2KB. This will inform users at the point of pushing the chart, allowing them to adjust the metadata size before uploading.Benefit:
By adding this validation to the
Push
operation, users will be able to resolve the issue earlier, avoiding troubleshooting time and preventing failures in subsequent operations such asreindex
.The text was updated successfully, but these errors were encountered: