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

Adding start of affected node status #143

Merged
merged 1 commit into from
Jan 23, 2025
Merged

Adding start of affected node status #143

merged 1 commit into from
Jan 23, 2025

Conversation

paigerube14
Copy link
Contributor

@paigerube14 paigerube14 commented Jan 10, 2025

This pr adds in the ability to track the timing of the nodes

@paigerube14
Copy link
Contributor Author

New telemetry section will look like this for a stop/start scenario

          "affected_nodes": [
                    {
                        "node_name": "ip-*.us-east-2.compute.internal",
                        "not_ready_time": 0.0,
                        "ready_time": 24.439035892486572,    
                        "unknown_time": 0.15461111068725586,
                        "stopped_time": 136.35135626792908,
                        "running_time": 15.249454021453857,
                        "terminating_time": 0.0
                    },
                    {
                        "node_name": "ip-*.us-east-2.compute.internal",
                        "not_ready_time": 0.0,
                        "ready_time": 23.62007188796997,
                        "unknown_time": 0.15206694602966309,
                        "stopped_time": 166.52288508415222,
                        "running_time": 15.330392122268677,
                        "terminating_time": 0.0
                    }
                ],

@paigerube14 paigerube14 changed the title Adding start of affected node status [WIP] Adding start of affected node status Jan 14, 2025
@paigerube14 paigerube14 force-pushed the node_timing branch 2 times, most recently from c08a01e to eaced50 Compare January 15, 2025 19:56
@paigerube14 paigerube14 changed the title [WIP] Adding start of affected node status Adding start of affected node status Jan 15, 2025
Signed-off-by: Paige Patton <[email protected]>
@chaitanyaenr chaitanyaenr merged commit a07496b into main Jan 23, 2025
4 checks passed
@tsebastiani tsebastiani deleted the node_timing branch January 24, 2025 08:39
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.

2 participants