Skip to content

Commit

Permalink
Automatic kubernetes api monitoring
Browse files Browse the repository at this point in the history
- Removes unnecessary checks
  • Loading branch information
fabwer committed Jan 4, 2022
1 parent be7d0ef commit 3ab6861
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,6 @@ func (r *AutomaticApiMonitoringReconciler) Reconcile() error {
}

func (r *AutomaticApiMonitoringReconciler) ensureSettingExists() (string, error) {
if r.name == "" {
return "", errors.New("no name given")
}
if r.kubeSystemUUID == "" {
return "", errors.New("no kube-system namespace UUID given")
}
Expand Down
1 change: 0 additions & 1 deletion src/controllers/dynakube/dynakube_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -314,7 +314,6 @@ func (r *ReconcileDynaKube) reconcileActiveGateCapabilities(dkState *status.Dyna

//start automatic config creation
if dkState.Instance.Status.KubeSystemUUID != "" &&
dkState.Instance.Name != "" &&
dkState.Instance.FeatureAutomaticKubernetesApiMonitoring() &&
dkState.Instance.KubernetesMonitoringMode() {
err := automaticapimonitoring.NewReconciler(dtc, dkState.Instance.Name, dkState.Instance.Status.KubeSystemUUID).
Expand Down
3 changes: 0 additions & 3 deletions src/dtclient/kubernetes_settings.go
Original file line number Diff line number Diff line change
Expand Up @@ -69,9 +69,6 @@ type constraintViolations []struct {
}

func (dtc *dynatraceClient) CreateKubernetesSetting(name, kubeSystemUUID, scope string) (string, error) {
if name == "" {
return "", errors.New("no name given")
}
if kubeSystemUUID == "" {
return "", errors.New("no kube-system namespace UUID given")
}
Expand Down

0 comments on commit 3ab6861

Please sign in to comment.