Skip to content

Conversation

@mfiedorowicz
Copy link
Member

This pull request extends the NetBox Diode SDK ingester to support ownership fields across various entity creation methods. The main focus is on adding owner and owner_group parameters to relevant functions, ensuring that ownership metadata can be set when creating entities. Additionally, support for new optional fields such as comments, profile, and sites is introduced for some entities. The changes also update the mapping of entity names to their key fields and refresh the code generation timestamp.

Ownership and Metadata Enhancements:

Other updates:

  • Updated the code generation timestamp in netboxlabs/diode/sdk/ingester.py to reflect the latest changes.
  • Removed an outdated reference to the examples directory in the README.md.

@github-actions
Copy link

github-actions bot commented Jan 12, 2026

Coverage

Coverage Report
FileStmtsMissCoverMissing
/opt/hostedtoolcache/Python/3.10.19/x64/lib/python3.10/site-packages/netboxlabs/diode/sdk
   client.py5094092%165–166, 185–187, 190–193, 482, 548, 553, 557, 643–648, 693–695, 700, 705, 710, 715, 725, 729, 733, 755, 776, 778, 852, 883, 891, 933, 969, 984–985, 992–993
   exceptions.py44393%69, 82–83
TOTAL5654392% 

Tests Skipped Failures Errors Time
175 0 💤 0 ❌ 0 🔥 1.714s ⏱️

@mfiedorowicz mfiedorowicz changed the title Feat netbox 4.5.0 support feat: NetBox 4.5.x support Jan 12, 2026
Copy link
Contributor

@jajeffries jajeffries left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mfiedorowicz mfiedorowicz merged commit 8e95f47 into develop Jan 12, 2026
6 checks passed
@mfiedorowicz mfiedorowicz deleted the feat-netbox-4.5.0-support branch January 12, 2026 18:40
@github-actions
Copy link

🎉 This PR is included in version 1.8.0 🎉

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.

3 participants