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

CI timeout #1816

Closed
Gamenot opened this issue Jan 27, 2023 · 2 comments · Fixed by #1941
Closed

CI timeout #1816

Gamenot opened this issue Jan 27, 2023 · 2 comments · Fixed by #1941
Assignees
Labels
bug Something isn't working Urgent urgent issue needs to be fixed asap

Comments

@Gamenot
Copy link
Collaborator

Gamenot commented Jan 27, 2023

BUG REPORT

High Level Description
CI noted the timeout of a particular run. This means that we have a rare infinite loop somewhere in the engine. This seems to be recent.

SMARTS version
1.0.1

Previous associated issues
N/A

Steps to reproduce the bug
Unsure.

Resulting and expected behaviour
The tests timed out but should roughly pass in under 12 minutes.

Error logs and screenshots
https://github.com/huawei-noah/SMARTS/actions/runs/4017689624/jobs/6902359722

System information
CI

  • Date when bug occurred
    2023/01/27

Impact [If known]
Potential issues with long running SMARTS.

@Gamenot Gamenot added bug Something isn't working Urgent urgent issue needs to be fixed asap labels Jan 27, 2023
@Gamenot
Copy link
Collaborator Author

Gamenot commented Jan 28, 2023

There seems to be an earlier instance of this happening with test_studio:test_scenario_generation_unchanged:

https://github.com/huawei-noah/SMARTS/actions/runs/3978346086/jobs/6820060923

@Gamenot
Copy link
Collaborator Author

Gamenot commented Jan 28, 2023

@saulfield It appears that scenario building might be causing issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Urgent urgent issue needs to be fixed asap
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants