Skip to content

Backport to 1.0 from main #47

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

Merged
merged 8 commits into from
Aug 6, 2021
Merged

Conversation

VijayanB
Copy link
Member

@VijayanB VijayanB commented Aug 5, 2021

Description

Backport #36 #39 #41 #43 #45 #46 #44

Issues Resolved

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has documentation added
  • Commits are signed as per the DCO using --signoff

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.

VijayanB and others added 8 commits August 5, 2021 14:29
Added version and major version to error message if cluster
is not compatible.

Signed-ofr-by: Vijayan Balasubramanian <balasvij@amazon.com>
Signed-off-by: Vijayan Balasubramanian <balasvij@amazon.com>
Signed-off-by: Vijayan Balasubramanian <balasvij@amazon.com>
…opensearch (opensearch-project#41)

* Add new docker building scripts for logstash

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* Move build related vars to the top

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
…oject#43)

* Add new docker building scripts for logstash

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* Move build related vars to the top

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* Tweak and improve the docker script comments

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* Resolve gem building incorrectly due to move of spec files

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
…ct#45)

* Add new docker building scripts for logstash

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* Move build related vars to the top

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* Tweak and improve the docker script comments

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* Resolve gem building incorrectly due to move of spec files

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* Remove the gem from docker image after installation

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
* Add the latest tag during the push

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* Make rm verbose and remove the # before plugin creation scripts

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
This gem is cryptographically signed. To be sure the gem you install hasn’t been tampered with:
Add certs/opensearch-rubygems.pem (if you haven’t already) as a trusted certificate
gem cert --add <(curl -Ls certs/opensearch-rubygems.pem)
gem install logstash-output-openserach -P HighSecurity
The HighSecurity trust profile will verify signed gems, and allow only if all the dependencies are signed.
This is created by following https://guides.rubygems.org/security/#building-gems

Signed-off-by: Vijayan Balasubramanian <balasvij@amazon.com>
@VijayanB VijayanB self-assigned this Aug 5, 2021
@VijayanB VijayanB requested review from jmazanec15 and vamshin August 5, 2021 21:34
@VijayanB VijayanB merged commit 8d27c6f into opensearch-project:1.0 Aug 6, 2021
@VijayanB VijayanB deleted the backport-1.0 branch August 6, 2021 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants