Skip to content

data carousel: update staged files immediately for to_pin requests#694

Open
mightqxc wants to merge 1 commit intomasterfrom
flin
Open

data carousel: update staged files immediately for to_pin requests#694
mightqxc wants to merge 1 commit intomasterfrom
flin

Conversation

@mightqxc
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Data Carousel staging flow so that to_pin requests immediately mark input files as staged in the DB, enabling dependent tasks to proceed without waiting for dataset-staging completion.

Changes:

  • Fail fast (with an error message) when updating the request status to staging in the DB fails.
  • For to_pin requests, call _update_staged_files() immediately after switching the request to staging.
  • Skip iDDS stage-in submission for to_pin requests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants