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

Nested workflow jobs with failure connections are very slow #2766

Closed
AlanCoding opened this issue Nov 20, 2018 · 3 comments
Closed

Nested workflow jobs with failure connections are very slow #2766

AlanCoding opened this issue Nov 20, 2018 · 3 comments

Comments

@AlanCoding
Copy link
Member

ISSUE TYPE
  • Bug Report
COMPONENT NAME
  • API
SUMMARY

Workflow job failure connections take a long time to run.

ENVIRONMENT
  • AWX version: X.Y.Z
  • AWX install method: openshift, minishift, docker on linux, docker for mac, boot2docker
  • Ansible version: X.Y.Z
  • Operating System:
  • Web Browser:
STEPS TO REPRODUCE

Create a workflow with a long chain of failure nodes.

For purposes of illustrations, make these nodes call the workflow itself, so that it fails right away

screen shot 2018-11-20 at 2 46 17 pm

Run this

EXPECTED RESULTS

The entire workflow should finish in under 5 seconds.

ACTUAL RESULTS

Takes about 2 minutes in example

ADDITIONAL INFORMATION

I think this is addressed by hooks added in #2729

@shanemcd
Copy link
Member

shanemcd commented Apr 4, 2023

I think this can be closed if I traced this correctly.... #5768 got merged.

@shanemcd shanemcd closed this as completed Apr 4, 2023
@AlanCoding
Copy link
Member Author

I hate to re-open, but the steps are very clear and really quick to reproduce.

Create a workflow with a long chain of failure nodes.
For purposes of illustrations, make these nodes call the workflow itself, so that it fails right away

I made one with 3 nodes in it (again, the nodes are itself).

Started 4/5/2023, 8:26:37 AM
Finished 4/5/2023, 8:27:24 AM

That's a time of about 50 seconds. It should really be about 1 second.

@AlanCoding
Copy link
Member Author

Patched merged

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

No branches or pull requests

2 participants