Make it possible to disable drift-detection feature for each kind of application from Piped configuration

[nghialv]

What would you like to be added:

Why is this needed:

For example, in Terraform, that feature is still under Alpha status. It could cause numerous bugs and performance downgrades. Therefore, it should be disabled by default.

[kentakozuka]

While this feature is enabled by default now, disabling it by default affect existing users.
So how about adding an option to opt-out drift detections?
Which level of config is better to set?;

• piped config (if disabled, all terraform drift detections are disabled)
• application config (if disalbed, drift detection of the target application is disabled)

[kentakozuka]

@khanhtc1202 wdyt?

[khanhtc1202]

@kentakozuka Yes, it should be a configurable option for users who want to make this feature turning off. 👍

Which level of config is better to set?;

I think it's better to make it application config first, then add to piped configuration (if needed) since the piped is designed to control many applications at once, and disabling this feature of one piped may affect many unexpected applications due to users may forget which applications is handling by that piped. Of course, in the current usecase, we want to off that feature due to the piped usage, but the feature showing on the UI is part of the application, thus this is easier to understand if we make it on/off switch by the application configuration.

Nah, changed my mind. I think it should be piped configuration, and make it turning on by default.

[kentakozuka]

OK. Let me add the option to piped config 👍

[github-actions]

This issue is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 7 days.

[github-actions]

This issue was closed because it has been stalled for 7 days with no activity. Feel free to reopen if still applicable.