[9.0] (backport #10259) [OTel] Fix CLI example to use proper syntax (#10271)

* [OTel] Fix CLI example to use proper syntax (#10259)

This uses the expected `::` syntax instead of `.` for the `--set`
argument.

(cherry picked from commit e917e0a)

* Remove tests that would require a breaking change to fix

---------

Co-authored-by: Chris Earle <[email protected]>
Co-authored-by: Chris Earle <[email protected]>

Author: mergify[bot]

internal/pkg/agent/cmd/otel_flags.go

@@ -26,7 +26,7 @@ func setupOtelFlags(flags *pflag.FlagSet) {
 		" single location can be set per flag entry e.g. `--config=file:/path/to/first --config=file:path/to/second`.")
 
 	flags.StringArray(otelSetFlagName, []string{}, "Set arbitrary component config property. The component has to be defined in the config file and the flag"+
-		" has a higher precedence. Array config properties are overridden and maps are joined. Example --set=processors.batch.timeout=2s")
+		" has a higher precedence. Array config properties are overridden and maps are joined. Example --set \"processors::batch::timeout=2s\"")
 
 	goFlags := new(flag.FlagSet)
 	featuregate.GlobalRegistry().RegisterFlags(goFlags)

internal/pkg/agent/cmd/validate_test.go

@@ -23,6 +23,11 @@ func TestValidateCommand(t *testing.T) {
 			[]string{filepath.Join("testdata", "otel", "otel.yml")},
 			false,
 		},
+		{
+			"otel config with set",
+			[]string{filepath.Join("testdata", "otel", "otel.yml"), "yaml:processors::resource::attributes: [{ key: service.name, action: insert, value: elastic-otel-test1 }]"},
+			false,
+		},
 		{
 			"agent config",
 			[]string{filepath.Join("testdata", "otel", "elastic-agent.yml")},
@@ -33,7 +38,12 @@ func TestValidateCommand(t *testing.T) {
 	for _, tc := range tt {
 		t.Run(tc.Name, func(t *testing.T) {
 			err := validateOtelConfig(context.Background(), tc.ConfigPaths)
-			require.Equal(t, tc.ExpectingErr, err != nil)
+
+			if tc.ExpectingErr {
+				require.Error(t, err)
+			} else {
+				require.NoError(t, err)
+			}
 		})
 	}
 }