Skip to content

Commit

Permalink
Fix liniting
Browse files Browse the repository at this point in the history
  • Loading branch information
0sewa0 committed Nov 25, 2024
1 parent f0d46bd commit 264f5bc
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 4 deletions.
8 changes: 6 additions & 2 deletions pkg/clients/dynatrace/agent_version.go
Original file line number Diff line number Diff line change
Expand Up @@ -90,10 +90,14 @@ func (dtc *dynatraceClient) GetAgentVersions(ctx context.Context, os, installerT
return nil, errors.New("os or installerType is empty")
}

flavor = determineFlavor(installerType)
_flavor := determineFlavor(installerType)
if flavor != "" {
_flavor = flavor
}

_arch := determineArch(installerType)

url := dtc.getAgentVersionsUrl(os, installerType, flavor, _arch)
url := dtc.getAgentVersionsUrl(os, installerType, _flavor, _arch)
err := dtc.makeRequestAndUnmarshal(ctx, url, dynatracePaaSToken, &response)

return response.AvailableVersions, errors.WithStack(err)
Expand Down
1 change: 0 additions & 1 deletion pkg/clients/dynatrace/endpoints.go
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@ func (dtc *dynatraceClient) getAgentVersionsUrl(os, installerType, flavor, arch

return fmt.Sprintf("%s/v1/deployment/installer/agent/versions/%s/%s?flavor=%s",
dtc.url, os, installerType, flavor)

}

func (dtc *dynatraceClient) getOneAgentConnectionInfoUrl() string {
Expand Down
1 change: 1 addition & 0 deletions pkg/controllers/dynakube/oneagent/daemonset/affinity.go
Original file line number Diff line number Diff line change
Expand Up @@ -13,5 +13,6 @@ func (b *builder) affinity() *corev1.Affinity {
} else {
affinity = node.Affinity()
}

return &affinity
}
2 changes: 1 addition & 1 deletion pkg/util/kubeobjects/node/affinity.go
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ func Affinity() corev1.Affinity {
}

// AMDOnlyAffinity provides an affinity that will only allow deployment on AMD64 nodes.
// This if manly needed for the Dynatrace tenant-registry as it only has AMD64 images.
// This is manly needed for the Dynatrace tenant-registry as it only has AMD64 images.
func AMDOnlyAffinity() corev1.Affinity {
return AffinityForArches(arch.AMDImage)
}
Expand Down

0 comments on commit 264f5bc

Please sign in to comment.