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.

Sign up for GitHub

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

Jump to bottom

Regression introduced on AMI set processing UID loader #217

Closed
DiegoPino opened this issue Sep 5, 2024 · 6 comments
Closed

Regression introduced on AMI set processing UID loader #217

DiegoPino opened this issue Sep 5, 2024 · 6 comments
Assignees
@DiegoPino
Labels
bug Something isn't working CSV Processing Things we do here and there to keep the tabulated goddesses happy Ingest Setup Knobs and Levers you move while thinking about feelings and metadata and CSV files
Milestone
0.8.0

Comments

@DiegoPino
Copy link
Member

What?

CSV AMI Queue added a challenge, which is the AMI set needs to run as the user submitting to the queue. That was fixed by #215 and #216 but that introduced now the need to also have on a Direct submit the UID present.
@DiegoPino DiegoPino added bug Something isn't working Ingest Setup Knobs and Levers you move while thinking about feelings and metadata and CSV files CSV Processing Things we do here and there to keep the tabulated goddesses happy labels Sep 5, 2024
@DiegoPino DiegoPino added this to the 0.8.0 milestone Sep 5, 2024
@DiegoPino DiegoPino self-assigned this Sep 5, 2024
@DiegoPino DiegoPino mentioned this issue Sep 5, 2024
Merged
@patdunlavey
Copy link
Contributor

Confirmed that #218 resolves this problem:

  1. Immediate processing of AMI set works
  2. Delete processed ADOs works
  3. Enqueued csv expansion and ADO processing still works.

@DiegoPino
Copy link
Member Author

@patdunlavey thanks a lot!

@patdunlavey
Copy link
Contributor

Thank YOU!!!

@patdunlavey
Copy link
Contributor

@DiegoPino Do you need a PR for 0.9.0?

@DiegoPino
Copy link
Member Author

DiegoPino commented Sep 5, 2024 via email

@DiegoPino
Copy link
Member Author

resolved via cf70fd9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DiegoPino DiegoPino

Labels
bug Something isn't working CSV Processing Things we do here and there to keep the tabulated goddesses happy Ingest Setup Knobs and Levers you move while thinking about feelings and metadata and CSV files
Projects
None yet
Milestone
0.8.0
Development

No branches or pull requests
3 participants
@patdunlavey @DiegoPino and others