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.

Sign up for GitHub

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

Add observability, reporting, query workbench and gantt chart to manifests #3048

Merged
merged 6 commits into from
Jan 9, 2023
Merged

Add observability, reporting, query workbench and gantt chart to manifests #3048

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
da9450c
added dashboard observability, reports and gantt chart to manifest
mengweieric Jan 5, 2023
e1d1dd2
added opensearch observability and reports to manifest
mengweieric Jan 5, 2023
6990f24
resolve linting issue
mengweieric Jan 5, 2023
3aad1a5
reslove conflicts after rebasing
mengweieric Jan 8, 2023
0fbc29f
Merge branch 'main' into add-insight-manifests
mengweieric Jan 9, 2023
a64d189
Change ref to 2.5 branch for OS
zelinh Jan 9, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 10 additions & 2 deletions manifests/2.5.0/opensearch-2.5.0.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ ci:
components:
- name: OpenSearch
repository: https://github.com/opensearch-project/OpenSearch.git
ref: 2.x
ref: '2.5'
checks:
- gradle:publish
- gradle:properties:version
Expand Down Expand Up @@ -108,7 +108,6 @@ components:
platforms:
- linux
- windows
working_directory: opensearch-observability
Copy link
Member

Choose a reason for hiding this comment

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

Have we removed all front end folders in this repo and make it exclusive for backend? If not, removing the working_directory may cause build failure.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

there's a PR ready to merge which removes all front end stuff, will get it merged soon

checks:
- gradle:properties:version
- gradle:dependencies:opensearch.version
Expand All @@ -121,6 +120,15 @@ components:
checks:
- gradle:properties:version
- gradle:dependencies:opensearch.version: alerting
- name: opensearch-reports
repository: https://github.com/opensearch-project/reporting
ref: 2.x
platforms:
- linux
- windows
checks:
- gradle:properties:version
- gradle:dependencies:opensearch.version
- name: index-management
repository: https://github.com/opensearch-project/index-management.git
ref: 2.x
Expand Down
13 changes: 9 additions & 4 deletions manifests/2.5.0/opensearch-dashboards-2.5.0.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,16 +14,21 @@ components:
repository: https://github.com/opensearch-project/dashboards-search-relevance.git
ref: 2.x
- name: queryWorkbenchDashboards
repository: https://github.com/opensearch-project/sql.git
working_directory: workbench
repository: https://github.com/opensearch-project/dashboards-query-workbench.git
ref: 2.x
- name: notificationsDashboards
repository: https://github.com/opensearch-project/notifications.git
working_directory: dashboards-notifications
ref: 2.x
- name: observabilityDashboards
repository: https://github.com/opensearch-project/observability.git
working_directory: dashboards-observability
repository: https://github.com/opensearch-project/dashboards-observability.git
ref: 2.x
- name: reportsDashboards
repository: https://github.com/opensearch-project/dashboards-reporting.git
Copy link
Member

Choose a reason for hiding this comment

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

This repo is still not public yet therefore cause the manifest CI checks failed.

Copy link
Contributor

Choose a reason for hiding this comment

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

This repo is public now

Copy link
Member

Choose a reason for hiding this comment

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

@zelinh Can you please re-run the failing tests?

Copy link
Member

Choose a reason for hiding this comment

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

@mengweieric Could you rebase to resolve the conflicts?

ref: 2.x
- name: ganttChartDashboards
repository: https://github.com/opensearch-project/dashboards-visualizations.git
working_directory: gantt-chart
ref: 2.x
- name: anomalyDetectionDashboards
repository: https://github.com/opensearch-project/anomaly-detection-dashboards-plugin
Expand Down