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
Whilst building a CSV is still currently a manual process and therefore is error prone, taking the deploy/crds/cr.yaml file, converting to JSON, and pasting it under the alm-examples: field simply yields a type dict (the entire CR contents encapsulated in { }). If this is left as is, running operator-courier verify on the resulting bundle yields the following output: string indices must be integers.
Adding [ ] around the { } resolves this issue by converting alm-examples: JSON value from a type dict to a type list/array (but this value is of yaml type string as denoted by the | ). Can this be explicitly caught by the courier, perhaps adding a suggestion as with other missing or mis-populated fields?
Note that, this might not cause a deployment failure in OLM as with #104 as the courier verify exits abnormally (and this is done first in the CVP pipeline prior to deployment), but our partners (users) writing their operator using helm or ansible likely won't have a clue what to do to fix their CSV in the dev/testing stage if they get the error noted above.
The text was updated successfully, but these errors were encountered:
Whilst building a CSV is still currently a manual process and therefore is error prone, taking the
deploy/crds/cr.yaml
file, converting to JSON, and pasting it under thealm-examples:
field simply yields a type dict (the entire CR contents encapsulated in{ }
). If this is left as is, runningoperator-courier verify
on the resulting bundle yields the following output:string indices must be integers
.Adding
[ ]
around the{ }
resolves this issue by convertingalm-examples:
JSON value from a type dict to a type list/array (but this value is of yaml type string as denoted by the|
). Can this be explicitly caught by the courier, perhaps adding a suggestion as with other missing or mis-populated fields?Note that, this might not cause a deployment failure in OLM as with #104 as the courier verify exits abnormally (and this is done first in the CVP pipeline prior to deployment), but our partners (users) writing their operator using helm or ansible likely won't have a clue what to do to fix their CSV in the dev/testing stage if they get the error noted above.
The text was updated successfully, but these errors were encountered: