Skip to content

Object missing metadata fields to store the chart metadata information  #49

@scbizu

Description

@scbizu

Related to chartmuseum's #22 and #220 , we still have a bug of resolving chart deletion between cache and the storage , we temporarily fix it by treating the cache one as no content chart file and parsing the delete chart version by ourselves to find the correct version to delete.

But parsing the version correctly is hard even if the stored chart version is non-semver , I raise a pr#363 to fix the issue but it can not cover all the cases. So I think parsing the delete version by ourselves maybe is not the right way to fix this issue.

The root cause of this issue is the storage.Object can not store the metadata of our chart file , if we can add the such field into the storage.Object, the issue can be fixed in the right way .

/cc @jdolitsky , if ok , I can work on another PR to fix this issue .

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions