Skip to content

Commit 84ba9b9

Browse files
#RI-3971 - add one node recommendations
1 parent 9588cc1 commit 84ba9b9

File tree

2 files changed

+13
-2
lines changed

2 files changed

+13
-2
lines changed

redisinsight/api/src/constants/recommendations.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,3 +19,11 @@ export const RECOMMENDATION_NAMES = Object.freeze({
1919
SEARCH_INDEXES: 'searchIndexes',
2020
DANGEROUS_COMMANDS: 'dangerousCommands',
2121
});
22+
23+
export const ONE_NODE_RECOMMENDATIONS = [
24+
RECOMMENDATION_NAMES.LUA_SCRIPT,
25+
RECOMMENDATION_NAMES.DANGEROUS_COMMANDS,
26+
RECOMMENDATION_NAMES.AVOID_LOGICAL_DATABASES,
27+
RECOMMENDATION_NAMES.RTS,
28+
RECOMMENDATION_NAMES.REDIS_VERSION,
29+
];

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

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import { HttpException, Injectable, Logger } from '@nestjs/common';
22
import { isNull, flatten, concat } from 'lodash';
33
import { RecommendationService } from 'src/modules/recommendation/recommendation.service';
44
import { catchAclError } from 'src/utils';
5+
import { ONE_NODE_RECOMMENDATIONS } from 'src/constants';
56
import { DatabaseAnalyzer } from 'src/modules/database-analysis/providers/database-analyzer';
67
import { plainToClass } from 'class-transformer';
78
import { DatabaseAnalysis, ShortDatabaseAnalysis } from 'src/modules/database-analysis/models';
@@ -55,7 +56,7 @@ export class DatabaseAnalysisService {
5556

5657
let recommendationToExclude = [];
5758

58-
const recommendations = await scanResults.reduce(async (previousPromise, nodeResult) => {
59+
const recommendations = await scanResults.reduce(async (previousPromise, nodeResult, idx) => {
5960
const jobsArray = await previousPromise;
6061
const nodeRecommendations = await this.recommendationService.getRecommendations({
6162
client: nodeResult.client,
@@ -64,7 +65,9 @@ export class DatabaseAnalysisService {
6465
globalClient: client,
6566
exclude: recommendationToExclude,
6667
});
67-
// recommendationToExclude = concat(recommendationToExclude, [RECOMMENDATION_NAMES.RTS]);
68+
if (idx === 0) {
69+
recommendationToExclude = concat(recommendationToExclude, ONE_NODE_RECOMMENDATIONS);
70+
}
6871
const foundedRecommendations = nodeRecommendations.filter((recommendation) => !isNull(recommendation));
6972
const foundedRecommendationNames = foundedRecommendations.map(({ name }) => name);
7073
recommendationToExclude = concat(recommendationToExclude, foundedRecommendationNames);

0 commit comments

Comments
 (0)