Skip to content

Comments

bug(nimbus): don't unset published_dto for experiments moving from preview to live#14187

Merged
jaredlockhart merged 1 commit intomainfrom
14184
Dec 12, 2025
Merged

bug(nimbus): don't unset published_dto for experiments moving from preview to live#14187
jaredlockhart merged 1 commit intomainfrom
14184

Conversation

@jaredlockhart
Copy link
Collaborator

Becuase

  • We recently discovered a live experiment with no published_dto
  • This breaks the state of the experiment and its ability to pause enrollment
  • Looking at the changelogs it looks as if it was moved from preview to live very quickly
  • It's possible the preview unpublish task accidentally unset the published_dto after the experiment was marked live

This commit

  • Only unsets published_dto for experiments in draft when moving out of preview
  • Wraps all kinto tasks in atomic transactions
  • Generates a changelog for all kinto changes that modify the database for easier retroactive debugging

fixes #14184

…eview to live

Becuase

* We recently discovered a live experiment with no published_dto
* This breaks the state of the experiment and its ability to pause enrollment
* Looking at the changelogs it looks as if it was moved from preview to live very quickly
* It's possible the preview unpublish task accidentally unset the published_dto after the experiment was marked live

This commit

* Only unsets published_dto for experiments in draft when moving out of preview
* Wraps all kinto tasks in atomic transactions
* Generates a changelog for all kinto changes that modify the database for easier retroactive debugging

fixes #14184
@jaredlockhart
Copy link
Collaborator Author

Followup #14188

@jaredlockhart jaredlockhart added this pull request to the merge queue Dec 12, 2025
Merged via the queue into main with commit f0f47a2 Dec 12, 2025
16 checks passed
@jaredlockhart jaredlockhart deleted the 14184 branch December 12, 2025 21:36
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.

Paused experiment shows as still enrolling

2 participants