chore: use ignored containers in reconcile

cli/cmd/install.go

@@ -7,6 +7,7 @@ import (
 	"time"
 
 	"github.com/odigos-io/odigos/cli/pkg/autodetect"
+	"github.com/odigos-io/odigos/common/consts"
 
 	"github.com/odigos-io/odigos/cli/pkg/labels"
 
@@ -17,8 +18,6 @@ import (
 	"github.com/odigos-io/odigos/cli/pkg/kube"
 	"github.com/odigos-io/odigos/cli/pkg/log"
 	"github.com/odigos-io/odigos/common"
-	"github.com/odigos-io/odigos/common/consts"
-	"github.com/odigos-io/odigos/common/utils"
 	k8sconsts "github.com/odigos-io/odigos/k8sutils/pkg/consts"
 
 	"github.com/spf13/cobra"
@@ -205,10 +204,8 @@ func validateUserInputProfiles(tier common.OdigosTier) {
 }
 
 func createOdigosConfig(odigosTier common.OdigosTier) common.OdigosConfiguration {
-	fullIgnoredContainers := utils.MergeDefaultIgnoreWithUserInput(userInputIgnoredContainers, consts.IgnoredContainers)
 
 	selectedProfiles := []common.ProfileName{}
-
 	for _, profile := range userInputInstallProfiles {
 		selectedProfiles = append(selectedProfiles, common.ProfileName(profile))
 	}
@@ -218,7 +215,7 @@ func createOdigosConfig(odigosTier common.OdigosTier) common.OdigosConfiguration
 		TelemetryEnabled:          telemetryEnabled,
 		OpenshiftEnabled:          openshiftEnabled,
 		IgnoredNamespaces:         userInputIgnoredNamespaces,
-		IgnoredContainers:         fullIgnoredContainers,
+		IgnoredContainers:         userInputIgnoredContainers,
 		SkipWebhookIssuerCreation: skipWebhookIssuerCreation,
 		Psp:                       psp,
 		ImagePrefix:               imagePrefix,
@@ -253,8 +250,8 @@ func init() {
 	installCmd.Flags().StringVar(&autoScalerImage, "autoscaler-image", "keyval/odigos-autoscaler", "autoscaler container image name")
 	installCmd.Flags().StringVar(&imagePrefix, "image-prefix", "", "prefix for all container images. used when your cluster doesn't have access to docker hub")
 	installCmd.Flags().BoolVar(&psp, "psp", false, "enable pod security policy")
-	installCmd.Flags().StringSliceVar(&userInputIgnoredNamespaces, "ignore-namespace", consts.DefaultIgnoredNamespaces, "namespaces not to show in odigos ui")
-	installCmd.Flags().StringSliceVar(&userInputIgnoredContainers, "ignore-container", consts.IgnoredContainers, "container names to exclude from instrumentation (useful for sidecar container)")
+	installCmd.Flags().StringSliceVar(&userInputIgnoredNamespaces, "ignore-namespace", k8sconsts.DefaultIgnoredNamespaces, "namespaces not to show in odigos ui")
+	installCmd.Flags().StringSliceVar(&userInputIgnoredContainers, "ignore-container", k8sconsts.DefaultIgnoredContainers, "container names to exclude from instrumentation (useful for sidecar container)")
 	installCmd.Flags().StringSliceVar(&userInputInstallProfiles, "profile", []string{}, "install preset profiles with a specific configuration")
 
 	if OdigosVersion != "" {

common/consts/consts.go

@@ -41,8 +41,3 @@ const (
 var (
 	ErrorPodsNotFound = errors.New("could not find a ready pod")
 )
-
-var (
-	DefaultIgnoredNamespaces = []string{"kube-system", "local-path-storage", "istio-system", "linkerd", "kube-node-lease"}
-	IgnoredContainers        = []string{"istio-proxy", "vault-agent", "filebeat", "linkerd-proxy", "fluentd", "akeyless-init"}
-)

helm/odigos/values.yaml

@@ -2,22 +2,16 @@ image:
   tag: ""
 imagePullSecrets: []
 
-# namespaces not to show in odigos ui
+# namespaces list not to show in odigos ui
+# set by default: odigos-system, kube-system, local-path-storage, istio-system, linkerd, kube-node-lease, odigos-system
+# you can add additional namespaces to ignore by adding them to the list
 ignoredNamespaces:
-  - odigos-system
-  - kube-system
-  - local-path-storage
-  - istio-system
-  - linkerd
-  - kube-node-lease
 
+# container names to never instrument
+# useful for sidecars which are not interesting to be instrumented
+# set by default: istio-proxy, vault-agent, filebeat, linkerd-proxy, fluentd, akeyless-init
+# you can add additional container names to ignore by adding them to the list
 ignoredContainers:
-  - istio-proxy
-  - vault-agent
-  - filebeat
-  - linkerd-proxy
-  - fluentd
-  - akeyless-init
 
 collectorGateway:
   # the memory request for the cluster gateway collector deployment.

k8sutils/pkg/consts/consts.go

@@ -2,6 +2,11 @@ package consts
 
 import "k8s.io/apimachinery/pkg/util/version"
 
+var (
+	DefaultIgnoredNamespaces = []string{"kube-system", "local-path-storage", "istio-system", "linkerd", "kube-node-lease"}
+	DefaultIgnoredContainers = []string{"istio-proxy", "vault-agent", "filebeat", "linkerd-proxy", "fluentd", "akeyless-init"}
+)
+
 type CollectorRole string
 
 const (
@@ -69,4 +74,4 @@ var (
 const (
 	OdigosCloudApiKeySecretKey = "odigos-cloud-api-key"
 	OdigosOnpremTokenSecretKey = "odigos-onprem-token"
-)
+)

scheduler/controllers/odigosconfig/odigosconfig_controller.go

@@ -5,6 +5,7 @@ import (
 
 	"github.com/odigos-io/odigos/common"
 	"github.com/odigos-io/odigos/common/consts"
+	k8sconsts "github.com/odigos-io/odigos/k8sutils/pkg/consts"
 	"github.com/odigos-io/odigos/k8sutils/pkg/env"
 	corev1 "k8s.io/api/core/v1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
@@ -28,9 +29,12 @@ func (r *odigosConfigController) Reconcile(ctx context.Context, _ ctrl.Request)
 	}
 
 	// make sure the default ignored namespaces are always present
-	odigosConfig.IgnoredNamespaces = mergeIgnoredItemLists(odigosConfig.IgnoredNamespaces, consts.DefaultIgnoredNamespaces)
+	odigosConfig.IgnoredNamespaces = mergeIgnoredItemLists(odigosConfig.IgnoredNamespaces, k8sconsts.DefaultIgnoredNamespaces)
 	odigosConfig.IgnoredNamespaces = append(odigosConfig.IgnoredNamespaces, env.GetCurrentNamespace())
 
+	// make sure the default ignored containers are always present
+	odigosConfig.IgnoredContainers = mergeIgnoredItemLists(odigosConfig.IgnoredContainers, k8sconsts.DefaultIgnoredContainers)
+
 	err = r.persistEffectiveConfig(ctx, odigosConfig)
 	if err != nil {
 		return ctrl.Result{}, err