Skip to content

Commit 042de69

Browse files
committed
fix: fix container names for sdk, apache, nginx
Found through code inspection, the ApacheHttpd, Nginx and SDK injectors do not honor their container-names annotations. This is a breaking change if anyone is using the enablement flag with container names for these 3 injectors, which I believe would work as truthy would treat a list of container names as true. Signed-off-by: Alan Clucas <[email protected]>
1 parent e82f595 commit 042de69

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

pkg/instrumentation/podmutator.go

+3-3
Original file line numberDiff line numberDiff line change
@@ -232,15 +232,15 @@ func (langInsts *languageInstrumentations) setLanguageSpecificContainers(ns meta
232232
},
233233
{
234234
iwc: &langInsts.ApacheHttpd,
235-
annotation: annotationInjectApacheHttpd,
235+
annotation: annotationInjectApacheHttpdContainersName,
236236
},
237237
{
238238
iwc: &langInsts.Nginx,
239-
annotation: annotationInjectNginx,
239+
annotation: annotationInjectNginxContainersName,
240240
},
241241
{
242242
iwc: &langInsts.Sdk,
243-
annotation: annotationInjectSdk,
243+
annotation: annotationInjectSdkContainersName,
244244
},
245245
}
246246

0 commit comments

Comments
 (0)