make blocks helm, kustomize, directory optional

[AkakievKD]

The point of the feature is that the implementation is similar to the sync_policy block, if the automated block and the sync_options field do not have any values, then the entire sync_policy block is removed from the final description resources during output this feature will allow you to deploy charts with default values

[mkilchhofer]

All your mentioned blocks (helm, kustomize and directory) are already optional.

Helm:

terraform-provider-argocd/argocd/schema_application.go

Lines 1307 to 1313 in f0b27f3

	"helm": {
	Type: schema.TypeList,
	Description: "Helm specific options.",
	MaxItems: 1,
	MinItems: 1,
	Optional: true,
	Elem: &schema.Resource{

Kustomize:

terraform-provider-argocd/argocd/schema_application.go

Lines 1399 to 1404 in f0b27f3

	"kustomize": {
	Type: schema.TypeList,
	Description: "Kustomize specific options.",
	MaxItems: 1,
	MinItems: 1,
	Optional: true,

Directory:

terraform-provider-argocd/argocd/schema_application.go

Lines 1466 to 1468 in f0b27f3

	MaxItems: 1,
	MinItems: 1,
	Optional: true,

In my opinion it looks more like a syntax error inside your IDE (VScode):

• The source block is not closed (missing curly bracket).

Can you confirm that, @Nello-Angelo ?

[AkakievKD]

yes i did mistake

[AkakievKD]

here I have a problem with the fact that if the block remains empty then the provider panic

#492

@mkilchhofer

[AkakievKD]

maybe you should add a condition if the values ​​of these variables are empty and nothing is passed to them, then simply ignore this block and perhaps indicate in the code that they are optional

the same behavior in sync_policy block try to do the same with other blocks with panic if empty

I mean that this block corresponds to the described behavior with removal from the final terraform output