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

SQL & Observability Repo's split need build support for the new structure #3044

Closed
YANG-DB opened this issue Jan 5, 2023 · 4 comments
Closed
Labels
enhancement New Enhancement

Comments

@YANG-DB
Copy link
Member

YANG-DB commented Jan 5, 2023

Is your feature request related to a problem? Please describe

Repository split as discussed here...
See opensearch-project/sql#640
See opensearch-project/observability#803
See #2188

Describe the solution you'd like

Recently the next changes have taken place to allow better utilization of the existing repositories

SQL was split into the following repos:

Observability was split into the following repos:

Reporting was split into the following repos:

Describe alternatives you've considered

No response

Additional context

New plugins need to be build with the same composition as the former plugins assembed

  • SQL (backend repo)

    • sql-jdbc
    • sql-odbc
    • sql-cli
    • dashboards-query-workbench (frontend)
  • Observability (backend repo)

    • dashboards-observability (frontend)
  • Reporting (backend repo)

    • reporting-cli
    • dashboards-reporting (frontend)
@YANG-DB YANG-DB added enhancement New Enhancement untriaged Issues that have not yet been triaged labels Jan 5, 2023
@pjfitzgibbons
Copy link

As a clarification to build team :

The "backend" repos (those with no prefix - observability, reporting ) have been updated to remove the front-end section of the codebase. These repos are likely actively failing builds right now.

The "frontend" repos dashboards-observability, dashboards-reporting are "new to you", and need to be checked for proper GH workflow steps as well as verify the now "alternate" artifact paths for front-end are accounted for in the build system.

@prudhvigodithi
Copy link
Member

[Triage] Hey @YANG-DB @pjfitzgibbons we will get back on this after having a team level discussion.

@bbarani
Copy link
Member

bbarani commented Feb 2, 2023

@prudhvigodithi Can we close this issue?

@prudhvigodithi
Copy link
Member

prudhvigodithi commented Feb 2, 2023

Hey Closing this issue, the 2.5.0 release is already done with the repo split and with new naming convention.
https://github.com/opensearch-project/opensearch-build/blob/main/manifests/2.5.0/opensearch-dashboards-2.5.0.yml#L25-L29.
@YANG-DB @pjfitzgibbons closing this issue, please reopen if you see any issue.
Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New Enhancement
Projects
None yet
Development

No branches or pull requests

4 participants