Skip to content

Commit c3de745

Browse files
authored
Feature/runners lambdas add env to filter (#10)
* Add environment to filter * Add environment to filter (fix)
1 parent 8007a15 commit c3de745

File tree

3 files changed

+13
-2
lines changed

3 files changed

+13
-2
lines changed

modules/runners/lambdas/scale-runners/src/scale-runners/handler.test.ts

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,10 @@ describe('handler', () => {
9191

9292
it('gets the current org level runners', async () => {
9393
await handle('aws:sqs', TEST_DATA);
94-
expect(listRunners).toBeCalledWith({ repoName: undefined });
94+
expect(listRunners).toBeCalledWith({
95+
environment: 'unit-test-environment',
96+
repoName: undefined,
97+
});
9598
});
9699

97100
it('does not create a token when maximum runners has been reached', async () => {
@@ -126,7 +129,10 @@ describe('handler', () => {
126129

127130
it('gets the current repo level runners', async () => {
128131
await handle('aws:sqs', TEST_DATA);
129-
expect(listRunners).toBeCalledWith({ repoName: `${TEST_DATA.repositoryOwner}/${TEST_DATA.repositoryName}` });
132+
expect(listRunners).toBeCalledWith({
133+
environment: 'unit-test-environment',
134+
repoName: `${TEST_DATA.repositoryOwner}/${TEST_DATA.repositoryName}`,
135+
});
130136
});
131137

132138
it('does not create a token when maximum runners has been reached', async () => {

modules/runners/lambdas/scale-runners/src/scale-runners/handler.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ export const handle = async (eventSource: string, payload: ActionRequestMessage)
5151

5252
if (queuedWorkflows.data.total_count > 0) {
5353
const currentRunners = await listRunners({
54+
environment: environment,
5455
repoName: enableOrgLevel ? undefined : `${payload.repositoryOwner}/${payload.repositoryName}`,
5556
});
5657
console.info(

modules/runners/lambdas/scale-runners/src/scale-runners/runners.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ export interface RunnerInfo {
1010
export interface ListRunnerFilters {
1111
repoName?: string;
1212
orgName?: string;
13+
environment?: string;
1314
}
1415

1516
export async function listRunners(filters: ListRunnerFilters | undefined = undefined): Promise<RunnerInfo[]> {
@@ -19,6 +20,9 @@ export async function listRunners(filters: ListRunnerFilters | undefined = undef
1920
{ Name: 'instance-state-name', Values: ['running', 'pending'] },
2021
];
2122
if (filters) {
23+
if (filters.environment !== undefined) {
24+
ec2Filters.push({ Name: 'tag:Environment', Values: [filters.environment] });
25+
}
2226
if (filters.repoName !== undefined) {
2327
ec2Filters.push({ Name: 'tag:Repo', Values: [filters.repoName] });
2428
}

0 commit comments

Comments
 (0)