-
Notifications
You must be signed in to change notification settings - Fork 481
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
Stabilize test_streaming.py by scanning the data_dir, not the parent of data_dir #3641
Conversation
Actually, I changed the build section of the workflow to "ubuntu-20.04" attempting to solve a build problem with the "py39_2" branch, but the testing sections are still "ubuntu-latest". So I'm not sure why test_streaming.py is not causing more trouble at head-of-line. |
14c9b43
to
2703628
Compare
@moodyjon can you explain the problem a little more? i don't quite understand what this is fixing. what is the value of |
When running tests in the GitHub action, I had some debugging changes in place to print the dir contents. What I saw appeared to the the contents of /tmp. We don't want to be looking in /tmp, we want to be looking in the test-specific /tmp, or perhaps one of the subdirs like <data_dir>/blobfiles. <data_dir>/uploads. It's quite possible the test is looking in the wrong place even now. Should it be looking under /blobfiles? On MacOS, its not /tmp, but /var/folders/NN/NNNNNNNNNNNNNNN/T seems to play that role:
|
Some of the other tests take a different approach. They start with the full_path of the stream, then scan the contents of dirname(full_path).
Looks to me as if |
@moodyjon okay, that makes sense now. please rebase and I'll merge. |
2703628
to
227683e
Compare
Unbundled from #3623
Parent of "data_dir" seems to be "/tmp" and contents are always changing which disturbs test.
Apparently it's not causing a problem due to OS version and behavior of that OS version? It's causing major problems on upgrade to 20.04 for testing.