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

Use GH CI to upload test output for any job that fails tests #2606

Merged
merged 2 commits into from
Jun 12, 2020

Conversation

lgritz
Copy link
Collaborator

@lgritz lgritz commented Jun 11, 2020

This will make it much easier to investigate failures or update ref
output when failures only happen on CI -- any test that fail tests
will produce a downloadable "artifact" consisting of all tests'
captured console output.

Signed-off-by: Larry Gritz lg@larrygritz.com

This will make it much easier to investigate failures or update ref
output when failures only happen on CI -- any test that fail tests
will produce a downloadable "artifact" consisting of all tests'
captured console output.

Signed-off-by: Larry Gritz <lg@larrygritz.com>
@fpsunflower
Copy link
Contributor

Do we have any tests that output images? Or might we have any in the future? Maybe capture out.* instead of only out.txt ?

@lgritz
Copy link
Collaborator Author

lgritz commented Jun 11, 2020

Most tests (other than a few utility unit tests) output images. But then they usually compare to a reference image and print the results. Lots of tests are printing metadata or hashes to verify images.

Because of how large the artifact would be if I included all image output, and that my first important use of this was for text output (separate PR coming soon), I omitted it for now, thinking that I could add if/when we need it. But if you think we should just be proactive and bundle them all now, it's an easy change.

Yeah, sure, that makes sense. I'll amend.

I'm gonna add the same bits to OSL's CI as well, this is super useful, especially for those test differences that are acceptable differences, but just ever so slightly varying for each platform, so I can't quite get the same results on the machines I have direct access to.

Signed-off-by: Larry Gritz <lg@larrygritz.com>
@lgritz
Copy link
Collaborator Author

lgritz commented Jun 12, 2020

Amended: We now upload all outputs (including images), but only from the actual tests that fail.

@fpsunflower
Copy link
Contributor

Sweet - will be very handy on the OSL side too. I was just thinking about the sampler change there and how to update the windows/mac ref images ...

@lgritz
Copy link
Collaborator Author

lgritz commented Jun 12, 2020

I'll submit the OSL version of this right away.

@lgritz lgritz merged commit ee07290 into AcademySoftwareFoundation:master Jun 12, 2020
@lgritz lgritz deleted the lg-artifact-ghci branch June 12, 2020 05:38
lgritz added a commit to lgritz/OpenImageIO that referenced this pull request Jul 9, 2020
…SoftwareFoundation#2606)

This will make it much easier to investigate failures or update ref
output when failures only happen on CI -- any test that fail tests
will produce a downloadable "artifact" consisting of the outputs
of all the failed tests.

Signed-off-by: Larry Gritz <lg@larrygritz.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants