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

Rebase flint #963

Merged
merged 126 commits into from
Sep 6, 2023
Merged

Conversation

derek-ho
Copy link
Collaborator

@derek-ho derek-ho commented Aug 31, 2023

Description

Rebase flint feature branch off latest changes in main

Issues Resolved

[List any issues this PR will resolve]

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.

TackAdam and others added 30 commits June 13, 2023 15:23
Signed-off-by: Simeon Widdis <[email protected]>
Signed-off-by: Simeon Widdis <[email protected]>
Signed-off-by: Simeon Widdis <[email protected]>
Signed-off-by: Simeon Widdis <[email protected]>
Signed-off-by: TackAdam <[email protected]>
* add capability to build the plugin and add it into the docker of the Opensearch-Dashboard

update the developer-guid to reflect the correct bootstraping process
add docker-compose support and tutorial

Signed-off-by: YANGDB <[email protected]>

* Update Dockerfile

* Update Dockerfile and adding .env

* Update Using-Docker.md

Co-authored-by: Joshua Li <[email protected]>
Signed-off-by: YANGDB <[email protected]>

---------

Signed-off-by: YANGDB <[email protected]>
Co-authored-by: Joshua Li <[email protected]>
* Merge in kibana backend from osints/dev

Signed-off-by: Simeon Widdis <[email protected]>

* Add integration type to .kibana from osints/dev

Signed-off-by: Simeon Widdis <[email protected]>

* Re-add license header

Signed-off-by: Simeon Widdis <[email protected]>

* Fix integrations type

Signed-off-by: Simeon Widdis <[email protected]>

---------

Signed-off-by: Simeon Widdis <[email protected]>
…e-paths

Copy path sanitization from osints/dev
Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: Eric Wei <[email protected]>
* refactor naming

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

* refactor vis workspace

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

* config panel refactoring

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

* refactor workspace

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

---------

Signed-off-by: Eric Wei <[email protected]>
[BugFix] Application Analytics - Save Visualization Missing Index
Cypress v12.8.1 - Fixes for panels/Toast message fix
anirudha and others added 7 commits August 18, 2023 21:47
…earch-project#943)

* Refactor validation logic with a deeper interface

Signed-off-by: Simeon Widdis <[email protected]>

* Remove redundant test.

This test is unneeded after 12c4bcf

Signed-off-by: Simeon Widdis <[email protected]>

* Add tests for new validators

Signed-off-by: Simeon Widdis <[email protected]>

* Make better failure mode for invalid objects

Signed-off-by: Simeon Widdis <[email protected]>

* Convert validator methods to use result types

Signed-off-by: Simeon Widdis <[email protected]>

---------

Signed-off-by: Simeon Widdis <[email protected]>
* Apply yarn auto-lints

Signed-off-by: Simeon Widdis <[email protected]>

* Fix equalities

Signed-off-by: Simeon Widdis <[email protected]>

* Fix naming conventions

Signed-off-by: Simeon Widdis <[email protected]>

* Fix non-export issues

Signed-off-by: Simeon Widdis <[email protected]>

* Fix default exports

Signed-off-by: Simeon Widdis <[email protected]>

* Remove unused dependency

Signed-off-by: Simeon Widdis <[email protected]>

---------

Signed-off-by: Simeon Widdis <[email protected]>
* Update mocha

Signed-off-by: Simeon Widdis <[email protected]>

* Update mocha to 10.2.0

Signed-off-by: Simeon Widdis <[email protected]>

* Revert mocha to 10.1.0 for OSD compat.

Signed-off-by: Simeon Widdis <[email protected]>

* Remove random added yaml dep

Signed-off-by: Simeon Widdis <[email protected]>

---------

Signed-off-by: Simeon Widdis <[email protected]>
…conventions (opensearch-project#514)

* trace analytics v2 first remove expensive dashboard page

Signed-off-by: Derek Ho <[email protected]>

* get trace groups request working and mapping correctly

Signed-off-by: Derek Ho <[email protected]>

* get trace group filter fully working

Signed-off-by: Derek Ho <[email protected]>

* try some service map adjustments

Signed-off-by: Derek Ho <[email protected]>

* change to up down to fully show labels

Signed-off-by: Derek Ho <[email protected]>

* put dashboard in accordian at bottom of services and only make calls if it is open

Signed-off-by: Derek Ho <[email protected]>

* some changes to support dashboard content in accordian

Signed-off-by: Derek Ho <[email protected]>

* demo code

Signed-off-by: Derek Ho <[email protected]>

* fix tests

Signed-off-by: Derek Ho <[email protected]>

* remove dashboard from app analytics

Signed-off-by: Derek Ho <[email protected]>

* fix up with tests

Signed-off-by: Derek Ho <[email protected]>

* fix test

Signed-off-by: Derek Ho <[email protected]>

* revert only and extract to constant

Signed-off-by: Derek Ho <[email protected]>

* add auth const

Signed-off-by: Derek Ho <[email protected]>

* address self-PR review

Signed-off-by: Derek Ho <[email protected]>

* remove unecessary code

Signed-off-by: Derek Ho <[email protected]>

* change timeout to 25 seconds

Signed-off-by: Derek Ho <[email protected]>

* change to row in accordian

Signed-off-by: Derek Ho <[email protected]>

* update labels and tests

Signed-off-by: Derek Ho <[email protected]>

* update cypress test

Signed-off-by: Derek Ho <[email protected]>

* verify cypress tests work with changes

Signed-off-by: Derek Ho <[email protected]>

* remove accidental only

Signed-off-by: Derek Ho <[email protected]>

* address PR review comments

Signed-off-by: Derek Ho <[email protected]>

* make missing configuration more clear

Signed-off-by: Derek Ho <[email protected]>

* update snapshots

Signed-off-by: Derek Ho <[email protected]>

* update snapshot

Signed-off-by: Derek Ho <[email protected]>

---------

Signed-off-by: Derek Ho <[email protected]>
sumukhswamy and others added 3 commits September 5, 2023 11:06
opensearch-project#928)

* added code for the change of design of the sidebar, fixed overriding spinner bug, changed font size

Signed-off-by: sumukhswamy <[email protected]>

* addressed PR comments

Signed-off-by: sumukhswamy <[email protected]>

* code for overriding pattern

Signed-off-by: sumukhswamy <[email protected]>

* updated snapshots

Signed-off-by: sumukhswamy <[email protected]>

---------

Signed-off-by: sumukhswamy <[email protected]>
@codecov
Copy link

codecov bot commented Sep 6, 2023

Codecov Report

Merging #963 (952bab6) into feature/flint-UI (fe66220) will increase coverage by 0.58%.
The diff coverage is n/a.

@@                 Coverage Diff                  @@
##           feature/flint-UI     #963      +/-   ##
====================================================
+ Coverage             43.12%   43.71%   +0.58%     
====================================================
  Files                   303      313      +10     
  Lines                 18008    18714     +706     
  Branches               4386     4562     +176     
====================================================
+ Hits                   7766     8180     +414     
- Misses                10201    10492     +291     
- Partials                 41       42       +1     
Flag Coverage Δ
dashboards-observability 43.71% <ø> (+0.58%) ⬆️

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

see 73 files with indirect coverage changes

@derek-ho derek-ho merged commit 5fc3cc2 into opensearch-project:feature/flint-UI Sep 6, 2023
5 checks passed
@derek-ho derek-ho deleted the rebase-flint branch September 6, 2023 15:42
derek-ho added a commit that referenced this pull request Sep 6, 2023
@derek-ho derek-ho mentioned this pull request Sep 6, 2023
@derek-ho derek-ho restored the rebase-flint branch September 6, 2023 16:07
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.