Always evaluate templates

  - This is required because certain agents might start before their
    labels are synced. If some of the snippets assigned to them contain
    template variables, loading the config will fail.

Author: jcreixell

pkg/config/agentmanagement_remote_config.go

@@ -164,11 +164,6 @@ func appendExternalLabels(c *Config, externalLabels map[string]string) {
 }
 
 func evaluateTemplate(config string, templateVariables map[string]any) (string, error) {
-	// Avoid doing anything if there are no template variables
-	if len(templateVariables) == 0 {
-		return config, nil
-	}
-
 	tpl, err := template.New("config").Parse(config)
 	if err != nil {
 		return "", err