Allow Running Locally and Add Test Matrix #8
Conversation
|
When nothing breaks feel free to merge. In the past we did run the test matrix on the ATCD scoreboard but it was disabled at some point because we ran out of disk space at Vanderbilt. |
|
I would guess that would be caused by the database part. I removed that in DOCGroup/autobuild#74. |
|
So I'm pretty confident that this should work since it works for me locally. The only thing I could see breaking is if there's something weird about the system like it's pretty old. |
|
Integrated page is now empty and the new links to the matrix pages give an error |
|
What is the output the |
|
So I ran it locally again and I don't have the same results, but the issue I saw was that some of our builds weren't available anymore. At the very least I have to change the test matrix to handle builds being missing. I still would like to see the output of that script though. |
|
Okay, another thought I had is the way I think I also see why the integrated page broke. I added a new elements to the scoreboard build XML and for some reason the integrated page generator uses a very similar, but completely separate XML parser for the very same file. I'm still not sure why I didn't see error this before I merged the PR though. |
See DOCGroup#8. These changes try to make the script more robust by not exiting on error immediately and rerunning itself after an update.
See DOCGroup/scoreboard#8 These changes fix the parsing in the integrated page generator and try to make matrix.py more robust by inserting skipped tests when builds or test results cause an error instead of crashing.
|
I've created #9 and DOCGroup/autobuild#77. The script changes in this repo are not strictly necessary, but I thought it was a good idea to make the script a little more robust. You'll should to do a manual pull on this repo before running it again. |
Yeah, that's the same error I saw, so the autobuild PR should fix that. From the current scoreboard, it looks like you already ran with the fixed version. |
|
Yes, I ran it when I noticed your merge, it now all seems to work again, thanks for the improvements! |
Requires DOCGroup/autobuild#76