Open tchiotludo opened 1 year ago
Or make the opposite : use terraform fields and internally generate this mandatory part of the YAML. Makes sense to define these values with TF params to me.
WDYT ?
I've an issue, the goal keep_original_source is to kept exact source of the flow, if the people want to add comment everywhere (at the beginning, end, between namespace and id properties), it must be able to it, when we unserialize, serialize a yaml string, most of parsers will break that (remove comment, changing quote, ...), so it appear to be not the good way
Okey, and what about doing it in the API call : https://kestra.io/docs/api-guide/#post-/api/v1/flows.
Using values from content
for everything except id
and namespace
if var provided using TF ressource, otherwise use all values from content
.
tldr : not having to unserialize, serialize a yaml string
In the end it's more a question about how we see the terraform provider.
If no logic at all should be implemented within the provider then we go with only content
& keep_original_source
params.
Otherwise we could also have flow_id
, namespace
and some other future fields.
As discuss with Ludo long times ago, this would be tricky, so we decided that the change will be done when we will fully change the API to create flows
As discuss with Ludo long times ago, this would be tricky, so we decided that the change will be done when we will fully change the API to create flows
OK yes. Can't wait for this. Do you have an idea when it will be released?
@aballiet We just deprecated methods using JSON to create/edit flow, so I can't really tell
right now, we could read it from the yaml like that
but we are able to read it directly from the yaml since the
id
andnamespace
are mandatory