DOP-4599: Deploy-all slack button deploys all active versions of repos

.github/workflows/deploy-stg-ecs.yml

@@ -3,6 +3,7 @@ on:
     branches:
       - "main"
       - "integration"
+      - "DOP-4599"
 concurrency:
   group: environment-stg-${{ github.ref }}
   cancel-in-progress: true
@@ -16,7 +17,7 @@ jobs:
         with:
           node-version: '18.x'
       - name: Install Serverless Framework
-        run: npm install -g serverless
+        run: npm install -g serverless@3
       - name: Serverless AWS authentication
         run: sls config credentials --provider aws --key ${{ secrets.AWS_ACCESS_KEY_ID }} --secret ${{ secrets.AWS_SECRET_ACCESS_KEY }}
       - name: Configure AWS credentials

api/controllers/v1/slack.ts

@@ -15,6 +15,8 @@ import {
 } from '../../handlers/slack';
 import { DocsetsRepository } from '../../../src/repositories/docsetsRepository';
 import { Payload } from '../../../src/entities/job';
+import { ProjectsRepository } from '../../../src/repositories/projectsRepository';
+import { DOCS_METADATA } from '../../../src/constants';
 
 export const DisplayRepoOptions = async (event: APIGatewayEvent): Promise<APIGatewayProxyResult> => {
   const consoleLogger = new ConsoleLogger();
@@ -34,21 +36,34 @@ export const DisplayRepoOptions = async (event: APIGatewayEvent): Promise<APIGat
   const client = new mongodb.MongoClient(c.get('dbUrl'));
   await client.connect();
   const db = client.db(process.env.DB_NAME);
+  const projectsRepository = new ProjectsRepository(client.db(DOCS_METADATA), c, consoleLogger);
   const repoEntitlementRepository = new RepoEntitlementsRepository(db, c, consoleLogger);
   const repoBranchesRepository = new RepoBranchesRepository(db, c, consoleLogger);
   const key_val = getQSString(event.body);
-  const entitlement = await repoEntitlementRepository.getRepoEntitlementsBySlackUserId(key_val['user_id']);
-  if (!isUserEntitled(entitlement) || isRestrictedToDeploy(key_val['user_id'])) {
-    const { restrictedProdDeploy } = c.get<any>('prodDeploy');
-    const response = restrictedProdDeploy
-      ? 'Production freeze in place - please notify DOP if seeing this past 3/26'
-      : 'User is not entitled!';
-    return prepResponse(401, 'text/plain', response);
-  }
 
   const isAdmin = await repoEntitlementRepository.getIsAdmin(key_val['user_id']);
+  let entitledRepos: any[] = [];
+  //if user has admin permissions, they can deploy all repo branches
+  if (isAdmin) {
+    const repos = await repoBranchesRepository.getProdDeployableRepoBranches();
+    for (const repo of repos) {
+      const projectEntry = await projectsRepository.getProjectEntry(repo.project);
+      const repoOwner = projectEntry?.github?.organization;
+      if (repoOwner) entitledRepos.push(`${repoOwner}/${repo.repoName}`);
+    }
+  } else {
+    const entitlements = await repoEntitlementRepository.getRepoEntitlementsBySlackUserId(key_val['user_id']);
+    if (!isUserEntitled(entitlements) || isRestrictedToDeploy(key_val['user_id'])) {
+      const { restrictedProdDeploy } = c.get<any>('prodDeploy');
+      const response = restrictedProdDeploy
+        ? 'Production freeze in place - please notify DOP if seeing this past 3/26'
+        : 'User is not entitled!';
+      return prepResponse(401, 'text/plain', response);
+    }
+    entitledRepos = entitlements.repos;
+  }
 
-  const entitledBranches = await buildEntitledGroupsList(entitlement, repoBranchesRepository);
+  const entitledBranches = await buildEntitledGroupsList(entitledRepos, repoBranchesRepository);
   const resp = await slackConnector.displayRepoOptions(entitledBranches, key_val['trigger_id'], isAdmin);
   if (resp?.status == 200 && resp?.data?.ok) {
     return {
@@ -65,8 +80,9 @@ export const DisplayRepoOptions = async (event: APIGatewayEvent): Promise<APIGat
 async function deployRepo(deployable: Array<any>, logger: ILogger, jobRepository: JobRepository, jobQueueUrl) {
   try {
     await jobRepository.insertBulkJobs(deployable, jobQueueUrl);
+    console.error('testing console error logging');
   } catch (err) {
-    logger.error('deployRepo', err);
+    console.error('Deploy repo error');
   }
 }
 
@@ -86,25 +102,17 @@ export const getDeployableJobs = async (
   const deployable = [];
 
   for (let i = 0; i < values?.repo_option?.length; i++) {
-    let jobTitle: string, repoOwner: string, repoName: string, branchName: string, directory: string | undefined;
-    if (values.deploy_option == 'deploy_all') {
-      repoOwner = 'mongodb';
-      branchName = 'master';
-      repoName = values.repo_option[i].repoName;
-      jobTitle = `Slack deploy: ${repoOwner}/${repoName}/${branchName}, by ${entitlement.github_username}`;
+    let repoOwner: string, repoName: string, branchName: string, directory: string | undefined;
+    const splitValues = values.repo_option[i].value.split('/');
+    const jobTitle = `Slack deploy: ${values.repo_option[i].value}, by ${entitlement.github_username}`;
+    if (splitValues.length === 3) {
+      // e.g. mongodb/docs-realm/master => (owner/repo/branch)
+      [repoOwner, repoName, branchName] = splitValues;
+    } else if (splitValues.length === 4 && process.env.FEATURE_FLAG_MONOREPO_PATH === 'true') {
+      // e.g. 10gen/docs-monorepo/cloud-docs/master => (owner/monorepo/repoDirectory/branch)
+      [repoOwner, repoName, directory, branchName] = splitValues;
     } else {
-      const splitValues = values.repo_option[i].value.split('/');
-      jobTitle = `Slack deploy: ${values.repo_option[i].value}, by ${entitlement.github_username}`;
-
-      if (splitValues.length === 3) {
-        // e.g. mongodb/docs-realm/master => (owner/repo/branch)
-        [repoOwner, repoName, branchName] = splitValues;
-      } else if (splitValues.length === 4 && process.env.FEATURE_FLAG_MONOREPO_PATH === 'true') {
-        // e.g. 10gen/docs-monorepo/cloud-docs/master => (owner/monorepo/repoDirectory/branch)
-        [repoOwner, repoName, directory, branchName] = splitValues;
-      } else {
-        throw Error('Selected entitlement value is configured incorrectly. Check user entitlements!');
-      }
+      throw Error('Selected entitlement value is configured incorrectly. Check user entitlements!');
     }
 
     const hashOption = values?.hash_option ?? null;
@@ -115,11 +123,12 @@ export const getDeployableJobs = async (
     const non_versioned = repoInfo.branches.length === 1;
 
     const branchObject = await repoBranchesRepository.getRepoBranchAliases(repoName, branchName, repoInfo.project);
-    if (!branchObject?.aliasObject) continue;
+    if (branchObject.status == 'failure' || !branchObject?.aliasObject)
+      return prepResponse(401, 'text/plain', 'Branch not found in repos branches repository');
 
     const publishOriginalBranchName: boolean = branchObject.aliasObject.publishOriginalBranchName;
     const aliases: string[] | null = branchObject.aliasObject.urlAliases;
-    let urlSlug: string = branchObject.aliasObject.urlSlug; // string or null, string must match value in urlAliases or gitBranchName
+    let urlSlug: string = branchObject?.aliasObject.urlSlug; // string or null, string must match value in urlAliases or gitBranchName
     const isStableBranch = !!branchObject.aliasObject.isStableBranch; // bool or Falsey, add strong typing
 
     if (!urlSlug || !urlSlug.trim()) {
@@ -142,7 +151,7 @@ export const getDeployableJobs = async (
       directory
     );
 
-    if (!aliases || aliases.length === 0) {
+    if (!aliases || !aliases.length) {
       if (non_versioned) {
         newPayload.urlSlug = '';
       }
@@ -214,16 +223,24 @@ export const DeployRepo = async (event: any = {}): Promise<any> => {
 
   let values = [];
   const isAdmin = await repoEntitlementRepository.getIsAdmin(parsed.user.id);
+  const optionGroups = parsed.view.blocks[0]?.element?.option_groups;
   try {
-    values = await slackConnector.parseSelection(stateValues, isAdmin, repoBranchesRepository);
+    values = await slackConnector.parseSelection(stateValues, isAdmin, optionGroups);
   } catch (e) {
-    console.log(`Error parsing selection: ${e}`);
     return prepResponse(401, 'text/plain', e);
   }
-  const deployable = await getDeployableJobs(values, entitlement, repoBranchesRepository, docsetsRepository);
-
+  let deployable;
+  try {
+    deployable = await getDeployableJobs(values, entitlement, repoBranchesRepository, docsetsRepository);
+  } catch (e) {
+    return prepResponse(401, 'text/plain', `${e} error within get deployable jobs`);
+  }
   if (deployable.length > 0) {
-    await deployRepo(deployable, consoleLogger, jobRepository, c.get('jobsQueueUrl'));
+    try {
+      await deployRepo(deployable, consoleLogger, jobRepository, c.get('jobsQueueUrl'));
+    } catch (e) {
+      return prepResponse(401, 'text/plain', `${e} error deploying repos`);
+    }
   }
   return {
     statusCode: 200,

api/handlers/slack.ts

@@ -19,9 +19,9 @@ export function prepResponse(statusCode, contentType, body) {
 }
 
 //if person is admin, get all prod deployable repos
-export async function buildEntitledGroupsList(entitlement: any, repoBranchesRepository: RepoBranchesRepository) {
-  const repoOptions: any[] = [];
-  for (const repo of entitlement.repos) {
+export async function buildEntitledGroupsList(entitledRepos: any, repoBranchesRepository: RepoBranchesRepository) {
+  const entitledBranches: any[] = [];
+  for (const repo of entitledRepos) {
     const [repoOwner, repoName, directoryPath] = repo.split('/');
     const branches = await repoBranchesRepository.getRepoBranches(repoName, directoryPath);
 
@@ -62,10 +62,10 @@ export async function buildEntitledGroupsList(entitlement: any, repoBranchesRepo
             .localeCompare(branchOne.text.text.toString().replace(/\d+/g, (n) => +n + 100000))
         ),
       };
-      repoOptions.push(repoOption);
+      entitledBranches.push(repoOption);
     }
   }
-  return repoOptions.sort((repoOne, repoTwo) => repoOne.label.text.localeCompare(repoTwo.label.text));
+  return entitledBranches.sort((repoOne, repoTwo) => repoOne.label.text.localeCompare(repoTwo.label.text));
 }
 
 export function getQSString(qs: string) {

src/constants.ts

@@ -0,0 +1 @@
+export const DOCS_METADATA = 'docs_metadata';

src/repositories/projectsRepository.ts

@@ -7,7 +7,7 @@ import { ILogger } from '../services/logger';
 
 export class ProjectsRepository extends BaseRepository {
   constructor(db: mongodb.Db, config: IConfig, logger: ILogger) {
-    super(config, logger, 'ProjectsRepository', db.collection(process.env.PROJECTS_COL_NAME || ''));
+    super(config, logger, 'ProjectsRepository', db.collection(process.env.PROJECTS_COL_NAME || 'projects'));
   }
 
   async getProjectEntry(name: string): Promise<mongodb.WithId<mongodb.BSON.Document> | null> {

src/repositories/repoBranchesRepository.ts

@@ -34,7 +34,10 @@ export class RepoBranchesRepository extends BaseRepository {
 
   async getProdDeployableRepoBranches(): Promise<Document[]> {
     const reposArray = await this._collection
-      .aggregate([{ $match: { prodDeployable: true, internalOnly: false } }, { $project: { _id: 0, repoName: 1 } }])
+      .aggregate([
+        { $match: { prodDeployable: true, internalOnly: false } },
+        { $project: { _id: 0, repoName: 1, project: 1 } },
+      ])
       .toArray();
     return reposArray ?? [];
   }
@@ -49,7 +52,6 @@ export class RepoBranchesRepository extends BaseRepository {
         { $project: { branches: 1 } },
       ])
       .toArray();
-
     if (aliasArray.length === 1) {
       returnObject['aliasObject'] = aliasArray[0].branches;
       returnObject.status = 'success';

src/services/slack.ts

@@ -2,7 +2,6 @@ import axios from 'axios';
 import { ILogger } from './logger';
 import { IConfig } from 'config';
 import * as crypto from 'crypto';
-import { RepoBranchesRepository } from '../repositories/repoBranchesRepository';
 export const axiosApi = axios.create();
 
 function bufferEqual(a: Buffer, b: Buffer) {
@@ -24,7 +23,7 @@ function timeSafeCompare(a: string, b: string) {
 export interface ISlackConnector {
   validateSlackRequest(payload: any): boolean;
   displayRepoOptions(repos: Array<string>, triggerId: string, isAdmin: boolean): Promise<any>;
-  parseSelection(payload: any, isAdmin: boolean, repoBranchesRepository: RepoBranchesRepository): any;
+  parseSelection(payload: any, isAdmin: boolean, optionGroups: any[]): any;
   sendMessage(message: any, user: string): Promise<any>;
 }
 
@@ -54,11 +53,7 @@ export class SlackConnector implements ISlackConnector {
     return {};
   }
 
-  async parseSelection(
-    stateValues: any,
-    isAdmin: boolean,
-    repoBranchesRepository: RepoBranchesRepository
-  ): Promise<any> {
+  async parseSelection(stateValues: any, isAdmin: boolean, optionGroups: any[]): Promise<any> {
     const values = {};
     const inputMapping = {
       block_repo_option: 'repo_option',
@@ -72,7 +67,16 @@ export class SlackConnector implements ISlackConnector {
       }
 
       values['deploy_option'] = 'deploy_all';
-      values['repo_option'] = await repoBranchesRepository.getProdDeployableRepoBranches();
+      //go through all options in dropdown by option group
+      //append version to repo_option if active
+      values['repo_option'] = [];
+      for (const group of optionGroups) {
+        values['repo_option'].push(
+          ...group.options.filter((option) => {
+            !option.text.text.startsWith('(!inactive)');
+          })
+        );
+      }
       return values;
     }
 
@@ -207,6 +211,7 @@ export class SlackConnector implements ISlackConnector {
               },
               option_groups: repos,
             },
+            optional: true,
           },
           {
             type: 'input',