Skip to content

[epic] operator-controller rejects helm charts that use helm hooks #995

Open
@joelanford

Description

@joelanford

In order to reduce the scope of an initial implementation of helm support in operator-controller, should we proactively reject helm charts that ultimately include chart hooks in their manifests?

Supporting helm chart hooks generally means that we are using helm's libraries to manage the lifecycle of the objects, which has other implications (e.g. see #923).

It also means that there are additional complexities due to the stages introduced by chart hooks (e.g. it is generally not acceptable to block in a Reconcile implementation).

NOTE: This is not immediately important while we are only supporting registry+v1 bundles because operator-controller generates helm charts on-the-fly from registry+v1 bundle content, and it does not use chart hooks.

Metadata

Metadata

Assignees

No one assigned

    Labels

    epicv1.xIssues related to OLMv1 features that come after 1.0

    Type

    No type

    Projects

    Status

    No status

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions