Message ID | 20240203092538.5491-1-alexis.lothore@bootlin.com |
---|---|
Headers | show |
Series | testimage: enable artifacts retrieval for failed tests | expand |
On 2/3/24 10:25, Alexis Lothoré via lists.openembedded.org wrote: > From: Alexis Lothoré <alexis.lothore@bootlin.com> > > TESTIMAGE_FAILED_QA_ARTIFACTS currently sets a default list of files to be > saved whenever some tests fail. Unfortunately, this default value is very > easily discarded, because TESTIMAGE_FAILED_QA_ARTIFACTS is expected to be > enriched by some core recipes (example: ptest images) which are often > parsed before testimage.bbclass. Those core recipes could still manage to > get the default value AND add some files by using override syntax, but then > it makes it difficult for downstream recipes or bbappend files to tune this > variable. > > Allow to set this default value and make sure it is not overwritten by > recipes wanting to tune it, by setting the default value in > core-image. > While doing so, set it as a default value instead of a weak default value. And I realize that I forgot to remove "minimal" from the commit message, which is not true anymore for this version. I'll wait a bit before resending, it looks like I have some formatting issues detected by patchtest too