Added versioning to inventory_view_saved_object

src/core/server/integration_tests/ci_checks/saved_objects/check_registered_types.test.ts

@@ -129,7 +129,7 @@ describe('checking migration metadata changes on all registered SO types', () =>
         "ingest-outputs": "55988d5f778bbe0e76caa7e6468707a0a056bdd8",
         "ingest-package-policies": "60d43f475f91417d14d9df05476acf2e63e99435",
         "ingest_manager_settings": "111a616eb72627c002029c19feb9e6c439a10505",
-        "inventory-view": "b8683c8e352a286b4aca1ab21003115a4800af83",
+        "inventory-view": "fd2b7fe713956f261018dded00d8f8c986417763",
         "kql-telemetry": "93c1d16c1a0dfca9c8842062cf5ef8f62ae401ad",
         "legacy-url-alias": "9b8cca3fbb2da46fd12823d3cd38fdf1c9f24bc8",
         "lens": "5cfa2c52b979b4f8df56dd13c477e152183468b9",

x-pack/solutions/observability/plugins/infra/server/saved_objects/inventory_view/inventory_view_saved_object.test.ts

@@ -0,0 +1,61 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License
+ * 2.0; you may not use this file except in compliance with the Elastic License
+ * 2.0.
+ */
+import {
+  createModelVersionTestMigrator,
+  type ModelVersionTestMigrator,
+} from '@kbn/core-test-helpers-model-versions';
+
+import { inventoryViewSavedObjectType } from './inventory_view_saved_object';
+
+describe('invetoryViewSavedObject model version transformation', () => {
+  let migrator: ModelVersionTestMigrator;
+  beforeEach(() => {
+    migrator = createModelVersionTestMigrator({ type: inventoryViewSavedObjectType });
+  });
+
+  describe('model veresion 2', () => {
+    const inventoryViewV2 = {
+      id: 'someId',
+      type: 'inventory-view',
+      attributes: {
+        metric: { type: 'cpu' },
+        sort: { by: 'name', direction: 'desc' },
+        groupBy: [],
+        nodeType: 'host',
+        view: 'map',
+        customOptions: [],
+        customMetrics: [],
+        boundsOverride: { min: 0, max: 1 },
+        autoBounds: true,
+        accountId: '',
+        region: '',
+        autoReload: false,
+        filterQuery: { expression: '', kind: 'kuery' },
+        legend: { palette: 'cool', reverseColors: false, steps: 18 },
+        timelineOpen: false,
+        name: 'test',
+      },
+      references: [],
+    };
+
+    it('should clamp legend.steps to 18 when converting from v1 to v2', () => {
+      const inventoryViewV1 = JSON.parse(JSON.stringify(inventoryViewV2));
+      inventoryViewV1.attributes.legend.steps = 20;
+      const migrated = migrator.migrate({
+        document: {
+          ...inventoryViewV1,
+          attributes: {
+            ...inventoryViewV1.attributes,
+          },
+        },
+        fromVersion: 1,
+        toVersion: 2,
+      });
+      expect(migrated.attributes).toEqual(inventoryViewV2.attributes);
+    });
+  });
+});

x-pack/solutions/observability/plugins/infra/server/saved_objects/inventory_view/inventory_view_saved_object.ts

@@ -5,6 +5,7 @@
  * 2.0.
  */
 
+import { schema } from '@kbn/config-schema';
 import { fold } from 'fp-ts/lib/Either';
 import { pipe } from 'fp-ts/lib/pipeable';
 import type { SavedObject, SavedObjectsType } from '@kbn/core/server';
@@ -21,6 +22,11 @@ const getInventoryViewTitle = (savedObject: SavedObject<unknown>) =>
     )
   );
 
+const schemaV1 = schema.object({}, { unknowns: 'allow' });
+const schemaV2 = schema.object({
+  legend: schema.object({ steps: schema.number({ max: 18, min: 2 }) }),
+});
+
 export const inventoryViewSavedObjectType: SavedObjectsType = {
   name: inventoryViewSavedObjectName,
   hidden: false,
@@ -36,4 +42,31 @@ export const inventoryViewSavedObjectType: SavedObjectsType = {
     dynamic: false,
     properties: {},
   },
+  modelVersions: {
+    1: {
+      changes: [],
+      schemas: {
+        create: schemaV1,
+      },
+    },
+    2: {
+      changes: [
+        {
+          type: 'unsafe_transform',
+          transformFn: (document) => {
+            if (document.attributes.legend.steps > 18) {
+              document.attributes.legend.steps = 18;
+            } else if (document.attributes.legend.steps < 2) {
+              document.attributes.legend.steps = 2;
+            }
+            return { document };
+          },
+        },
+      ],
+      schemas: {
+        forwardCompatibility: schemaV2.extends({}, { unknowns: 'ignore' }),
+        create: schemaV2,
+      },
+    },
+  },
 };

x-pack/solutions/observability/plugins/infra/tsconfig.json

@@ -115,7 +115,8 @@
     "@kbn/config",
     "@kbn/observability-utils-common",
     "@kbn/charts-theme",
-    "@kbn/response-ops-rule-params"
+    "@kbn/response-ops-rule-params",
+    "@kbn/core-test-helpers-model-versions"
   ],
   "exclude": ["target/**/*"]
 }