Skip to content
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

[Backport 2.x] [MD]Add data source config to opensearch-dashboards-docker #2563

Merged
merged 1 commit into from
Oct 13, 2022

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 8e8b89b from #2557

Signed-off-by: Kristen Tian <tyarong@amazon.com>

Signed-off-by: Kristen Tian <tyarong@amazon.com>
(cherry picked from commit 8e8b89b)
@codecov-commenter
Copy link

codecov-commenter commented Oct 13, 2022

Codecov Report

Merging #2563 (954ed64) into 2.x (954ed64) will not change coverage.
The diff coverage is n/a.

❗ Current head 954ed64 differs from pull request most recent head 0251a55. Consider uploading reports for the commit 0251a55 to get more accurate results

@@           Coverage Diff           @@
##              2.x    #2563   +/-   ##
=======================================
  Coverage   66.71%   66.71%           
=======================================
  Files        3204     3204           
  Lines       60964    60964           
  Branches     9267     9267           
=======================================
  Hits        40673    40673           
  Misses      18073    18073           
  Partials     2218     2218           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@kavilla
Copy link
Member

kavilla commented Oct 13, 2022

Nice @kristenTian,

Do we have a follow-up to add it the same: https://github.com/opensearch-project/opensearch-build/blob/main/docker/release/config/opensearch-dashboards/opensearch-dashboards-docker-entrypoint.sh#L156

I know this is duplicated and we should fix this in the build repo to pull from our versioned docker stuff.

@kavilla kavilla merged commit 7c1475f into 2.x Oct 13, 2022
@github-actions github-actions bot deleted the backport/backport-2557-to-2.x branch October 13, 2022 16:25
kristenTian added a commit to kristenTian/opensearch-build that referenced this pull request Oct 18, 2022
kristenTian added a commit to kristenTian/opensearch-build that referenced this pull request Nov 1, 2022
peterzhuamazon pushed a commit to opensearch-project/opensearch-build that referenced this pull request Nov 1, 2022
Related PR: opensearch-project/OpenSearch-Dashboards#2563

Signed-off-by: Kristen Tian <tyarong@amazon.com>

Signed-off-by: Kristen Tian <tyarong@amazon.com>
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