[PR #13823] [MERGED] 13813 fix virtual chassis member count #14253

Closed
opened 2025-12-29 23:23:26 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/13823
Author: @arthanson
Created: 9/19/2023
Status: Merged
Merged: 9/20/2023
Merged by: @jeremystretch

Base: developHead: 13813-virtual-chassis-member-count


📝 Commits (2)

  • e691baa 13813 fix virtual chassis member count
  • 9a02d5a 13813 add test

📊 Changes

2 files changed (+11 additions, -2 deletions)

View changed files

📝 netbox/utilities/counters.py (+3 -2)
📝 netbox/utilities/tests/test_counters.py (+8 -0)

📄 Description

Fixes: #13813

Fixes VirtualChassis member count - issue was in counter fields where the tracked object (Device) existed but had the FK pointer that was being tracked as None which caused the increment to get bypassed. Added test for this condition.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/netbox-community/netbox/pull/13823 **Author:** [@arthanson](https://github.com/arthanson) **Created:** 9/19/2023 **Status:** ✅ Merged **Merged:** 9/20/2023 **Merged by:** [@jeremystretch](https://github.com/jeremystretch) **Base:** `develop` ← **Head:** `13813-virtual-chassis-member-count` --- ### 📝 Commits (2) - [`e691baa`](https://github.com/netbox-community/netbox/commit/e691baa6dfe54e0938e27d5ed612bc6a1b68fa59) 13813 fix virtual chassis member count - [`9a02d5a`](https://github.com/netbox-community/netbox/commit/9a02d5aaebec6f860005c540111760bf355fbb7c) 13813 add test ### 📊 Changes **2 files changed** (+11 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `netbox/utilities/counters.py` (+3 -2) 📝 `netbox/utilities/tests/test_counters.py` (+8 -0) </details> ### 📄 Description ### Fixes: #13813 Fixes VirtualChassis member count - issue was in counter fields where the tracked object (Device) existed but had the FK pointer that was being tracked as None which caused the increment to get bypassed. Added test for this condition. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 23:23:26 +01:00
adam closed this issue 2025-12-29 23:23:26 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#14253