Skip to content

Commit d80a846

Browse files
authored
Merge pull request #113 from amazon-ospo/timeoutFix
feat: Updating memory and timeout for RateLimitTracker
2 parents 10db7c0 + 3780abd commit d80a846

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

src/packages/app-framework/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -403,7 +403,7 @@ and can be used to track installation lifecycles reliably.
403403

404404
### Rate Limit Scheduler
405405

406-
Scans GitHub App Rate Limit usages every 5 minutes
406+
Scans GitHub App Rate Limit usages every 10 minutes
407407
and generates corresponding metrics.
408408
These metrics are accessible through the rate limit dashboard.
409409
This enables proper tracking of API usage for each GitHub App.

src/packages/app-framework/src/credential-manager/rate-limit-tracker/index.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,12 @@ export class RateLimitTracker extends Construct {
3131
props.InstallationTable.tableName,
3232
},
3333
description: 'Tracks GitHub API Rate Limits',
34-
memorySize: 512,
34+
memorySize: 1024,
35+
timeout: Duration.minutes(5),
3536
});
3637

3738
new Rule(scope, 'rateLimitTrackerRule', {
38-
schedule: Schedule.rate(Duration.minutes(5)),
39+
schedule: Schedule.rate(Duration.minutes(10)),
3940
enabled: true,
4041
targets: [new LambdaFunction(this.lambdaHandler)],
4142
});

0 commit comments

Comments
 (0)