fix(controlplane): clarify endpoint discard and tooltip diagnostics - #2865
Merged
Merged
Conversation
This file contains hidden or 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
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.
Endpoint tooltips currently open when any part of their table row is hovered or focused, and the first row's tooltips are clipped above the table. Scope hover and focus to each tooltip and position endpoint status and failure-rate explanations below their triggers.
Also persist a reason when queue matching discards a delivery because its endpoint is paused/inactive or authentication is unavailable. These deliveries previously had an empty description and no HTTP attempts; delivery policy and retry behavior are unchanged.
Validation: six ChromeHeadless tooltip tests, including a rendered regression that fails against the previous implementation; local endpoint-page checks against the Docker API; production dashboard build; worker race tests for both delivery modes; Go vet and repository lint. The dashboard build retains existing size warnings.