Add observability, reporting, query workbench and gantt chart to manifests #3048
Conversation
Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: Eric Wei <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #3048 +/- ##
=======================================
Coverage 93.17% 93.17%
=======================================
Files 167 167
Lines 4602 4602
=======================================
Hits 4288 4288
Misses 314 314 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Signed-off-by: Eric Wei <[email protected]>
| @@ -108,7 +108,6 @@ components: | |||
| platforms: | |||
| - linux | |||
| - windows | |||
| working_directory: opensearch-observability | |||
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
there's a PR ready to merge which removes all front end stuff, will get it merged soon
| repository: https://github.com/opensearch-project/dashboards-observability.git | ||
| ref: 2.x | ||
| - name: reportsDashboards | ||
| repository: https://github.com/opensearch-project/dashboards-reporting.git |
There was a problem hiding this comment.
This repo is still not public yet therefore cause the manifest CI checks failed.
There was a problem hiding this comment.
@zelinh Can you please re-run the failing tests?
There was a problem hiding this comment.
@mengweieric Could you rebase to resolve the conflicts?
zelinh
force-pushed
the
add-insight-manifests
branch
from
c559908 to
d840b15
Compare
Signed-off-by: Eric Wei <[email protected]>
mengweieric
force-pushed
the
add-insight-manifests
branch
from
d840b15 to
3aad1a5
Compare
mengweieric
changed the title
|
CI is failing as OpenSearch has bumped to 2.6.0 for 2.x branch. Will need to rebase after this PR merged. #3059 |
|
@peterzhuamazon @zelinh can we review and merge this soon ? |
Signed-off-by: Zelin Hao <[email protected]>
@anirudha Yes, once the CI checks completed, I could help merge this. |
Description
Add observability, reporting and gantt chart to manifests
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
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.