Skip to content

Conversation

@dmarticus
Copy link
Contributor

rolled the percentage to 100%. Let's get some real data.

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

This PR rolls out the new /flags endpoint to 100% of /decide traffic while protecting top customers from potential disruption by excluding them from the rollout.

  • Updates ROLLOUT_PERCENTAGE from 0.1 to 1 in posthog/client.py for full rollout
  • Adds EXCLUDED_HASHES set in posthog/client.py containing API keys of top 10 customers to maintain stability
  • Bumps version to 3.25.0 in posthog/version.py to reflect the significant change
  • Documents rollout strategy in CHANGELOG.md with clear exclusion criteria

3 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

Copy link
Collaborator

@haacked haacked left a comment

Choose a reason for hiding this comment

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

:shipit:

@dmarticus dmarticus merged commit a1583f6 into master Apr 15, 2025
6 checks passed
@dmarticus dmarticus deleted the chore/roll-flags-to-everyone-but-the-special-snowflakes branch April 15, 2025 21:13
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.

3 participants