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

red-165 - add sync details page #6

Closed
wants to merge 1 commit into from
Closed

red-165 - add sync details page #6

wants to merge 1 commit into from

Conversation

magonjr
Copy link
Contributor

@magonjr magonjr commented Jul 4, 2024

No description provided.

Copy link

linear bot commented Jul 4, 2024

RED-165 Monitor improvements - OOS Detection Page

ISSUE SUMMARY:

https://drive.google.com/file/d/1jM0TQlmKnM4hoi7he2TlIZMfToIio-Om/view?usp=sharing

it needs to be a new page that you open up
in this page we have  a 2d grid / columns
IP port  / node ID on the left.   nodes are default sorted by node ID
the first column can be insync    (the general flag).   then you can have three more columns for the stats:
export interface IsInsyncResult {
  radixes: any[]
  stats: { good: number; bad: number; total: number }
  insync: boolean
}
then you have  N columns for all the radix values
but these are sparse arrays
so a   node will only have some of the data
if it does not have any radix data it leaves that radix blank
if it has insync data then green
oos then magenta
Also should try to represent edge vs consensus
maybe   E or C    in black text
and the green and magenta are cell backgrounds
radix values may be “n” chars long
num columns is of course    n^16
ie. radix length 2...    00  , 01, 02,,......   FD,  FE, FF 


ISSUE REPRO STEPS:

<HINT: Add steps to list as-needed. If interaction is complex, add screenshots or a Slack screen-capture video (just drag and drop)>

  1. <<TODO: Replace with repro step GOLD-82 feat: display queue details #1>>
  2. <<TODO: Replace with repro step Clean up large-network view fixes PM-70 #2>>
  3. Observe <<TODO: Describe unintended behavior.>>

EXPECTED RESULT:

<<TODO: Replace this with your expected results.>>


PULL REQUESTS:

<HINT: If your fix requires changes in multiple repos, add the following info per-repository.>

<<TODO: Enter Repository Name>>

Pull Request Link: <<TODO: Insert PR-LINK>>

GPT Review Link: <<TODO: Insert GPT-Review-Link>>

Jenkins Test Link: <<TODO: Insert Jenkins Test Job Link>>


ADDITIONAL INSTRUCTIONS:

<HINT: Add any additional instructions needed for the assignee. If you have specific requirements for how the task should be implemented or fixed, enter them or link them here.>

<<TODO: Insert additional instructions for assignee.>>

@afostr afostr closed this Jul 24, 2024
@afostr afostr deleted the red-165 branch July 24, 2024 00:14
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.

3 participants