Particular / ServicePulse

Production monitoring for distributed systems.
https://docs.particular.net/servicepulse/
Other
33 stars 27 forks source link

Custom checks not shown and no statistics #349

Open ramonsmits opened 8 years ago

ramonsmits commented 8 years ago

I created a custom check but it is not listed in the custom checks:

image

I expected all custom checks to be visible. Not only failed checks.

I expected to see statistics too like when was the last time the custom check send a result to service control. What was the last time it failed or maybe even all failure periods.

WojcikMike commented 8 years ago

ATM the custom checks screen display only failed checks which I find confusing. I think ti should be changed to include all ckecks with an indication if the check is success or failure and probably more detaild indication what was the last reported time.

@Particular/servicepulse-maintainers please have a look

WilliamBZA commented 8 years ago

I propose the following tasks for this as a starting point:

Following on from this the UI should probably be revisited, cleaned up and brought in line with the rest of the ServicePulse UI.

sergioc commented 8 years ago

indicate their failed state and no details

is that because we have no control over the format of the details? E.g. they can be unstructured and/or long?

grouped by category

how are categories defined?

Do you have any idea how many custom checks do customers tend to configure?

WilliamBZA commented 8 years ago

is that because we have no control over the format of the details? E.g. they can be unstructured and/or long?

Correct. There doesn't appear to be a limit anywhere.

how are categories defined?

Each Custom Check defines it's category at compilation time

Do you have any idea how many custom checks do customers tend to configure?

No, so paging is essential

sergioc commented 8 years ago

Do you have any idea how many custom checks do customers tend to configure? No, so paging is essential

No, so paging is essential

How about contacting some TAMs and ask if they have any idea how are customers using custom checks?

Each Custom Check defines it's category at compilation time

I'm not sure I understand the purpose of categories. Is there a doco for this?

WilliamBZA commented 8 years ago

I'm not sure I understand the purpose of categories. Is there a doco for this?

Nothing at all. Perhaps it's time to create some?

sergioc commented 8 years ago

Perhaps it's time to create some?

Probably a good idea, but it'll likely take time.

Can you summarize the concept and purpose here?

WilliamBZA commented 8 years ago

I don't think that's within the scope of this issue.

sergioc commented 8 years ago

It's important that we understand what categories are to validate the requirement for having the display "grouped by categories".

WilliamBZA commented 8 years ago

If you'd prefer we can remove the group by categories and simply list the custom checks. I don't really understand the concern though. Can you elaborate and perhaps we can come to a solution?

sergioc commented 8 years ago

The concern is that I don't understand categories here. That's why I asked for a quick summary.

WojcikMike commented 8 years ago

As currently categories are not used, we can define the meaning. The way how I would use it would be either: business alignement:

Or as technical alignement:

That would allow to display all checks together. However I would suggest creating separate issue for categories, and for now just display it on a screen.

sergioc commented 8 years ago

So you just assign a specific category to a specific custom check. Got it.

If the data is available, I think it's not wise to eye-drop categories to another issue. It doesn't seem that complex. It's just better to design it all together.