Skip to content

Errors as values update book plan#1651

Merged
JBorrow merged 8 commits into
masterfrom
errors_as_values_update_book_plan
May 29, 2026
Merged

Errors as values update book plan#1651
JBorrow merged 8 commits into
masterfrom
errors_as_values_update_book_plan

Conversation

@JBorrow
Copy link
Copy Markdown
Member

@JBorrow JBorrow commented May 29, 2026

Allows progress on other tubes for update_book_plan even when we error out for a specific tube. Fixes #1650.

Also includes updates from monitor-and-profile-update_g3thk_database which allows for monitoring and profiling of the g3thk_database flow.

@JBorrow JBorrow requested a review from kmharrington May 29, 2026 13:39
@JBorrow
Copy link
Copy Markdown
Member Author

JBorrow commented May 29, 2026

My only concern here is whether this causes other problems further down the line, I don't think so as the tubes should be completely independent and they get bound into separate books?

Copy link
Copy Markdown
Member

@kmharrington kmharrington left a comment

Choose a reason for hiding this comment

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

the timecode and hk books have independent checking to decide if they're ready to be registered, so I don't expect this to cause any errors farther down

@JBorrow
Copy link
Copy Markdown
Member Author

JBorrow commented May 29, 2026

Given that this is running happily, I am ok to merge at this stage.

@JBorrow JBorrow merged commit 9baa1f1 into master May 29, 2026
5 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.

Data Packaging: update_book_plan should check all tubes before throwing errors

2 participants