Skip to content

qaagent: avoid parsing errors by removing stderr from --json output#3060

Merged
nikw9944 merged 1 commit intomainfrom
nikw9944/infra-638
Feb 20, 2026
Merged

qaagent: avoid parsing errors by removing stderr from --json output#3060
nikw9944 merged 1 commit intomainfrom
nikw9944/infra-638

Conversation

@nikw9944
Copy link
Contributor

@nikw9944 nikw9944 commented Feb 20, 2026

Summary of Changes

  • qaagent: avoid parsing errors by removing stderr from --json output
  • Prevents QA test failures in case the client recommends a version upgrade:
2026-02-20T17:34:39.7173092Z     qa_test.go:68: �[2m2026-02-20T17:34:39.715Z�[0m DBG Collect `doublezero latency` for each client
2026-02-20T17:34:40.0577082Z     qa_alldevices_unicast_test.go:107: 
2026-02-20T17:34:40.0578365Z         	Error Trace:	/home/runner/_work/infra/infra/doublezero/e2e/qa_alldevices_unicast_test.go:107
2026-02-20T17:34:40.0579340Z         	Error:      	Received unexpected error:
2026-02-20T17:34:40.0583686Z         	            	failed to get latency on host <qa-hostname>: rpc error: code = Unknown desc = failed to unmarshal latency response: error: invalid character 'A' looking for beginning of value, output: A new version of the client is available: 0.8.9 → 0.8.10
2026-02-20T17:34:40.0586209Z         	            	We recommend updating to the latest version for the best experience.
2026-02-20T17:34:40.0586945Z         	            	[
2026-02-20T17:34:40.0587389Z         	            	  {
2026-02-20T17:34:40.0588589Z         	            	    "device_pk": "6HDniG2dEQuuGGz14G4DxJ3J1WQRauARCTT18ckh5mpP",

Testing Verification

  • Tested in devnet

@nikw9944 nikw9944 changed the title qaagent: avoid parsing erros by removing stderr from --json output qaagent: avoid parsing errors by removing stderr from --json output Feb 20, 2026
@nikw9944 nikw9944 marked this pull request as ready for review February 20, 2026 18:45
@nikw9944 nikw9944 requested a review from snormore February 20, 2026 18:46
@nikw9944 nikw9944 enabled auto-merge (squash) February 20, 2026 18:47
@nikw9944 nikw9944 self-assigned this Feb 20, 2026
@nikw9944 nikw9944 merged commit 97076bc into main Feb 20, 2026
40 of 43 checks passed
@nikw9944 nikw9944 deleted the nikw9944/infra-638 branch February 20, 2026 18:50
nikw9944 added a commit that referenced this pull request Feb 20, 2026
…3060)

## Summary of Changes
* qaagent: avoid parsing errors by removing stderr from --json output
* Prevents QA test failures in case the client recommends a version
upgrade:
```
2026-02-20T17:34:39.7173092Z     qa_test.go:68: �[2m2026-02-20T17:34:39.715Z�[0m DBG Collect `doublezero latency` for each client
2026-02-20T17:34:40.0577082Z     qa_alldevices_unicast_test.go:107: 
2026-02-20T17:34:40.0578365Z         	Error Trace:	/home/runner/_work/infra/infra/doublezero/e2e/qa_alldevices_unicast_test.go:107
2026-02-20T17:34:40.0579340Z         	Error:      	Received unexpected error:
2026-02-20T17:34:40.0583686Z         	            	failed to get latency on host <qa-hostname>: rpc error: code = Unknown desc = failed to unmarshal latency response: error: invalid character 'A' looking for beginning of value, output: A new version of the client is available: 0.8.9 → 0.8.10
2026-02-20T17:34:40.0586209Z         	            	We recommend updating to the latest version for the best experience.
2026-02-20T17:34:40.0586945Z         	            	[
2026-02-20T17:34:40.0587389Z         	            	  {
2026-02-20T17:34:40.0588589Z         	            	    "device_pk": "6HDniG2dEQuuGGz14G4DxJ3J1WQRauARCTT18ckh5mpP",
```

## Testing Verification
* Tested in devnet
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants