Skip to content

feat(nimbus): add is_holdback, do_rerun, do_rerun_timestamp fields to model and v8 API#16096

Merged
yashikakhurana merged 2 commits into
mainfrom
16087
Jun 30, 2026
Merged

feat(nimbus): add is_holdback, do_rerun, do_rerun_timestamp fields to model and v8 API#16096
yashikakhurana merged 2 commits into
mainfrom
16087

Conversation

@yashikakhurana

@yashikakhurana yashikakhurana commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Because

  • Holdback experiments need to be flagged in Experimenter and re-run by downstream analysis. Jetstream reads these fields from the v8 API to decide when to rerun an experiment, and metric-hub uses them for long-term holdback analysis.

This commit

  • Adds is_holdback, do_rerun, and do_rerun_timestamp fields to the NimbusExperiment model and exposes them in the v8 API as isHoldback, doRerun, and doRerunTimestamp.
  • Updates the v8 serializer and changelog serializer test expectations and regenerates the OpenAPI schema and Swagger docs.

Related

fixes #16087

Yashika Khurana added 2 commits June 23, 2026 15:24
Add isHoldback/doRerun/doRerunTimestamp to the v8 serializer test schema
and is_holdback/do_rerun/do_rerun_timestamp to the changelog serializer
expected dicts, and regenerate the OpenAPI schema and Swagger docs to
include the new fields.
Comment on lines +1584 to +1586
"doRerunTimestamp": {
"type": "string",
"readOnly": true

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should this also include "format": "date-time"?

Suggested change
"doRerunTimestamp": {
"type": "string",
"readOnly": true
"doRerunTimestamp": {
"type": "string",
"readOnly": true
"format": "date-time",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

its auto generated @moibra05

@mikewilli mikewilli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM -- And forgot that the API uses camelCase instead of snake so I think I'll have to make an update on the Jetstream side.

@yashikakhurana yashikakhurana added this pull request to the merge queue Jun 30, 2026
Merged via the queue into main with commit 420addd Jun 30, 2026
25 checks passed
@yashikakhurana yashikakhurana deleted the 16087 branch June 30, 2026 17:11
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.

Add holdback fields to NimbusExperiment model and v8 API

3 participants