Reconcile master with deployed openhtf + add per-test result label#1298
Closed
uh-oh-monteiro wants to merge 4 commits into
Closed
Reconcile master with deployed openhtf + add per-test result label#1298uh-oh-monteiro wants to merge 4 commits into
uh-oh-monteiro wants to merge 4 commits into
Conversation
* Add DDNS health indicator to single-station dashboard status bar Render the tri-state DDNS record health reported by the station server in the single-station status bar, below the IP line: MATCH -> green check (resolves to the station's current IP) MISMATCH -> red cross (resolves, but to a different/stale IP) UNKNOWN -> grey question (could not resolve: NXDOMAIN, timeout, no FQDN) The line is omitted entirely when the server reports no DDNS information (the multi-station dashboard path and older servers do not send these fields), so the change degrades safely against an un-bumped backend. The fields (ddns_hostname / ddns_resolved_ip / ddns_status) are produced by the backend counterpart in halter-corp/hardware-test-framework#1567. NOTE: the committed dist/ bundle is NOT regenerated in this commit. The Angular 4 / webpack 3 / node-sass 7 toolchain could not be built in the authoring environment (Node 25 / Python 3.12 lacks distutils, node-sass native build fails). Run "npm install && npm run build" in web_gui with a compatible Node before deploying, and commit the regenerated dist/. See the accompanying APPLY.md. Co-Authored-By: clank-halter[bot] <2696940+clank-halter[bot]@users.noreply.github.com> Clank-Job-Id: 6ae51244-e605-4ce0-9e50-ebf172b7b87d * Pin Node 16 in web_ui CI job so node-sass can build The web_ui job used setup-node without a version, picking up the runner's default (now Node 22). node-sass@7 can't compile its native addon on Node >= 18, so npm install failed. Pin Node 16 to unblock CI. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> --------- Co-authored-by: kurtmc <2661907+kurtmc@users.noreply.github.com> Co-authored-by: clank-halter[bot] <2696940+clank-halter[bot]@users.noreply.github.com> Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
Squashed halter fork baseline on top of upstream + DDNS: extended DUT id, unmanaged plugs + child tests, multislot/restart/ clickable-phase UI, multiplug-instantiation fix, DutIdentifier serialisable, Jan-2025 upstream bump, child-test reporting fixes.
Squashed operator-fault-report-escalate: GUI fault panel, report relay + escalate-to-PagerDuty, station_server.set_fault_report_handler / set_fault_escalate_handler, node16 CI pin.
Adds a generic, optional metadata['label'] surfaced as a badge on the left of each History row (styled like the running status badge, DUT id centred). Null when unset, so unlabelled tests render unchanged. Used by the P5 ICT 6-up plans to show each child test's slot number.
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
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
2 participants
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.
Master was only upstream + the DDNS PR, while the stations actually run the fork (via the escalate branch). This brings the deployed state onto master as clean, squashed commits and adds one new feature.
Fast-forwards onto master (linear on origin/master).
Commits
upstream bump, child-test reporting fixes.
Notes