Uh oh!
There was an error while loading. Please reload this page.
bpo-36511: Fix Windows arm32 buildbot, pythoninfo, scp, and ssh - #91
Conversation
zooba
commented
May 20, 2019
I think this could also be done by handling the flag specially in the test.bat file, yes? Rather than making more modifications here. I'll leave it to @zware as to where he'd like the split to be - I don't have a strong preference. |
paulmon
commented
May 20, 2019
I found that test.pythoninfo is called directly from a buildbot step without calling any of the .bat files in tools\buildbot currently. This is why I didn't catch it before. |
| ShellCommand( | ||
| name="pythoninfo", | ||
| description="pythoninfo", | ||
| command=self.python_command + ["-m", "test.pythoninfo"], |
There was a problem hiding this comment.
This line doesn't call any .bat files. It ends up executing an arm32 binary on the x86/x64 build host which doesn't work
paulmon
commented
May 22, 2019
The must-have parts of this PR are that the test files must be copied to the test device before pythoninfo and pythoninfo must be run on the test device, not the build host. Currently it's not possible to change where pythoninfo runs without changing the buildmaster-config |
paulmon
commented
May 29, 2019
In my manual testing it looks to me like this change with these 2 changes will get the arm32 buildbot to zero test failures I will be away until Mon Jun 3, but I'd like to get this wrapped up soon if possible. Thanks, |
zooba
commented
Jun 11, 2019
@zware Need you to approve this, but it looks okay to me. |
zware
commented
Jun 13, 2019
Sorry for the delay. This was not the solution I'd have preferred for this issue, but I'm not going to have time to work that out and it's not fair to try to make you do it when I'm not even sure it would work :). This will work and will solve the issue, so we'll go with it. |
paulmon
commented
Jun 13, 2019
Thanks! If you have suggestions for making it better I am interested in helping out. |
zware
commented
Jun 13, 2019
What I think I'd like to see is separate builders for the 'build' and 'test' portions running on separate workers with the 'test' builders triggered by success on the 'build' builders, so that everything can just be native commands without dealing with SSH or special batch files. This does require |
paulmon
commented
Jun 19, 2019
That sounds like a great idea. I can't get to right now, but I'll keep it mind |
When running the buildbot for Windows ARM32 test.pythoninfo must be run remotely.
Also seperating deploy step from both pythoninfo and test.bat
This requires a cpython PR to work: python/cpython#13454
I think this should work. Feedback is appreciated.
@zooba@zware