W-17017575-insights-hyperforce-duke #314
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
6 tasks
dukesphere
commented
Nov 2, 2024
dukesphere
commented
Nov 2, 2024
dukesphere
commented
Nov 2, 2024
dukesphere
commented
Nov 2, 2024
dukesphere
commented
Nov 5, 2024
dukesphere
commented
Nov 12, 2024
| + | ||
| This code comes from Runtime Manager. | ||
|
|
||
| URL:: |
There was a problem hiding this comment.
Currently "Uri" in the UI (Mule app detail header)
Can we change to "URL"? @sathishpaulleo
There was a problem hiding this comment.
Suggested change
| URL:: | |
| URL:: |
dukesphere
commented
Nov 12, 2024
| [[performance-trends-overview]] | ||
| ==== Performance Trends | ||
|
|
||
| Entities with Slow Response Times:: |
There was a problem hiding this comment.
UI currently says Entities with slow Response Times
slow -> Slow
@sathishpaulleo
dukesphere
commented
Nov 12, 2024
| // | ||
| //H3: ENTITIES LIST | ||
| [[entities-list]] | ||
| === Entities List |
There was a problem hiding this comment.
Suggest removing "List"
UX is generally to avoid unnecessary terms like this (self-identification that is obvious)
dukesphere
commented
Nov 12, 2024
dukesphere
commented
Nov 12, 2024
dukesphere
commented
Nov 12, 2024
Comment on lines
+244
to
+246
| //TODO_TODO: to avoid duplication (see chart with same name this one) CHANGE NAME to something like Policy Violation Volume (or similar) | ||
| Policy Violations:: | ||
| Number of API policy violations |
There was a problem hiding this comment.
Suggested change
| //TODO_TODO: to avoid duplication (see chart with same name this one) CHANGE NAME to something like Policy Violation Volume (or similar) | |
| Policy Violations:: | |
| Number of API policy violations | |
| //TODO_TODO: to avoid duplication (see chart with same name this one) CHANGE NAME to something like Policy Violation Volume (or similar) | |
| Policy Violations:: | |
| Number of API policy violations |
dukesphere
commented
Nov 12, 2024
Comment on lines
+16
to
+19
| //TODO: suggest Runtime Version conform on Runtime (App) vs. Runtime Version (API) | ||
|
|
||
| Runtime:: | ||
| Mule runtime engine version of this Mule app |
There was a problem hiding this comment.
Suggested change
| //TODO: suggest Runtime Version conform on Runtime (App) vs. Runtime Version (API) | |
| Runtime:: | |
| Mule runtime engine version of this Mule app | |
| //TODO: suggest Runtime Version conform on Runtime (App) vs. Runtime Version (API) | |
| Runtime:: | |
| Mule runtime engine version of this Mule app |
dukesphere
commented
Nov 12, 2024
Comment on lines
+22
to
+23
| Replicas:: | ||
| Virtual core fraction and count of this Mule app, for example, *0.1 vCores x 1* (= 10% of 1 vCore) |
There was a problem hiding this comment.
Suggested change
| Replicas:: | |
| Virtual core fraction and count of this Mule app, for example, *0.1 vCores x 1* (= 10% of 1 vCore) | |
| Replicas:: | |
| Virtual core fraction and count of this Mule app, for example, *0.1 vCores x 1* (= 10% of 1 vCore) |
pls verify description of Replicas, @sathishpaulleo
dukesphere
commented
Nov 12, 2024
Comment on lines
+218
to
+220
| //TODO_TODO_TODO: What if empty but seems to be running? | ||
| API Instance ID:: | ||
| Instance identifier for an API implementation incorporated into a Flex Gateway or API Gateway runtime |
There was a problem hiding this comment.
Suggested change
| //TODO_TODO_TODO: What if empty but seems to be running? | |
| API Instance ID:: | |
| Instance identifier for an API implementation incorporated into a Flex Gateway or API Gateway runtime | |
| //TODO_TODO_TODO: What if value empty but seems to be Active? | |
| API Instance ID:: | |
| Instance identifier for an API implementation incorporated into a Flex Gateway or API Gateway runtime |
dukesphere
commented
Nov 12, 2024
Comment on lines
+222
to
+224
| //TODO: capitalize "version"? | ||
| API version:: | ||
| Version of the API implementation |
There was a problem hiding this comment.
Suggested change
| //TODO: capitalize "version"? | |
| API version:: | |
| Version of the API implementation | |
| //TODO: capitalize "version"? | |
| API version:: | |
| Version of the API implementation |
for consistency @sathishpaulleo
dukesphere
commented
Nov 12, 2024
Comment on lines
+282
to
+284
| // TODO: capitalize "paths" -> "Paths" and "N" to "20" if that's the limit (SATHISH) | ||
| Request Volume by Top N paths:: | ||
| Number of requests over time from endpoints with the most requests |
There was a problem hiding this comment.
Suggested change
| // TODO: capitalize "paths" -> "Paths" and "N" to "20" if that's the limit (SATHISH) | |
| Request Volume by Top N paths:: | |
| Number of requests over time from endpoints with the most requests | |
| // TODO: capitalize "paths" -> "Paths" and "N" to "20" if that's the limit (SATHISH) | |
| Request Volume by Top N paths:: | |
| Number of requests over time from endpoints with the most requests |
dukesphere
commented
Nov 12, 2024
Comment on lines
+294
to
+298
| // TODO: downloads available? | ||
| Failed Requests Volume by HTTP Method:: | ||
| Number of unsuccessful requests over time, grouped by the HTTP method used | ||
| //+ | ||
| //include::partial$include-insights-charts.adoc[tag="download-as"] |
There was a problem hiding this comment.
Suggested change
| // TODO: downloads available? | |
| Failed Requests Volume by HTTP Method:: | |
| Number of unsuccessful requests over time, grouped by the HTTP method used | |
| //+ | |
| //include::partial$include-insights-charts.adoc[tag="download-as"] | |
| // TODO: downloads available? | |
| Failed Requests Volume by HTTP Method:: | |
| Number of unsuccessful requests over time, grouped by the HTTP method used | |
| //+ | |
| //include::partial$include-insights-charts.adoc[tag="download-as"] |
dukesphere
commented
Nov 12, 2024
Comment on lines
+300
to
+304
| // TODO: downloads available? | ||
| Failed Requests Volume by Response Code:: | ||
| Number of unsuccessful requests over time, grouped by the response code (such as *2xx*) | ||
| //+ | ||
| //include::partial$include-insights-charts.adoc[tag="download-as"] |
There was a problem hiding this comment.
Suggested change
| // TODO: downloads available? | |
| Failed Requests Volume by Response Code:: | |
| Number of unsuccessful requests over time, grouped by the response code (such as *2xx*) | |
| //+ | |
| //include::partial$include-insights-charts.adoc[tag="download-as"] | |
| // TODO: downloads available? | |
| Failed Requests Volume by Response Code:: | |
| Number of unsuccessful requests over time, grouped by the response code (such as *2xx*) | |
| //+ | |
| //include::partial$include-insights-charts.adoc[tag="download-as"] |
dukesphere
commented
Nov 12, 2024
Comment on lines
+306
to
+311
| // TODO: capitalize "paths" -> "Paths" and "N" to "20" if that's the limit (SATHISH) | ||
| // TODO: downloads available? | ||
| Failed Requests by Top N paths:: | ||
| Number of unsuccessful requests over time from endpoints with the most requests | ||
| //+ | ||
| //include::partial$include-insights-charts.adoc[tag="download-as"] |
There was a problem hiding this comment.
Suggested change
| // TODO: capitalize "paths" -> "Paths" and "N" to "20" if that's the limit (SATHISH) | |
| // TODO: downloads available? | |
| Failed Requests by Top N paths:: | |
| Number of unsuccessful requests over time from endpoints with the most requests | |
| //+ | |
| //include::partial$include-insights-charts.adoc[tag="download-as"] | |
| // TODO: capitalize "paths" -> "Paths" and "N" to "20" if that's the limit (SATHISH) | |
| // TODO: downloads available? | |
| Failed Requests by Top N paths:: | |
| Number of unsuccessful requests over time from endpoints with the most requests | |
| //+ | |
| //include::partial$include-insights-charts.adoc[tag="download-as"] |
dukesphere
commented
Nov 12, 2024
Comment on lines
+321
to
+324
| //TODO: RESPONSE TIME UNIT MISSING from chart | ||
| //can download as CSV, XLS | ||
| Top 20 Paths by Average Response Time:: | ||
| Average response time of the 20 most requested endpoints. |
There was a problem hiding this comment.
Suggested change
| //TODO: RESPONSE TIME UNIT MISSING from chart | |
| //can download as CSV, XLS | |
| Top 20 Paths by Average Response Time:: | |
| Average response time of the 20 most requested endpoints. | |
| //TODO: RESPONSE TIME UNIT MISSING from chart | |
| //can download as CSV, XLS | |
| Top 20 Paths by Average Response Time:: | |
| Average response time of the 20 most requested endpoints. |
dukesphere
commented
Nov 12, 2024
Comment on lines
+328
to
+332
| //TODO: 95p or capitalize "percentile" -> "Percentile" | ||
| //TODO: Can we say "Bottom 95p Response Time" or similar? | ||
| //You can download as CSV, XLS | ||
| Top 20 Paths by 95th percentile Response Time:: | ||
| For the 20 most requested endpoints, the average response time of 95% of requests; the response time of the remaining 5% is typically greater than this number |
There was a problem hiding this comment.
Suggested change
| //TODO: 95p or capitalize "percentile" -> "Percentile" | |
| //TODO: Can we say "Bottom 95p Response Time" or similar? | |
| //You can download as CSV, XLS | |
| Top 20 Paths by 95th percentile Response Time:: | |
| For the 20 most requested endpoints, the average response time of 95% of requests; the response time of the remaining 5% is typically greater than this number | |
| //TODO: 95p or capitalize "percentile" -> "Percentile" | |
| //TODO: Response time units missing | |
| Top 20 Paths by 95th percentile Response Time:: | |
| For the 20 most requested endpoints, the average response time of 95% of requests; the response time of the remaining 5% is typically greater than this number |
dukesphere
commented
Nov 12, 2024
Comment on lines
+370
to
+372
| //TODO: "Requests" -> "Request", and can we remove "Volume"? or change to "Request Total" | ||
| Total Requests Volume:: | ||
| Total number of requests |
There was a problem hiding this comment.
Suggested change
| //TODO: "Requests" -> "Request", and can we remove "Volume"? or change to "Request Total" | |
| Total Requests Volume:: | |
| Total number of requests | |
| //TODO: "Requests" -> "Request", and can we remove "Volume"? or change to "Request Total" | |
| Total Requests Volume:: | |
| Total number of requests |
dukesphere
commented
Nov 12, 2024
dukesphere
commented
Nov 12, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Writer's Quality Checklist
Before merging your PR, did you: