Skip to content

Commit

Permalink
Removing JMX boolean which was added and not used
Browse files Browse the repository at this point in the history
  • Loading branch information
carlosroman committed Nov 15, 2024
1 parent f3a33cc commit 3466bcf
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 9 deletions.
6 changes: 1 addition & 5 deletions components/datadog/agent/docker_image.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ const (
defaultOTAgentImageTag = "nightly-ot-beta-main"
)

func dockerAgentFullImagePath(e config.Env, repositoryPath, imageTag string, otel, jmx bool) string {
func dockerAgentFullImagePath(e config.Env, repositoryPath, imageTag string, otel bool) string {
// return agent image path if defined
if e.AgentFullImagePath() != "" {
return e.AgentFullImagePath()
Expand Down Expand Up @@ -53,10 +53,6 @@ func dockerAgentFullImagePath(e config.Env, repositoryPath, imageTag string, ote
imageTag = dockerAgentImageTag(e, config.AgentSemverVersion)
}

if jmx {
imageTag = fmt.Sprintf("%s-jmx", imageTag)
}

return utils.BuildDockerImagePath(repositoryPath, imageTag)
}

Expand Down
2 changes: 1 addition & 1 deletion components/datadog/agent/ecs.go
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ func ecsLinuxAgentSingleContainerDefinition(e config.Env, apiKeySSMParamName pul
Cpu: pulumi.IntPtr(200),
Memory: pulumi.IntPtr(512),
Name: pulumi.String("datadog-agent"),
Image: pulumi.String(dockerAgentFullImagePath(e, "public.ecr.aws/datadog/agent", "", false, false)),
Image: pulumi.String(dockerAgentFullImagePath(e, "public.ecr.aws/datadog/agent", "", false)),
Essential: pulumi.BoolPtr(true),
LinuxParameters: ecs.TaskDefinitionLinuxParametersArgs{
Capabilities: ecs.TaskDefinitionKernelCapabilitiesArgs{
Expand Down
4 changes: 2 additions & 2 deletions components/datadog/agent/ecsFargate.go
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import (

func ECSFargateLinuxContainerDefinition(e config.Env, image string, apiKeySSMParamName pulumi.StringInput, fakeintake *fakeintake.Fakeintake, logConfig ecs.TaskDefinitionLogConfigurationPtrInput) *ecs.TaskDefinitionContainerDefinitionArgs {
if image == "" {
image = dockerAgentFullImagePath(e, "public.ecr.aws/datadog/agent", "", false, false)
image = dockerAgentFullImagePath(e, "public.ecr.aws/datadog/agent", "", false)
}
return &ecs.TaskDefinitionContainerDefinitionArgs{
Cpu: pulumi.IntPtr(0),
Expand Down Expand Up @@ -96,7 +96,7 @@ func ECSFargateLinuxContainerDefinition(e config.Env, image string, apiKeySSMPar
// https://docs.aws.amazon.com/AmazonECS/latest/developerguide/tutorial-deploy-fluentbit-on-windows.html
func ECSFargateWindowsContainerDefinition(e config.Env, image string, apiKeySSMParamName pulumi.StringInput, fakeintake *fakeintake.Fakeintake) *ecs.TaskDefinitionContainerDefinitionArgs {
if image == "" {
image = dockerAgentFullImagePath(e, "public.ecr.aws/datadog/agent", "", false, false)
image = dockerAgentFullImagePath(e, "public.ecr.aws/datadog/agent", "", false)
}
return &ecs.TaskDefinitionContainerDefinitionArgs{
Cpu: pulumi.IntPtr(0),
Expand Down
2 changes: 1 addition & 1 deletion components/datadog/agent/kubernetes_helm.go
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ func NewHelmInstallation(e config.Env, args HelmInstallationArgs, opts ...pulumi
}

// Compute some values
agentImagePath := dockerAgentFullImagePath(e, "", "", args.OTelAgent, false)
agentImagePath := dockerAgentFullImagePath(e, "", "", args.OTelAgent)
if args.AgentFullImagePath != "" {
agentImagePath = args.AgentFullImagePath
}
Expand Down

0 comments on commit 3466bcf

Please sign in to comment.