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

feat: add a way to debug stores in troubleshooting page #3121

Merged
merged 1 commit into from Jul 5, 2023

Conversation

benoitf
Copy link
Collaborator

@benoitf benoitf commented Jul 5, 2023

What does this PR do?

Improve self-troubleshooting page

With the refactoring of the stores, #3101

now we're able to get more data/statistics from them

Number of elements in the store
event that triggered the update
time when it was updated last time
number of elements fetched at that time

allow to clear all history of events and manually fetch again the data to repopulate the store

Screenshot/screencast of this PR

bLMmrTO2jR.mp4

What issues does this PR fix or reference?

Fixes #3053

How to test this PR?

Go to the troubleshooting page, and you can see time to update the stores, events that triggered the update, etc.

also component tests added

fixes containers#3053

Signed-off-by: Florent Benoit <fbenoit@redhat.com>
@benoitf benoitf requested a review from a team as a code owner July 5, 2023 13:51
@benoitf benoitf requested review from jeffmaury and cdrage and removed request for a team July 5, 2023 13:51
Copy link
Contributor

@cdrage cdrage left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code LGTM and I was able to test it out! Here's below of testing it out on just a Kubernetes connection.

I also tested against Docker Desktop and Podman.

LGTM!

Screen.Recording.2023-07-05.at.2.04.08.PM.mov

@benoitf benoitf merged commit 7176fb0 into containers:main Jul 5, 2023
8 checks passed
@podman-desktop-bot podman-desktop-bot added this to the 1.2.0 milestone Jul 5, 2023
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.

Troubleshooting Step2: Add the ability to see the stores and refresh them.
3 participants