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

Jump to bottom

[BUG] Metrics Analytics breaks after removing prometheus datasource #1108

Closed
derek-ho opened this issue Oct 11, 2023 · 1 comment
Closed

[BUG] Metrics Analytics breaks after removing prometheus datasource #1108

derek-ho opened this issue Oct 11, 2023 · 1 comment
Assignees
@pjfitzgibbons
Labels
bug Something isn't working untriaged

Comments

@derek-ho
Copy link
Collaborator

What is the bug?

  • Metrics analytics working as expected (empty)
  • Add prometheus data source
  • Metrics analytics working as expected (showing some data)
  • Remove prometheus data source
  • Metrics analytics breaking
    See screenshot below:
Screenshot 2023-10-11 at 10 04 02 AM

How can one reproduce the bug?
Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

What is the expected behavior?
A clear and concise description of what you expected to happen.

What is your host/environment?

  • OS: [e.g. iOS]
  • Version [e.g. 22]
  • Plugins

Do you have any screenshots?
If applicable, add screenshots to help explain your problem.

Do you have any additional context?
Add any other context about the problem.
@derek-ho derek-ho added bug Something isn't working untriaged labels Oct 11, 2023
@derek-ho
Copy link
Collaborator Author

Seems like a browser caching issue

mengweieric pushed a commit to mengweieric/dashboards-observability that referenced this issue Oct 30, 2023
@derek-ho
feat: enable windows and macos builds (opensearch-project#1108)
0e65c32 
* fix: initial code to add windows support for opensearch-observability

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

* add build for windows in dashboards-observability

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

* change windows command and add windows build for UI modules

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

* try gradlew bat for windows

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

* try to exclude tests

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

* fix jacoco

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

* do for mac and change name

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

* try to run the tests for windows and mac

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

* try to add gitattributes file

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

* add formatting rule

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

* try to enable both

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

* try to add git config to fix

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

* autocrlf false for windwos and remove comments

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

* comment out bwc tests,

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

* add line

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

* fix up using matrix and fix test

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

Signed-off-by: Derek Ho <[email protected]>
JulianQuispel pushed a commit to JulianQuispel/dashboards-observability that referenced this issue Jan 24, 2024
@derek-ho
[BACKPORT 1.3] feat: enable windows and macos builds (opensearch-proj…
39c34ea 
…ect#1108) (opensearch-project#1328)

* feat: enable windows and macos builds (opensearch-project#1108)

* fix: initial code to add windows support for opensearch-observability

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

* add build for windows in dashboards-observability

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

* change windows command and add windows build for UI modules

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

* try gradlew bat for windows

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

* try to exclude tests

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

* fix jacoco

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

* do for mac and change name

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

* try to run the tests for windows and mac

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

* try to add gitattributes file

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

* add formatting rule

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

* try to enable both

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

* try to add git config to fix

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

* autocrlf false for windwos and remove comments

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

* comment out bwc tests,

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

* add line

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

* fix up using matrix and fix test

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

Signed-off-by: Derek Ho <[email protected]>
(cherry picked from commit 0e65c32)
Signed-off-by: Derek Ho <[email protected]>

* remove java 17

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

* remove -d for windows

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

* Update .github/workflows/dashboards-observability-test-and-build-workflow.yml

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

* fix fail fast

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

* try to do fix step for mac

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

* remove mac

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

Signed-off-by: Derek Ho <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pjfitzgibbons pjfitzgibbons

Labels
bug Something isn't working untriaged
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pjfitzgibbons @derek-ho