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

Add BWC test for Observability #312

Closed
wants to merge 21 commits into from

Conversation

kavithacm
Copy link
Collaborator

Description

  • Add bwc specific Integration Test
  • Update gradle.build to run bwc tests from opensearch 1.1 to opensearch 1.2
  • Included snapshot of Notebooks 1.1 and Observability 1.2

Issues Resolved

#276

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • Commits are signed 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.

kavithacm and others added 21 commits October 18, 2021 13:55
* Bump prismjs from 1.24.1 to 1.25.0 (opensearch-project#137)

Bumps [prismjs](https:/PrismJS/prism) from 1.24.1 to 1.25.0.
- [Release notes](https:/PrismJS/prism/releases)
- [Changelog](https:/PrismJS/prism/blob/master/CHANGELOG.md)
- [Commits](PrismJS/prism@v1.24.1...v1.25.0)

---
updated-dependencies:
- dependency-name: prismjs
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump immer from 9.0.5 to 9.0.6 (opensearch-project#136)

Bumps [immer](https:/immerjs/immer) from 9.0.5 to 9.0.6.
- [Release notes](https:/immerjs/immer/releases)
- [Commits](immerjs/immer@v9.0.5...v9.0.6)

---
updated-dependencies:
- dependency-name: immer
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Redirect legacy notebooks URL to current observability one (opensearch-project#141)

* Move observability frontend to a sub directory (opensearch-project#142)

* Add minimal plugin for backend observability (opensearch-project#143)

* Add models for objects and requests (opensearch-project#144)

* Add CRUD actions and index operations for observability objects (opensearch-project#145)

* Update data modal and enable CI (opensearch-project#148)

* Panels' visualization design change (opensearch-project#149)

* modified common files

* modified UI components

* modified panel adaptor and router

* moved loading to post http call

* fixed prettier quotes

* Feature timestamp (opensearch-project#152)

* timestamp related changes

Signed-off-by: Eric Wei <[email protected]>

* cleanup

Signed-off-by: Eric Wei <[email protected]>

* removed loggings and change to console error

Signed-off-by: Eric Wei <[email protected]>

* remove unused sidebar component

Signed-off-by: Eric Wei <[email protected]>

* adding cancel button for edit & panel actions (opensearch-project#153)

* Feature toasts errors handling (opensearch-project#155)

* timestamp related changes

Signed-off-by: Eric Wei <[email protected]>

* cleanup

Signed-off-by: Eric Wei <[email protected]>

* removed loggings and change to console error

Signed-off-by: Eric Wei <[email protected]>

* remove unused sidebar component

Signed-off-by: Eric Wei <[email protected]>

* added toasts

Signed-off-by: Eric Wei <[email protected]>

* resolved build failure

Signed-off-by: Eric Wei <[email protected]>

* couple of error handling changes

Signed-off-by: Eric Wei <[email protected]>

* added types

Signed-off-by: Eric Wei <[email protected]>

* Autocomplete only displays current command (opensearch-project#157)

* Only show current command in suggestion
Signed-off by: Eugene Lee <[email protected]>

* Remove console logs
Signed-off by: Eugene Lee <[email protected]>

* Only display suggestion
Signed-off by: Eugene Lee <[email protected]>

* Removed commas
Signed-off by: Eugene Lee <[email protected]>

* Remove import
Signed-off by: Eugene Lee <[email protected]>

* changed to support query without 'search' prefix (opensearch-project#158)

Signed-off-by: Eric Wei <[email protected]>

* Remove app analytics (opensearch-project#154)

* removing app analytics from sidenav

* removed unused headers

* added trace analytics as default page

* Use JS API to redirect legacy notebooks URL (opensearch-project#162)

* Panels bug fix#1 (opensearch-project#159)

* fixed UI bugs #1

* folder name typo fix

* changes for adopting new sql artifact (opensearch-project#165)

* changes for adopting new sql artifact

Signed-off-by: Eric Wei <[email protected]>

* minor changes

Signed-off-by: Eric Wei <[email protected]>

* Improve reindex handling for .opensearch-notebooks (opensearch-project#163)

* inherited datepicker format from settings (opensearch-project#164)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Joshua <[email protected]>
Co-authored-by: Shenoy Pratik <[email protected]>
Co-authored-by: Eric Wei <[email protected]>
Co-authored-by: eugenesk24 <[email protected]>
Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>
Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>
Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>
@kavithacm kavithacm requested a review from a team December 10, 2021 02:07
@kavithacm kavithacm self-assigned this Dec 10, 2021
@codecov-commenter
Copy link

codecov-commenter commented Dec 10, 2021

Codecov Report

Merging #312 (b93be0f) into main (da0cc10) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #312   +/-   ##
=========================================
  Coverage     71.86%   71.86%           
  Complexity      269      269           
=========================================
  Files            81       81           
  Lines          2804     2804           
  Branches        334      334           
=========================================
  Hits           2015     2015           
  Misses          624      624           
  Partials        165      165           
Flag Coverage Δ
dashboards-observability 63.87% <ø> (ø)
opensearch-observability 74.91% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...rg/opensearch/observability/ObservabilityPlugin.kt 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update da0cc10...b93be0f. Read the comment docs.

@kavithacm kavithacm closed this Dec 10, 2021
@kavithacm kavithacm deleted the new-bwc-test branch December 10, 2021 18:09
joshuali925 pushed a commit to joshuali925/observability that referenced this pull request Jul 20, 2022
…debuginfo_api

debuginfo: Implement file hash to debug info API calls
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.

2 participants