[PR #19807] [MERGED] Fixes #19806: Introduce JobFailed exception to allow marking background jobs as failed #15725

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

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/19807
Author: @jeremystretch
Created: 7/2/2025
Status: Merged
Merged: 7/2/2025
Merged by: @jnovinger

Base: mainHead: 19806-job-status-failed


📝 Commits (1)

  • e90d0cc Fixes #19806: Introduce JobFailed exception to allow marking background jobs as failed

📊 Changes

6 files changed (+37 additions, -10 deletions)

View changed files

📝 docs/features/background-jobs.md (+1 -1)
📝 docs/plugins/development/background-jobs.md (+2 -1)
📝 netbox/core/exceptions.py (+12 -2)
📝 netbox/core/models/jobs.py (+3 -4)
📝 netbox/netbox/jobs.py (+8 -1)
📝 netbox/netbox/tests/test_jobs.py (+11 -1)

📄 Description

Fixes: #19806

  • Introduce the JobFailed exception
  • Add the test_handle_failed() test
  • Update development documentation

🔄 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/19807 **Author:** [@jeremystretch](https://github.com/jeremystretch) **Created:** 7/2/2025 **Status:** ✅ Merged **Merged:** 7/2/2025 **Merged by:** [@jnovinger](https://github.com/jnovinger) **Base:** `main` ← **Head:** `19806-job-status-failed` --- ### 📝 Commits (1) - [`e90d0cc`](https://github.com/netbox-community/netbox/commit/e90d0cc9406e98176d9a3c4bd3d876415f80266c) Fixes #19806: Introduce JobFailed exception to allow marking background jobs as failed ### 📊 Changes **6 files changed** (+37 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `docs/features/background-jobs.md` (+1 -1) 📝 `docs/plugins/development/background-jobs.md` (+2 -1) 📝 `netbox/core/exceptions.py` (+12 -2) 📝 `netbox/core/models/jobs.py` (+3 -4) 📝 `netbox/netbox/jobs.py` (+8 -1) 📝 `netbox/netbox/tests/test_jobs.py` (+11 -1) </details> ### 📄 Description ### Fixes: #19806 - Introduce the `JobFailed` exception - Add the `test_handle_failed()` test - Update development documentation --- <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:23:39 +01:00
adam closed this issue 2025-12-30 00:23:39 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#15725