Skip to content

Conversation

@gagantrivedi
Copy link
Member

@gagantrivedi gagantrivedi commented Nov 3, 2025

Thanks for submitting a PR! Please check the boxes below:

  • I have added information to docs/ if required so people know about the feature!
  • I have filled in the "Changes" section below?
  • I have filled in the "How did you test this code" section below?
  • I have used a Conventional Commit title for this Pull Request

Changes

Add AWS_SSE_LOGS_BUCKET_NAME environment variable to re-enable SSE usage tracking.
Was disabled here
For more details, ref: #6224

How did you test this code?

n/a

@gagantrivedi gagantrivedi requested a review from a team as a code owner November 3, 2025 03:28
@gagantrivedi gagantrivedi requested review from emyller and removed request for a team November 3, 2025 03:28
@vercel
Copy link

vercel bot commented Nov 3, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
docs Ignored Ignored Preview Nov 3, 2025 11:51am
flagsmith-frontend-preview Ignored Ignored Preview Nov 3, 2025 11:51am
flagsmith-frontend-staging Ignored Ignored Preview Nov 3, 2025 11:51am

Copy link
Contributor

@matthewelwell matthewelwell left a comment

Choose a reason for hiding this comment

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

@gagantrivedi can you also add information into the description here to provide context on why we disabled it, and why we're re-enabling it? Please link to any issues, discussions or other relevant PRs.

@gagantrivedi
Copy link
Member Author

@gagantrivedi can you also add information into the description here to provide context on why we disabled it, and why we're re-enabling it? Please link to any issues, discussions or other relevant PRs.

Done

@matthewelwell
Copy link
Contributor

@gagantrivedi I assume then, that these have been done?

  • Move existing SSE usage data to a new bucket
  • Clean up the current bucket

@axolo-co
Copy link

axolo-co bot commented Nov 3, 2025

gagantrivedi commented:
We’ll handle this after the release is deployed; otherwise, Fastly will continue pushing data into the bucket, and it will keep accumulating since we’re not processing it.

Copy link
Contributor

Then the issue needs updating with the correct steps in the correct order.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants