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.

Sign up for GitHub

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

Jump to bottom

CI: 'Upload Surefire reports' also for cancelled() #16644

Merged
merged 1 commit into from
Apr 20, 2021
Merged

CI: 'Upload Surefire reports' also for cancelled() #16644

merged 1 commit into from
Apr 20, 2021

Conversation

famod
Copy link
Member

@famod famod commented Apr 19, 2021

@quarkus-bot quarkus-bot bot added the area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure label Apr 19, 2021
@quarkus-bot
Copy link

quarkus-bot bot commented Apr 20, 2021

Failing Jobs - Building 520bf68

Status Name Step Test failures Logs Raw logs
✔️ JVM Tests - JDK 11
⌛ JVM Tests - JDK 11 Windows Build ⚠️ Check → Logs Raw logs
✔️ JVM Tests - JDK 15

@famod
Copy link
Member Author

famod commented Apr 20, 2021 •
edited
Loading

@gsmet two things here:

  • The upload step kicked in for JVM Tests - JDK 11 Windows, which is good, but the bot seems to ignore the uploaded reports.
  • I'm not sure how it will look like when the entire workflow is aborted in case of a (rebase) push. It might be irritating if test failures are shown that have popped up in the run.

@famod
Copy link
Member Author

famod commented Apr 20, 2021

The upload step kicked in for JVM Tests - JDK 11 Windows, which is good, but the bot seems to ignore the uploaded reports.

Nevermind, there is no test failure. Need coffee.

@gsmet gsmet merged commit f0777b5 into quarkusio:main Apr 20, 2021
@quarkus-bot quarkus-bot bot added this to the 2.0 - main milestone Apr 20, 2021
@famod famod deleted the ci-surefire-reports-cancelled branch April 20, 2021 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gsmet gsmet gsmet approved these changes

Assignees
No one assigned
Labels
area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure
Projects
None yet
Milestone
2.0.0.Alpha1
Development

Successfully merging this pull request may close these issues.
2 participants
@famod @gsmet