-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[OpenSearch Playground] Adds GitHub action file for deployment #42
Conversation
Signed-off-by: Tao liu <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm. Left 1 minor comment
- 'config/playground/helm/prod/**' | ||
|
||
OS-OSD-Dev-Deployment: | ||
if: ${{ needs.Pre-Deployment.outputs.config_change_dev == 'true'}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I saw this has been used in many places, maybe consider making it an env variable if possible?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to pass the environment variable to different jobs, not the same job. If we pass it in the same jobs, we can create a environment for it. the way that we pass the value to different jobs is leveraging outputs from job. the reference is here https://docs.github.com/en/actions/learn-github-actions/environment-variables#passing-values-between-steps-and-jobs-in-a-workflow
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see, that makes sense.
Signed-off-by: Tao liu <[email protected]>
Signed-off-by: Tao liu [email protected]
Description
Adds GitHub action yml file, which was used for deploying OpenSearch and OpenSearch Dashboards.
NOTE, this PR is initial request for the action file, it didn't include the full deployment steps as Helm configuration file was not checked in, the related PR is here
Issues Resolved
Resolve #21
Check List
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.