Add validation, deployment and indexing workflows #142
Conversation
gaiksaya
requested review from
bbarani,
TheAlgo,
DandyDeveloper,
peterzhuamazon and
prudhvigodithi
as code owners
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #142 +/- ##
=======================================
Coverage ? 83.33%
=======================================
Files ? 1
Lines ? 24
Branches ? 4
=======================================
Hits ? 20
Misses ? 4
Partials ? 0 ☔ View full report in Codecov by Sentry. |
| opensearch_url_latest=https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/${DIST_VERSION}/latest/linux/x64/tar/dist/opensearch/opensearch-${DIST_VERSION}-linux-x64.tar.gz | ||
| opensearch_dashboards_url_latest=https://ci.opensearch.org/ci/dbc/distribution-build-opensearch-dashboards/${DIST_VERSION}/latest/linux/x64/tar/dist/opensearch-dashboards/opensearch-dashboards-${DIST_VERSION}-linux-x64.tar.gz | ||
|
|
||
| opensearch_build_id=$(curl -w "%{url_effective}\n" -I -L -f -S $opensearch_url_latest -o /dev/null | cut -d / -f 8) |
There was a problem hiding this comment.
how about reading the latest dist yml file, and parsing that using yq to fetch distribution urls, build id etc?
e.g. https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/2.11.0/8649/linux/x64/tar/dist/opensearch/manifest.yml
There was a problem hiding this comment.
Thanks for the suggestion. That works much better than what I implemented. Added the change.
| with: | ||
| node-version: 16.x | ||
|
|
||
| - name: Deploy nightly playground |
There was a problem hiding this comment.
I am assuming if validation fails then the flow won't even reach here, right?
Signed-off-by: Sayali Gaikawad <[email protected]>
Signed-off-by: Sayali Gaikawad <[email protected]>
Signed-off-by: Sayali Gaikawad <[email protected]>
Description
This PR adds below:
Issues Resolved
resolves #131
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.