-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Multiple tests fail: [Errno 2] No such file or directory: '/home/cloudtest/.azure/dummy_cli_config_dir/xGYaEIdmTuUx36nO/commandIndex.json' #28848
Comments
Thank you for opening this issue, we will look into it. |
This issue now even happens on Interesting thing is that it only happens on instance 3 and certain tests:
|
Possibly caused by #28673. I am able to repro locally with
It seems test classes are collected by However, this still can't answer why previous tests didn't fail, such as https://dev.azure.com/azclitools/public/_build/results?buildId=152726&view=logs&j=d3f69986-121a-5641-0308-9a539278aa7f&t=2538cf34-13d6-5659-841c-bf258a998985 |
This error doesn't happen without
azure-cli/src/azure-cli-core/azure/cli/core/_session.py Lines 88 to 110 in fa307d6
Without When running |
It seems that this is caused by the internal change in #28807 succeeded but #28817 failed.
I added some log in test initialization code. It seems that each test is initialized and executed in different thread, and the random config dir env is not polluted. In 8.2.0, the |
This is very interesting discovery. By injecting import os
print('__init__', os.getpid(), method_name) at azure-cli/src/azure-cli-testsdk/azure/cli/testsdk/base.py Lines 83 to 86 in 2750c65
we can see different versions of pytest show different output: With pytest 8.1.1:
With pytest 8.2.0:
We can see pytest 8.2.0:
|
I reported an issue to pytest: pytest-dev/pytest#12289 |
https://dev.azure.com/azclitools/public/_build/results?buildId=153071&view=logs&j=f127618a-a658-5219-f5b0-4a245c9227b8&t=60a7c4e7-f35f-5fef-6efb-f1b54b8cb075&l=1659
The text was updated successfully, but these errors were encountered: