Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unexpected Default Behavior Change in enable_search_index_orders #575

Closed
robin-hps opened this issue Mar 12, 2025 · 1 comment
Closed

Unexpected Default Behavior Change in enable_search_index_orders #575

robin-hps opened this issue Mar 12, 2025 · 1 comment
Labels
question Further information is requested

Comments

@robin-hps
Copy link

I wasn’t sure whether this should be a bug report, as in hindsight, it might be expected behavior. However, I wanted to share it for visibility.

After upgrading to Terraform provider version 1.19.2, we noticed that order search was unexpectedly disabled in our Commercetools setup. Upon investigating, we found that the enable_search_index_orders setting now defaults to true only when explicitly defined. Since we hadn’t specified this setting in our configuration, the default behavior changed, causing order searches to stop working.

That said, this may not technically be a bug, as our Terraform plan did indicate the change—we just overlooked it. My assumption is that this behavior is related to this change, but I’m not very proficient in Go, so I can’t say for certain.

Anyway, thanks for the good work 👍

Cheers,
Robin

@robin-hps robin-hps added question Further information is requested triage Needs triage labels Mar 12, 2025
@demeyerthom
Copy link
Member

Hi @robin-hps yeah this was an additional feature that had some impact on pre-existing environments. We tried improving the handling of these values by explicitly setting defaults to the same as the commercetools api, but I guess in pre-existing deployments that had different values where this was not specified in the config it would reset them to defaults under the hood.

Apologies for the inconvenience!

@demeyerthom demeyerthom removed the triage Needs triage label Mar 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants