Skip to content

Show reprocess ID after triggering reprocessing - #360

Merged
leowerneck merged 1 commit into
IMAP-Science-Operations-Center:mainfrom
vineetbansal:vb/issue1676
Oct 8, 2026
Merged

leowerneck merged 1 commit into
IMAP-Science-Operations-Center:mainfrom
vineetbansal:vb/issue1676

Conversation

@vineetbansal

@vineetbansal vineetbansal commented Oct 2, 2026 •

Copy link
Copy Markdown
Collaborator

Partially addresses #1676 in sds-data-manager (PR #1677 addresses the other half).

If available from upstream, returns reprocess_id as the return value of the reprocess function, which is then displayed on the terminal.

Same behavior as before if no such key is available in the response json.

@leowerneck leowerneck 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.

One question: do we want to support the custom tag syntax -tag TAG discussed in sds-data-manager#1676?

@vineetbansal

vineetbansal commented Oct 2, 2026 •

Copy link
Copy Markdown
Collaborator Author

I'd say the custom tag stuff can be left for later, once we get used to searching by tags and making sure it is practical - the way to search for tags is to say something like "imap/<key>=<value>" in the filters in the dagster UI, which will get tricky unless we're sticking to some discipline as to both the keys and values.

@jaredclaypoole jaredclaypoole 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.

Looks good!

@leowerneck
leowerneck merged commit afbfb20 into IMAP-Science-Operations-Center:main Oct 8, 2026
16 checks passed
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