Improve logging: reduce noise, add labguide tracking, add session met…#162
Open
tatanithin wants to merge 1 commit intonested/2025.1from
Open
Improve logging: reduce noise, add labguide tracking, add session met…#162tatanithin wants to merge 1 commit intonested/2025.1from
tatanithin wants to merge 1 commit intonested/2025.1from
Conversation
…adata Noise reduction (~94% fewer logs per session): - Remove per-device status check logs (2400+/session) - Remove 'Found N devices' poll cycle log - Internal/client gRPC checks: log only state transitions - CVP token error: downgrade to warning, suppress repeats - Client gRPC test error: downgrade to warning (client firewall) - CVP status down: log only transitions in atd-ws.js - Add cvp_status_up event for CVP ready transition Labguide page tracking (unified-header.js): - Track every labguide page view via cloudLog - Detect context: standalone, cvp, terminal, coder, browser, etc. - Capture parent page path for iframe context - Capture hash fragment as section label - Filter Sphinx utility pages (genindex, search) - Guard against double-fire from sub_filter_once off Session metadata logging (session_logger.py): - Log lab session details at uilanding startup - Captures: lab_name, topology, course_name, lab_type, user_email, user_name, labguides_modules, login_user, eos_type, zone - Handles missing customer_details gracefully ClientLogHandler (pages.py): - Extract page, context, parent_page, section labels from client logs
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.
…adata
Noise reduction (~94% fewer logs per session):
Labguide page tracking (unified-header.js):
Session metadata logging (session_logger.py):
ClientLogHandler (pages.py):