fix(manager/kubernetes): don't extract unknown kubernetes objects (#18230)

lib/modules/datasource/kubernetes-api/index.ts

@@ -4,15 +4,17 @@ import * as kubernetesApiVersioning from '../../versioning/kubernetes-api';
 import { Datasource } from '../datasource';
 import type { GetReleasesConfig, ReleaseResult } from '../types';
 
+const apiData: Record<string, string[]> = JSON5.parse(
+  dataFiles.get('data/kubernetes-api.json5')!
+);
+
+export const supportedApis = new Set(Object.keys(apiData));
+
 export class KubernetesApiDatasource extends Datasource {
   static readonly id = 'kubernetes-api';
-  private readonly kubernetesApiVersions: Record<string, string[]>;
 
   constructor() {
     super(KubernetesApiDatasource.id);
-    this.kubernetesApiVersions = JSON5.parse(
-      dataFiles.get('data/kubernetes-api.json5')!
-    );
   }
 
   override defaultVersioning = kubernetesApiVersioning.id;
@@ -20,7 +22,7 @@ export class KubernetesApiDatasource extends Datasource {
   getReleases({
     packageName,
   }: GetReleasesConfig): Promise<ReleaseResult | null> {
-    const versions = this.kubernetesApiVersions[packageName];
+    const versions = apiData[packageName];
     if (versions) {
       const releases = versions.map((version) => ({ version }));
       return Promise.resolve({ releases });

lib/modules/manager/kubernetes/extract.spec.ts

@@ -13,16 +13,9 @@ describe('modules/manager/kubernetes/extract', () => {
       expect(extractPackageFile('', 'file.yaml', {})).toBeNull();
     });
 
-    it('returns only API version', () => {
+    it('does not return unknown kind', () => {
       const res = extractPackageFile(kubernetesConfigMapFile, 'file.yaml', {});
-      expect(res?.deps).toStrictEqual([
-        {
-          currentValue: 'v1',
-          datasource: 'kubernetes-api',
-          depName: 'ConfigMap',
-          versioning: 'kubernetes-api',
-        },
-      ]);
+      expect(res).toBeNull();
     });
 
     it('extracts multiple Kubernetes configurations', () => {

lib/modules/manager/kubernetes/extract.ts

@@ -2,7 +2,10 @@ import is from '@sindresorhus/is';
 import { loadAll } from 'js-yaml';
 import { logger } from '../../../logger';
 import { newlineRegex, regEx } from '../../../util/regex';
-import { KubernetesApiDatasource } from '../../datasource/kubernetes-api';
+import {
+  KubernetesApiDatasource,
+  supportedApis,
+} from '../../datasource/kubernetes-api';
 import * as kubernetesApiVersioning from '../../versioning/kubernetes-api';
 import { getDep } from '../dockerfile/extract';
 import type { ExtractConfig, PackageDependency, PackageFile } from '../types';
@@ -57,7 +60,7 @@ function extractImages(
 }
 
 function extractApis(content: string, fileName: string): PackageDependency[] {
-  let doc: KubernetesConfiguration[] | undefined;
+  let doc: KubernetesConfiguration[];
 
   try {
     doc = loadAll(content) as KubernetesConfiguration[];
@@ -73,6 +76,7 @@ function extractApis(content: string, fileName: string): PackageDependency[] {
         is.nonEmptyStringAndNotWhitespace(m.kind) &&
         is.nonEmptyStringAndNotWhitespace(m.apiVersion)
     )
+    .filter((m) => supportedApis.has(m.kind))
     .map((configuration) => ({
       depName: configuration.kind,
       currentValue: configuration.apiVersion,