Skip to content

release - #112

Merged
mfiedorowicz merged 2 commits into
releasefrom
develop
Sep 14, 2026
Merged

mfiedorowicz merged 2 commits into
releasefrom
develop

Conversation

@ldrozdz93

Copy link
Copy Markdown
Contributor

No description provided.

ldrozdz93 and others added 2 commits September 11, 2026 09:37
…rtualMachine

VirtualMachine(cluster=..., site=...) filled the cluster's scope_site whenever
HasField("scope_site") was False. Cluster.scope is a oneof, so that guard is
also False when the caller already scoped the cluster to a SiteGroup, Location
or Region, and CopyFrom then silently cleared it.

ingester.py is regenerated output: this is the artifact produced by the
generator fix in netboxlabs/eng-observability-skunkworks#57, whose only change
over the current NetBox v4.7.0 generation is that guard.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Sep 14, 2026

Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
/opt/hostedtoolcache/Python/3.10.21/x64/lib/python3.10/site-packages/netboxlabs/diode/sdk
   chunking.py33197%112
   client.py5674293%202–203, 222–224, 227–230, 522, 590, 595, 599, 682–687, 732–734, 764, 768, 772, 794, 815, 817, 891, 922, 930, 992–993, 1002, 1035, 1062, 1067–1068, 1097, 1102–1105
   exceptions.py44393%69, 82–83
TOTAL6594693% 

Tests Skipped Failures Errors Time
214 0 💤 0 ❌ 0 🔥 2.000s ⏱️

@nbl-ai-review

nbl-ai-review Bot commented Sep 14, 2026

Copy link
Copy Markdown

AI Code Review — risk tier: Lite · dimensions run: correctness, security

Advisory only. A human owns the merge decision.

Summary — This is a release merge (develop → release) whose only substantive change fixes the Cluster scope oneof handling by switching from a single HasField check to WhichOneof, with matching test coverage. No security-sensitive surface is touched, and no issues were found. Net positive change.

🤖 AI Code Review · run · prompts: code-review/v1 · models: claude-sonnet-5/medium · context: no description · 1 comment(s) · 0 prior AI review(s) · usage: $0.63 · 874k in (89% cached) · 2.4k out

@mfiedorowicz
mfiedorowicz merged commit 2fc7ea6 into release Sep 14, 2026
13 checks passed
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.14.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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