Closed doronkg closed 5 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 40.96%. Comparing base (
852a6f9
) to head (bd207f9
).
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Basic OpenShift installation comes with 60+ namespaces beginning with
openshift-
prefix, which doesn't include additional namespaces created by OpenShift operators or customized installations, that would also be created with that prefix.
There are various unused resources in 30+ default openshift-
namespaces.
Attaching the output of kor all
, not including the exclusions listed in this PR.
Awaiting #249 to be merged and then I'll apply that logic in this PR.
What this PR does / why we need it
This PR excludes the default resources created in basic OpenShift installations.
PR Checklist
GitHub Issue
Closes #240
Notes for your reviewers
In
pkg/kor/secrets.go
, a new exception secret type was added -kubernetes.io/dockercfg
, which is the OpenShift equivalent ofkubernetes.io/dockerconfigjson
.This PR addresses all default namespaces that doesn't begin with
openshift-
prefix. Meaning, it covers the following namespaces:openshift
,default
,kube-system
,kube-public
,kube-node-lease
&assisted-installer
.Basic OpenShift installation comes with 60+ namespaces beginning with
openshift-
prefix, which doesn't include additional namespaces created by OpenShift operators or customized installations, that would also be created with that prefix.As I see it, there are 3 options to address this case:
openshift-
prefix (flexible).@yonahd please share your thoughts, I'm leaning towards option no. (1).