gaiksaya commented 3 years ago

Description of changes: The infrastructure team is separating the production and staging locations into different AWS accounts. Plugins need to modify their workflows to publish to the new locations.

This PR changes the CD workflow to add a build number and write the zip, deb, and rpm plugin artifacts to staging.artifacts.opendistroforelasticsearch.amazon.com.

Test Result: https://github.com/opendistro-for-elasticsearch/k-NN/runs/1685141609?check_suite_focus=true

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

codecov[bot] commented 3 years ago

Codecov Report

Merging #301 (8685bf4) into master (79f7748) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #301   +/-   ##
=========================================
  Coverage     80.51%   80.51%           
  Complexity      388      388           
=========================================
  Files            62       62           
  Lines          1468     1468           
  Branches        130      130           
=========================================
  Hits           1182     1182           
  Misses          239      239           
  Partials         47       47

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 79f7748...8685bf4. Read the comment docs.

VijayanB commented 3 years ago

LGTM

opendistro-for-elasticsearch / k-NN

Change CD workflow to use new staging bucket for artifacts #301

Codecov Report