rm-disable-compression: use pflag's MarkDeprecated

One small downside of using MarkDeprecated is that pflag doesn't use our
logr.Logger, so it is printed as pain text rather than the format
specified in --logging-format.

Author: maelvls

pkg/agent/config.go

@@ -167,10 +167,6 @@ type AgentCmdFlags struct {
 
 	// Prometheus (--enable-metrics) enables the Prometheus metrics server.
 	Prometheus bool
-
-	// DisableCompression (--disable-compression) is deprecated and no longer
-	// has an effect.
-	DisableCompression bool
 }
 
 func InitAgentCmdFlags(c *cobra.Command, cfg *AgentCmdFlags) {
@@ -297,13 +293,15 @@ func InitAgentCmdFlags(c *cobra.Command, cfg *AgentCmdFlags) {
 		"Enables Prometheus metrics server on the agent (port: 8081).",
 	)
 
+	var dummy bool
 	c.PersistentFlags().BoolVar(
-		&cfg.DisableCompression,
+		&dummy,
 		"disable-compression",
 		false,
 		"Deprecated. No longer has an effect.",
 	)
-	c.PersistentFlags().MarkHidden("disable-compression")
+	c.PersistentFlags().MarkDeprecated("disable-compression", "no longer has an effect")
+
 }
 
 type AuthMode string
@@ -585,13 +583,6 @@ func ValidateAndCombineConfig(log logr.Logger, cfg Config, flags AgentCmdFlags)
 		}
 	}
 
-	// Validation of --disable-compression.
-	{
-		if flags.DisableCompression {
-			log.Info("The flag --disable-compression has been deprecated an no longer has any effect.")
-		}
-	}
-
 	// Validation of the config fields exclude_annotation_keys_regex and
 	// exclude_label_keys_regex.
 	{