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

Replace jacoco Jenkins plugin by new plugin #2867

Merged
merged 11 commits into from
Feb 5, 2025
Merged

Replace jacoco Jenkins plugin by new plugin #2867

merged 11 commits into from
Feb 5, 2025

Conversation

DaGeRe
Copy link
Contributor

@DaGeRe DaGeRe commented Jan 2, 2025

Pull Request

Contribution

This pull request provides the following contribution to Kieker

  • Contribution 1 Update jacoco call to new Jenkins plugin

Code Quality Thresholds

  • It was necessary to raise any code quality thresholds
  • If yes, which had to be raised and why was it necessary?
    ..

DaGeRe and others added 4 commits January 2, 2025 18:11
Bumps [ch.qos.logback:logback-classic](https://github.com/qos-ch/logback) from 1.5.12 to 1.5.16.
- [Commits](qos-ch/logback@v_1.5.12...v_1.5.16)

---
updated-dependencies:
- dependency-name: ch.qos.logback:logback-classic
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [ch.qos.logback:logback-access](https://github.com/qos-ch/logback-access) from 1.5.12 to 1.5.16.
- [Commits](https://github.com/qos-ch/logback-access/commits)

---
updated-dependencies:
- dependency-name: ch.qos.logback:logback-access
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@shinhyungyang
Copy link
Contributor

Quick update: we still get this error log messages:

java.io.IOException: No such file or directory
[Pipeline] recordCoverage
[Coverage] [-ERROR-] Errors during parsing
[Coverage] [-ERROR-] No files found for pattern '**/jacoco.xml'. Configuration error?
[Coverage] Recording coverage results
[Coverage] Creating parser for JaCoCo Coverage Reports
[Coverage] Using default pattern '**/jacoco.xml' since user defined pattern is not set
[Coverage] Searching for all files in '/home/jenkins-agent/workspace/kieker-dev_PR-2867' that match the pattern '**/jacoco.xml'
[Coverage] Traversing of symbolic links: disabled
[Coverage] Ignore errors and continue processing

@shinhyungyang
Copy link
Contributor

@DaGeRe Hi, could you have a look?

@DaGeRe
Copy link
Contributor Author

DaGeRe commented Feb 5, 2025

Thanks for the update, looks good. Nevertheless, we'll need to include the release checks again. I'll do so.

Copy link
Contributor

@shinhyungyang shinhyungyang left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@DaGeRe DaGeRe merged commit 65dc70d into main Feb 5, 2025
29 checks passed
@DaGeRe
Copy link
Contributor Author

DaGeRe commented Feb 5, 2025

Thanks, I've merged. We'll now need to rebase the other PRs to main and let them build.

@DaGeRe DaGeRe deleted the GH-5-Fix-Jacoco branch February 5, 2025 15:54
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.

2 participants