[PR #18929] [MERGED] Fixes #18928: Fix support for cascading deletions when cleaning up expired changelog records #15476

Closed
opened 2025-12-30 00:22:10 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/18929
Author: @jeremystretch
Created: 3/17/2025
Status: Merged
Merged: 3/19/2025
Merged by: @bctiemann

Base: mainHead: 18928-housekeeping-changelog-cleanup


📝 Commits (1)

  • c5801f9 Fixes #18928: Fix support for cascading deletions when cleaning up expired changelog records

📊 Changes

1 file changed (+1 additions, -2 deletions)

View changed files

📝 netbox/extras/management/commands/housekeeping.py (+1 -2)

📄 Description

Fixes: #18928

  • Use the conventional delete() method rather than _raw_delete()

Note: This will have some performance impact, but it should be negligible in most circumstances. It's not expected to be concerning as the housekeeping task is typically run in the background anyway.


🔄 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/18929 **Author:** [@jeremystretch](https://github.com/jeremystretch) **Created:** 3/17/2025 **Status:** ✅ Merged **Merged:** 3/19/2025 **Merged by:** [@bctiemann](https://github.com/bctiemann) **Base:** `main` ← **Head:** `18928-housekeeping-changelog-cleanup` --- ### 📝 Commits (1) - [`c5801f9`](https://github.com/netbox-community/netbox/commit/c5801f988127cdb764d765b1452e1b4d46f7c834) Fixes #18928: Fix support for cascading deletions when cleaning up expired changelog records ### 📊 Changes **1 file changed** (+1 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `netbox/extras/management/commands/housekeeping.py` (+1 -2) </details> ### 📄 Description ### Fixes: #18928 - Use the conventional `delete()` method rather than `_raw_delete()` Note: This _will_ have some performance impact, but it should be negligible in most circumstances. It's not expected to be concerning as the housekeeping task is typically run in the background anyway. --- <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-30 00:22:10 +01:00
adam closed this issue 2025-12-30 00:22:10 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#15476