Skip to content

Remove unused Airtable Cloud Function support directories#5228

Open
fsalemi wants to merge 1 commit intomainfrom
remove-airtable-cloud-function-directories
Open

Remove unused Airtable Cloud Function support directories#5228
fsalemi wants to merge 1 commit intomainfrom
remove-airtable-cloud-function-directories

Conversation

@fsalemi
Copy link
Copy Markdown
Contributor

@fsalemi fsalemi commented May 4, 2026

Description

Remove unused Cloud Function support directories following successful cleanup of Airtable issue automation infrastructure.

All Cloud Function and Cloud Scheduler resources have been:

  • Removed from Terraform configuration (PR1)
  • Destroyed or deleted in GCP

This PR removes remaining unused directories:

  • cloud-functions/
  • iac/cal-itp-data-infra/cloud-functions/

Resolves #5217

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature
  • Breaking change (removes unused infrastructure code)
  • Documentation

How has this been tested?

  • Verified Cloud Function and Scheduler are no longer present in GCP
  • Confirmed Terraform state no longer contains relevant resources
  • This PR only removes unused files/directories

Post-merge follow-ups

  • No action required
  • Actions required (specified below)

@erikamov
Copy link
Copy Markdown
Contributor

erikamov commented May 5, 2026

Terraform plan did not run any plan to delete the code because the PR is deleting all the Terraform files. You need to just remove your python code, sql and lines where it points to it, but keep those Terraform files and the rest of its content.

@fsalemi
Copy link
Copy Markdown
Contributor Author

fsalemi commented May 5, 2026 via email

@erikamov
Copy link
Copy Markdown
Contributor

erikamov commented May 5, 2026

I’ve already done that step. This is the second PR.

On Tue, May 5, 2026 at 12:22 PM Erika @.> wrote: erikamov left a comment (cal-itp/data-infra#5228) <#5228 (comment)> Terraform plan did not run any plan to delete the code because the PR is deleting all the Terraform files. You need to just remove your python code, sql and lines where it points to it, but keep those Terraform files and the rest of its content. — Reply to this email directly, view it on GitHub <#5228 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AM4UBZK4TFXJVL352TUSXE34ZI5PXAVCNFSM6AAAAACYQYIEYSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DGOBSGMYDIMZRHA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you authored the thread.Message ID: @.>

I’ve already done that step. This is the second PR.

So you don't need this one. You can keep all the Terraform settings in case someone else needs to create a cloud function in the future.

@fsalemi
Copy link
Copy Markdown
Contributor Author

fsalemi commented May 5, 2026 via email

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.

Remove deprecated Cloud Function and Cloud Scheduler resources

2 participants