Feature/ri 6265 update list of databases

redisinsight/api/config/features-config.json

@@ -1,5 +1,5 @@
 {
-  "version": 2.56,
+  "version": 2.57,
   "features": {
     "redisDataIntegration": {
       "flag": true,
@@ -111,6 +111,10 @@
       "data": {
         "strategy": "ioredis"
       }
+    },
+    "enhancedCloudUI": {
+      "flag": true,
+      "perc": [[0, 50]]
     }
   }
 }

redisinsight/api/migration/1733740794737-database-createdAt.ts

@@ -0,0 +1,20 @@
+import { MigrationInterface, QueryRunner } from "typeorm";
+
+export class DatabaseCreatedAt1733740794737 implements MigrationInterface {
+    name = 'DatabaseCreatedAt1733740794737'
+
+    public async up(queryRunner: QueryRunner): Promise<void> {
+        await queryRunner.query(`CREATE TABLE "temporary_database_instance" ("id" varchar PRIMARY KEY NOT NULL, "host" varchar NOT NULL, "port" integer NOT NULL, "name" varchar NOT NULL, "username" varchar, "password" varchar, "tls" boolean, "verifyServerCert" boolean, "lastConnection" datetime, "caCertId" varchar, "clientCertId" varchar, "connectionType" varchar NOT NULL DEFAULT ('STANDALONE'), "nodes" varchar DEFAULT ('[]'), "nameFromProvider" varchar, "sentinelMasterName" varchar, "sentinelMasterUsername" varchar, "sentinelMasterPassword" varchar, "provider" varchar DEFAULT ('UNKNOWN'), "modules" varchar NOT NULL DEFAULT ('[]'), "db" integer, "encryption" varchar, "tlsServername" varchar, "new" boolean, "ssh" boolean, "timeout" integer, "compressor" varchar NOT NULL DEFAULT ('NONE'), "version" varchar, "createdAt" datetime DEFAULT (datetime('now')), CONSTRAINT "FK_d1bc747b5938e22b4b708d8e9a5" FOREIGN KEY ("caCertId") REFERENCES "ca_certificate" ("id") ON DELETE SET NULL ON UPDATE NO ACTION, CONSTRAINT "FK_3b9b625266c00feb2d66a9f36e4" FOREIGN KEY ("clientCertId") REFERENCES "client_certificate" ("id") ON DELETE SET NULL ON UPDATE NO ACTION)`);
+        await queryRunner.query(`INSERT INTO "temporary_database_instance"("id", "host", "port", "name", "username", "password", "tls", "verifyServerCert", "lastConnection", "caCertId", "clientCertId", "connectionType", "nodes", "nameFromProvider", "sentinelMasterName", "sentinelMasterUsername", "sentinelMasterPassword", "provider", "modules", "db", "encryption", "tlsServername", "new", "ssh", "timeout", "compressor", "version") SELECT "id", "host", "port", "name", "username", "password", "tls", "verifyServerCert", "lastConnection", "caCertId", "clientCertId", "connectionType", "nodes", "nameFromProvider", "sentinelMasterName", "sentinelMasterUsername", "sentinelMasterPassword", "provider", "modules", "db", "encryption", "tlsServername", "new", "ssh", "timeout", "compressor", "version" FROM "database_instance"`);
+        await queryRunner.query(`DROP TABLE "database_instance"`);
+        await queryRunner.query(`ALTER TABLE "temporary_database_instance" RENAME TO "database_instance"`);
+    }
+
+    public async down(queryRunner: QueryRunner): Promise<void> {
+        await queryRunner.query(`ALTER TABLE "database_instance" RENAME TO "temporary_database_instance"`);
+        await queryRunner.query(`CREATE TABLE "database_instance" ("id" varchar PRIMARY KEY NOT NULL, "host" varchar NOT NULL, "port" integer NOT NULL, "name" varchar NOT NULL, "username" varchar, "password" varchar, "tls" boolean, "verifyServerCert" boolean, "lastConnection" datetime, "caCertId" varchar, "clientCertId" varchar, "connectionType" varchar NOT NULL DEFAULT ('STANDALONE'), "nodes" varchar DEFAULT ('[]'), "nameFromProvider" varchar, "sentinelMasterName" varchar, "sentinelMasterUsername" varchar, "sentinelMasterPassword" varchar, "provider" varchar DEFAULT ('UNKNOWN'), "modules" varchar NOT NULL DEFAULT ('[]'), "db" integer, "encryption" varchar, "tlsServername" varchar, "new" boolean, "ssh" boolean, "timeout" integer, "compressor" varchar NOT NULL DEFAULT ('NONE'), "version" varchar, CONSTRAINT "FK_d1bc747b5938e22b4b708d8e9a5" FOREIGN KEY ("caCertId") REFERENCES "ca_certificate" ("id") ON DELETE SET NULL ON UPDATE NO ACTION, CONSTRAINT "FK_3b9b625266c00feb2d66a9f36e4" FOREIGN KEY ("clientCertId") REFERENCES "client_certificate" ("id") ON DELETE SET NULL ON UPDATE NO ACTION)`);
+        await queryRunner.query(`INSERT INTO "database_instance"("id", "host", "port", "name", "username", "password", "tls", "verifyServerCert", "lastConnection", "caCertId", "clientCertId", "connectionType", "nodes", "nameFromProvider", "sentinelMasterName", "sentinelMasterUsername", "sentinelMasterPassword", "provider", "modules", "db", "encryption", "tlsServername", "new", "ssh", "timeout", "compressor", "version") SELECT "id", "host", "port", "name", "username", "password", "tls", "verifyServerCert", "lastConnection", "caCertId", "clientCertId", "connectionType", "nodes", "nameFromProvider", "sentinelMasterName", "sentinelMasterUsername", "sentinelMasterPassword", "provider", "modules", "db", "encryption", "tlsServername", "new", "ssh", "timeout", "compressor", "version" FROM "temporary_database_instance"`);
+        await queryRunner.query(`DROP TABLE "temporary_database_instance"`);
+    }
+
+}

redisinsight/api/migration/index.ts

@@ -44,6 +44,7 @@ import { Rdi1716370509836 } from './1716370509836-rdi';
 import { AiHistory1718260230164 } from './1718260230164-ai-history';
 import { CloudSession1729085495444 } from './1729085495444-cloud-session';
 import { CommandExecution1726058563737 } from './1726058563737-command-execution';
+import { DatabaseCreatedAt1733740794737 } from './1733740794737-database-createdAt';
 
 export default [
   initialMigration1614164490968,
@@ -92,4 +93,5 @@ export default [
   AiHistory1718260230164,
   CloudSession1729085495444,
   CommandExecution1726058563737,
+  DatabaseCreatedAt1733740794737,
 ];

redisinsight/api/src/modules/database/database.service.spec.ts

@@ -530,6 +530,8 @@ describe('DatabaseService', () => {
         mockSessionMetadata,
         omit({ ...mockDatabase, username: 'new-name', timeout: 40_000 }, ['sshOptions.id']),
       );
+      expect(databaseRepository.get)
+        .toHaveBeenCalledWith(mockSessionMetadata, mockDatabase.id, false, ['id', 'sshOptions.id', 'createdAt']);
     });
 
     it('should create new database with merged ssh options', async () => {

redisinsight/api/src/modules/database/database.service.ts

@@ -294,7 +294,7 @@ export class DatabaseService {
   public async clone(sessionMetadata: SessionMetadata, id: string, dto: UpdateDatabaseDto): Promise<Database> {
     this.logger.log('Clone existing database');
     const database = await this.merge(
-      await this.get(sessionMetadata, id, false, ['id', 'sshOptions.id']),
+      await this.get(sessionMetadata, id, false, ['id', 'sshOptions.id', 'createdAt']),
       dto,
     );
     if (DatabaseService.isConnectionAffected(dto)) {

redisinsight/api/src/modules/database/entities/database.entity.ts

@@ -1,5 +1,5 @@
 import {
-  Column, Entity, ManyToOne, OneToOne, PrimaryGeneratedColumn,
+  Column, CreateDateColumn, Entity, ManyToOne, OneToOne, PrimaryGeneratedColumn,
 } from 'typeorm';
 import { CaCertificateEntity } from 'src/modules/certificate/entities/ca-certificate.entity';
 import { ClientCertificateEntity } from 'src/modules/certificate/entities/client-certificate.entity';
@@ -172,6 +172,12 @@ export class DatabaseEntity {
   @Column({ type: 'datetime', nullable: true })
   lastConnection: Date;
 
+  @CreateDateColumn({
+    nullable: true,
+  })
+  @Expose()
+  createdAt: Date;
+
   @Expose()
   @Column({
     nullable: true,
@@ -231,4 +237,4 @@ export class DatabaseEntity {
   @Expose()
   @Column({ nullable: true })
   version: string;
-}
+}

redisinsight/api/src/modules/database/models/database.ts

@@ -146,6 +146,13 @@ export class Database {
   @Expose()
   lastConnection: Date;
 
+  @ApiProperty({
+    description: 'Date of creation',
+    type: Date,
+  })
+  @Expose()
+  createdAt?: Date;
+
   @ApiPropertyOptional({
     description: 'Redis OSS Sentinel master group.',
     type: SentinelMaster,

redisinsight/api/src/modules/database/repositories/local.database.repository.ts

@@ -96,6 +96,7 @@ export class LocalDatabaseRepository extends DatabaseRepository {
       .select([
         'd.id', 'd.name', 'd.host', 'd.port', 'd.db', 'd.new', 'd.timeout',
         'd.connectionType', 'd.modules', 'd.lastConnection', 'd.provider', 'd.version', 'cd',
+        'd.createdAt',
       ])
       .getMany();
 
@@ -262,4 +263,4 @@ export class LocalDatabaseRepository extends DatabaseRepository {
       }
     }
   }
-}
+}

redisinsight/api/src/modules/feature/constants/index.ts

@@ -29,6 +29,7 @@ export enum KnownFeatures {
   DatabaseChat = 'databaseChat',
   Rdi = 'redisDataIntegration',
   HashFieldExpiration = 'hashFieldExpiration',
+  EnhancedCloudUI = 'enhancedCloudUI',
 }
 
 export interface IFeatureFlag {

redisinsight/api/src/modules/feature/constants/known-features.ts

@@ -39,4 +39,8 @@ export const knownFeatures: Record<KnownFeatures, IFeatureFlag> = {
     name: KnownFeatures.Rdi,
     storage: FeatureStorage.Database,
   },
+  [KnownFeatures.EnhancedCloudUI]: {
+    name: KnownFeatures.EnhancedCloudUI,
+    storage: FeatureStorage.Database,
+  },
 };

redisinsight/api/src/modules/feature/providers/feature-flag/feature-flag.provider.ts

@@ -60,6 +60,10 @@ export class FeatureFlagProvider {
       this.featuresConfigService,
       this.settingsService,
     ));
+    this.strategies.set(KnownFeatures.EnhancedCloudUI, new WithDataFlagStrategy(
+      this.featuresConfigService,
+      this.settingsService,
+    ));
   }
 
   getStrategy(name: string): FeatureFlagStrategy {

redisinsight/api/test/api/database/GET-databases.test.ts

@@ -17,6 +17,7 @@ const responseSchema = Joi.array().items(Joi.object().keys({
   compressor: Joi.string().valid('NONE', 'LZ4', 'GZIP', 'ZSTD', 'SNAPPY').allow(null),
   connectionType: Joi.string().valid('STANDALONE', 'SENTINEL', 'CLUSTER', 'NOT CONNECTED').required(),
   lastConnection: Joi.string().isoDate().allow(null).required(),
+  createdAt: Joi.string().isoDate(),
   version: Joi.string().allow(null).required(),
   modules: Joi.array().items(Joi.object().keys({
     name: Joi.string().required(),

redisinsight/api/test/api/database/PATCH-databases-id.test.ts

@@ -233,7 +233,7 @@ describe(`PATCH /databases/:id`, () => {
           after: async () => {
             newDatabase = await localDb.getInstanceById(constants.TEST_INSTANCE_ID_3);
             expect(newDatabase).to.contain({
-              ..._.omit(oldDatabase, ['modules', 'provider', 'lastConnection', 'new', 'timeout', 'compressor', 'version']),
+              ..._.omit(oldDatabase, ['modules', 'provider', 'lastConnection', 'new', 'timeout', 'compressor', 'version', 'createdAt']),
               host: constants.TEST_REDIS_HOST,
               port: constants.TEST_REDIS_PORT,
             });

redisinsight/api/test/api/database/POST-databases-clone-id.test.ts

@@ -161,7 +161,7 @@ describe(`POST /databases/clone/:id`, () => {
           after: async () => {
             newDatabase = await localDb.getInstanceByName('some name');
             expect(newDatabase).to.contain({
-              ..._.omit(oldDatabase, ['id', 'modules', 'name', 'provider', 'lastConnection', 'new', 'timeout', 'compressor', 'version']),
+              ..._.omit(oldDatabase, ['id', 'modules', 'name', 'provider', 'lastConnection', 'new', 'timeout', 'compressor', 'version', 'createdAt']),
               host: constants.TEST_REDIS_HOST,
               port: constants.TEST_REDIS_PORT,
             });

redisinsight/api/test/api/database/constants.ts

@@ -36,6 +36,7 @@ export const databaseSchema = Joi.object().keys({
   compressor: Joi.string().valid('NONE', 'LZ4', 'GZIP', 'ZSTD', 'SNAPPY').required(),
   nameFromProvider: Joi.string().allow(null),
   lastConnection: Joi.string().isoDate().allow(null),
+  createdAt: Joi.string().isoDate(),
   provider: Joi.string().valid(...providers),
   new: Joi.boolean().allow(null),
   tls: Joi.boolean().allow(null),

redisinsight/ui/src/components/form-dialog/FormDialog.spec.tsx

@@ -0,0 +1,23 @@
+import React from 'react'
+import { render, screen } from 'uiSrc/utils/test-utils'
+
+import FormDialog from './FormDialog'
+
+describe('FormDialog', () => {
+  it('should render', () => {
+    render(
+      <FormDialog
+        isOpen
+        onClose={jest.fn()}
+        header={(<div data-testid="header" />)}
+        footer={(<div data-testid="footer" />)}
+      >
+        <div data-testid="body" />
+      </FormDialog>
+    )
+
+    expect(screen.getByTestId('header')).toBeInTheDocument()
+    expect(screen.getByTestId('footer')).toBeInTheDocument()
+    expect(screen.getByTestId('body')).toBeInTheDocument()
+  })
+})

redisinsight/ui/src/components/form-dialog/FormDialog.tsx

@@ -0,0 +1,40 @@
+import React from 'react'
+import { EuiModal, EuiModalBody, EuiModalFooter, EuiModalHeader, EuiModalHeaderTitle } from '@elastic/eui'
+import { Nullable } from 'uiSrc/utils'
+
+import styles from './styles.module.scss'
+
+export interface Props {
+  isOpen: boolean
+  onClose: () => void
+  header: Nullable<React.ReactNode>
+  footer?: Nullable<React.ReactNode>
+  children: Nullable<React.ReactNode>
+}
+
+const FormDialog = (props: Props) => {
+  const { isOpen, onClose, header, footer, children } = props
+
+  if (!isOpen) return null
+
+  return (
+    <EuiModal
+      className={styles.modal}
+      onClose={onClose}
+    >
+      <EuiModalHeader>
+        <EuiModalHeaderTitle id="formModalHeader">
+          {header}
+        </EuiModalHeaderTitle>
+      </EuiModalHeader>
+      <EuiModalBody>
+        {children}
+      </EuiModalBody>
+      <EuiModalFooter>
+        {footer}
+      </EuiModalFooter>
+    </EuiModal>
+  )
+}
+
+export default FormDialog

redisinsight/ui/src/components/form-dialog/index.ts

@@ -0,0 +1,3 @@
+import FormDialog from './FormDialog'
+
+export default FormDialog