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

"Export to BCO" internal server error 500 #15961

Closed
agbiotec opened this issue Apr 19, 2023 · 4 comments · Fixed by galaxyproject/gxformat2#89
Closed

"Export to BCO" internal server error 500 #15961

agbiotec opened this issue Apr 19, 2023 · 4 comments · Fixed by galaxyproject/gxformat2#89

Comments

@agbiotec
Copy link

At https://usegalaxy.org/ public server,

the "Export to BCO" for completed workflow invocations returns internal server error 500.

For example this internal server error is observed with
workflow with invocation id# 7de4365c4cae6031 has been successfully completed (all green),
that puts data on history id# 643ea0e16359e250 where all the jobs have been successfully completed.

Screenshots of these on links attached. I believe there was a problem with the "Export to BCO" tool 2-3 weeks back, which was fixed, and in fact I was able to use it to export a BCO successfully for a workflow before it seems to have broken again.

Thank you
screenshot-3
screenshot-2
screenshot-1

@martenson
Copy link
Member

related sentry item and trace for this seems to be

 File "galaxy/webapps/galaxy/api/workflows.py", line 1487, in download_invocation_bco
    bco = self._deprecated_generate_bco(trans, invocation_id, merge_history_metadata)
  File "galaxy/webapps/galaxy/api/workflows.py", line 1506, in _deprecated_generate_bco
    return self.invocations_service.deprecated_generate_invocation_bco(trans, invocation_id, export_options)
  File "galaxy/webapps/galaxy/services/invocations.py", line 235, in deprecated_generate_invocation_bco
    export_target.seek(0)
  File "galaxy/model/store/__init__.py", line 2364, in __exit__
    self._finalize()
  File "galaxy/model/store/__init__.py", line 2517, in _finalize
    super()._finalize()
  File "galaxy/model/store/__init__.py", line 2347, in _finalize
    self.app.workflow_contents_manager.store_workflow_artifacts(
  File "galaxy/managers/workflows.py", line 853, in store_workflow_artifacts
    self.app.workflow_contents_manager.store_workflow_to_path(path, workflow.stored_workflow, workflow, **kwd)
  File "galaxy/managers/workflows.py", line 873, in store_workflow_to_path
    abstract_dict = from_dict(wf_dict)
  File "gxformat2/abstract.py", line 32, in from_dict
    normalized_workflow = NormalizedWorkflow(workflow_dict)
  File "gxformat2/normalize.py", line 84, in __init__
    normalized_workflow["inputs"] = inputs_as_normalized_steps(normalized_workflow)
  File "gxformat2/model.py", line 94, in inputs_as_normalized_steps
    for input_def_raw in with_step_ids(inputs):
  File "gxformat2/model.py", line 43, in with_step_ids
    assert step["id"] is not None

@KKrampis
Copy link

KKrampis commented May 5, 2023

Hello and thank you for looking into this, I really appreciate your time! I was wondering if there are any updates with fixing the issue ?

Best regards

@mvdbeek
Copy link
Member

mvdbeek commented May 12, 2023

That'll be fixed by #16081

@agbiotec
Copy link
Author

agbiotec commented Jun 8, 2023

My apologies to bother again, and I really appreciate the effort and pointers to the code for fixing this bug !

Would you know the fix was applied on usegalaxy.org, if not, when would it be rolled out on the public Galaxy ?

Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants