Skip to content

fix: extend gp-news timeout; remove clawbench dns#179

Merged
Perry2004 merged 1 commit into
mainfrom
fix/extend-gp-news-timeout
Apr 7, 2026
Merged

fix: extend gp-news timeout; remove clawbench dns#179
Perry2004 merged 1 commit into
mainfrom
fix/extend-gp-news-timeout

Conversation

@Perry2004
Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings April 7, 2026 05:12
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the gp-news Lambda’s execution timeout to accommodate longer runs and removes Terraform-managed Route53 DNS records for the clawbench subdomain under the shared common/dns stack.

Changes:

  • Increase apps/gp-news/usw2dev Lambda timeout from 60s to 180s.
  • Remove clawbench Purelymail-related DNS records from common/dns.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
common/dns/clawbench.tf Deletes Route53 records previously managing clawbench email/DNS entries.
apps/gp-news/usw2dev/lambda.tf Extends Lambda timeout to reduce risk of premature termination for longer scrapes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Perry2004 Perry2004 merged commit 2b48185 into main Apr 7, 2026
8 checks passed
@Perry2004 Perry2004 deleted the fix/extend-gp-news-timeout branch April 7, 2026 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants