This repository was archived by the owner on Aug 5, 2022. It is now read-only.
CI Jenkinsfile,tester-exec: streamline, fix issue of lost test results #102
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
okartau
force-pushed
the
try-parallel-tests
branch
9 times, most recently
from
440cebd to
10ef480
Compare
okartau
changed the title
Contributor
Author
|
eh, bad turn, 3rd run left some results out, so moving xunit step into finally block is not magical fix |
okartau
force-pushed
the
try-parallel-tests
branch
from
9cf7d17 to
d6d989d
Compare
Contributor
Author
|
updated junit plugin (plus restart of jenkins), lets see now.. test this please |
okartau
force-pushed
the
try-parallel-tests
branch
5 times, most recently
from
9ad738d to
c21b37c
Compare
okartau
changed the title
Contributor
Author
|
test this please, now looks promising with locking added. just get more coverage |
okartau
changed the title
okartau
changed the title
okartau
force-pushed
the
try-parallel-tests
branch
2 times, most recently
from
b619b39 to
ae79de3
Compare
Split and use first elem can be done on one line. Comment made shorted and more concise. Signed-off-by: Olev Kartau <olev.kartau@intel.com>
Tester archived artifacts using "any path" wildcard **/ which has been dragged along from some older life of this script. Currently all result files are in top level directory. Signed-off-by: Olev Kartau <olev.kartau@intel.com>
This improves readability and shortens lines, also makes it easier to write repeated tester blocks. Signed-off-by: Olev Kartau <olev.kartau@intel.com>
No functional change Signed-off-by: Olev Kartau <olev.kartau@intel.com>
It was discovered we lost one or more tester result file sets from xUnit processing every now and then. Log files analysis shows that when multiple testers attempted xUnit step in nearly same time, it caused loss of data. We can avoid same-time run by locking this section. Signed-off-by: Olev Kartau <olev.kartau@intel.com>
Few symlinks exist after testing which point out of local tree, created by tested device. These become broken when exported to CI master, and will show as garbage. Signed-off-by: Olev Kartau <olev.kartau@intel.com>
This tarball has no use. Signed-off-by: Olev Kartau <olev.kartau@intel.com>
If a variable is used only once, or just decodes part of name by adding it as part of variable name, it is not really helpful. Reduce amount of lines by not defining such variables. Signed-off-by: Olev Kartau <olev.kartau@intel.com>
okartau
force-pushed
the
try-parallel-tests
branch
from
ae79de3 to
15346e8
Compare
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Streamline, cleanup, restructure.
5. commit solves the recently found issue of missing xUnit result blocks in some CI jobs
Signed-off-by: Olev Kartau olev.kartau@intel.com