Skip to content

Commit 3345c9a

Browse files
committed
fix: remove managed fields
1 parent a291314 commit 3345c9a

File tree

1 file changed

+20
-8
lines changed

1 file changed

+20
-8
lines changed

discovery/describers/kubernetes.go

Lines changed: 20 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,8 @@ func KubernetesNode(ctx context.Context, client model.Client, extra string, stre
2020

2121
for _, node := range nodes.Items {
2222
var resource models.Resource
23-
23+
// We don't need to include the managed fields in the description, also it causes issues in elastic search mapping generation
24+
node.ManagedFields = nil
2425
resource = models.Resource{
2526
ID: fmt.Sprintf("node/%s", node.Name),
2627
Name: node.Name,
@@ -53,7 +54,8 @@ func KubernetesPersistentVolume(ctx context.Context, client model.Client, extra
5354

5455
for _, pv := range pvs.Items {
5556
var resource models.Resource
56-
57+
// We don't need to include the managed fields in the description, also it causes issues in elastic search mapping generation
58+
pv.ManagedFields = nil
5759
resource = models.Resource{
5860
ID: fmt.Sprintf("persistentvolume/%s", pv.Name),
5961
Name: pv.Name,
@@ -85,7 +87,8 @@ func KubernetesPersistentVolumeClaim(ctx context.Context, client model.Client, e
8587

8688
for _, pvc := range pvcs.Items {
8789
var resource models.Resource
88-
90+
// We don't need to include the managed fields in the description, also it causes issues in elastic search mapping generation
91+
pvc.ManagedFields = nil
8992
resource = models.Resource{
9093
ID: fmt.Sprintf("persistentvolumeclaim/%s/%s", pvc.Namespace, pvc.Name),
9194
Name: fmt.Sprintf("%s/%s", pvc.Namespace, pvc.Name),
@@ -117,7 +120,8 @@ func KubernetesPod(ctx context.Context, client model.Client, extra string, strea
117120

118121
for _, pod := range pods.Items {
119122
var resource models.Resource
120-
123+
// We don't need to include the managed fields in the description, also it causes issues in elastic search mapping generation
124+
pod.ManagedFields = nil
121125
resource = models.Resource{
122126
ID: fmt.Sprintf("pod/%s/%s", pod.Namespace, pod.Name),
123127
Name: fmt.Sprintf("%s/%s", pod.Namespace, pod.Name),
@@ -149,7 +153,8 @@ func KubernetesService(ctx context.Context, client model.Client, extra string, s
149153

150154
for _, service := range services.Items {
151155
var resource models.Resource
152-
156+
// We don't need to include the managed fields in the description, also it causes issues in elastic search mapping generation
157+
service.ManagedFields = nil
153158
resource = models.Resource{
154159
ID: fmt.Sprintf("service/%s/%s", service.Namespace, service.Name),
155160
Name: fmt.Sprintf("%s/%s", service.Namespace, service.Name),
@@ -184,6 +189,8 @@ func KubernetesSecret(ctx context.Context, client model.Client, extra string, st
184189
// Do not include the data in the secret
185190
secret.Data = nil
186191
secret.StringData = nil
192+
// We don't need to include the managed fields in the description, also it causes issues in elastic search mapping generation
193+
secret.ManagedFields = nil
187194
resource = models.Resource{
188195
ID: fmt.Sprintf("secret/%s/%s", secret.Namespace, secret.Name),
189196
Name: fmt.Sprintf("%s/%s", secret.Namespace, secret.Name),
@@ -219,6 +226,8 @@ func KubernetesConfigMap(ctx context.Context, client model.Client, extra string,
219226
// Do not include the data in the configmap
220227
configMap.Data = nil
221228
configMap.BinaryData = nil
229+
// We don't need to include the managed fields in the description, also it causes issues in elastic search mapping generation
230+
configMap.ManagedFields = nil
222231
resource = models.Resource{
223232
ID: fmt.Sprintf("configmap/%s/%s", configMap.Namespace, configMap.Name),
224233
Name: fmt.Sprintf("%s/%s", configMap.Namespace, configMap.Name),
@@ -250,7 +259,8 @@ func KubernetesServiceAccount(ctx context.Context, client model.Client, extra st
250259

251260
for _, serviceAccount := range serviceAccounts.Items {
252261
var resource models.Resource
253-
262+
// We don't need to include the managed fields in the description, also it causes issues in elastic search mapping generation
263+
serviceAccount.ManagedFields = nil
254264
resource = models.Resource{
255265
ID: fmt.Sprintf("serviceaccount/%s/%s", serviceAccount.Namespace, serviceAccount.Name),
256266
Name: fmt.Sprintf("%s/%s", serviceAccount.Namespace, serviceAccount.Name),
@@ -282,7 +292,8 @@ func KubernetesDeployment(ctx context.Context, client model.Client, extra string
282292

283293
for _, deployment := range deployments.Items {
284294
var resource models.Resource
285-
295+
// We don't need to include the managed fields in the description, also it causes issues in elastic search mapping generation
296+
deployment.ManagedFields = nil
286297
resource = models.Resource{
287298
ID: fmt.Sprintf("deployment/%s/%s", deployment.Namespace, deployment.Name),
288299
Name: fmt.Sprintf("%s/%s", deployment.Namespace, deployment.Name),
@@ -314,7 +325,8 @@ func KubernetesStatefulSet(ctx context.Context, client model.Client, extra strin
314325

315326
for _, statefulSet := range statefulSets.Items {
316327
var resource models.Resource
317-
328+
// We don't need to include the managed fields in the description, also it causes issues in elastic search mapping generation
329+
statefulSet.ManagedFields = nil
318330
resource = models.Resource{
319331
ID: fmt.Sprintf("statefulset/%s/%s", statefulSet.Namespace, statefulSet.Name),
320332
Name: fmt.Sprintf("%s/%s", statefulSet.Namespace, statefulSet.Name),

0 commit comments

Comments
 (0)