Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

As a user, I want to force an upload of file that is already in S3 or the Registry #100

Open
jordanpadams opened this issue Apr 26, 2024 · 3 comments
Assignees
Labels
B15.1 p.must-have requirement the current issue is a requirement

Comments

@jordanpadams
Copy link
Member

Checked for duplicates

Yes - I've already checked

πŸ§‘β€πŸ”¬ User Persona(s)

Node Operator

πŸ’ͺ Motivation

...so that I can overwrite data that has already been loaded into the PDC

πŸ“– Additional Details

No response

Acceptance Criteria

Given a file that has already been loaded into the registry
When I perform a DUM upload with the --overwrite flag enabled
Then I expect the data to overwrite the existing file in the system, and DUM to note that this occurred in the logs

βš™οΈ Engineering Details

No response

@tloubrieu-jpl
Copy link
Member

from the breakout meeting today:
The API need to provide a simple end-point for DUM to retrieve files, critical information to provide is:

file name/path
discipline node
md5sum

We are not sure yet what the best key should provided by the API, either:

option1: node + file path β†’ returns md5sum, lidvid
option2: md5sum β†’ returns node, file path and lidvid

With option 1, a new end-point for the API could be:

/files/{node}/{file_path} which would return the {md5sum}Β 

The issue is that the file_path is not always the same, on the staging bucket or where the file is archived eventually.Β 

@ramesh-maddegoda, @viviant100 could you investigate how the path on the archive bucket (ODR) is being created from the path in the staging bucket ?

@tloubrieu-jpl
Copy link
Member

Waiting for Jordan for validation

@tloubrieu-jpl
Copy link
Member

Validated with endpoint /files/{md5sum} to be developed in the API.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B15.1 p.must-have requirement the current issue is a requirement
Projects
Status: Sprint Backlog
Status: ToDo
Development

No branches or pull requests

3 participants