.vscode/settings.json

@@ -1,19 +1,13 @@
 {
   "cSpell.words": [
-    "Mutex",
     "Repos",
-    "amqp",
     "boardcat",
-    "errout",
-    "fanout",
     "gitea",
-    "golevelup",
     "lpush",
     "lrange",
     "metatype",
     "pmessage",
     "psubscribe",
-    "rabbitmq",
     "rpop",
     "rpush"
   ]

config.yml.example

@@ -14,7 +14,5 @@ db:
     port: 6379
     password:
     prefix: fennec
-  rabbitmq:
-    uri: 'amqp://fennec:fennec@192.168.31.194:5672'
 workspaces:
   root: '/Users/ivanli/Projects/fennec/workspaces'

package.json

@@ -1,6 +1,6 @@
 {
   "name": "fennec-be",
-  "version": "0.1.0",
+  "version": "0.0.1",
   "description": "",
   "author": "",
   "private": true,
@@ -21,7 +21,6 @@
     "test:e2e": "jest --config ./test/jest-e2e.json"
   },
   "dependencies": {
-    "@golevelup/nestjs-rabbitmq": "^1.16.1",
     "@nestjs/bull": "^0.3.1",
     "@nestjs/common": "^7.5.1",
     "@nestjs/config": "^0.6.2",
@@ -29,7 +28,6 @@
     "@nestjs/graphql": "^7.9.8",
     "@nestjs/platform-express": "^7.5.1",
     "@nestjs/typeorm": "^7.1.5",
-    "@types/amqplib": "^0.8.0",
     "@types/bull": "^3.15.0",
     "@types/ramda": "^0.27.38",
     "apollo-server-express": "^2.19.2",
@@ -43,11 +41,9 @@
     "graphql-tools": "^7.0.2",
     "ioredis": "^4.25.0",
     "js-yaml": "^4.0.0",
-    "nestjs-pino": "^1.4.0",
     "nestjs-redis": "^1.2.8",
     "observable-to-async-generator": "^1.0.1-rc",
     "pg": "^8.5.1",
-    "pino-pretty": "^4.7.1",
     "ramda": "^0.27.1",
     "reflect-metadata": "^0.1.13",
     "rimraf": "^3.0.2",
@@ -66,7 +62,6 @@
     "@types/jest": "^26.0.15",
     "@types/js-yaml": "^4.0.0",
     "@types/node": "^14.14.6",
-    "@types/pino-pretty": "^4.7.0",
     "@types/supertest": "^2.0.10",
     "@typescript-eslint/eslint-plugin": "^4.6.1",
     "@typescript-eslint/parser": "^4.6.1",

src/app.module.ts

@@ -16,32 +16,13 @@ import { RawBodyMiddleware } from './commons/middlewares/raw-body.middleware';
 import { GiteaWebhooksController } from './webhooks/gitea-webhooks.controller';
 import { ParseBodyMiddleware } from './commons/middlewares/parse-body.middleware';
 import { BullModule } from '@nestjs/bull';
-import { LoggerModule } from 'nestjs-pino';
+import { PubSubModule } from './commons/pub-sub/pub-sub.module';
-
-import pinoPretty from 'pino-pretty';
 
 @Module({
   imports: [
     ConfigModule.forRoot({
       load: [configuration],
     }),
-    LoggerModule.forRootAsync({
-      imports: [ConfigModule],
-      useFactory: (configService: ConfigService) => {
-        const isDev = configService.get<'dev' | 'prod'>('env') === 'dev';
-        return {
-          pinoHttp: {
-            prettyPrint: isDev
-              ? {
-                  levelFirst: true,
-                }
-              : false,
-            prettifier: pinoPretty,
-          },
-        };
-      },
-      inject: [ConfigService],
-    }),
     TypeOrmModule.forRootAsync({
       imports: [ConfigModule],
       useFactory: (configService: ConfigService) => ({
@@ -77,6 +58,17 @@ import pinoPretty from 'pino-pretty';
       }),
       inject: [ConfigService],
     }),
+    PubSubModule.forRootAsync({
+      imports: [ConfigModule],
+      useFactory: (configService: ConfigService) => ({
+        redis: {
+          host: configService.get<string>('db.redis.host', 'localhost'),
+          port: configService.get<number>('db.redis.port', undefined),
+          password: configService.get<string>('db.redis.password', undefined),
+        },
+      }),
+      inject: [ConfigService],
+    }),
     ProjectsModule,
     ReposModule,
     PipelinesModule,

src/commons/commons.module.ts

@@ -1,10 +1,10 @@
 import { Module } from '@nestjs/common';
 import { PasswordConverter } from './services/password-converter';
-import { RedisMutexModule } from './redis-mutex/redis-mutex.module';
+import { PubSubModule } from './pub-sub/pub-sub.module';
 
 @Module({
   providers: [PasswordConverter],
-  exports: [PasswordConverter, RedisMutexModule],
+  exports: [PasswordConverter],
-  imports: [RedisMutexModule],
+  imports: [PubSubModule],
 })
 export class CommonsModule {}

src/commons/exceptions/application.exception.ts

@@ -1,11 +1,11 @@
-import { pick } from 'ramda';
-
 export class ApplicationException extends Error {
   code: number;
   error: Error;
 
   constructor(
-    message: string | { error?: Error; message?: string | any; code?: number },
+    message:
+      | string
+      | { error?: Error; message?: string | object; code?: number },
   ) {
     if (message instanceof Object) {
       super();
@@ -18,8 +18,4 @@ export class ApplicationException extends Error {
       super((message as unknown) as any);
     }
   }
-
-  toJSON() {
-    return pick(['code', 'message'], this);
-  }
 }

src/commons/pub-sub/decorators/inject-pub-sub.decorator.ts

@@ -0,0 +1,5 @@
+import { Inject } from '@nestjs/common';
+import { getPubSubToken } from '../utils/token';
+
+export const InjectPubSub = (name?: string): ParameterDecorator =>
+  Inject(getPubSubToken(name));

src/commons/pub-sub/interfaces/pub-sub-async-config.interface.ts

@@ -0,0 +1,8 @@
+import { ModuleMetadata } from '@nestjs/common';
+import { PubSubOptions } from './pub-sub-options.interface';
+
+export interface PubSubAsyncConfig extends Pick<ModuleMetadata, 'imports'> {
+  name?: string;
+  useFactory: (...args: any[]) => Promise<PubSubOptions> | PubSubOptions;
+  inject?: any[];
+}

src/commons/pub-sub/interfaces/pub-sub-options.interface.ts

@@ -0,0 +1,6 @@
+import { RedisOptions } from 'ioredis';
+
+export interface PubSubOptions {
+  name?: string;
+  redis: RedisOptions;
+}

src/commons/pub-sub/interfaces/pub-sub-raw-message.interface.ts

@@ -0,0 +1,15 @@
+export type PubSubRawMessage<T> =
+  | PubSubRawNextMessage<T>
+  | PubSubRawErrorMessage<T>
+  | PubSubRawCompleteMessage<T>;
+export interface PubSubRawNextMessage<T> {
+  type: 'next';
+  message: T;
+}
+export interface PubSubRawErrorMessage<T> {
+  type: 'error';
+  error: string;
+}
+export interface PubSubRawCompleteMessage<T> {
+  type: 'complete';
+}

src/commons/pub-sub/pub-sub.constants.ts

@@ -0,0 +1 @@
+export const DEFAULT_PUB_SUB_NAME = 'default';

src/commons/pub-sub/pub-sub.module.ts

@@ -0,0 +1,48 @@
+import { DynamicModule, Module } from '@nestjs/common';
+import { PubSubService } from './pub-sub.service';
+import {
+  createAsyncPubSubProviders,
+  createPubSubProvider,
+} from './pub-sub.providers';
+import { PubSubOptions } from './interfaces/pub-sub-options.interface';
+import { PubSubAsyncConfig } from './interfaces/pub-sub-async-config.interface';
+import { getPubSubConfigToken } from './utils/token';
+
+@Module({
+  providers: [PubSubService],
+})
+export class PubSubModule {
+  public static forRoot(options: PubSubOptions): DynamicModule {
+    const providers = [
+      {
+        provide: getPubSubConfigToken(options.name),
+        useValue: options,
+      },
+      createPubSubProvider(options.name),
+    ];
+    return {
+      global: true,
+      module: PubSubModule,
+      providers,
+      exports: providers,
+    };
+  }
+  public static forRootAsync(...configs: PubSubAsyncConfig[]): DynamicModule {
+    const providers = createAsyncPubSubProviders(configs);
+    return {
+      global: true,
+      module: PubSubModule,
+      imports: configs
+        .map((config) => config.imports)
+        .flat()
+        .filter((o, i, a) => a.indexOf(o) === i),
+      providers,
+      exports: providers,
+    };
+  }
+  public static forFeature(): DynamicModule {
+    return {
+      module: PubSubModule,
+    };
+  }
+}

src/commons/pub-sub/pub-sub.providers.ts

@@ -0,0 +1,31 @@
+import { Provider } from '@nestjs/common';
+import { PubSubAsyncConfig } from './interfaces/pub-sub-async-config.interface';
+import { PubSubOptions } from './interfaces/pub-sub-options.interface';
+import { PubSub } from './pub-sub';
+import { getPubSubConfigToken, getPubSubToken } from './utils/token';
+
+export function createPubSubProvider(name: string): Provider {
+  return {
+    provide: getPubSubToken(name),
+    useFactory: (option: PubSubOptions) => new PubSub(option),
+    inject: [getPubSubConfigToken(name)],
+  };
+}
+
+export function createOptionsProvider(config: PubSubAsyncConfig): Provider {
+  return {
+    provide: getPubSubConfigToken(config.name),
+    useFactory: config.useFactory,
+    inject: config.inject || [],
+  };
+}
+export function createAsyncPubSubProviders(
+  configs: PubSubAsyncConfig[],
+): Provider[] {
+  return configs
+    .map((config) => [
+      createOptionsProvider(config),
+      createPubSubProvider(config.name),
+    ])
+    .flat();
+}

src/commons/pub-sub/pub-sub.service.spec.ts

@@ -1,15 +1,15 @@
 import { Test, TestingModule } from '@nestjs/testing';
-import { RedisMutexService } from './redis-mutex.service';
+import { PubSubService } from './pub-sub.service';
 
-describe('RedisMutexService', () => {
+describe('PubsubService', () => {
-  let service: RedisMutexService;
+  let service: PubSubService;
 
   beforeEach(async () => {
     const module: TestingModule = await Test.createTestingModule({
-      providers: [RedisMutexService],
+      providers: [PubSubService],
     }).compile();
 
-    service = module.get<RedisMutexService>(RedisMutexService);
+    service = module.get<PubSubService>(PubSubService);
   });
 
   it('should be defined', () => {

src/commons/pub-sub/pub-sub.service.ts

@@ -0,0 +1,9 @@
+import { Injectable } from '@nestjs/common';
+import { PubSubOptions } from './interfaces/pub-sub-options.interface';
+
+@Injectable()
+export class PubSubService {
+  private options = new Map<string, PubSubOptions>();
+  private pubClient;
+  private;
+}

src/commons/pub-sub/pub-sub.spec.ts

@@ -0,0 +1,87 @@
+import debug from 'debug';
+import { tap } from 'rxjs/operators';
+import { PubSub } from './pub-sub';
+
+debug.enable('app:pubsub:*');
+
+describe('PubSub', () => {
+  let instance: PubSub;
+
+  beforeEach(async () => {
+    instance = new PubSub({
+      name: 'default',
+      redis: {
+        host: 'localhost',
+      },
+    });
+  });
+
+  it('should be defined', () => {
+    expect(instance).toBeDefined();
+  });
+
+  it('should can send and receive message', async () => {
+    const arr = new Array(10)
+      .fill(undefined)
+      .map(() => Math.random().toString(36).slice(2, 8));
+    const results: string[] = [];
+    instance
+      .message$<string>('test')
+      .pipe(
+        tap((val) => {
+          console.log(val);
+        }),
+      )
+      .subscribe((val) => results.push(val));
+    await new Promise((r) => setTimeout(r, 1000));
+    await Promise.all([...arr.map((str) => instance.publish('test', str))]);
+    await new Promise((r) => setTimeout(r, 1000));
+    expect(results).toMatchObject(arr);
+  });
+
+  it('should complete', async () => {
+    const arr = new Array(10)
+      .fill(undefined)
+      .map(() => Math.random().toString(36).slice(2, 8));
+    const results: string[] = [];
+    instance
+      .message$<string>('test')
+      .pipe(
+        tap((val) => {
+          console.log(val);
+        }),
+      )
+      .subscribe((val) => results.push(val));
+    await new Promise((r) => setTimeout(r, 1000));
+    await Promise.all([...arr.map((str) => instance.publish('test', str))]);
+    await instance.finish('test');
+    await Promise.all([...arr.map((str) => instance.publish('test', str))]);
+    await new Promise((r) => setTimeout(r, 1000));
+    expect(results).toMatchObject(arr);
+  });
+  it('should error', async () => {
+    const arr = new Array(10)
+      .fill(undefined)
+      .map(() => Math.random().toString(36).slice(2, 8));
+    const results: string[] = [];
+    let error: string;
+    instance
+      .message$<string>('test')
+      .pipe(
+        tap((val) => {
+          console.log(val);
+        }),
+      )
+      .subscribe({
+        next: (val) => results.push(val),
+        error: (err) => (error = err.message),
+      });
+    await new Promise((r) => setTimeout(r, 1000));
+    await Promise.all([...arr.map((str) => instance.publish('test', str))]);
+    await instance.throwError('test', 'TEST ERROR MESSAGE');
+    await Promise.all([...arr.map((str) => instance.publish('test', str))]);
+    await new Promise((r) => setTimeout(r, 1000));
+    expect(results).toMatchObject(arr);
+    expect(error).toEqual('TEST ERROR MESSAGE');
+  });
+});

src/commons/pub-sub/pub-sub.ts

@@ -0,0 +1,115 @@
+import debug from 'debug';
+import { EventEmitter } from 'events';
+import IORedis, { Redis } from 'ioredis';
+import { from, fromEvent, Observable } from 'rxjs';
+import { filter, map, share, switchMap, takeWhile, tap } from 'rxjs/operators';
+import { ApplicationException } from '../exceptions/application.exception';
+import { PubSubOptions } from './interfaces/pub-sub-options.interface';
+import {
+  PubSubRawMessage,
+  PubSubRawNextMessage,
+} from './interfaces/pub-sub-raw-message.interface';
+
+const log = debug('fennec:pubsub:instance');
+export class PubSub extends EventEmitter {
+  pubRedis: Redis;
+  pSubRedis: Redis;
+  channels: string[] = [];
+  event$: Observable<[string, string, any]>;
+
+  constructor(private readonly options: PubSubOptions) {
+    super();
+    this.pubRedis = new IORedis(this.options.redis);
+    this.pSubRedis = new IORedis(this.options.redis);
+
+    this.pSubRedis.on('pmessage', (...args) =>
+      log.extend('raw')('%s %s %o', ...args),
+    );
+
+    this.event$ = fromEvent<[string, string, string]>(
+      this.pSubRedis,
+      'pmessage',
+    ).pipe(
+      map((ev) => {
+        try {
+          ev[2] = JSON.parse(ev[2]);
+        } catch (err) {
+          log('WARN: is not json');
+          return null;
+        }
+        log('on message: %s %s %o', ...ev);
+        return ev;
+      }),
+      filter((v) => !!v),
+      share(),
+    );
+  }
+
+  async disconnect() {
+    log('disconnecting to redis...');
+    this.pubRedis.disconnect();
+    this.pSubRedis.disconnect();
+    log('disconnected');
+  }
+
+  async registerChannel(channel: string) {
+    if (this.channels.includes(channel)) {
+      return;
+    }
+    this.channels.push(channel);
+
+    return await this.pSubRedis.psubscribe(channel);
+  }
+
+  private async redisPublish<T>(channel: string, message: PubSubRawMessage<T>) {
+    log.extend('publish')('channel: %s, message: %O', channel, message);
+    return await this.pubRedis.publish(channel, JSON.stringify(message));
+  }
+
+  async publish(channel: string, message: any): Promise<number> {
+    return await this.redisPublish(channel, {
+      type: 'next',
+      message,
+    });
+  }
+
+  async finish(channel: string): Promise<number> {
+    return await this.redisPublish(channel, {
+      type: 'complete',
+    });
+  }
+
+  async throwError(channel: string, error: string): Promise<number> {
+    return await this.redisPublish(channel, {
+      type: 'error',
+      error,
+    });
+  }
+
+  message$ = <T>(channel: string): Observable<T> => {
+    return from(this.registerChannel(channel))
+      .pipe(switchMap(() => this.event$))
+      .pipe(
+        filter(([pattern]) => pattern === channel),
+        tap(([pattern, channel, message]) => {
+          log.extend('subscribe')(
+            'channel: %s, match: %s, message: %O',
+            channel,
+            pattern,
+            message,
+          );
+        }),
+        takeWhile(([pattern, channel, message]) => {
+          if (pattern === channel) {
+            if (message.type === 'error') {
+              throw new ApplicationException(message.error);
+            }
+            return message.type !== 'complete';
+          }
+          return true;
+        }),
+        map((ev) => ev[2] as PubSubRawMessage<T>),
+        map((message: PubSubRawNextMessage<T>) => message.message),
+      );
+  };
+}

src/commons/pub-sub/utils/token.ts

@@ -0,0 +1,8 @@
+import { DEFAULT_PUB_SUB_NAME } from '../pub-sub.constants';
+
+export function getPubSubToken(name = DEFAULT_PUB_SUB_NAME) {
+  return `app:pub-usb:${name || DEFAULT_PUB_SUB_NAME}`;
+}
+export function getPubSubConfigToken(name = DEFAULT_PUB_SUB_NAME) {
+  return `app:pub-usb:config:${name || DEFAULT_PUB_SUB_NAME}`;
+}

src/commons/redis-mutex/redis-mutex.module.ts

@@ -1,10 +0,0 @@
-import { Module } from '@nestjs/common';
-import { RedisMutexService } from './redis-mutex.service';
-import { RedisModule } from 'nestjs-redis';
-
-@Module({
-  imports: [RedisModule],
-  providers: [RedisMutexService],
-  exports: [RedisMutexService],
-})
-export class RedisMutexModule {}

src/commons/redis-mutex/redis-mutex.service.ts

@@ -1,71 +0,0 @@
-import { Injectable } from '@nestjs/common';
-import { RedisService } from 'nestjs-redis';
-import * as uuid from 'uuid';
-import { ApplicationException } from '../exceptions/application.exception';
-
-export interface RedisMutexOption {
-  /**
-   * seconds
-   */
-  expires?: number;
-  /**
-   * seconds
-   */
-  timeout?: number | null;
-  /**
-   * milliseconds
-   */
-  retryDelay?: number;
-}
-
-@Injectable()
-export class RedisMutexService {
-  constructor(private readonly redisClient: RedisService) {}
-
-  public async lock(
-    key: string,
-    { expires = 100, timeout = 10, retryDelay = 100 }: RedisMutexOption = {
-      expires: 100,
-      timeout: 10,
-      retryDelay: 100,
-    },
-  ) {
-    const redisKey = `${'mutex-lock'}:${key}`;
-    const redis = this.redisClient.getClient();
-    const value = uuid.v4();
-    const timeoutAt = timeout ? Date.now() + timeout * 1000 : null;
-
-    while (
-      !(await redis
-        .set(redisKey, value, 'EX', expires, 'NX')
-        .then(() => true)
-        .catch(() => false))
-    ) {
-      if (timeoutAt && timeoutAt > Date.now()) {
-        throw new ApplicationException('lock timeout');
-      }
-      await new Promise((resolve) => setTimeout(resolve, retryDelay));
-    }
-
-    const renewTimer = setInterval(() => {
-      redis.expire(redisKey, expires);
-    }, (expires * 1000) / 2);
-
-    return async () => {
-      clearInterval(renewTimer);
-      await redis.eval(
-        `
-      if redis.call("get", KEYS[1]) == ARGV[1]
-      then
-          return redis.call("del", KEYS[1])
-      else
-          return 0
-      end
-    `,
-        1,
-        redisKey,
-        value,
-      );
-    };
-  }
-}

src/commons/services/base-db.service.ts

@@ -53,22 +53,8 @@ export class BaseDbService<Entity extends AppBaseEntity> extends TypeormHelper {
   async isDuplicateEntityForUpdate<Dto extends Entity>(
     id: string,
     dto: Partial<Dto>,
-    extendsFields?: Array<keyof Dto & string>,
-  ): Promise<false | never>;
-  async isDuplicateEntityForUpdate<Dto extends Entity>(
-    old: Entity,
-    dto: Partial<Dto>,
-    extendsFields?: Array<keyof Dto & string>,
-  ): Promise<false | never>;
-  async isDuplicateEntityForUpdate<Dto extends Entity>(
-    id: string | Entity,
-    dto: Partial<Dto>,
     extendsFields: Array<keyof Dto & string> = [],
   ): Promise<false | never> {
-    if (typeof id !== 'string') {
-      dto = Object.assign({}, id, dto);
-      id = id.id;
-    }
     const qb = this.repository.createQueryBuilder('entity');
     const compareFields = this.getCompareFields(dto, [
       ...this.uniqueFields,

src/commons/utils/rabbit-mq.ts

@@ -1,21 +0,0 @@
-import { hostname } from 'os';
-
-export function getInstanceName() {
-  return hostname();
-}
-
-export function getSelfInstanceRouteKey(key: string) {
-  return getAppInstanceRouteKey(key, getInstanceName());
-}
-
-export function getAppInstanceRouteKey(key: string, appInstance?: string) {
-  return appInstance ? `${key}.${appInstance}` : key;
-}
-
-export function getSelfInstanceQueueKey(key: string) {
-  return getAppInstanceQueueKey(key, getInstanceName());
-}
-
-export function getAppInstanceQueueKey(key: string, appInstance?: string) {
-  return appInstance ? `${key}.${appInstance}` : key;
-}

src/pipeline-tasks/enums/task-statuses.enum.ts

@@ -11,5 +11,3 @@ registerEnumType(TaskStatuses, {
   name: 'TaskStatuses',
   description: '任务状态',
 });
-
-export const terminalTaskStatuses = [TaskStatuses.success, TaskStatuses.failed];

src/pipeline-tasks/models/pipeline-task-event.ts

@@ -1,25 +0,0 @@
-import { Field, ObjectType } from '@nestjs/graphql';
-import { PipelineUnits } from '../enums/pipeline-units.enum';
-import { TaskStatuses } from '../enums/task-statuses.enum';
-import { Type } from 'class-transformer';
-
-@ObjectType()
-export class PipelineTaskEvent {
-  @Field()
-  taskId: string;
-  @Field()
-  pipelineId: string;
-  @Field()
-  projectId: string;
-  @Field(() => PipelineUnits, { nullable: true })
-  unit: PipelineUnits | null;
-  @Field()
-  @Type(() => Date)
-  emittedAt: Date;
-  @Field()
-  message: string;
-  @Field()
-  messageType: 'stdout' | 'stderr' | 'stdin';
-  @Field(() => TaskStatuses)
-  status: TaskStatuses;
-}

src/pipeline-tasks/models/work-unit-metadata.model.ts

@@ -1,10 +1,9 @@
-import { Field, InputType, Int, ObjectType } from '@nestjs/graphql';
+import { InputType, ObjectType } from '@nestjs/graphql';
 import { WorkUnit } from './work-unit.model';
 
 @InputType('WorkUnitMetadataInput')
 @ObjectType()
 export class WorkUnitMetadata {
-  @Field(() => Int)
   version = 1;
   units: WorkUnit[];
 }

src/pipeline-tasks/pipeline-task-flush.service.spec.ts

@@ -1,88 +0,0 @@
-import { AmqpConnection } from '@golevelup/nestjs-rabbitmq';
-import { Test, TestingModule } from '@nestjs/testing';
-import { RedisService } from 'nestjs-redis';
-import { PipelineTaskFlushService } from './pipeline-task-flush.service';
-import { PipelineTaskEvent } from './models/pipeline-task-event';
-import { TaskStatuses } from './enums/task-statuses.enum';
-import {
-  EXCHANGE_PIPELINE_TASK_TOPIC,
-  ROUTE_PIPELINE_TASK_DONE,
-} from './pipeline-tasks.constants';
-
-describe('PipelineTaskFlushService', () => {
-  let service: PipelineTaskFlushService;
-  let redisService: RedisService;
-  let amqpConnection: AmqpConnection;
-
-  beforeEach(async () => {
-    const redisClient = {
-      rpush: jest.fn(() => Promise.resolve()),
-      lrange: jest.fn(() => Promise.resolve()),
-      expire: jest.fn(() => Promise.resolve()),
-    };
-    const module: TestingModule = await Test.createTestingModule({
-      providers: [
-        PipelineTaskFlushService,
-        {
-          provide: RedisService,
-          useValue: {
-            getClient() {
-              return redisClient;
-            },
-          },
-        },
-        {
-          provide: AmqpConnection,
-          useValue: {
-            request: jest.fn(() => Promise.resolve()),
-          },
-        },
-      ],
-    }).compile();
-
-    service = module.get<PipelineTaskFlushService>(PipelineTaskFlushService);
-    redisService = module.get<RedisService>(RedisService);
-    amqpConnection = module.get<AmqpConnection>(AmqpConnection);
-  });
-
-  it('should be defined', () => {
-    expect(service).toBeDefined();
-  });
-
-  describe('write', () => {
-    const amqpMsg = {
-      properties: { headers: { sender: 'test' } },
-    } as any;
-    it('normal', async () => {
-      const testEvent = new PipelineTaskEvent();
-      testEvent.taskId = 'test';
-      testEvent.status = TaskStatuses.working;
-      const rpush = jest.spyOn(redisService.getClient(), 'rpush');
-      const request = jest.spyOn(amqpConnection, 'request');
-      await service.write(testEvent, amqpMsg);
-      expect(rpush).toBeCalledTimes(1);
-      expect(rpush.mock.calls[0][0]).toEqual('p-task:log:test');
-      expect(rpush.mock.calls[0][1]).toEqual(JSON.stringify(testEvent));
-      expect(request).toBeCalledTimes(1);
-    });
-    it('event for which task done', async () => {
-      const testEvent = new PipelineTaskEvent();
-      testEvent.taskId = 'test';
-      testEvent.status = TaskStatuses.success;
-      const rpush = jest.spyOn(redisService.getClient(), 'rpush');
-      const request = jest.spyOn(amqpConnection, 'request');
-      await service.write(testEvent, amqpMsg);
-      expect(rpush).toBeCalledTimes(1);
-      expect(request).toBeCalledTimes(1);
-      expect(request.mock.calls[0][0]).toMatchObject({
-        exchange: EXCHANGE_PIPELINE_TASK_TOPIC,
-        routingKey: ROUTE_PIPELINE_TASK_DONE,
-        payload: {
-          taskId: 'test',
-          status: TaskStatuses.success,
-          runOn: 'test',
-        },
-      });
-    });
-  });
-});

src/pipeline-tasks/pipeline-task-flush.service.ts

@@ -1,66 +0,0 @@
-import { AmqpConnection, RabbitSubscribe } from '@golevelup/nestjs-rabbitmq';
-import { Injectable } from '@nestjs/common';
-import { ConsumeMessage } from 'amqplib';
-import { deserialize } from 'class-transformer';
-import { RedisService } from 'nestjs-redis';
-import { isNil } from 'ramda';
-import { getSelfInstanceQueueKey } from '../commons/utils/rabbit-mq';
-import { PipelineTaskEvent } from './models/pipeline-task-event';
-import {
-  EXCHANGE_PIPELINE_TASK_TOPIC,
-  ROUTE_PIPELINE_TASK_DONE,
-} from './pipeline-tasks.constants';
-import {
-  EXCHANGE_PIPELINE_TASK_FANOUT,
-  ROUTE_PIPELINE_TASK_LOG,
-  QUEUE_WRITE_PIPELINE_TASK_LOG,
-} from './pipeline-tasks.constants';
-
-@Injectable()
-export class PipelineTaskFlushService {
-  constructor(
-    private readonly redisService: RedisService,
-    private readonly amqpConnection: AmqpConnection,
-  ) {}
-
-  @RabbitSubscribe({
-    exchange: EXCHANGE_PIPELINE_TASK_FANOUT,
-    routingKey: ROUTE_PIPELINE_TASK_LOG,
-    queue: getSelfInstanceQueueKey(QUEUE_WRITE_PIPELINE_TASK_LOG),
-    queueOptions: {
-      autoDelete: true,
-      durable: true,
-    },
-  })
-  async write(message: PipelineTaskEvent, amqpMsg: ConsumeMessage) {
-    const client = this.redisService.getClient();
-    await client.rpush(this.getKey(message.taskId), JSON.stringify(message));
-    await client.expire(this.getKey(message.taskId), 600); // ten minutes
-    if (isNil(message.unit)) {
-      try {
-        await this.amqpConnection.request({
-          exchange: EXCHANGE_PIPELINE_TASK_TOPIC,
-          routingKey: ROUTE_PIPELINE_TASK_DONE,
-          payload: {
-            taskId: message.taskId,
-            status: message.status,
-            runOn: amqpMsg.properties.headers.sender,
-          },
-        });
-      } catch (error) {
-        console.log(error);
-      }
-    }
-  }
-
-  async read(taskId: string) {
-    const raw = await this.redisService
-      .getClient()
-      .lrange(this.getKey(taskId), 0, -1);
-    return raw.map((it) => deserialize(PipelineTaskEvent, it));
-  }
-
-  private getKey(taskId: string) {
-    return `p-task:log:${taskId}`;
-  }
-}

src/pipeline-tasks/pipeline-task-logs.service.spec.ts

@@ -0,0 +1,30 @@
+import { Test, TestingModule } from '@nestjs/testing';
+import { PipelineTaskLogsService } from './pipeline-task-logs.service';
+import { RedisService } from 'nestjs-redis';
+import { getPubSubToken } from '../commons/pub-sub/utils/token';
+
+describe('PipelineTaskLogsService', () => {
+  let service: PipelineTaskLogsService;
+
+  beforeEach(async () => {
+    const module: TestingModule = await Test.createTestingModule({
+      providers: [
+        PipelineTaskLogsService,
+        {
+          provide: RedisService,
+          useValue: {},
+        },
+        {
+          provide: getPubSubToken(),
+          useValue: {},
+        },
+      ],
+    }).compile();
+
+    service = module.get<PipelineTaskLogsService>(PipelineTaskLogsService);
+  });
+
+  it('should be defined', () => {
+    expect(service).toBeDefined();
+  });
+});

src/pipeline-tasks/pipeline-task-logs.service.ts

@@ -0,0 +1,81 @@
+import { Injectable } from '@nestjs/common';
+import { observableToAsyncIterable } from 'graphql-tools';
+import { RedisService } from 'nestjs-redis';
+import { find, omit, propEq } from 'ramda';
+import { InjectPubSub } from '../commons/pub-sub/decorators/inject-pub-sub.decorator';
+import { PubSub } from '../commons/pub-sub/pub-sub';
+import { PipelineUnits } from './enums/pipeline-units.enum';
+import { TaskStatuses } from './enums/task-statuses.enum';
+import { PipelineTaskLogMessage } from './models/pipeline-task-log-message.module';
+import { PipelineTaskLogs } from './models/pipeline-task-logs.model';
+import { PipelineTask } from './pipeline-task.entity';
+
+const LOG_TIMEOUT_SECONDS = 10_000;
+
+@Injectable()
+export class PipelineTaskLogsService {
+  constructor(
+    private readonly redisService: RedisService,
+    @InjectPubSub()
+    private readonly pubSub: PubSub,
+  ) {}
+
+  get redis() {
+    return this.redisService.getClient();
+  }
+
+  getKeys(task: PipelineTask) {
+    return `ptl:${task.id}`;
+  }
+
+  async recordLog(log: PipelineTaskLogMessage) {
+    const logDto = omit(['task'], log);
+    await Promise.all([
+      this.pubSub.publish(this.getKeys(log.task), logDto),
+      this.redis
+        .expire(this.getKeys(log.task), LOG_TIMEOUT_SECONDS)
+        .then(() =>
+          this.redis.rpush(this.getKeys(log.task), JSON.stringify(logDto)),
+        ),
+    ]);
+  }
+
+  async readLog(task: PipelineTask): Promise<PipelineTaskLogMessage[]> {
+    return await this.redis.lrange(this.getKeys(task), 0, -1).then((items) =>
+      items.map((item) => {
+        const log = JSON.parse(item) as PipelineTaskLogMessage;
+        log.task = task;
+        log.time = new Date(log.time);
+        return log;
+      }),
+    );
+  }
+
+  async readLogsAsPipelineTaskLogs(
+    task: PipelineTask,
+  ): Promise<PipelineTaskLogs[]> {
+    const logs = await this.readLog(task);
+    const taskLogs: PipelineTaskLogs[] = [];
+    for (const log of logs) {
+      const taskLog = find<PipelineTaskLogs>(
+        propEq('unit', log.unit),
+        taskLogs,
+      );
+      if (!taskLog) {
+        taskLogs.push({
+          unit: (log.unit as unknown) as PipelineUnits,
+          status: TaskStatuses.working,
+          startedAt: log.time,
+          logs: log.message,
+        });
+      } else {
+        taskLog.logs += log.message;
+      }
+    }
+    return taskLogs;
+  }
+
+  watchLogs(task: PipelineTask) {
+    return observableToAsyncIterable(this.pubSub.message$(this.getKeys(task)));
+  }
+}

src/pipeline-tasks/pipeline-task.consumer.spec.ts

@@ -0,0 +1,240 @@
+import { Test, TestingModule } from '@nestjs/testing';
+import { Job } from 'bull';
+import { join } from 'path';
+import { ReposService } from '../repos/repos.service';
+import { PipelineUnits } from './enums/pipeline-units.enum';
+import { PipelineTaskConsumer } from './pipeline-task.consumer';
+import { PipelineTask } from './pipeline-task.entity';
+import { PipelineTasksService } from './pipeline-tasks.service';
+import { PipelineTaskLogMessage } from './models/pipeline-task-log-message.module';
+import { Pipeline } from '../pipelines/pipeline.entity';
+import { Project } from '../projects/project.entity';
+import { TaskStatuses } from './enums/task-statuses.enum';
+import { PipelineTaskLogsService } from './pipeline-task-logs.service';
+import { ApplicationException } from '../commons/exceptions/application.exception';
+
+describe('PipelineTaskConsumer', () => {
+  let consumer: PipelineTaskConsumer;
+  let tasksService: PipelineTasksService;
+  let logsService: PipelineTaskLogsService;
+  const getJob = () =>
+    ({
+      data: {
+        pipelineId: 'test',
+        units: [PipelineUnits.checkout, PipelineUnits.test],
+      },
+    } as Job<PipelineTask>);
+
+  beforeEach(async () => {
+    const module: TestingModule = await Test.createTestingModule({
+      providers: [
+        {
+          provide: PipelineTasksService,
+          useValue: {
+            doNextTask: () => undefined,
+            updateTask: async (value) => value,
+          },
+        },
+        {
+          provide: ReposService,
+          useValue: {
+            getWorkspaceRootByTask: () => 'workspace-root',
+            checkout: async () => undefined,
+          },
+        },
+        {
+          provide: PipelineTaskLogsService,
+          useValue: {
+            recordLog: async () => undefined,
+            readLogsAsPipelineTaskLogs: async () => [],
+          },
+        },
+        PipelineTaskConsumer,
+      ],
+    }).compile();
+
+    tasksService = module.get(PipelineTasksService);
+    logsService = module.get(PipelineTaskLogsService);
+    consumer = module.get(PipelineTaskConsumer);
+  });
+
+  it('should be defined', () => {
+    expect(consumer).toBeDefined();
+  });
+
+  describe('onCompleted', () => {
+    it('should call doNextTask()', () => {
+      const job = getJob();
+      const doNextTask = jest.spyOn(tasksService, 'doNextTask');
+      consumer.onCompleted(job);
+      expect(doNextTask).toHaveBeenCalledTimes(1);
+    });
+  });
+
+  describe('runScript', () => {
+    let logText: string;
+    let errorText: string;
+    let recordLog: jest.SpyInstance;
+    beforeEach(() => {
+      logText = '';
+      errorText = '';
+      recordLog = jest
+        .spyOn(logsService, 'recordLog')
+        .mockImplementation(async (log: PipelineTaskLogMessage) => {
+          logText += log.message;
+          if (log.isError) {
+            errorText += log.message;
+          }
+        });
+    });
+    it('should success and log right message', async () => {
+      await consumer.runScript(
+        'node one-second-work.js',
+        join(__dirname, '../../test/data'),
+      );
+      expect(logText).toMatch(/10.+20.+30.+40.+50.+60.+70.+80.+90/s);
+      expect(recordLog).toHaveBeenCalledTimes(10);
+      expect(
+        ((recordLog.mock.calls[8][0] as unknown) as PipelineTaskLogMessage)
+          .message,
+      ).toMatch(/^90/);
+    });
+    it('should failed and log right message', async () => {
+      await expect(
+        consumer.runScript(
+          'node bad-work.js',
+          join(__dirname, '../../test/data'),
+        ),
+      ).rejects.toThrowError(/exec script failed/);
+      expect(errorText).toMatch(/Error Message/);
+      const logs = recordLog.mock.calls
+        .map((call) => ((call[0] as unknown) as PipelineTaskLogMessage).message)
+        .join('');
+      expect(logs).toMatch(/10.+20.+30.+40.+50/s);
+    });
+    it('should log with task', async () => {
+      const task = new PipelineTask();
+      task.id = 'test';
+
+      const recordLog = jest.spyOn(logsService, 'recordLog');
+      await expect(
+        consumer.runScript(
+          'node bad-work.js',
+          join(__dirname, '../../test/data'),
+          task,
+        ),
+      ).rejects.toThrowError(/exec script failed/);
+
+      expect(errorText).toMatch(/Error Message 2/);
+      expect(
+        ((recordLog.mock.calls[2][0] as unknown) as PipelineTaskLogMessage)
+          .task,
+      ).toMatchObject(task);
+    });
+  });
+
+  describe('doTask', () => {
+    let task: PipelineTask;
+
+    beforeEach(() => {
+      task = new PipelineTask();
+      task.id = 'test-id';
+      task.logs = [];
+      task.pipeline = new Pipeline();
+      task.pipeline.workUnitMetadata = {
+        version: 1,
+        units: [
+          {
+            type: PipelineUnits.checkout,
+            scripts: [],
+          },
+          {
+            type: PipelineUnits.installDependencies,
+            scripts: ["echo ' Hello, Fennec!'"],
+          },
+        ],
+      };
+      task.units = task.pipeline.workUnitMetadata.units.map(
+        (unit) => unit.type,
+      );
+      task.pipeline.project = new Project();
+      task.pipeline.project.name = 'test-project';
+    });
+
+    it('success and update task on db', async () => {
+      const job: Job = ({
+        data: task,
+        update: jest.fn().mockImplementation(() => undefined),
+      } as unknown) as Job;
+
+      jest
+        .spyOn(consumer, 'runScript')
+        .mockImplementation(async () => undefined);
+      const updateTask = jest.spyOn(tasksService, 'updateTask');
+
+      await consumer.doTask(job);
+
+      expect(updateTask).toHaveBeenCalledTimes(4);
+      expect(updateTask.mock.calls[0][0].startedAt).toBeDefined();
+      expect(updateTask.mock.calls[1][0].endedAt).toBeDefined();
+      expect(updateTask.mock.calls[1][0].status).toEqual(TaskStatuses.success);
+    });
+    it('failed and update task on db', async () => {
+      const job: Job = ({
+        data: task,
+        update: jest.fn().mockImplementation(() => undefined),
+      } as unknown) as Job;
+
+      jest.spyOn(consumer, 'runScript').mockImplementation(async () => {
+        throw new ApplicationException('exec script failed');
+      });
+      const updateTask = jest.spyOn(tasksService, 'updateTask');
+
+      await consumer.doTask(job);
+
+      expect(updateTask).toHaveBeenCalledTimes(4);
+      expect(updateTask.mock.calls[0][0].startedAt).toBeDefined();
+      expect(updateTask.mock.calls[1][0].endedAt).toBeDefined();
+      expect(updateTask.mock.calls[1][0].status).toEqual(TaskStatuses.failed);
+    });
+    it('should do all task', async () => {
+      const job: Job = ({
+        data: task,
+        update: jest.fn().mockImplementation(() => undefined),
+      } as unknown) as Job;
+
+      const runScript = jest
+        .spyOn(consumer, 'runScript')
+        .mockImplementation(async () => undefined);
+      const updateTask = jest.spyOn(tasksService, 'updateTask');
+
+      await consumer.doTask(job);
+
+      expect(runScript).toHaveBeenCalledTimes(1);
+      expect(updateTask).toHaveBeenCalledTimes(4);
+      const taskDto: PipelineTask = updateTask.mock.calls[0][0];
+      expect(taskDto.logs).toHaveLength(2);
+      expect(taskDto.logs[0].status).toEqual(TaskStatuses.success);
+      expect(taskDto.logs[0].unit).toEqual(PipelineUnits.checkout);
+    });
+    it('should log error message', async () => {
+      const job: Job = ({
+        data: task,
+        update: jest.fn().mockImplementation(() => undefined),
+      } as unknown) as Job;
+
+      jest.spyOn(consumer, 'runScript').mockImplementation(async () => {
+        throw new Error('bad message');
+      });
+      const updateTask = jest.spyOn(tasksService, 'updateTask');
+
+      await consumer.doTask(job);
+
+      expect(updateTask).toHaveBeenCalledTimes(4);
+      const taskDto: PipelineTask = updateTask.mock.calls[0][0];
+      expect(taskDto.logs).toHaveLength(2);
+      expect(taskDto.logs[0].status).toEqual(TaskStatuses.success);
+      expect(taskDto.logs[1].status).toEqual(TaskStatuses.failed);
+    });
+  });
+});

src/pipeline-tasks/pipeline-task.consumer.ts

@@ -0,0 +1,159 @@
+import { PipelineTaskLogs } from './models/pipeline-task-logs.model';
+import { ReposService } from './../repos/repos.service';
+import {
+  OnQueueCompleted,
+  OnQueueFailed,
+  Process,
+  Processor,
+} from '@nestjs/bull';
+import { Job } from 'bull';
+import { spawn } from 'child_process';
+import { PipelineTask } from './pipeline-task.entity';
+import { PIPELINE_TASK_QUEUE } from './pipeline-tasks.constants';
+import { PipelineTasksService } from './pipeline-tasks.service';
+import { ApplicationException } from '../commons/exceptions/application.exception';
+import { PipelineUnits } from './enums/pipeline-units.enum';
+import { PipelineTaskLogMessage } from './models/pipeline-task-log-message.module';
+import { TaskStatuses } from './enums/task-statuses.enum';
+import { PipelineTaskLogsService } from './pipeline-task-logs.service';
+import debug from 'debug';
+
+const log = debug('fennec:pipeline-tasks:consumer');
+
+@Processor(PIPELINE_TASK_QUEUE)
+export class PipelineTaskConsumer {
+  constructor(
+    private readonly service: PipelineTasksService,
+    private readonly reposService: ReposService,
+    private readonly logsService: PipelineTaskLogsService,
+  ) {}
+  @Process()
+  async doTask(job: Job<PipelineTask>) {
+    let task = job.data;
+    if (task.pipeline.workUnitMetadata.version !== 1) {
+      throw new ApplicationException(
+        'work unit metadata version is not match.',
+      );
+    }
+
+    task.startedAt = new Date();
+    task.status = TaskStatuses.working;
+    task = await this.service.updateTask(task);
+    log('start job');
+    await job.update(task);
+
+    const workspaceRoot = this.reposService.getWorkspaceRootByTask(task);
+
+    const units = task.units.map(
+      (type) =>
+        task.pipeline.workUnitMetadata.units.find(
+          (unit) => unit.type === type,
+        ) ?? { type: type, scripts: [] },
+    );
+
+    log('task have [%o] units', units);
+    try {
+      for (const unit of units) {
+        const unitLog = new PipelineTaskLogs();
+        unitLog.unit = unit.type;
+        unitLog.startedAt = new Date();
+        log('curr unit is %s', unit.type);
+        try {
+          // 检出代码前执行 git checkout
+          if (unit.type === PipelineUnits.checkout) {
+            log('begin checkout');
+            await this.reposService.checkout(task, workspaceRoot);
+            unitLog.status = TaskStatuses.success;
+            log('end checkout');
+          }
+          for (const script of unit.scripts) {
+            unitLog.logs += `[RUN SCRIPT] ${script}`;
+            log('begin runScript %s', script);
+            await this.runScript(script, workspaceRoot, task, unit.type);
+            log('end runScript %s', script);
+          }
+          unitLog.status = TaskStatuses.success;
+        } catch (err) {
+          unitLog.status = TaskStatuses.failed;
+          unitLog.logs += err.message;
+          throw err;
+        } finally {
+          unitLog.endedAt = new Date();
+          unitLog.logs = await this.logsService
+            .readLogsAsPipelineTaskLogs(task)
+            .then(
+              (taskLogs) =>
+                taskLogs.find((tl) => tl.unit === unit.type)?.logs ?? '',
+            );
+          task.logs.push(unitLog);
+          task = await this.service.updateTask(task);
+          await job.update(task);
+        }
+      }
+
+      task.status = TaskStatuses.success;
+    } catch (err) {
+      task.status = TaskStatuses.failed;
+      log('task is failed', err);
+    } finally {
+      task.endedAt = new Date();
+      task = await this.service.updateTask(task);
+      await job.update(task);
+    }
+  }
+
+  async runScript(
+    script: string,
+    workspaceRoot: string,
+    task?: PipelineTask,
+    unit?: PipelineUnits,
+  ): Promise<void> {
+    return new Promise((resolve, reject) => {
+      const sub = spawn(script, {
+        shell: true,
+        cwd: workspaceRoot,
+      });
+      let loggingCount = 0; // semaphore
+
+      sub.stderr.on('data', (data: Buffer) => {
+        const str = data.toString();
+        loggingCount++;
+        this.logsService
+          .recordLog(PipelineTaskLogMessage.create(task, unit, str, true))
+          .finally(() => loggingCount--);
+      });
+      sub.stdout.on('data', (data: Buffer) => {
+        const str = data.toString();
+        loggingCount++;
+        this.logsService
+          .recordLog(PipelineTaskLogMessage.create(task, unit, str, false))
+          .finally(() => loggingCount--);
+      });
+      sub.addListener('close', async (code) => {
+        await new Promise<void>(async (resolve) => {
+          for (let i = 0; i < 10 && loggingCount > 0; i++) {
+            await new Promise((resolve) => setTimeout(resolve, 500));
+            log('waiting logging... (%dx500ms)', i);
+          }
+          resolve();
+        });
+        if (code === 0) {
+          return resolve();
+        }
+        return reject(new ApplicationException('exec script failed'));
+      });
+    });
+  }
+
+  @OnQueueCompleted()
+  onCompleted(job: Job<PipelineTask>) {
+    log('queue onCompleted');
+    this.service.doNextTask(job.data.pipeline);
+  }
+
+  @OnQueueFailed()
+  onFailed(job: Job<PipelineTask>) {
+    log('queue onFailed');
+    this.service.doNextTask(job.data.pipeline);
+  }
+}

src/pipeline-tasks/pipeline-task.entity.ts

@@ -36,7 +36,4 @@ export class PipelineTask extends AppBaseEntity {
 
   @Column({ nullable: true })
   endedAt?: Date;
-
-  @Column({ nullable: true })
-  runOn: string;
 }

src/pipeline-tasks/pipeline-task.logger.spec.ts

@@ -1,74 +0,0 @@
-import { Test, TestingModule } from '@nestjs/testing';
-import { PipelineTaskLogger } from './pipeline-task.logger';
-import { PipelineTaskEvent } from './models/pipeline-task-event';
-import { take, timeout } from 'rxjs/operators';
-
-describe('PipelineTaskRunner', () => {
-  let logger: PipelineTaskLogger;
-  let module: TestingModule;
-
-  beforeEach(async () => {
-    module = await Test.createTestingModule({
-      providers: [PipelineTaskLogger],
-    }).compile();
-
-    logger = module.get(PipelineTaskLogger);
-  });
-
-  it('should be defined', () => {
-    expect(logger).toBeDefined();
-  });
-
-  describe('getMessage$', () => {
-    it('normal', async () => {
-      const event = new PipelineTaskEvent();
-      event.taskId = 'test';
-      const emittedAt = new Date();
-      event.emittedAt = emittedAt.toISOString() as any;
-      const message$ = logger.getMessage$('test');
-
-      let receiveEvent;
-      message$.pipe(take(1)).subscribe((value) => (receiveEvent = value));
-      await logger.handleEvent(event);
-      expect(receiveEvent).toMatchObject({
-        ...event,
-        emittedAt,
-      });
-    });
-    it('no match', async () => {
-      const event = new PipelineTaskEvent();
-      event.taskId = 'test';
-      const message$ = logger.getMessage$('other');
-      setTimeout(() => {
-        logger.handleEvent(event);
-      });
-      expect(message$.pipe(take(1), timeout(100)).toPromise()).rejects.toMatch(
-        'timeout',
-      );
-    });
-    it('multiple subscribers', async () => {
-      const event = new PipelineTaskEvent();
-      event.taskId = 'test';
-      const message$ = logger.getMessage$('test');
-      const message2$ = logger.getMessage$('test');
-      setTimeout(() => {
-        logger.handleEvent(event);
-      });
-      expect(message$.pipe(take(1), timeout(100)).toPromise()).resolves.toEqual(
-        event,
-      );
-      expect(
-        message2$.pipe(take(1), timeout(100)).toPromise(),
-      ).resolves.toEqual(event);
-    });
-  });
-
-  describe('onModuleDestroy', () => {
-    it('complete observable when destroying module', async () => {
-      logger.onModuleDestroy();
-      await expect(
-        (logger as any).message$.toPromise(),
-      ).resolves.toBeUndefined();
-    });
-  });
-});

src/pipeline-tasks/pipeline-task.logger.ts

@@ -1,37 +0,0 @@
-import { RabbitSubscribe } from '@golevelup/nestjs-rabbitmq';
-import { Injectable, OnModuleDestroy } from '@nestjs/common';
-import { plainToClass } from 'class-transformer';
-import { Observable, Subject } from 'rxjs';
-import { filter } from 'rxjs/operators';
-import { PipelineTaskEvent } from './models/pipeline-task-event';
-import {
-  EXCHANGE_PIPELINE_TASK_FANOUT,
-  QUEUE_HANDLE_PIPELINE_TASK_LOG_EVENT,
-  ROUTE_PIPELINE_TASK_LOG,
-} from './pipeline-tasks.constants';
-
-@Injectable()
-export class PipelineTaskLogger implements OnModuleDestroy {
-  private readonly messageSubject = new Subject<PipelineTaskEvent>();
-  private readonly message$: Observable<PipelineTaskEvent> = this.messageSubject.pipe();
-
-  @RabbitSubscribe({
-    exchange: EXCHANGE_PIPELINE_TASK_FANOUT,
-    routingKey: ROUTE_PIPELINE_TASK_LOG,
-    queue: QUEUE_HANDLE_PIPELINE_TASK_LOG_EVENT,
-    queueOptions: {
-      autoDelete: true,
-    },
-  })
-  async handleEvent(message: PipelineTaskEvent) {
-    this.messageSubject.next(plainToClass(PipelineTaskEvent, message));
-  }
-
-  getMessage$(taskId: string) {
-    return this.message$.pipe(filter((event) => event.taskId === taskId));
-  }
-
-  onModuleDestroy() {
-    this.messageSubject.complete();
-  }
-}

src/pipeline-tasks/pipeline-task.runner.spec.ts

@@ -1,326 +0,0 @@
-import { Test, TestingModule } from '@nestjs/testing';
-import { ReposService } from '../repos/repos.service';
-import { PipelineUnits } from './enums/pipeline-units.enum';
-import { PipelineTask } from './pipeline-task.entity';
-import { Pipeline } from '../pipelines/pipeline.entity';
-import { Project } from '../projects/project.entity';
-import { TaskStatuses } from './enums/task-statuses.enum';
-import { getLoggerToken, PinoLogger } from 'nestjs-pino';
-import { PipelineTaskRunner } from './pipeline-task.runner';
-import { WorkUnitMetadata } from './models/work-unit-metadata.model';
-import { AmqpConnection } from '@golevelup/nestjs-rabbitmq';
-describe('PipelineTaskRunner', () => {
-  let runner: PipelineTaskRunner;
-  let reposService: ReposService;
-
-  beforeEach(async () => {
-    const module: TestingModule = await Test.createTestingModule({
-      providers: [
-        {
-          provide: ReposService,
-          useValue: {
-            getWorkspaceRootByTask: () => 'workspace-root',
-            checkout: async () => undefined,
-          },
-        },
-        {
-          provide: getLoggerToken(PipelineTaskRunner.name),
-          useValue: new PinoLogger({}),
-        },
-        {
-          provide: 'spawn',
-          useValue: () => undefined,
-        },
-        PipelineTaskRunner,
-        {
-          provide: AmqpConnection,
-          useValue: {},
-        },
-      ],
-    }).compile();
-
-    reposService = module.get(ReposService);
-    runner = module.get(PipelineTaskRunner);
-  });
-
-  it('should be defined', () => {
-    expect(runner).toBeDefined();
-  });
-
-  it('onNewTask', async () => {
-    const task = new PipelineTask();
-    let tmpTask;
-    const doTask = jest
-      .spyOn(runner, 'doTask')
-      .mockImplementation(async (task) => {
-        tmpTask = task;
-      });
-    await runner.onNewTask(task);
-    expect(tmpTask).toEqual(task);
-    expect(doTask).toBeCalledTimes(1);
-  });
-
-  describe('test biz', () => {
-    let emitEvent: jest.SpyInstance;
-    beforeEach(() => {
-      emitEvent = jest
-        .spyOn(runner, 'emitEvent')
-        .mockImplementation((..._) => Promise.resolve());
-    });
-
-    describe('doTask', () => {
-      let checkout: jest.SpyInstance;
-      let doTaskUnit: jest.SpyInstance;
-
-      beforeEach(() => {
-        checkout = jest
-          .spyOn(runner, 'checkout')
-          .mockImplementation((..._) => Promise.resolve('/null'));
-        doTaskUnit = jest
-          .spyOn(runner, 'doTaskUnit')
-          .mockImplementation((..._) => Promise.resolve());
-      });
-
-      it('only checkout', async () => {
-        const task = new PipelineTask();
-        (task.id = 'taskId'), (task.pipeline = new Pipeline());
-        task.units = [PipelineUnits.checkout];
-        task.pipeline.id = 'pipelineId';
-        task.pipeline.project = new Project();
-        task.pipeline.project.id = 'projectId';
-        task.pipeline.workUnitMetadata = new WorkUnitMetadata();
-        task.pipeline.workUnitMetadata.version = 1;
-        task.pipeline.workUnitMetadata.units = [
-          {
-            type: PipelineUnits.checkout,
-            scripts: [],
-          },
-        ];
-
-        await runner.doTask(task);
-
-        expect(checkout).toBeCalledTimes(1);
-        expect(doTaskUnit).toBeCalledTimes(0);
-        expect(emitEvent).toBeCalledTimes(2);
-        expect(emitEvent.mock.calls[0][0]).toMatchObject(task);
-        expect(emitEvent.mock.calls[0][1]).toBeNull();
-        expect(emitEvent.mock.calls[0][2]).toEqual(TaskStatuses.working);
-        expect(emitEvent.mock.calls[1][0]).toMatchObject(task);
-        expect(emitEvent.mock.calls[1][1]).toBeNull();
-        expect(emitEvent.mock.calls[1][2]).toEqual(TaskStatuses.success);
-      });
-
-      it('many units', async () => {
-        const task = new PipelineTask();
-        (task.id = 'taskId'), (task.pipeline = new Pipeline());
-        task.units = [
-          PipelineUnits.checkout,
-          PipelineUnits.test,
-          PipelineUnits.deploy,
-        ];
-        task.pipeline.id = 'pipelineId';
-        task.pipeline.project = new Project();
-        task.pipeline.project.id = 'projectId';
-        task.pipeline.workUnitMetadata = new WorkUnitMetadata();
-        task.pipeline.workUnitMetadata.version = 1;
-        task.pipeline.workUnitMetadata.units = [
-          {
-            type: PipelineUnits.checkout,
-            scripts: [],
-          },
-          {
-            type: PipelineUnits.installDependencies,
-            scripts: ['pwd'],
-          },
-          {
-            type: PipelineUnits.test,
-            scripts: ['pwd'],
-          },
-          {
-            type: PipelineUnits.deploy,
-            scripts: ['pwd', 'uname'],
-          },
-        ];
-
-        await runner.doTask(task);
-
-        expect(checkout).toBeCalledTimes(1);
-        expect(doTaskUnit).toBeCalledTimes(2);
-        expect(emitEvent).toBeCalledTimes(2);
-      });
-
-      it('unit work failed', async () => {
-        const task = new PipelineTask();
-        (task.id = 'taskId'), (task.pipeline = new Pipeline());
-        task.units = [PipelineUnits.checkout, PipelineUnits.test];
-        task.pipeline.id = 'pipelineId';
-        task.pipeline.project = new Project();
-        task.pipeline.project.id = 'projectId';
-        task.pipeline.workUnitMetadata = new WorkUnitMetadata();
-        task.pipeline.workUnitMetadata.version = 1;
-        task.pipeline.workUnitMetadata.units = [
-          {
-            type: PipelineUnits.checkout,
-            scripts: [],
-          },
-          {
-            type: PipelineUnits.test,
-            scripts: ['pwd'],
-          },
-        ];
-
-        doTaskUnit = jest
-          .spyOn(runner, 'doTaskUnit')
-          .mockImplementation((..._) =>
-            Promise.reject(new Error('test error')),
-          );
-        await runner.doTask(task);
-
-        expect(checkout).toBeCalledTimes(1);
-        expect(doTaskUnit).toBeCalledTimes(1);
-        expect(emitEvent).toBeCalledTimes(2);
-        expect(emitEvent.mock.calls[1][0]).toMatchObject(task);
-        expect(emitEvent.mock.calls[1][1]).toBeNull();
-        expect(emitEvent.mock.calls[1][2]).toEqual(TaskStatuses.failed);
-      });
-    });
-
-    describe('doTaskUnit', () => {
-      it('success', async () => {
-        const runScript = jest
-          .spyOn(runner, 'runScript')
-          .mockImplementation((..._) => Promise.resolve());
-        const task = new PipelineTask();
-
-        const unit = PipelineUnits.test;
-        const workspacePath = '/null';
-        await runner.doTaskUnit(unit, ['pwd'], task, workspacePath);
-
-        expect(emitEvent.mock.calls[0][0]).toEqual(task);
-        expect(emitEvent.mock.calls[0][1]).toEqual(unit);
-        expect(emitEvent.mock.calls[0][2]).toEqual(TaskStatuses.working);
-        expect(emitEvent.mock.calls[1][0]).toEqual(task);
-        expect(emitEvent.mock.calls[1][1]).toEqual(unit);
-        expect(emitEvent.mock.calls[1][2]).toEqual(TaskStatuses.success);
-        expect(runScript.mock.calls[0][0]).toEqual('pwd');
-        expect(runScript.mock.calls[0][1]).toEqual(workspacePath);
-        expect(runScript.mock.calls[0][2]).toEqual(task);
-        expect(runScript.mock.calls[0][3]).toEqual(unit);
-      });
-      it('failed', async () => {
-        const runScript = jest
-          .spyOn(runner, 'runScript')
-          .mockImplementation((..._) =>
-            Promise.reject(new Error('test error')),
-          );
-        const task = new PipelineTask();
-
-        const unit = PipelineUnits.test;
-        const workspacePath = '/null';
-        await expect(
-          runner.doTaskUnit(unit, ['pwd'], task, workspacePath),
-        ).rejects.toThrow('test error');
-
-        expect(emitEvent.mock.calls[1]?.[0]).toEqual(task);
-        expect(emitEvent.mock.calls[1]?.[1]).toEqual(unit);
-        expect(emitEvent.mock.calls[1]?.[2]).toEqual(TaskStatuses.failed);
-        expect(runScript).toBeCalledTimes(1);
-      });
-    });
-
-    describe('runScript', () => {
-      it('normal', async () => {
-        const spawn = jest.fn((..._: any[]) => ({
-          stdout: {
-            on: () => undefined,
-          },
-          stderr: {
-            on: () => undefined,
-          },
-          addListener: (_: any, fn: (code: number) => void) => {
-            fn(0);
-          },
-        }));
-        (runner as any).spawn = spawn;
-
-        const task = new PipelineTask();
-        task.id = 'taskId';
-        const unit = PipelineUnits.deploy;
-
-        await runner.runScript('script name', 'workspaceRoot', task, unit);
-        expect(spawn).toHaveBeenCalledTimes(1);
-        expect(spawn.mock.calls[0][0]).toEqual('script name');
-        expect(spawn.mock.calls[0][1]).toMatchObject({
-          shell: true,
-          cwd: 'workspaceRoot',
-        });
-      });
-      it('failed', async () => {
-        const spawn = jest.fn((..._: any[]) => ({
-          stdout: {
-            on: () => undefined,
-          },
-          stderr: {
-            on: () => undefined,
-          },
-          addListener: (_: any, fn: (code: number) => void) => {
-            fn(1);
-          },
-        }));
-        (runner as any).spawn = spawn;
-
-        const task = new PipelineTask();
-        task.id = 'taskId';
-        const unit = PipelineUnits.deploy;
-
-        expect(
-          runner.runScript('script name', 'workspaceRoot', task, unit),
-        ).rejects.toThrowError();
-      });
-      it('wait emit message done', async () => {
-        let finishedFn: () => void;
-        const on = jest.fn((_: any, fn: (buff: Buffer) => void) => {
-          setTimeout(() => {
-            fn(Buffer.from('message 1'));
-            setTimeout(() => {
-              fn(Buffer.from('message 2'));
-              setTimeout(() => {
-                fn(Buffer.from('message 3'));
-                finishedFn();
-              }, 1000);
-            }, 10);
-          }, 10);
-        });
-        const spawn = jest.fn((..._: any[]) => ({
-          stdout: {
-            on,
-          },
-          stderr: {
-            on,
-          },
-          addListener: (_: any, fn: (code: number) => void) => {
-            finishedFn = () => fn(0);
-          },
-        }));
-
-        let emitSuccessCount = 0;
-        jest.spyOn(runner, 'emitEvent').mockImplementation((..._: any[]) => {
-          return new Promise((resolve) => {
-            setTimeout(() => {
-              emitSuccessCount++;
-              resolve();
-            }, 1000);
-          });
-        });
-        (runner as any).spawn = spawn;
-
-        const task = new PipelineTask();
-        task.id = 'taskId';
-        const unit = PipelineUnits.deploy;
-
-        await runner.runScript('script name', 'workspaceRoot', task, unit);
-        expect(emitSuccessCount).toEqual(1 + 6);
-      });
-    });
-  });
-});

src/pipeline-tasks/pipeline-task.runner.ts

@@ -1,301 +0,0 @@
-import { ReposService } from '../repos/repos.service';
-import { spawn, ChildProcessWithoutNullStreams } from 'child_process';
-import { PipelineTask } from './pipeline-task.entity';
-import { ApplicationException } from '../commons/exceptions/application.exception';
-import { PipelineUnits } from './enums/pipeline-units.enum';
-import { TaskStatuses } from './enums/task-statuses.enum';
-import { InjectPinoLogger, PinoLogger } from 'nestjs-pino';
-import {
-  AmqpConnection,
-  RabbitRPC,
-  RabbitSubscribe,
-} from '@golevelup/nestjs-rabbitmq';
-import { PipelineTaskEvent } from './models/pipeline-task-event';
-import { last } from 'ramda';
-import { Inject } from '@nestjs/common';
-import {
-  EXCHANGE_PIPELINE_TASK_TOPIC,
-  QUEUE_PIPELINE_TASK_KILL,
-  ROUTE_PIPELINE_TASK_KILL,
-} from './pipeline-tasks.constants';
-import {
-  EXCHANGE_PIPELINE_TASK_FANOUT,
-  ROUTE_PIPELINE_TASK_LOG,
-} from './pipeline-tasks.constants';
-import {
-  getInstanceName,
-  getSelfInstanceQueueKey,
-  getSelfInstanceRouteKey,
-} from '../commons/utils/rabbit-mq';
-
-type Spawn = typeof spawn;
-
-export class PipelineTaskRunner {
-  readonly processes = new Map<string, ChildProcessWithoutNullStreams>();
-  readonly stopTaskIds = new Set<string>();
-
-  constructor(
-    private readonly reposService: ReposService,
-    @InjectPinoLogger(PipelineTaskRunner.name)
-    private readonly logger: PinoLogger,
-    @Inject('spawn')
-    private readonly spawn: Spawn,
-    private readonly amqpConnection: AmqpConnection,
-  ) {}
-  @RabbitSubscribe({
-    exchange: 'new-pipeline-task',
-    routingKey: 'mac',
-    queue: 'mac.new-pipeline-task',
-  })
-  async onNewTask(task: PipelineTask) {
-    this.logger.info({ task }, 'on new task [%s].', task.id);
-    try {
-      await this.doTask(task);
-    } catch (err) {
-      this.logger.error({ task, err }, err.message);
-    }
-  }
-  @RabbitRPC({
-    exchange: EXCHANGE_PIPELINE_TASK_TOPIC,
-    routingKey: getSelfInstanceRouteKey(ROUTE_PIPELINE_TASK_KILL),
-    queue: getSelfInstanceQueueKey(QUEUE_PIPELINE_TASK_KILL),
-    queueOptions: {
-      autoDelete: true,
-      durable: true,
-    },
-  })
-  async onStopTask(task: PipelineTask) {
-    this.logger.info({ task }, 'on stop task [%s].', task.id);
-    this.stopTaskIds.add(task.id);
-    const process = this.processes.get(task.id);
-    if (process) {
-      this.logger.info({ task }, 'send signal SIGINT to child process.');
-      process.kill('SIGINT');
-
-      setTimeout(() => {
-        setTimeout(() => {
-          this.stopTaskIds.delete(task.id);
-        }, 10_000);
-        if (process === this.processes.get(task.id)) {
-          this.logger.info({ task }, 'send signal SIGKILL to child process.');
-          process.kill('SIGKILL');
-          return;
-        }
-        if (this.processes.has(task.id)) {
-          this.logger.error(
-            { task },
-            'this pipeline task not stop yet. there is a new process running, maybe is a bug about error capture',
-          );
-        }
-      }, 10_000);
-    } else {
-      this.logger.info({ task }, 'child process is not running.');
-    }
-    return true;
-  }
-
-  async doTask(task: PipelineTask) {
-    if (task.pipeline.workUnitMetadata.version !== 1) {
-      throw new ApplicationException(
-        'work unit metadata version is not match.',
-      );
-    }
-    await this.emitEvent(
-      task,
-      null,
-      TaskStatuses.working,
-      `[start task]`,
-      'stdout',
-    );
-
-    this.logger.info('running task [%s].', task.id);
-    try {
-      const workspaceRoot = await this.checkout(task);
-      const units = task.units
-        .filter((unit) => unit !== PipelineUnits.checkout)
-        .map(
-          (type) =>
-            task.pipeline.workUnitMetadata.units.find(
-              (unit) => unit.type === type,
-            ) ?? { type: type, scripts: [] },
-        );
-      this.logger.info({ units }, 'begin run units.');
-      for (const unit of units) {
-        await this.doTaskUnit(unit.type, unit.scripts, task, workspaceRoot);
-      }
-      await this.emitEvent(
-        task,
-        null,
-        TaskStatuses.success,
-        `[finished task] success`,
-        'stdout',
-      );
-      this.logger.info({ task }, 'task [%s] completed.', task.id);
-    } catch (err) {
-      await this.emitEvent(
-        task,
-        null,
-        TaskStatuses.failed,
-        `[finished unit] ${err.message}`,
-        'stderr',
-      );
-      this.logger.error({ task, error: err }, 'task [%s] failed.', task.id);
-    } finally {
-    }
-  }
-
-  async doTaskUnit(
-    unit: PipelineUnits,
-    scripts: string[],
-    task: PipelineTask,
-    workspaceRoot: string,
-  ) {
-    await this.emitEvent(
-      task,
-      unit,
-      TaskStatuses.working,
-      `[begin unit] ${unit}`,
-      'stdin',
-    );
-    this.logger.info({ task }, 'curr unit is %s', unit);
-    try {
-      for (const script of scripts) {
-        this.logger.debug('begin runScript %s', script);
-        if (this.stopTaskIds.has(task.id)) {
-          throw new ApplicationException('Task is be KILLED');
-        }
-        await this.runScript(script, workspaceRoot, task, unit);
-        this.logger.debug('end runScript %s', script);
-      }
-
-      await this.emitEvent(
-        task,
-        unit,
-        TaskStatuses.success,
-        `[finished unit] ${unit}`,
-        'stdout',
-      );
-    } catch (err) {
-      await this.emitEvent(
-        task,
-        unit,
-        TaskStatuses.failed,
-        `[finished unit] ${err.message}`,
-        'stderr',
-      );
-      throw err;
-    }
-  }
-
-  async checkout(task: PipelineTask) {
-    await this.emitEvent(
-      task,
-      PipelineUnits.checkout,
-      TaskStatuses.working,
-      '[begin unit] checkout',
-      'stdin',
-    );
-    try {
-      const path = await this.reposService.checkout4Task(task);
-      await this.emitEvent(
-        task,
-        PipelineUnits.checkout,
-        TaskStatuses.success,
-        'checkout success.',
-        'stdout',
-      );
-      return path;
-    } catch (err) {
-      await this.emitEvent(
-        task,
-        PipelineUnits.checkout,
-        TaskStatuses.failed,
-        'checkout failed.',
-        'stderr',
-      );
-    }
-  }
-
-  async emitEvent(
-    task: PipelineTask,
-    unit: PipelineUnits | null,
-    status: TaskStatuses,
-    message: string,
-    messageType: 'stderr' | 'stdout' | 'stdin',
-  ) {
-    const event: PipelineTaskEvent = {
-      taskId: task.id,
-      pipelineId: task.pipeline.id,
-      projectId: task.pipeline.project.id,
-      unit,
-      emittedAt: new Date(),
-      message: last(message) === '\n' ? message : message + '\n',
-      messageType,
-      status,
-    };
-    this.amqpConnection
-      .publish(EXCHANGE_PIPELINE_TASK_FANOUT, ROUTE_PIPELINE_TASK_LOG, event, {
-        headers: {
-          sender: getInstanceName(),
-        },
-      })
-      .catch((error) => {
-        this.logger.error(
-          { error, event },
-          'send event message to queue failed. %s',
-          error.message,
-        );
-      });
-  }
-
-  async runScript(
-    script: string,
-    workspaceRoot: string,
-    task: PipelineTask,
-    unit: PipelineUnits,
-  ): Promise<void> {
-    await this.emitEvent(task, unit, TaskStatuses.working, script, 'stdin');
-    return new Promise((resolve, reject) => {
-      const sub = this.spawn(script, {
-        shell: true,
-        cwd: workspaceRoot,
-      });
-      this.processes.set(task.id, sub);
-      let loggingCount = 0; // semaphore
-
-      sub.stderr.on('data', (data: Buffer) => {
-        const str = data.toString();
-        loggingCount++;
-
-        this.emitEvent(task, unit, TaskStatuses.working, str, 'stdout').finally(
-          () => loggingCount--,
-        );
-      });
-      sub.stdout.on('data', (data: Buffer) => {
-        const str = data.toString();
-        loggingCount++;
-
-        this.emitEvent(task, unit, TaskStatuses.working, str, 'stderr').finally(
-          () => loggingCount--,
-        );
-      });
-      sub.addListener('close', async (code) => {
-        this.processes.delete(task.id);
-        await new Promise<void>(async (resolve) => {
-          for (let i = 0; i < 10 && loggingCount > 0; i++) {
-            await new Promise((resolve) => setTimeout(resolve, 500));
-            this.logger.debug('waiting logging... (%dx500ms)', i);
-          }
-          resolve();
-        });
-        if (code === 0) {
-          return resolve();
-        }
-        if (this.stopTaskIds.has(task.id)) {
-          throw reject(new ApplicationException('Task is be KILLED'));
-        }
-        return reject(new ApplicationException('exec script failed'));
-      });
-    });
-  }
-}

src/pipeline-tasks/pipeline-tasks.constants.ts

@@ -1,9 +1,2 @@
-export const EXCHANGE_PIPELINE_TASK_TOPIC = 'pipeline-task.topic';
+export const PIPELINE_TASK_QUEUE = 'PIPELINE_TASK_QUEUE';
-export const EXCHANGE_PIPELINE_TASK_FANOUT = 'pipeline-task.fanout';
+export const PIPELINE_TASK_LOG_QUEUE = 'PIPELINE_TASK_LOG_QUEUE';
-export const ROUTE_PIPELINE_TASK_LOG = 'pipeline-task-log';
-export const QUEUE_HANDLE_PIPELINE_TASK_LOG_EVENT = 'pipeline-task-log';
-export const QUEUE_WRITE_PIPELINE_TASK_LOG = 'write-pipeline-task-log';
-export const ROUTE_PIPELINE_TASK_DONE = 'pipeline-task-done';
-export const QUEUE_PIPELINE_TASK_DONE = 'pipeline-task-done';
-export const ROUTE_PIPELINE_TASK_KILL = 'pipeline-task-kill';
-export const QUEUE_PIPELINE_TASK_KILL = 'pipeline-task-kill';

src/pipeline-tasks/pipeline-tasks.module.ts

@@ -6,66 +6,27 @@ import { PipelineTask } from './pipeline-task.entity';
 import { Pipeline } from '../pipelines/pipeline.entity';
 import { ReposModule } from '../repos/repos.module';
 import { RedisModule } from 'nestjs-redis';
-import { RabbitMQModule } from '@golevelup/nestjs-rabbitmq';
+import { BullModule } from '@nestjs/bull';
-import { ConfigModule, ConfigService } from '@nestjs/config';
+import { PipelineTaskConsumer } from './pipeline-task.consumer';
-import { PipelineTaskRunner } from './pipeline-task.runner';
+import { PIPELINE_TASK_QUEUE } from './pipeline-tasks.constants';
-import { spawn } from 'child_process';
+import { PipelineTaskLogsService } from './pipeline-task-logs.service';
-import {
+import { PubSubModule } from '../commons/pub-sub/pub-sub.module';
-  EXCHANGE_PIPELINE_TASK_FANOUT,
-  EXCHANGE_PIPELINE_TASK_TOPIC,
-} from './pipeline-tasks.constants';
-import { PipelineTaskLogger } from './pipeline-task.logger';
-import { PipelineTaskFlushService } from './pipeline-task-flush.service';
 
 @Module({
   imports: [
     TypeOrmModule.forFeature([PipelineTask, Pipeline]),
+    BullModule.registerQueue({
+      name: PIPELINE_TASK_QUEUE,
+    }),
+    PubSubModule.forFeature(),
     RedisModule,
     ReposModule,
-    RabbitMQModule.forRootAsync(RabbitMQModule, {
-      imports: [ConfigModule],
-      useFactory: (configService: ConfigService) => ({
-        uri: configService.get<string>('db.rabbitmq.uri'),
-        exchanges: [
-          {
-            name: 'new-pipeline-task',
-            type: 'fanout',
-            options: {
-              durable: true,
-              autoDelete: true,
-            },
-          },
-          {
-            name: EXCHANGE_PIPELINE_TASK_FANOUT,
-            type: 'fanout',
-            options: {
-              durable: false,
-              autoDelete: true,
-            },
-          },
-          {
-            name: EXCHANGE_PIPELINE_TASK_TOPIC,
-            type: 'topic',
-            options: {
-              durable: false,
-              autoDelete: true,
-            },
-          },
-        ],
-      }),
-      inject: [ConfigService],
-    }),
   ],
   providers: [
     PipelineTasksService,
     PipelineTasksResolver,
-    PipelineTaskRunner,
+    PipelineTaskConsumer,
-    PipelineTaskLogger,
+    PipelineTaskLogsService,
-    {
-      provide: 'spawn',
-      useValue: spawn,
-    },
-    PipelineTaskFlushService,
   ],
   exports: [PipelineTasksService],
 })

src/pipeline-tasks/pipeline-tasks.resolver.spec.ts

@@ -1,6 +1,6 @@
 import { Test, TestingModule } from '@nestjs/testing';
-import { PipelineTaskLogger } from './pipeline-task.logger';
 import { PipelineTasksResolver } from './pipeline-tasks.resolver';
+import { PipelineTaskLogsService } from './pipeline-task-logs.service';
 import { PipelineTasksService } from './pipeline-tasks.service';
 
 describe('PipelineTasksResolver', () => {
@@ -15,7 +15,7 @@ describe('PipelineTasksResolver', () => {
           useValue: {},
         },
         {
-          provide: PipelineTaskLogger,
+          provide: PipelineTaskLogsService,
           useValue: {},
         },
       ],

src/pipeline-tasks/pipeline-tasks.resolver.ts

@@ -2,17 +2,16 @@ import { Resolver, Args, Mutation, Subscription, Query } from '@nestjs/graphql';
 import { PipelineTask } from './pipeline-task.entity';
 import { PipelineTasksService } from './pipeline-tasks.service';
 import { CreatePipelineTaskInput } from './dtos/create-pipeline-task.input';
+import { PipelineTaskLogMessage } from './models/pipeline-task-log-message.module';
 import { PipelineTaskLogArgs } from './dtos/pipeline-task-log.args';
+import { PipelineTaskLogsService } from './pipeline-task-logs.service';
 import { plainToClass } from 'class-transformer';
-import { PipelineTaskLogger } from './pipeline-task.logger';
-import { observableToAsyncIterable } from '@graphql-tools/utils';
-import { PipelineTaskEvent } from './models/pipeline-task-event';
 
 @Resolver()
 export class PipelineTasksResolver {
   constructor(
     private readonly service: PipelineTasksService,
-    private readonly taskLogger: PipelineTaskLogger,
+    private readonly logsService: PipelineTaskLogsService,
   ) {}
 
   @Mutation(() => PipelineTask)
@@ -20,17 +19,16 @@ export class PipelineTasksResolver {
     return await this.service.addTask(taskDto);
   }
 
-  @Subscription(() => PipelineTaskEvent, {
+  @Subscription(() => PipelineTaskLogMessage, {
     resolve: (value) => {
-      const data = plainToClass(PipelineTaskEvent, value);
+      const data = plainToClass(PipelineTaskLogMessage, value);
       return data;
     },
   })
-  async pipelineTaskEvent(@Args() args: PipelineTaskLogArgs) {
+  async pipelineTaskLog(@Args() args: PipelineTaskLogArgs) {
     const task = await this.service.findTaskById(args.taskId);
-    return observableToAsyncIterable<PipelineTaskEvent>(
+    const asyncIterator = this.logsService.watchLogs(task);
-      this.taskLogger.getMessage$(task.id),
+    return asyncIterator;
-    );
   }
 
   @Subscription(() => PipelineTask, {
@@ -39,7 +37,7 @@ export class PipelineTasksResolver {
     },
   })
   async pipelineTaskChanged(@Args('id') id: string) {
-    // return await this.service.watchTaskUpdated(id);
+    return await this.service.watchTaskUpdated(id);
   }
 
   @Query(() => [PipelineTask])
@@ -48,14 +46,7 @@ export class PipelineTasksResolver {
   }
 
   @Query(() => PipelineTask)
-  async pipelineTask(@Args('id') id: string) {
+  async findPipelineTask(@Args('id') id: string) {
     return await this.service.findTaskById(id);
   }
-
-  @Mutation(() => Boolean)
-  async stopPipelineTask(@Args('id') id: string) {
-    const task = await this.service.findTaskById(id);
-    await this.service.stopTask(task);
-    return true;
-  }
 }

src/pipeline-tasks/pipeline-tasks.service.spec.ts

@@ -2,19 +2,52 @@ import { Test, TestingModule } from '@nestjs/testing';
 import { PipelineTasksService } from './pipeline-tasks.service';
 import { getRepositoryToken } from '@nestjs/typeorm';
 import { PipelineTask } from './pipeline-task.entity';
+import { PIPELINE_TASK_QUEUE } from './pipeline-tasks.constants';
+import { getQueueToken } from '@nestjs/bull';
+import { RedisService } from 'nestjs-redis';
 import { Pipeline } from '../pipelines/pipeline.entity';
+import { EntityNotFoundError } from 'typeorm/error/EntityNotFoundError';
 import { Repository } from 'typeorm';
-import { AmqpConnection } from '@golevelup/nestjs-rabbitmq';
+import { Queue } from 'bull';
-import { PipelineTaskFlushService } from './pipeline-task-flush.service';
+import { LockFailedException } from '../commons/exceptions/lock-failed.exception';
-import { getLoggerToken, PinoLogger } from 'nestjs-pino';
+import { getPubSubToken } from '../commons/pub-sub/utils/token';
 
 describe('PipelineTasksService', () => {
   let service: PipelineTasksService;
   let module: TestingModule;
   let taskRepository: Repository<PipelineTask>;
   let pipelineRepository: Repository<Pipeline>;
+  const getBasePipeline = () =>
+    ({
+      id: 'test',
+      name: '测试流水线',
+      branch: 'master',
+      workUnitMetadata: {},
+      project: {
+        id: 'test-project',
+      },
+    } as Pipeline);
+  let redisClient;
+  let taskQueue: Queue;
+  const getTask = () =>
+    ({
+      pipelineId: 'test',
+      commit: 'test',
+      pipeline: { branch: 'master' },
+      units: [],
+    } as PipelineTask);
 
   beforeEach(async () => {
+    redisClient = (() => ({
+      set: jest.fn().mockImplementation(async () => 'OK'),
+      del: jest.fn().mockImplementation(async () => 'test'),
+      get: jest.fn().mockImplementation(async () => 'test'),
+      lpush: jest.fn().mockImplementation(async () => 1),
+      rpop: jest.fn().mockImplementation(async () => JSON.stringify(getTask())),
+    }))() as any;
+    taskQueue = (() => ({
+      add: jest.fn().mockImplementation(async () => null),
+    }))() as any;
     module = await Test.createTestingModule({
       providers: [
         PipelineTasksService,
@@ -27,16 +60,18 @@ describe('PipelineTasksService', () => {
           useValue: new Repository(),
         },
         {
-          provide: AmqpConnection,
+          provide: getQueueToken(PIPELINE_TASK_QUEUE),
-          useValue: {},
+          useValue: taskQueue,
         },
         {
-          provide: PipelineTaskFlushService,
+          provide: RedisService,
-          useValue: {},
+          useValue: {
+            getClient: jest.fn(() => redisClient),
+          },
         },
         {
-          provide: getLoggerToken(PipelineTasksService.name),
+          provide: getPubSubToken(),
-          useValue: new PinoLogger({}),
+          useValue: {},
         },
       ],
     }).compile();
@@ -57,43 +92,119 @@ describe('PipelineTasksService', () => {
     expect(service).toBeDefined();
   });
 
-  // describe('addTask', () => {
+  describe('addTask', () => {
-  //   beforeEach(() => {
+    beforeEach(() => {
-  //     jest
+      jest
-  //       .spyOn(pipelineRepository, 'findOneOrFail')
+        .spyOn(pipelineRepository, 'findOneOrFail')
-  //       .mockImplementation(async () => getBasePipeline());
+        .mockImplementation(async () => getBasePipeline());
-  //   });
+    });
-  //   it('pipeline not found', async () => {
+    it('pipeline not found', async () => {
-  //     jest.spyOn(taskRepository, 'findOneOrFail').mockImplementation(() => {
+      jest.spyOn(taskRepository, 'findOneOrFail').mockImplementation(() => {
-  //       throw new EntityNotFoundError(Pipeline, {});
+        throw new EntityNotFoundError(Pipeline, {});
-  //     });
+      });
-  //     await expect(
+      await expect(
-  //       service.addTask({ pipelineId: 'test', commit: 'test', units: [] }),
+        service.addTask({ pipelineId: 'test', commit: 'test', units: [] }),
-  //     ).rejects;
+      ).rejects;
-  //   });
+    });
-  //   it('create task on db', async () => {
+    it('create task on db', async () => {
-  //     const save = jest
+      const save = jest
-  //       .spyOn(taskRepository, 'save')
+        .spyOn(taskRepository, 'save')
-  //       .mockImplementation(async (data: any) => data);
+        .mockImplementation(async (data: any) => data);
-  //     const findOne = jest.spyOn(taskRepository, 'findOne');
+      const findOne = jest.spyOn(taskRepository, 'findOne');
-  //     await service.addTask({ pipelineId: 'test', commit: 'test', units: [] }),
+      jest
-  //       expect(save.mock.calls[0][0]).toMatchObject({
+        .spyOn(service, 'doNextTask')
-  //         pipelineId: 'test',
+        .mockImplementation(async () => undefined);
-  //         commit: 'test',
+      await service.addTask({ pipelineId: 'test', commit: 'test', units: [] }),
-  //         units: [],
+        expect(save.mock.calls[0][0]).toMatchObject({
-  //       });
+          pipelineId: 'test',
-  //     expect(findOne).toBeCalled();
+          commit: 'test',
-  //   });
+          units: [],
-  //   it('add task', async () => {
+        });
-  //     const lpush = jest.spyOn(redisClient, 'lpush');
+      expect(findOne).toBeCalled();
-  //     await service.addTask({ pipelineId: 'test', commit: 'test', units: [] });
+    });
-  //     expect(typeof lpush.mock.calls[0][1] === 'string').toBeTruthy();
+    it('add task', async () => {
-  //     expect(JSON.parse(lpush.mock.calls[0][1] as string)).toMatchObject({
+      const lpush = jest.spyOn(redisClient, 'lpush');
-  //       pipelineId: 'test',
+      const doNextTask = jest.spyOn(service, 'doNextTask');
-  //       commit: 'test',
+      jest
-  //       units: [],
+        .spyOn(service, 'doNextTask')
-  //       pipeline: getBasePipeline(),
+        .mockImplementation(async () => undefined);
-  //     });
+      await service.addTask({ pipelineId: 'test', commit: 'test', units: [] });
-  //   });
+      expect(typeof lpush.mock.calls[0][1] === 'string').toBeTruthy();
-  // });
+      expect(JSON.parse(lpush.mock.calls[0][1] as string)).toMatchObject({
+        pipelineId: 'test',
+        commit: 'test',
+        units: [],
+        pipeline: getBasePipeline(),
+      });
+      expect(doNextTask).toHaveBeenCalledWith(getBasePipeline());
+    });
+  });
+
+  describe('doNextTask', () => {
+    it('add task to queue', async () => {
+      let lckValue: string;
+      const set = jest
+        .spyOn(redisClient, 'set')
+        .mockImplementation(async (...args) => (lckValue = args[3] as string));
+      const get = jest
+        .spyOn(redisClient, 'get')
+        .mockImplementation(async () => lckValue);
+      const del = jest.spyOn(redisClient, 'del');
+      const rpop = jest.spyOn(redisClient, 'rpop');
+      const add = jest.spyOn(taskQueue, 'add');
+
+      await service.doNextTask(getBasePipeline());
+
+      expect(add).toHaveBeenCalledWith(getTask());
+      expect(set).toHaveBeenCalledTimes(1);
+      expect(rpop).toHaveBeenCalledTimes(1);
+      expect(get).toHaveBeenCalledTimes(1);
+      expect(del).toHaveBeenCalledTimes(1);
+    });
+    it('pipeline is busy', async () => {
+      let remainTimes = 3;
+
+      let lckValue: string;
+      const set = jest
+        .spyOn(redisClient, 'set')
+        .mockImplementation(async (...args) => {
+          if (remainTimes-- > 0) {
+            throw new Error();
+          } else {
+            lckValue = args[3] as string;
+          }
+        });
+      const get = jest
+        .spyOn(redisClient, 'get')
+        .mockImplementation(async () => lckValue);
+      const del = jest.spyOn(redisClient, 'del');
+      const rpop = jest.spyOn(redisClient, 'rpop');
+      const add = jest.spyOn(taskQueue, 'add');
+
+      await service.doNextTask(getBasePipeline());
+
+      expect(rpop).toHaveBeenCalledTimes(1);
+      expect(set).toHaveBeenCalledTimes(4);
+      expect(get).toHaveBeenCalledTimes(1);
+      expect(del).toHaveBeenCalledTimes(1);
+      expect(add).toHaveBeenCalledWith(getTask());
+    }, 10_000);
+    it('pipeline always busy and timeout', async () => {
+      const set = jest
+        .spyOn(redisClient, 'set')
+        .mockImplementation(async () => {
+          throw new Error();
+        });
+      const get = jest.spyOn(redisClient, 'get');
+      const del = jest.spyOn(redisClient, 'del');
+
+      await expect(
+        service.doNextTask(getBasePipeline()),
+      ).rejects.toBeInstanceOf(LockFailedException);
+
+      expect(set).toHaveBeenCalledTimes(5);
+      expect(get).toHaveBeenCalledTimes(0);
+      expect(del).toHaveBeenCalledTimes(0);
+    }, 15_000);
+  });
 });

src/pipeline-tasks/pipeline-tasks.service.ts

@@ -1,23 +1,20 @@
-import { BadRequestException, Injectable } from '@nestjs/common';
+import { ConflictException, Injectable } from '@nestjs/common';
 import { InjectRepository } from '@nestjs/typeorm';
 import { PipelineTask } from './pipeline-task.entity';
-import { Repository } from 'typeorm';
+import { In, Repository } from 'typeorm';
 import { CreatePipelineTaskInput } from './dtos/create-pipeline-task.input';
+import { RedisService } from 'nestjs-redis';
 import { Pipeline } from '../pipelines/pipeline.entity';
+import { InjectQueue } from '@nestjs/bull';
+import { PIPELINE_TASK_QUEUE } from './pipeline-tasks.constants';
+import { Queue } from 'bull';
+import { LockFailedException } from '../commons/exceptions/lock-failed.exception';
+import { TaskStatuses } from './enums/task-statuses.enum';
+import { isNil } from 'ramda';
 import debug from 'debug';
-import { AmqpConnection, RabbitRPC } from '@golevelup/nestjs-rabbitmq';
+import { InjectPubSub } from '../commons/pub-sub/decorators/inject-pub-sub.decorator';
-import {
+import { PubSub } from '../commons/pub-sub/pub-sub';
-  EXCHANGE_PIPELINE_TASK_TOPIC,
+import { observableToAsyncIterable } from '@graphql-tools/utils';
-  QUEUE_PIPELINE_TASK_DONE,
-  ROUTE_PIPELINE_TASK_DONE,
-} from './pipeline-tasks.constants';
-import { PipelineTaskFlushService } from './pipeline-task-flush.service';
-import { find, isNil, propEq } from 'ramda';
-import { PipelineTaskLogs } from './models/pipeline-task-logs.model';
-import { TaskStatuses, terminalTaskStatuses } from './enums/task-statuses.enum';
-import { InjectPinoLogger, PinoLogger } from 'nestjs-pino';
-import { getAppInstanceRouteKey } from '../commons/utils/rabbit-mq';
-import { ROUTE_PIPELINE_TASK_KILL } from './pipeline-tasks.constants';
 
 const log = debug('fennec:pipeline-tasks:service');
 
@@ -28,32 +25,42 @@ export class PipelineTasksService {
     private readonly repository: Repository<PipelineTask>,
     @InjectRepository(Pipeline)
     private readonly pipelineRepository: Repository<Pipeline>,
-    private readonly amqpConnection: AmqpConnection,
+    @InjectQueue(PIPELINE_TASK_QUEUE)
-    private readonly eventFlushService: PipelineTaskFlushService,
+    private readonly queue: Queue<PipelineTask>,
-    @InjectPinoLogger(PipelineTasksService.name)
+    private readonly redis: RedisService,
-    private readonly logger: PinoLogger,
+    @InjectPubSub()
+    private readonly pubSub: PubSub,
   ) {}
   async addTask(dto: CreatePipelineTaskInput) {
     const pipeline = await this.pipelineRepository.findOneOrFail({
       where: { id: dto.pipelineId },
       relations: ['project'],
     });
-    // const hasUnfinishedTask = await this.repository
+    const hasUnfinishedTask = await this.repository
-    //   .findOne({
+      .findOne({
-    //     pipelineId: dto.pipelineId,
+        pipelineId: dto.pipelineId,
-    //     commit: dto.commit,
+        commit: dto.commit,
-    //     status: In([TaskStatuses.pending, TaskStatuses.working]),
+        status: In([TaskStatuses.pending, TaskStatuses.working]),
-    //   })
+      })
-    //   .then((val) => !isNil(val));
+      .then((val) => !isNil(val));
-    // if (hasUnfinishedTask) {
+    if (hasUnfinishedTask) {
-    //   throw new ConflictException(
+      throw new ConflictException(
-    //     'There are the same tasks among the unfinished tasks!',
+        'There are the same tasks among the unfinished tasks!',
-    //   );
+      );
-    // }
+    }
     const task = await this.repository.save(this.repository.create(dto));
     task.pipeline = pipeline;
 
-    this.amqpConnection.publish('new-pipeline-task', 'mac', task);
+    const tasksKey = this.getRedisTokens(pipeline)[1];
+    const redis = this.redis.getClient();
+    await redis.lpush(tasksKey, JSON.stringify(task));
+    log(
+      'add task %s:%s-%s',
+      task.id,
+      task.pipeline.branch,
+      task.commit.slice(0, 6),
+    );
+    await this.doNextTask(pipeline);
     return task;
   }
 
@@ -66,93 +73,65 @@ export class PipelineTasksService {
   }
 
   async listTasksByCommitHash(hash: string) {
-    return await this.repository.find({
+    return await this.repository.find({ commit: hash });
-      where: { commit: hash },
+  }
-      order: { createdAt: 'DESC' },
+
-    });
+  async doNextTask(pipeline: Pipeline) {
+    const [lckKey, tasksKey] = this.getRedisTokens(pipeline);
+    const redis = this.redis.getClient();
+
+    log('doNextTask()');
+    const unLck = await new Promise<() => Promise<void>>(
+      async (resolve, reject) => {
+        const lckValue = Date.now().toString();
+        for (let i = 0; i < 5; i++) {
+          if (
+            await redis
+              .set(lckKey, 0, 'EX', lckValue, 'NX')
+              .then(() => true)
+              .catch(() => false)
+          ) {
+            resolve(async () => {
+              if ((await redis.get(lckKey)) === lckValue) {
+                await redis.del(lckKey);
+              }
+            });
+            return;
+          }
+          await new Promise((resolve) => setTimeout(resolve, 2000));
+        }
+        reject(new LockFailedException(lckKey));
+      },
+    );
+
+    const task = JSON.parse(
+      (await redis.rpop(tasksKey).finally(() => unLck())) ?? 'null',
+    );
+    if (task) {
+      log(
+        'add task (%s:%s-%s) to queue',
+        task.id,
+        task.pipeline.branch,
+        task.commit.slice(0, 6),
+      );
+      await this.queue.add(task);
+    } else {
+      log('task is empty');
+    }
+  }
+
+  async updateTask(task: PipelineTask) {
+    this.pubSub.publish(`pipeline-task:${task.id}`, task);
+    return await this.repository.save(task);
+  }
+
+  async watchTaskUpdated(id: string) {
+    return observableToAsyncIterable(
+      this.pubSub.message$(`pipeline-task:${id}`),
+    );
   }
 
   getRedisTokens(pipeline: Pipeline): [string, string] {
     return [`pipeline-${pipeline.id}:lck`, `pipeline-${pipeline.id}:tasks`];
   }
-
-  @RabbitRPC({
-    exchange: EXCHANGE_PIPELINE_TASK_TOPIC,
-    routingKey: ROUTE_PIPELINE_TASK_DONE,
-    queue: QUEUE_PIPELINE_TASK_DONE,
-    queueOptions: {
-      autoDelete: true,
-      durable: true,
-    },
-  })
-  async updateByEvent({ taskId, runOn }: { taskId: string; runOn: string }) {
-    try {
-      const [events, task] = await Promise.all([
-        this.eventFlushService.read(taskId),
-        this.findTaskById(taskId),
-      ]);
-      this.logger.info('[updateByEvent] start. taskId: %s', taskId);
-
-      for (const event of events) {
-        if (isNil(event.unit)) {
-          if (
-            event.status !== TaskStatuses.pending &&
-            task.status === TaskStatuses.pending
-          ) {
-            task.startedAt = event.emittedAt;
-          } else if (terminalTaskStatuses.includes(event.status)) {
-            task.endedAt = event.emittedAt;
-          }
-          task.status = event.status;
-        } else {
-          let l: PipelineTaskLogs = find<PipelineTaskLogs>(
-            propEq('unit', event.unit),
-            task.logs,
-          );
-
-          if (isNil(l)) {
-            l = {
-              unit: event.unit,
-              startedAt: event.emittedAt,
-              endedAt: null,
-              logs: event.message,
-              status: event.status,
-            };
-
-            task.logs.push(l);
-          } else {
-            l.logs += event.message;
-          }
-
-          if (terminalTaskStatuses.includes(event.status)) {
-            l.endedAt = event.emittedAt;
-          }
-          l.status = event.status;
-        }
-      }
-      task.runOn = runOn;
-      await this.repository.update({ id: taskId }, task);
-      this.logger.info('[updateByEvent] success. taskId: %s', taskId);
-      return task;
-    } catch (error) {
-      this.logger.error(
-        { error },
-        '[updateByEvent] failed. taskId: %s',
-        taskId,
-      );
-    }
-  }
-
-  async stopTask(task: PipelineTask) {
-    if (isNil(task.runOn)) {
-      throw new BadRequestException(
-        "the task have not running instance on database. field 'runOn' is nil",
-      );
-    }
-    await this.amqpConnection.request({
-      exchange: EXCHANGE_PIPELINE_TASK_TOPIC,
-      routingKey: getAppInstanceRouteKey(ROUTE_PIPELINE_TASK_KILL, task.runOn),
-      payload: task,
-    });
-  }
 }

src/pipelines/commit-logs.resolver.ts

@@ -1,4 +1,3 @@
-import { Query } from '@nestjs/graphql';
 import {
   Args,
   Parent,
@@ -7,36 +6,28 @@ import {
   Subscription,
 } from '@nestjs/graphql';
 import { PipelineTasksService } from '../pipeline-tasks/pipeline-tasks.service';
-import { Commit, LogFields } from '../repos/dtos/log-list.model';
+import { LogFields, LogList } from '../repos/dtos/log-list.model';
 import { PipelinesService } from './pipelines.service';
 
-@Resolver(() => Commit)
+@Resolver(() => LogFields)
 export class CommitLogsResolver {
   constructor(
     private readonly service: PipelinesService,
     private readonly taskServices: PipelineTasksService,
   ) {}
-  @Subscription(() => String, { resolve: (val) => val, nullable: true })
+  @Subscription(() => LogList, {
-  async syncCommits(
+    resolve: (value) => {
-    @Args('pipelineId', { type: () => String })
+      return value;
-    pipelineId: string,
+    },
-    @Args('appInstance', { type: () => String, nullable: true })
+  })
-    appInstance?: string,
+  async listLogsForPipeline(@Args('id', { type: () => String }) id: string) {
-  ) {
+    const job = await this.service.listLogsForPipeline(id);
-    const pipeline = await this.service.findOneWithProject(pipelineId);
-    const syncCommitsPromise = this.service.syncCommits(pipeline, appInstance);
     return (async function* () {
-      yield await syncCommitsPromise;
+      yield await job.finished();
     })();
   }
   @ResolveField()
   async tasks(@Parent() commit: LogFields) {
     return await this.taskServices.listTasksByCommitHash(commit.hash);
   }
-
-  @Query(() => [Commit], { nullable: true })
-  async commits(@Args('pipelineId', { type: () => String }) id: string) {
-    const pipeline = await this.service.findOneWithProject(id);
-    return await this.service.listCommits(pipeline);
-  }
 }

src/pipelines/dtos/update-pipeline.input.ts

@@ -1,9 +1,5 @@
-import { InputType, OmitType } from '@nestjs/graphql';
+import { InputType } from '@nestjs/graphql';
 import { CreatePipelineInput } from './create-pipeline.input';
 
 @InputType()
-export class UpdatePipelineInput extends OmitType(CreatePipelineInput, [
+export class UpdatePipelineInput extends CreatePipelineInput {}
-  'projectId',
-]) {
-  id: string;
-}

src/pipelines/pipelines.module.ts

@@ -3,24 +3,18 @@ import { PipelinesResolver } from './pipelines.resolver';
 import { PipelinesService } from './pipelines.service';
 import { TypeOrmModule } from '@nestjs/typeorm';
 import { Pipeline } from './pipeline.entity';
+import { BullModule } from '@nestjs/bull';
+import { LIST_LOGS_TASK } from '../repos/repos.constants';
 import { CommitLogsResolver } from './commit-logs.resolver';
 import { PipelineTasksModule } from '../pipeline-tasks/pipeline-tasks.module';
-import { ReposModule } from '../repos/repos.module';
-import { RabbitMQModule } from '@golevelup/nestjs-rabbitmq';
-import { ConfigModule, ConfigService } from '@nestjs/config';
 
 @Module({
   imports: [
     TypeOrmModule.forFeature([Pipeline]),
-    PipelineTasksModule,
+    BullModule.registerQueue({
-    RabbitMQModule.forRootAsync(RabbitMQModule, {
+      name: LIST_LOGS_TASK,
-      imports: [ConfigModule],
-      useFactory: (configService: ConfigService) => ({
-        uri: configService.get<string>('db.rabbitmq.uri'),
-        exchanges: [],
-      }),
-      inject: [ConfigService],
     }),
+    PipelineTasksModule,
   ],
   providers: [PipelinesResolver, PipelinesService, CommitLogsResolver],
 })

src/pipelines/pipelines.resolver.ts

@@ -9,29 +9,30 @@ import { ListPipelineArgs } from './dtos/list-pipelines.args';
 export class PipelinesResolver {
   constructor(private readonly service: PipelinesService) {}
   @Query(() => [Pipeline])
-  async pipelines(@Args() dto: ListPipelineArgs) {
+  async listPipelines(@Args() dto: ListPipelineArgs) {
     return await this.service.list(dto);
   }
 
   @Query(() => Pipeline)
-  async pipeline(@Args('id', { type: () => String }) id: string) {
+  async findPipeline(@Args('id', { type: () => String }) id: string) {
     return await this.service.findOne(id);
   }
 
   @Mutation(() => Pipeline)
   async createPipeline(
     @Args('pipeline', { type: () => CreatePipelineInput })
-    dto: CreatePipelineInput,
+    dto: UpdatePipelineInput,
   ) {
     return await this.service.create(dto);
   }
 
   @Mutation(() => Pipeline)
-  async updatePipeline(
+  async modifyPipeline(
-    @Args('pipeline', { type: () => UpdatePipelineInput })
+    @Args('id', { type: () => String }) id: string,
+    @Args('Pipeline', { type: () => UpdatePipelineInput })
     dto: UpdatePipelineInput,
   ) {
-    const tmp = await this.service.update(dto);
+    const tmp = await this.service.update(id, dto);
     console.log(tmp);
     return tmp;
   }

src/pipelines/pipelines.service.spec.ts

@@ -2,14 +2,18 @@ import { Test, TestingModule } from '@nestjs/testing';
 import { PipelinesService } from './pipelines.service';
 import { Pipeline } from './pipeline.entity';
 import { getRepositoryToken } from '@nestjs/typeorm';
+import { getQueueToken } from '@nestjs/bull';
+import { LIST_LOGS_TASK } from '../repos/repos.constants';
 import { Repository } from 'typeorm';
 import { Project } from '../projects/project.entity';
-import { AmqpConnection } from '@golevelup/nestjs-rabbitmq';
+import { Job, Queue } from 'bull';
+import { ListLogsOption } from '../repos/models/list-logs.options';
 
 describe('PipelinesService', () => {
   let service: PipelinesService;
   let repository: Repository<Pipeline>;
   let pipeline: Pipeline;
+  let queue: Queue<ListLogsOption>;
 
   beforeEach(async () => {
     pipeline = Object.assign(new Pipeline(), {
@@ -33,17 +37,33 @@ describe('PipelinesService', () => {
           },
         },
         {
-          provide: AmqpConnection,
+          provide: getQueueToken(LIST_LOGS_TASK),
-          useValue: {},
+          useValue: {
+            add: jest.fn().mockImplementation(() => ({ id: 1 } as Job)),
+          },
         },
       ],
     }).compile();
 
     service = module.get<PipelinesService>(PipelinesService);
     repository = module.get(getRepositoryToken(Pipeline));
+    queue = module.get(getQueueToken(LIST_LOGS_TASK));
   });
 
   it('should be defined', () => {
     expect(service).toBeDefined();
   });
+
+  describe('listLogsForPipeline', () => {
+    it('should send task to queue.', async () => {
+      const add = jest.spyOn(queue, 'add');
+      await expect(
+        service.listLogsForPipeline('test-pipeline'),
+      ).resolves.toEqual({ id: 1 });
+      expect(add).toBeCalledWith({
+        project: pipeline.project,
+        branch: pipeline.branch,
+      });
+    });
+  });
 });

src/pipelines/pipelines.service.ts

@@ -6,16 +6,10 @@ import { BaseDbService } from '../commons/services/base-db.service';
 import { CreatePipelineInput } from './dtos/create-pipeline.input';
 import { UpdatePipelineInput } from './dtos/update-pipeline.input';
 import { ListPipelineArgs } from './dtos/list-pipelines.args';
-import {
+import { InjectQueue } from '@nestjs/bull';
-  EXCHANGE_REPO,
+import { LIST_LOGS_TASK } from '../repos/repos.constants';
-  ROUTE_FETCH,
+import { Queue } from 'bull';
-  ROUTE_LIST_COMMITS,
+import { ListLogsOption } from '../repos/models/list-logs.options';
-} from '../repos/repos.constants';
-import { AmqpConnection } from '@golevelup/nestjs-rabbitmq';
-import { Commit } from '../repos/dtos/log-list.model';
-import { getAppInstanceRouteKey } from '../commons/utils/rabbit-mq';
-import { ApplicationException } from '../commons/exceptions/application.exception';
-import { plainToClass } from 'class-transformer';
 
 @Injectable()
 export class PipelinesService extends BaseDbService<Pipeline> {
@@ -23,7 +17,8 @@ export class PipelinesService extends BaseDbService<Pipeline> {
   constructor(
     @InjectRepository(Pipeline)
     readonly repository: Repository<Pipeline>,
-    private readonly amqpConnection: AmqpConnection,
+    @InjectQueue(LIST_LOGS_TASK)
+    private readonly listLogsQueue: Queue<ListLogsOption>,
   ) {
     super(repository);
   }
@@ -31,48 +26,30 @@ export class PipelinesService extends BaseDbService<Pipeline> {
     return this.repository.find(dto);
   }
 
-  async findOneWithProject(id: string) {
-    return await this.repository.findOne({
-      where: { id },
-      relations: ['project'],
-    });
-  }
-
   async create(dto: CreatePipelineInput) {
     await this.isDuplicateEntity(dto);
     return await this.repository.save(this.repository.create(dto));
   }
 
-  async update(dto: UpdatePipelineInput) {
+  async update(id: string, dto: UpdatePipelineInput) {
-    const old = await this.findOne(dto.id);
+    await this.isDuplicateEntityForUpdate(id, dto);
-    await this.isDuplicateEntityForUpdate(old, dto);
+    const old = await this.findOne(id);
     return await this.repository.save(this.repository.merge(old, dto));
   }
 
   async remove(id: string) {
     return (await this.repository.softDelete({ id })).affected;
   }
-  async syncCommits(pipeline: Pipeline, appInstance?: string) {
+
-    return await this.amqpConnection.request<string | null>({
+  async listLogsForPipeline(id: string) {
-      exchange: EXCHANGE_REPO,
+    const pipeline = await this.repository.findOneOrFail({
-      routingKey: getAppInstanceRouteKey(ROUTE_FETCH, appInstance),
+      where: { id },
-      payload: pipeline,
+      relations: ['project'],
-      timeout: 120_000,
     });
-  }
+    const job = await this.listLogsQueue.add({
-  async listCommits(pipeline: Pipeline) {
+      project: pipeline.project,
-    return await this.amqpConnection
+      branch: pipeline.branch,
-      .request<[Error, Commit[]]>({
+    });
-        exchange: EXCHANGE_REPO,
+    return job;
-        routingKey: ROUTE_LIST_COMMITS,
-        payload: pipeline,
-        timeout: 30_000,
-      })
-      .then(([error, list]) => {
-        if (error) {
-          throw new ApplicationException(error);
-        }
-        return plainToClass(Commit, list);
-      });
   }
 }

src/projects/dtos/create-project.input.ts

@@ -21,10 +21,7 @@ export class CreateProjectInput {
   comment: string;
 
   @Matches(
-    /^(?:ssh:\/\/)?(?:[\w\d-_]+@)?(?:[\w\d-_]+\.)*\w{2,10}(?::\d{1,5})?(?:\/[\w\d-_.]+)*/,
+    /^(?:ssh:\/\/)?(?:[\w\d-_]+@)(?:[\w\d-_]+\.)*\w{2,10}(?::\d{1,5})?(?:\/[\w\d-_.]+)*/,
-    {
-      message: 'wrong ssh url',
-    },
   )
   @MaxLength(256)
   sshUrl: string;

src/projects/dtos/update-project.input.ts

@@ -1,9 +1,5 @@
 import { InputType } from '@nestjs/graphql';
-import { IsUUID } from 'class-validator';
 import { CreateProjectInput } from './create-project.input';
 
 @InputType()
-export class UpdateProjectInput extends CreateProjectInput {
+export class UpdateProjectInput extends CreateProjectInput {}
-  @IsUUID()
-  id: string;
-}

src/projects/projects.constants.ts

@@ -1,3 +0,0 @@
-export const EXCHANGE_PROJECT_TOPIC = 'project.topic';
-export const EXCHANGE_PROJECT_FANOUT = 'project.fanout';
-export const ROUTE_PROJECT_CHANGE = 'project-change';

src/projects/projects.module.ts

@@ -3,31 +3,9 @@ import { ProjectsService } from './projects.service';
 import { ProjectsResolver } from './projects.resolver';
 import { TypeOrmModule } from '@nestjs/typeorm';
 import { Project } from './project.entity';
-import { RabbitMQModule } from '@golevelup/nestjs-rabbitmq';
-import { ConfigModule, ConfigService } from '@nestjs/config';
-import { EXCHANGE_PROJECT_FANOUT } from './projects.constants';
 
 @Module({
-  imports: [
+  imports: [TypeOrmModule.forFeature([Project])],
-    TypeOrmModule.forFeature([Project]),
-    RabbitMQModule.forRootAsync(RabbitMQModule, {
-      imports: [ConfigModule],
-      useFactory: (configService: ConfigService) => ({
-        uri: configService.get<string>('db.rabbitmq.uri'),
-        exchanges: [
-          {
-            name: EXCHANGE_PROJECT_FANOUT,
-            type: 'fanout',
-            options: {
-              durable: false,
-              autoDelete: true,
-            },
-          },
-        ],
-      }),
-      inject: [ConfigService],
-    }),
-  ],
   providers: [ProjectsService, ProjectsResolver],
   exports: [ProjectsService],
 })

src/projects/projects.resolver.ts

@@ -8,12 +8,12 @@ import { ProjectsService } from './projects.service';
 export class ProjectsResolver {
   constructor(private readonly service: ProjectsService) {}
   @Query(() => [Project])
-  async projects() {
+  async findProjects() {
     return await this.service.list();
   }
 
   @Query(() => Project)
-  async project(@Args('id', { type: () => String }) id: string) {
+  async findProject(@Args('id', { type: () => String }) id: string) {
     return await this.service.findOne(id);
   }
 
@@ -26,17 +26,18 @@ export class ProjectsResolver {
   }
 
   @Mutation(() => Project)
-  async updateProject(
+  async modifyProject(
+    @Args('id', { type: () => String }) id: string,
     @Args('project', { type: () => UpdateProjectInput })
     dto: UpdateProjectInput,
   ) {
-    const tmp = await this.service.update(dto);
+    const tmp = await this.service.update(id, dto);
     console.log(tmp);
     return tmp;
   }
 
   @Mutation(() => Number)
-  async removeProject(@Args('id', { type: () => String }) id: string) {
+  async deleteProject(@Args('id', { type: () => String }) id: string) {
     return await this.service.remove(id);
   }
 }

src/projects/projects.service.spec.ts

@@ -2,7 +2,6 @@ import { Test, TestingModule } from '@nestjs/testing';
 import { ProjectsService } from './projects.service';
 import { getRepositoryToken } from '@nestjs/typeorm';
 import { Project } from './project.entity';
-import { AmqpConnection } from '@golevelup/nestjs-rabbitmq';
 
 describe('ProjectsService', () => {
   let service: ProjectsService;
@@ -15,10 +14,6 @@ describe('ProjectsService', () => {
           provide: getRepositoryToken(Project),
           useValue: {},
         },
-        {
-          provide: AmqpConnection,
-          useValue: {},
-        },
       ],
     }).compile();
 

src/projects/projects.service.ts

@@ -5,11 +5,6 @@ import { Repository } from 'typeorm';
 import { CreateProjectInput } from './dtos/create-project.input';
 import { Project } from './project.entity';
 import { UpdateProjectInput } from './dtos/update-project.input';
-import { AmqpConnection } from '@golevelup/nestjs-rabbitmq';
-import {
-  EXCHANGE_PROJECT_FANOUT,
-  ROUTE_PROJECT_CHANGE,
-} from './projects.constants';
 
 @Injectable()
 export class ProjectsService extends BaseDbService<Project> {
@@ -17,7 +12,6 @@ export class ProjectsService extends BaseDbService<Project> {
   constructor(
     @InjectRepository(Project)
     readonly repository: Repository<Project>,
-    private readonly amqpConnection: AmqpConnection,
   ) {
     super(repository);
   }
@@ -31,15 +25,10 @@ export class ProjectsService extends BaseDbService<Project> {
     return await this.repository.save(this.repository.create(dto));
   }
 
-  async update(dto: UpdateProjectInput) {
+  async update(id: string, dto: UpdateProjectInput) {
-    await this.isDuplicateEntityForUpdate(dto.id, dto);
+    await this.isDuplicateEntityForUpdate(id, dto);
-    const old = await this.findOne(dto.id);
+    const old = await this.findOne(id);
-    const project = await this.repository.save(this.repository.merge(old, dto));
+    return await this.repository.save(this.repository.merge(old, dto));
-    this.amqpConnection.publish(EXCHANGE_PROJECT_FANOUT, ROUTE_PROJECT_CHANGE, [
-      project,
-      old,
-    ]);
-    return project;
   }
 
   async remove(id: string) {

src/repos/dtos/log-list.model.ts

@@ -1,20 +1,7 @@
 import { ObjectType, Field } from '@nestjs/graphql';
-import { Type } from 'class-transformer';
 import { LogResult, DefaultLogFields } from 'simple-git';
 import { PipelineTask } from '../../pipeline-tasks/pipeline-task.entity';
 
-@ObjectType()
-export class Commit {
-  hash: string;
-  @Type(() => Date)
-  date: Date;
-  message: string;
-  refs: string;
-  body: string;
-  author_name: string;
-  author_email: string;
-  tasks: PipelineTask[];
-}
 @ObjectType()
 export class LogFields {
   hash: string;

src/repos/list-logs.consumer.ts

@@ -0,0 +1,14 @@
+import { ReposService } from './repos.service';
+import { Processor, Process } from '@nestjs/bull';
+import { Job } from 'bull';
+import { ListLogsOption } from './models/list-logs.options';
+import { LIST_LOGS_TASK } from './repos.constants';
+@Processor(LIST_LOGS_TASK)
+export class ListLogsConsumer {
+  constructor(private readonly service: ReposService) {}
+  @Process()
+  async listLogs(job: Job<ListLogsOption>) {
+    const logs = await this.service.listLogs(job.data);
+    return logs;
+  }
+}

src/repos/repos.constants.ts

@@ -1,6 +1,3 @@
-export const EXCHANGE_REPO = 'fennec.repo';
+export const LIST_LOGS_TASK = 'LIST_LOGS_TASK';
-export const ROUTE_FETCH = 'fetch';
+export const LIST_LOGS_PUB_SUB = 'LIST_LOGS_PUB_SUB';
-export const ROUTE_LIST_COMMITS = 'list-commits';
+export const LIST_LOGS_DONE = 'LIST_LOGS_DONE';
-export const QUEUE_LIST_COMMITS = 'list-commits';
-export const QUEUE_FETCH = 'repo-fetch';
-export const QUEUE_REFRESH_REPO = 'refresh-repo';

src/repos/repos.module.ts

@@ -3,37 +3,23 @@ import { TypeOrmModule } from '@nestjs/typeorm';
 import { Project } from '../projects/project.entity';
 import { ReposResolver } from './repos.resolver';
 import { ReposService } from './repos.service';
-import { ConfigModule, ConfigService } from '@nestjs/config';
+import { ConfigModule } from '@nestjs/config';
 import { ProjectsModule } from '../projects/projects.module';
-import { EXCHANGE_REPO } from './repos.constants';
+import { BullModule } from '@nestjs/bull';
-import { RabbitMQModule } from '@golevelup/nestjs-rabbitmq';
+import { LIST_LOGS_TASK, LIST_LOGS_PUB_SUB } from './repos.constants';
-import { CommonsModule } from '../commons/commons.module';
+import { PubSub } from 'graphql-subscriptions';
+import { ListLogsConsumer } from './list-logs.consumer';
 
 @Module({
   imports: [
     TypeOrmModule.forFeature([Project]),
     ConfigModule,
     ProjectsModule,
-    CommonsModule,
+    BullModule.registerQueue({
-    RabbitMQModule.forRootAsync(RabbitMQModule, {
+      name: LIST_LOGS_TASK,
-      imports: [ConfigModule],
-      useFactory: (configService: ConfigService) => ({
-        uri: configService.get<string>('db.rabbitmq.uri'),
-        exchanges: [
-          {
-            name: EXCHANGE_REPO,
-            type: 'topic',
-            options: {
-              durable: true,
-              autoDelete: true,
-            },
-          },
-        ],
-      }),
-      inject: [ConfigService],
     }),
   ],
-  providers: [ReposResolver, ReposService],
+  providers: [ReposResolver, ReposService, ListLogsConsumer],
   exports: [ReposService],
 })
 export class ReposModule {}

src/repos/repos.service.spec.ts

@@ -9,9 +9,6 @@ import configuration from '../commons/config/configuration';
 import { PipelineTask } from '../pipeline-tasks/pipeline-task.entity';
 import { join } from 'path';
 import { readFile } from 'fs/promises';
-import { getLoggerToken, PinoLogger } from 'nestjs-pino';
-import { Nack } from '@golevelup/nestjs-rabbitmq';
-import { getInstanceName } from '../commons/utils/rabbit-mq';
 
 const getTest1Project = () =>
   ({
@@ -48,10 +45,6 @@ describe('ReposService', () => {
           provide: getRepositoryToken(Project),
           useFactory: repositoryMockFactory,
         },
-        {
-          provide: getLoggerToken(ReposService.name),
-          useValue: new PinoLogger({}),
-        },
       ],
     }).compile();
 
@@ -146,57 +139,4 @@ describe('ReposService', () => {
       );
     });
   });
-
-  describe('fetch', () => {
-    it('success', async () => {
-      const project = new Project();
-      const pipeline = new Pipeline();
-      pipeline.branch = 'test';
-      const fetch = jest.fn((_: any) => Promise.resolve());
-      pipeline.project = project;
-      const getGit = jest.spyOn(service, 'getGit').mockImplementation(() =>
-        Promise.resolve({
-          fetch,
-        } as any),
-      );
-      await expect(service.fetch(pipeline)).resolves.toEqual(getInstanceName());
-      expect(getGit).toBeCalledTimes(1);
-      expect(getGit.mock.calls[0]?.[0]).toEqual(project);
-      expect(fetch).toBeCalledTimes(1);
-      expect(fetch.mock.calls[0]?.[0]).toMatchObject([
-        'origin',
-        'test',
-        '--depth=100',
-      ]);
-    });
-    it('failed a', async () => {
-      const project = new Project();
-      const pipeline = new Pipeline();
-      pipeline.branch = 'test';
-      const fetch = jest.fn((_: any) => Promise.resolve());
-      pipeline.project = project;
-      const getGit = jest
-        .spyOn(service, 'getGit')
-        .mockImplementation(() => Promise.reject('error'));
-      await expect(service.fetch(pipeline)).resolves.toMatchObject(new Nack());
-      expect(getGit).toBeCalledTimes(1);
-      expect(getGit.mock.calls[0]?.[0]).toEqual(project);
-      expect(fetch).toBeCalledTimes(0);
-    });
-    it('failed b', async () => {
-      const project = new Project();
-      const pipeline = new Pipeline();
-      pipeline.branch = 'test';
-      const fetch = jest.fn((_: any) => Promise.reject('error'));
-      pipeline.project = project;
-      const getGit = jest.spyOn(service, 'getGit').mockImplementation(() =>
-        Promise.resolve({
-          fetch,
-        } as any),
-      );
-      await expect(service.fetch(pipeline)).resolves.toMatchObject(new Nack());
-      expect(getGit).toBeCalledTimes(1);
-      expect(fetch).toBeCalledTimes(1);
-    });
-  });
 });

src/repos/repos.service.ts

@@ -1,4 +1,5 @@
 import { ListLogsOption } from './models/list-logs.options';
+import { Pipeline } from './../pipelines/pipeline.entity';
 import { PipelineTask } from './../pipeline-tasks/pipeline-task.entity';
 import { Injectable, NotFoundException } from '@nestjs/common';
 import { InjectRepository } from '@nestjs/typeorm';
@@ -9,31 +10,8 @@ import { gitP } from 'simple-git';
 import { Repository } from 'typeorm';
 import { Project } from '../projects/project.entity';
 import { ListBranchesArgs } from './dtos/list-branches.args';
+import { ListLogsArgs } from './dtos/list-logs.args';
 import { ConfigService } from '@nestjs/config';
-import { Commit } from './dtos/log-list.model';
-import { Nack, RabbitRPC, RabbitSubscribe } from '@golevelup/nestjs-rabbitmq';
-import { Pipeline } from '../pipelines/pipeline.entity';
-import { InjectPinoLogger, PinoLogger } from 'nestjs-pino';
-import {
-  EXCHANGE_REPO,
-  QUEUE_FETCH,
-  QUEUE_LIST_COMMITS,
-  QUEUE_REFRESH_REPO,
-  ROUTE_FETCH,
-  ROUTE_LIST_COMMITS,
-} from './repos.constants';
-import { getSelfInstanceQueueKey } from '../commons/utils/rabbit-mq';
-import {
-  getInstanceName,
-  getSelfInstanceRouteKey,
-} from '../commons/utils/rabbit-mq';
-import { ApplicationException } from '../commons/exceptions/application.exception';
-import {
-  EXCHANGE_PROJECT_FANOUT,
-  ROUTE_PROJECT_CHANGE,
-} from '../projects/projects.constants';
-import { RedisMutexService } from '../commons/redis-mutex/redis-mutex.service';
-import { rm } from 'fs/promises';
 
 const DEFAULT_REMOTE_NAME = 'origin';
 const INFO_PATH = '@info';
@@ -43,9 +21,6 @@ export class ReposService {
     @InjectRepository(Project)
     private readonly projectRepository: Repository<Project>,
     private readonly configService: ConfigService,
-    @InjectPinoLogger(ReposService.name)
-    private readonly logger: PinoLogger,
-    private readonly redisMutexService: RedisMutexService,
   ) {}
 
   getWorkspaceRoot(project: Project): string {
@@ -56,11 +31,7 @@ export class ReposService {
     );
   }
 
-  async getGit(
+  async getGit(project: Project, workspaceRoot?: string) {
-    project: Project,
-    workspaceRoot?: string,
-    { fetch = true } = {},
-  ) {
     if (!workspaceRoot) {
       workspaceRoot = this.getWorkspaceRoot(project);
     }
@@ -73,9 +44,7 @@ export class ReposService {
       await git.init();
       await git.addRemote(DEFAULT_REMOTE_NAME, project.sshUrl);
     }
-    if (fetch) {
+    await git.fetch();
-      await git.fetch();
-    }
     return git;
   }
 
@@ -120,113 +89,4 @@ export class ReposService {
       encodeURIComponent(`${task.pipeline.name}-${task.commit}`),
     );
   }
-
-  async checkout4Task(task: PipelineTask): Promise<string> {
-    const path = this.getWorkspaceRootByTask(task);
-    await this.checkout(task, path);
-    return path;
-  }
-
-  @RabbitRPC({
-    exchange: EXCHANGE_REPO,
-    routingKey: [
-      ROUTE_LIST_COMMITS,
-      getSelfInstanceRouteKey(ROUTE_LIST_COMMITS),
-    ],
-    queue: getSelfInstanceQueueKey(QUEUE_LIST_COMMITS),
-    queueOptions: {
-      autoDelete: true,
-    },
-  })
-  async listCommits(pipeline: Pipeline): Promise<[Error, Commit[]?]> {
-    const git = await this.getGit(pipeline.project, undefined, {
-      fetch: false,
-    });
-    try {
-      const data = await git.log([
-        '-100',
-        '--branches',
-        `remotes/origin/${pipeline.branch}`,
-        '--',
-      ]);
-      return [
-        null,
-        data.all.map(
-          (it) =>
-            ({
-              ...it,
-              date: new Date(it.date),
-            } as Commit),
-        ),
-      ];
-    } catch (error) {
-      this.logger.error(
-        { error, pipeline },
-        '[listCommits] %s',
-        error?.message,
-      );
-      return [new ApplicationException(error)];
-    }
-  }
-
-  @RabbitRPC({
-    exchange: EXCHANGE_REPO,
-    routingKey: [ROUTE_FETCH, getSelfInstanceRouteKey(ROUTE_FETCH)],
-    queue: getSelfInstanceQueueKey(QUEUE_FETCH),
-    queueOptions: {
-      autoDelete: true,
-    },
-  })
-  async fetch(pipeline: Pipeline): Promise<string | null | Nack> {
-    const unlock = await this.redisMutexService.lock(
-      `repo-project-${pipeline.projectId}`,
-    );
-    try {
-      const git = await this.getGit(pipeline.project, undefined, {
-        fetch: false,
-      });
-      await git.fetch(['origin', pipeline.branch, '--depth=100']);
-      return getInstanceName();
-    } catch (error) {
-      this.logger.error({ error, pipeline }, '[fetch] %s', error?.message);
-      return new Nack();
-    } finally {
-      await unlock();
-    }
-  }
-
-  @RabbitSubscribe({
-    exchange: EXCHANGE_PROJECT_FANOUT,
-    routingKey: ROUTE_PROJECT_CHANGE,
-    queue: QUEUE_REFRESH_REPO,
-    queueOptions: {
-      autoDelete: true,
-      durable: true,
-    },
-  })
-  async refreshRepo([project]: [Project]) {
-    this.logger.info({ project }, '[refreshRepo] start');
-    const unlock = await this.redisMutexService.lock(
-      `repo-project-${project.id}`,
-      {
-        timeout: null,
-      },
-    );
-    try {
-      const path = join(
-        this.configService.get<string>('workspaces.root'),
-        encodeURIComponent(project.name),
-      );
-      await rm(path, { recursive: true });
-      this.logger.info({ project }, '[refreshRepo] success');
-    } catch (error) {
-      this.logger.error(
-        { project, error },
-        '[refreshRepo] failed. $s',
-        error.message,
-      );
-    } finally {
-      await unlock();
-    }
-  }
 }