Move old jenkins to legacy and ignore tests #5607
Conversation
Signed-off-by: Sayali Gaikawad <[email protected]>
gaiksaya
requested review from
Divyaasm,
peterzhuamazon,
prudhvigodithi,
rishabh6788,
tianleh and
zelinh
as code owners
There was a problem hiding this comment.
Is this legacy as well now?
There was a problem hiding this comment.
Yes! Actual file in use https://github.com/opensearch-project/data-prepper/blob/main/jenkins/release.jenkinsFile
| import static com.lesfurets.jenkins.unit.global.lib.GitSource.gitSource | ||
|
|
||
| class TestCCRPerfTest extends BuildPipelineTest { | ||
| @Ignore class TestCCRPerfTest extends BuildPipelineTest { |
There was a problem hiding this comment.
Why ignore instead of remove here?
Unless you are thinking about potential re-use them later?
There was a problem hiding this comment.
There was a push back last time when I tried removing. Let me know what you prefer. I am okay with removing them as well.
github-project-automation
bot
moved this to 👀 In Review
in Engineering Effectiveness Board
github-project-automation
bot
moved this from 👀 In Review
to ✅ Done
in Engineering Effectiveness Board
Codecov ReportAll modified and coverable lines are covered by tests ✅
❌ Your project status has failed because the head coverage (0.00%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #5607 +/- ##
==========================================
- Coverage 92.01% 0 -92.02%
==========================================
Files 202 0 -202
Lines 7078 0 -7078
==========================================
- Hits 6513 0 -6513
+ Misses 565 0 -565 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I have updated the jenkinsfile file location for https://build.ci.opensearch.org/job/publish-to-maven/ workflow. |
Signed-off-by: Sayali Gaikawad <[email protected]>
2 participants
Description
Move old jenkins to legacy and ignore tests associated with those jenkins files.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.