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

integration.modules.test_saltutil tests failing on windows sometimes #56683

Closed
Ch3LL opened this issue Apr 17, 2020 · 5 comments
Closed

integration.modules.test_saltutil tests failing on windows sometimes #56683

Ch3LL opened this issue Apr 17, 2020 · 5 comments
Assignees
Labels
cannot-reproduce cannot be replicated with info/context provided Test-Failure Fix the CI failure ⛈
Milestone

Comments

@Ch3LL
Copy link
Contributor

Ch3LL commented Apr 17, 2020

Description
I'm seeing these tests fail on windows periodically:

integration.modules.test_saltutil.SaltUtilSyncModuleTest.test_sync_all
integration.modules.test_saltutil.SaltUtilSyncModuleTest.test_sync_all_blacklist
integration.modules.test_saltutil.SaltUtilSyncModuleTest.test_sync_all_whitelist

Here they fail on a PR test: https://jenkinsci.saltstack.com/job/pr-windows2019-py3/job/PR-56408/6/

and on branch tests: https://jenkinsci.saltstack.com/job/pr-windows2019-py3/job/master/737/

@Ch3LL Ch3LL added the Test-Failure Fix the CI failure ⛈ label Apr 17, 2020
@Ch3LL Ch3LL added this to the Approved milestone Apr 17, 2020
@xeacott
Copy link
Contributor

xeacott commented Apr 20, 2020

Looking at this now...

@xeacott
Copy link
Contributor

xeacott commented Apr 20, 2020

They seem to fail very sporadically with no real paper trail. There is a PR out right now with Windows2019 being built and the tests pass there:

[2020-04-20T17:48:16.371Z]          test_sync_all (integration.modules.test_saltutil.SaltUtilSyncModuleTest)
[2020-04-20T17:49:03.695Z]        [CPU:0.0%|MEM:61.1%]  ... ok (47.939s)
[2020-04-20T17:49:03.695Z]          test_sync_all_blacklist (integration.modules.test_saltutil.SaltUtilSyncModuleTest)
[2020-04-20T17:49:52.830Z]        [CPU:0.0%|MEM:61.1%]  ... ok (49.093s)
[2020-04-20T17:49:52.830Z]          test_sync_all_blacklist_and_whitelist (integration.modules.test_saltutil.SaltUtilSyncModuleTest)
[2020-04-20T17:50:41.872Z]        [CPU:0.0%|MEM:61.2%]  ... ok (49.111s)
[2020-04-20T17:50:41.872Z]          test_sync_all_whitelist (integration.modules.test_saltutil.SaltUtilSyncModuleTest)
[2020-04-20T17:51:31.767Z]        [CPU:0.0%|MEM:61.2%]  ... ok (48.783s)
[2020-04-20T17:51:31.767Z]          test_pillar_refresh (integration.modules.test_saltutil.SaltUtilSyncPillarTest)
[2020-04-20T17:51:31.767Z]        [CPU:0.0%|MEM:61.2%]  ... skip (0.000s)
[2020-04-20T17:51:31.767Z]          test_sync_grains (integration.modules.test_saltutil.SyncGrainsTest)
[2020-04-20T17:51:45.763Z]        [CPU:0.0%|MEM:61.2%]  ... ok (14.328s)

So immediate thinking is put sleeps in the test or skip them for now until a full test run can be run locally with the log file investigated. Otherwise can't repro locally and cant preproduce on a Windows 2019 VM.

@xeacott xeacott added team-windows cannot-reproduce cannot be replicated with info/context provided labels Apr 20, 2020
@xeacott
Copy link
Contributor

xeacott commented Apr 20, 2020

Link to that PR #56637

@Ch3LL
Copy link
Contributor Author

Ch3LL commented Apr 22, 2020

You should be able to look at the full log file on jenkins where the test fails in the build artifacts. Also if you see any way to make this test less flaky that is an approach you can take as well.

Also did you try running the full test suite to try to reproduce? It seems to be happening pretty consistently across tests. I want to guess 1 out of 3-5 runs.

@xeacott
Copy link
Contributor

xeacott commented Apr 24, 2020

I may look at making the test less flakey. I have a VM running nightly that is running the full suite, so im investigating those logs :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cannot-reproduce cannot be replicated with info/context provided Test-Failure Fix the CI failure ⛈
Projects
None yet
Development

No branches or pull requests

3 participants