Skip to content

YETUS-871. cleanup the junit output format#70

Merged
aw-was-here merged 2 commits intoapache:masterfrom
effectivemachines:YETUS-871
Aug 15, 2019
Merged

YETUS-871. cleanup the junit output format#70
aw-was-here merged 2 commits intoapache:masterfrom
effectivemachines:YETUS-871

Conversation

@aw-was-here
Copy link
Contributor

No description provided.

@aw-was-here aw-was-here changed the title YETUS-871. junit cleanup YETUS-871. cleanup the junit output format Aug 14, 2019
Copy link
Contributor

@busbey busbey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should change the various CI config files we have that still refer to --junit-results-xml in the same PR.

@aw-was-here
Copy link
Contributor Author

we should change the various CI config files we have that still refer to --junit-results-xml in the same PR.

Doh. Good catch. I'll let it run through all of the CI systems in my fork to see what happens.

@apache-yetus
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/yetus-github-multibranch/job/PR-70/1/console in case of problems.

@apache-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
💙 reexec 2m 37s Docker mode activated.
_ Prechecks _
💚 dupname 0m 0s No case conflicting files found.
💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
💙 mvndep 0m 15s Maven dependency ordering for branch
_ Patch Compile Tests _
💙 mvndep 0m 4s Maven dependency ordering for patch
💚 shellcheck 0m 0s There were no new shellcheck issues.
💚 shelldocs 0m 0s There were no new shelldocs issues.
💚 whitespace 0m 0s The patch has no whitespace issues.
💚 yamllint 0m 2s There were no new yamllint issues.
_ Other Tests _
💚 asflicense 0m 15s The patch does not generate ASF License warnings.
3m 35s
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/yetus-github-multibranch/job/PR-70/1/artifact/out/Dockerfile
GITHUB PR #70
Optional Tests dupname asflicense yamllint shellcheck shelldocs
uname Linux c397f359a946 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality precommit/src/main/shell/personality/yetus.sh
git revision master / e8f542b
Max. process+thread count 45 (vs. ulimit of 2000)
modules C: . precommit U: .
Console output https://builds.apache.org/job/yetus-github-multibranch/job/PR-70/1/console
versions git=2.7.4 maven=3.3.9 shellcheck=0.7.0 yamllint=1.12.1
Powered by Apache Yetus 0.11.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

@aw-was-here
Copy link
Contributor Author

Tested this on a variety of CI systems, all appear to be working as expected.

@aw-was-here
Copy link
Contributor Author

Thanks!

@aw-was-here aw-was-here merged commit 557b023 into apache:master Aug 15, 2019
@aw-was-here aw-was-here deleted the YETUS-871 branch August 15, 2019 05:48
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.

3 participants