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

author submissions: author creating is erroring, but gets stuck on PROCESSING #643

Open
DonHaul opened this issue Dec 18, 2024 · 1 comment
Open

author submissions: author creating is erroring, but gets stuck on PROCESSING #643

DonHaul opened this issue Dec 18, 2024 · 1 comment
Assignees
@DonHaul
Labels
project: backoffice type: bug Something isn't working

Comments

@DonHaul
Copy link
Collaborator

DonHaul commented Dec 18, 2024 •
edited
Loading

https://inspirebeta.net/backoffice/6972fef5-4a4a-4191-a4b9-4c728f06a53e

https://workflows.inspire.dev.siscern.org/dags/author_create_initialization_dag/grid?task_id=set_workflow_status_to_running&dag_run_id=6972fef5-4a4a-4191-a4b9-4c728f06a53e
@DonHaul DonHaul added project: backoffice type: bug Something isn't working labels Dec 18, 2024
DonHaul added a commit to DonHaul/inspirehep that referenced this issue Dec 18, 2024
@DonHaul
workflows: author submissions not erroring
ddf9869 
also some small config fixes for easy testing with docker

* ref: cern-sis/issues-inspire#643
@DonHaul DonHaul mentioned this issue Dec 18, 2024
Merged
DonHaul added a commit to DonHaul/inspirehep that referenced this issue Dec 18, 2024
@DonHaul
workflows: author submissions not erroring
8635d88 
also some small config fixes for easy testing with docker

* ref: cern-sis/issues-inspire#643
@DonHaul
Copy link
Collaborator Author

DonHaul commented Dec 18, 2024 •
edited
Loading

this issue was cause in
https://github.com/inspirehep/inspirehep/pull/3322/files#diff-5d4944ad0ac9fb01f90db61987c42a8ee43cdb85f9fb449ef5b3cf7ebf9f6c0dL56
where the data object was being converted to json automagicaly before, but now this had stopped happening
https://github.com/inspirehep/inspirehep/pull/3322/files#diff-0f92d6b19584a8cb7154eaa59b33541222bb73f8a9632c15adc58d6534b124b8R55

This change is good, because it makes the code more generic. but since there were no tests on the functions that used it. it started failing.
Adding tests to the set_workflow_status to make sure this doesn't make them fail

DonHaul added a commit to DonHaul/inspirehep that referenced this issue Dec 18, 2024
@DonHaul
workflows: author submissions not erroring
f465376 
also some small config fixes for easy testing with docker

* ref: cern-sis/issues-inspire#643
DonHaul added a commit to DonHaul/inspirehep that referenced this issue Dec 18, 2024
@DonHaul
workflows: author submissions not erroring
b13a5c8 
also some small config fixes for easy testing with docker

* ref: cern-sis/issues-inspire#643
DonHaul added a commit to DonHaul/inspirehep that referenced this issue Dec 18, 2024
@DonHaul
workflows: fix author submissions not erroring
798f4bb 
also some small config fixes for easy testing with docker

* ref: cern-sis/issues-inspire#643
DonHaul added a commit to DonHaul/inspirehep that referenced this issue Dec 18, 2024
@DonHaul
workflows: fix author submissions not erroring
9941ee3 
also some small config fixes for easy testing with docker

* ref: cern-sis/issues-inspire#643
DonHaul added a commit to DonHaul/inspirehep that referenced this issue Dec 19, 2024
@DonHaul
workflows: fix author submissions not erroring
60e2dc0 
also some small config fixes for easy testing with docker

* ref: cern-sis/issues-inspire#643
DonHaul added a commit to DonHaul/inspirehep that referenced this issue Dec 19, 2024
@DonHaul
workflows: fix author submissions not erroring
0c9df55 
also some small config fixes for easy testing with docker

* ref: cern-sis/issues-inspire#643
DonHaul added a commit to inspirehep/inspirehep that referenced this issue Dec 19, 2024
@DonHaul
workflows: fix author submissions not erroring
4342f1e 
also some small config fixes for easy testing with docker

* ref: cern-sis/issues-inspire#643
DonHaul added a commit to DonHaul/inspirehep that referenced this issue Dec 19, 2024
@DonHaul
workflows: fix creating ticket entry
a6f8655 
* ref: cern-sis/issues-inspire#643
drjova pushed a commit to inspirehep/inspirehep that referenced this issue Dec 19, 2024
@DonHaul @drjova
workflows: fix creating ticket entry
99a85c8 
* ref: cern-sis/issues-inspire#643
DonHaul added a commit to DonHaul/inspirehep that referenced this issue Dec 19, 2024
@DonHaul
workflows: fail properly for author wfs
79ef28b 
* ref: cern-sis/issues-inspire/issues/643
DonHaul added a commit to DonHaul/inspirehep that referenced this issue Dec 19, 2024
@DonHaul
workflows: add testing to wf callback failure
6e23601 
* ref: cern-sis/issues-inspire/issues/643
DonHaul added a commit to DonHaul/inspirehep that referenced this issue Dec 19, 2024
@DonHaul
workflows: add testing to wf callback failure
89cd90f 
* ref: cern-sis/issues-inspire/issues/643
DonHaul added a commit to DonHaul/inspirehep that referenced this issue Dec 19, 2024
@DonHaul
workflows: fail properly for author wfs
c1e251c 
* ref: cern-sis/issues-inspire/issues/643
DonHaul added a commit to DonHaul/inspirehep that referenced this issue Dec 19, 2024
@DonHaul
workflows: add testing to wf callback failure
348744d 
* ref: cern-sis/issues-inspire/issues/643
DonHaul added a commit to inspirehep/inspirehep that referenced this issue Dec 19, 2024
@DonHaul
workflows: fail properly for author wfs
1747759 
* ref: cern-sis/issues-inspire/issues/643
DonHaul added a commit to inspirehep/inspirehep that referenced this issue Dec 19, 2024
@DonHaul
workflows: add testing to wf callback failure
9e78629 
* ref: cern-sis/issues-inspire/issues/643
@DonHaul DonHaul self-assigned this Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DonHaul DonHaul

Labels
project: backoffice type: bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@DonHaul