Skip to content

Increase team-provisioner timeout to 300s#124

Merged
Alexanderamiri merged 1 commit intomainfrom
fix/provisioner-timeout
Apr 14, 2026
Merged

Increase team-provisioner timeout to 300s#124
Alexanderamiri merged 1 commit intomainfrom
fix/provisioner-timeout

Conversation

@Alexanderamiri
Copy link
Copy Markdown
Member

Summary

  • Increases team-provisioner Lambda timeout from 60s to 300s
  • With 54 members, the Lambda timed out during group sync after creating all accounts
  • Each member takes ~3s (check exists + create + forwarding attempt + welcome email), so 54 members alone need ~162s before group sync even starts

Context

The provision-groups run from registry PR #21 merge timed out at 60s. All accounts were created and welcome emails sent, but group membership sync didn't complete.

With 54 members, account creation + forwarding attempts + welcome
emails + group sync exceeds 60s. The Lambda timed out during group
sync after creating all accounts.
@Alexanderamiri Alexanderamiri requested a review from a team as a code owner April 14, 2026 20:36
@github-actions
Copy link
Copy Markdown

Terraform Plan

🚧 Changes detected — Plan: 0 to add, 1 to change, 0 to destroy.

Plan output

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  ~ update in-place

Terraform will perform the following actions:

  # module.lambdas.aws_lambda_function.team_provisioner will be updated in-place
  ~ resource "aws_lambda_function" "team_provisioner" {
        id                             = "javabin-team-provisioner"
        tags                           = {}
      ~ timeout                        = 60 -> 300
        # (22 unchanged attributes hidden)

        # (4 unchanged blocks hidden)
    }

Plan: 0 to add, 1 to change, 0 to destroy.

─────────────────────────────────────────────────────────────────────────────

Saved the plan to: tfplan

To perform exactly these actions, run the following command to apply:
    terraform apply "tfplan"

LLM Review

Risk: 🟢 LOW

Routine Lambda timeout configuration update for team_provisioner function from 60 to 300 seconds.

  • [routine] Lambda function timeout increased from 60 to 300 seconds - likely to accommodate longer provisioning operations
  • [routine] Single in-place update with no resource destruction or replacement
  • [routine] No security, IAM, or networking changes involved
  • 💰 [cost] Potential minor cost increase due to longer Lambda execution time, but likely necessary for operational reliability

@Alexanderamiri Alexanderamiri merged commit eb45d74 into main Apr 14, 2026
3 checks passed
@Alexanderamiri Alexanderamiri deleted the fix/provisioner-timeout branch April 14, 2026 20:37
Alexanderamiri added a commit that referenced this pull request May 9, 2026
## Summary
- Increases team-provisioner Lambda timeout from 60s to 300s
- With 54 members, the Lambda timed out during group sync after creating
all accounts
- Each member takes ~3s (check exists + create + forwarding attempt +
welcome email), so 54 members alone need ~162s before group sync even
starts

## Context
The provision-groups run from registry PR #21 merge timed out at 60s.
All accounts were created and welcome emails sent, but group membership
sync didn't complete.
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.

1 participant