Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove DownloadLogsAnalytics & SendPublicAPIEventsToGA Lambdas #1807

Merged
merged 1 commit into from
Nov 30, 2023

Conversation

deborahchua
Copy link
Contributor

These lambda functions are still running on an old version of Python (3.7) which is now end of life.

Due to previous attempts at upgrading to Python 3.8 or newer (and failing) we have agreed to remove them because:

  • the are not/under- used
  • they will break when Universal Analytics sunsets in June 2024
  • the Data Services team doesn't have the dev/engineer resource and AWS experience to fix and upgrade them themselves.

This is the console output for the Terraform plan run for production:

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

Trello card: https://trello.com/c/sxjNj31E/3360-archive-sendpublicapieventstoga-and-downloadlogsanalytics-lambda-functions-2

These lambda functions are still running on an old version of Python (3.7)
which is now end of life.

Due to previous attempts at upgrading to Python 3.8 or newer (and
failing) we have agreed to remove them because:

- the are not/under- used
- they will break when Universal Analytics sunsets in June 2024
- the Data Services team doesn't have the dev/engineer resource and
AWS experience to fix and upgrade them themselves.
@deborahchua deborahchua marked this pull request as ready for review November 30, 2023 11:28
Copy link
Contributor

@robinjam robinjam left a comment

Choose a reason for hiding this comment

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

LGTM, nice one 👍

@deborahchua deborahchua merged commit 1819705 into main Nov 30, 2023
4 checks passed
@deborahchua deborahchua deleted the archive-unused-lambdas branch November 30, 2023 13:45
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