From 9f1b9da49a6db805256285c332a0401264df2e09 Mon Sep 17 00:00:00 2001 From: Katarzyna Kulpa-Dybel <62446372+katarzyna-kulpa@users.noreply.github.com> Date: Wed, 20 Mar 2024 09:57:40 +0100 Subject: [PATCH] [issue-1122] Removing resolved TODOs (#217) --- .github/.codecov.yaml | 1 - charts/csi-baremetal-deployment/values.yaml | 2 -- charts/csi-baremetal-operator/templates/rbac.yaml | 2 -- 3 files changed, 5 deletions(-) diff --git a/.github/.codecov.yaml b/.github/.codecov.yaml index ed7ee5c5..0a3a7b93 100644 --- a/.github/.codecov.yaml +++ b/.github/.codecov.yaml @@ -6,7 +6,6 @@ coverage: project: default: threshold: 1% -# TODO: improve code coverage - https://github.com/dell/csi-baremetal/issues/658 ignore: - "pkg/node/node_daemonset.go" - "pkg/patcher/scheduler_patcher_vanilla.go" diff --git a/charts/csi-baremetal-deployment/values.yaml b/charts/csi-baremetal-deployment/values.yaml index 9727bedd..b10d2000 100644 --- a/charts/csi-baremetal-deployment/values.yaml +++ b/charts/csi-baremetal-deployment/values.yaml @@ -247,6 +247,4 @@ nodeController: path: /metrics port: 8787 -# TODO Move parameter to Operator chart - https://github.com/dell/csi-baremetal/issues/422 -# platform field must be set to the one of supported from pkg/constant/platforms.go platform: "vanilla" diff --git a/charts/csi-baremetal-operator/templates/rbac.yaml b/charts/csi-baremetal-operator/templates/rbac.yaml index 7e2ac637..468f3d1e 100644 --- a/charts/csi-baremetal-operator/templates/rbac.yaml +++ b/charts/csi-baremetal-operator/templates/rbac.yaml @@ -47,8 +47,6 @@ rules: verbs: - list - watch -# Required for Openshift only -# TODO Deploy only when OpenShift - https://github.com/dell/csi-baremetal/issues/422 - apiGroups: - config.openshift.io resources: