Skip to content

Conversation

@VijayanB
Copy link
Member

Signed-off-by: Vijayan Balasubramanian [email protected]

Description

Added script to build docker image.

Issues Resolved

[List any issues this PR will resolve]

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 VijayanB requested review from jmazanec15 and vamshin June 26, 2021 20:15
@VijayanB VijayanB self-assigned this Jun 26, 2021
Copy link
Member

@vamshin vamshin 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

@@ -0,0 +1,21 @@
#!/bin/bash

# This is intended to be run the plugin's root directory. `dockerfiles/run.sh`
Copy link
Member

Choose a reason for hiding this comment

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

When is this script used?

Copy link
Member Author

Choose a reason for hiding this comment

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

It is like build from source, here, instead of running in local, it will be executed inside the docker. I am planning to update the Developer guide to include this as well. Also, will use the same script without "docker-compose up" to publish the image to docker hub as well.

@VijayanB VijayanB force-pushed the docker-build branch 5 times, most recently from 50a8d9a to 2d17e0c Compare July 6, 2021 23:57
VijayanB added 2 commits July 6, 2021 17:02
Added script to build docker image based on host's architecture.
Supports only arm64 and x86_64, download corresponding base image and build.

Signed-off-by: Vijayan Balasubramanian <[email protected]>
Include release drafts similar to other plugins.

Signed-off-by: Vijayan Balasubramanian <[email protected]>
Since openseach supports only version 7, remove irrelevant warning
message.

Signed-off-by: Vijayan Balasubramanian <[email protected]>
@VijayanB VijayanB requested review from jmazanec15 and vamshin July 7, 2021 22:26
@VijayanB VijayanB added v1.0.0 v1.1.0 latest release labels Jul 7, 2021
Copy link
Member

@vamshin vamshin 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

@VijayanB VijayanB merged commit b2bf14f into opensearch-project:main Jul 8, 2021
@VijayanB VijayanB deleted the docker-build branch July 8, 2021 17:23
@VijayanB VijayanB restored the docker-build branch July 8, 2021 17:23
@VijayanB VijayanB added the Infrastructure Changes to infrastructure, testing, CI/CD, pipelines, etc. label Jul 9, 2021
This was referenced Jul 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Infrastructure Changes to infrastructure, testing, CI/CD, pipelines, etc. v1.0.0 v1.1.0 latest release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants