k6-operator icon indicating copy to clipboard operation
k6-operator copied to clipboard

Make `spec.paused` and `spec.quiet` a boolean in an upcoming version of the CRD

Open LCaparelli opened this issue 5 months ago • 1 comments

Feature Description

Unless there are plans to support other values and make it sort of an enum (which is also a breaking change), it probably makes more sense for spec.paused to be a bool. AFAICT from the code, we only use it as such anyway.

The current implementation leads to some awkwardness when using it, for example:

  • in Helm charts, as part of a template, for which we need to add quotes around user input
  • in Crossplane compositions, where we want to receive a bool from the user and then need to convert it to a string in order to match the current interface
  • probably any abstraction built on top of this that offers the field as a bool to users

Suggested Solution (optional)

No response

Already existing or connected issues / PRs (optional)

No response

LCaparelli avatar Aug 28 '24 21:08 LCaparelli