search

opendistro-for-elasticsearch / opendistro-build

🧰 Open Distro Build Scripts
https://opendistro.github.io/
Apache License 2.0
344 stars 175 forks source link

Publish helm chart using @helm/chart-releaser-action #774

Open mcbenjemaa opened 3 years ago

mcbenjemaa commented 3 years ago

Issue #294,:

Fixes #294

Description of changes:

Ref: https://github.com/helm/charts-repo-actions-demo

Add @helm/chart-testing-action - To lint chart on PRs Add @helm/chart-releaser-action - To publish Helm release on gh-pages

Works well on my Fork: https://github.com/mcbenjemaa/opendistro-build https://mcbenjemaa.github.io/opendistro-build/index.yaml

Test Results:

Updated README.md

Note: If this PR is related to Helm, please also update the README for related documentation changes. Thanks. https://github.com/opendistro-for-elasticsearch/opendistro-build/blob/master/helm/README.md

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 signing off your commits, please check here.

mcbenjemaa commented 3 years ago

/cc @DerekHeldtWerle @kalvinnchau

peterzhuamazon commented 3 years ago

Hi @mcbenjemaa we have moved to OpenSearch (opensearch.org) And we just merged the 1st version of Helm there https://github.com/opensearch-project/opensearch-devops/pull/4 Could you close this PR and migrate there? Thanks.

jankosecki commented 3 years ago

Hi @mcbenjemaa we have moved to OpenSearch (opensearch.org) And we just merged the 1st version of Helm there opensearch-project/opensearch-devops#4 Could you close this PR and migrate there? Thanks.

Do you think it would be worth to add a note about the move to the top of the README. If I haven't checked what's the status of the issue, I would be still using this repo. Is everything under opendistro-for-elasticsearch moving to opensearch-project?