Enable integration test failure notification via GitHub issues #3395
Conversation
gaiksaya
requested review from
dblock,
peterzhuamazon,
bbarani,
rishabh6788,
zelinh,
jordarlu,
prudhvigodithi,
Divyaasm and
tianleh
as code owners
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## main #3395 +/- ##
=======================================
Coverage 91.74% 91.74%
=======================================
Files 172 172
Lines 4991 4991
=======================================
Hits 4579 4579
Misses 412 412 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
…enSearch plugins Signed-off-by: Sayali Gaikawad <gaiksaya@amazon.com>
| repoUrl: buildManifestObj.getRepo("${local_component}"), | ||
| issueTitle: "Integration Test failed for component ${local_component}: ${version}", | ||
| issueBody: issueBodyMessage, | ||
| label: "autocut,v${version},test-failure" |
There was a problem hiding this comment.
Do we need to be more specific with label here? i.e integ-test-failure instead of test-failure? We will be creating GitHub issues based on other test failures (BWC, performance test) soon so I think it would be better to add a specific label along with test-failure? What do you think?
Signed-off-by: Sayali Gaikawad <gaiksaya@amazon.com>
|
Labels have been created for opensearch-plugins |
Description
Enable notification for failed integration tests for opensearch-plugins. This will create GitHub issues for example: opensearch-project/k-NN#860
To-Do: Createinteg-test-failurelabel across all opensearch plugins repos once this PR is approved.Issues Resolved
closes #3383
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.