Skip to content

Conversation

@peterzhuamazon
Copy link
Member

@peterzhuamazon peterzhuamazon commented Jul 29, 2021

Signed-off-by: Peter Zhu [email protected]

Description

Add new docker Multi-Arch image building scripts for logstash-output-opensearch

Issues Resolved

#42

Screen Shot 2021-07-29 at 3 37 30 PM

build-docker-multi-arch.log

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.

@peterzhuamazon peterzhuamazon added the enhancement Increases software capabilities beyond original client specifications label Jul 29, 2021
@peterzhuamazon peterzhuamazon changed the title Add new docker Multi-Arch image building scripts for logstash Add new docker Multi-Arch image building scripts for logstash-output-opensearch Jul 29, 2021
@VijayanB VijayanB requested a review from jmazanec15 July 29, 2021 19:52
Copy link
Member

@VijayanB VijayanB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks

@peterzhuamazon peterzhuamazon merged commit 175f957 into main Jul 29, 2021
@peterzhuamazon peterzhuamazon deleted the opensearch-logstash-multiarch branch July 29, 2021 20:07
VijayanB pushed a commit to VijayanB/logstash-output-opensearch that referenced this pull request Aug 5, 2021
…opensearch (opensearch-project#41)

* Add new docker building scripts for logstash

Signed-off-by: Peter Zhu <[email protected]>

* Move build related vars to the top

Signed-off-by: Peter Zhu <[email protected]>
@VijayanB VijayanB mentioned this pull request Aug 5, 2021
5 tasks
VijayanB pushed a commit to VijayanB/logstash-output-opensearch that referenced this pull request Aug 5, 2021
…opensearch (opensearch-project#41)

* Add new docker building scripts for logstash

Signed-off-by: Peter Zhu <[email protected]>

* Move build related vars to the top

Signed-off-by: Peter Zhu <[email protected]>
This was referenced Aug 5, 2021
VijayanB pushed a commit to VijayanB/logstash-output-opensearch that referenced this pull request Aug 5, 2021
…opensearch (opensearch-project#41)

* Add new docker building scripts for logstash

Signed-off-by: Peter Zhu <[email protected]>

* Move build related vars to the top

Signed-off-by: Peter Zhu <[email protected]>
@VijayanB VijayanB mentioned this pull request Aug 5, 2021
5 tasks
VijayanB pushed a commit that referenced this pull request Aug 6, 2021
…opensearch (#41)

* Add new docker building scripts for logstash

Signed-off-by: Peter Zhu <[email protected]>

* Move build related vars to the top

Signed-off-by: Peter Zhu <[email protected]>
VijayanB pushed a commit that referenced this pull request Aug 6, 2021
…opensearch (#41)

* Add new docker building scripts for logstash

Signed-off-by: Peter Zhu <[email protected]>

* Move build related vars to the top

Signed-off-by: Peter Zhu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Increases software capabilities beyond original client specifications

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Enhancement] Adding Multi-Arch Support for Docker Images in Logstash-output-opensearch

3 participants