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

[BUG] Dashboards-Observability flaky tests #897

Closed
ps48 opened this issue Sep 21, 2023 · 6 comments
Closed

[BUG] Dashboards-Observability flaky tests #897

ps48 opened this issue Sep 21, 2023 · 6 comments
Assignees
Labels
bug Something isn't working v2.11.0

Comments

@ps48
Copy link
Member

ps48 commented Sep 21, 2023

What is the bug?

Integration test for dashboards observability plugin is failing occasionally and should be made more robust. Currently below failing tests are marked as excluded in 2.10 from PR: #896

How can one reproduce the bug?

What is the expected behavior?

The above two tests must pass without the exclusion.

What is your host/environment?

  • OSD 2.10

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.

@wanglam
Copy link
Collaborator

wanglam commented Dec 19, 2023

Hi @ps48, do you have any plan to fix this issue in the coming release?

@wanglam
Copy link
Collaborator

wanglam commented Feb 1, 2024

Hi @ps48 , just find there are two PRs to related to this issue. The #1001 has been merged, and #1006 has been approved. Shall we close this issue after all the PRs merged?

@ps48
Copy link
Member Author

ps48 commented Feb 2, 2024

Yes let's merge the PR with backports and then we should be good to close here. It would be better to verify the CI for observability is green for panels and notebooks after all PRs are merged in.

@wanglam
Copy link
Collaborator

wanglam commented Feb 4, 2024

Hi @ps48 , It seems the observability tests workflow still failed in the 2.x branch. Could you help me take a look?

@ps48
Copy link
Member Author

ps48 commented Feb 6, 2024

@wanglam we are good to close the issue as observability tests are green and passing. We just need some reviews for backports to main: PR1 & PR2

@wanglam
Copy link
Collaborator

wanglam commented Feb 7, 2024

These backport PRs has been merged. Close this issue.

@wanglam wanglam closed this as completed Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working v2.11.0
Projects
None yet
Development

No branches or pull requests

3 participants