Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bulk-Delete: List Dependently-Deleted Entities #16191

Open
chadhpaine opened this issue May 18, 2024 · 0 comments
Open

Bulk-Delete: List Dependently-Deleted Entities #16191

chadhpaine opened this issue May 18, 2024 · 0 comments
Labels
complexity: high Expected to require a large amont of time and effort to implement relative to other tasks status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation type: feature Introduction of new functionality to the application

Comments

@chadhpaine
Copy link

chadhpaine commented May 18, 2024

NetBox version

v3.7.4

Feature type

New functionality

Proposed functionality

When performing a bulk-deletion of entities, Netbox should ask the user to confirm the deletion of both the set of entities selected explicitly for deletion as well as the set of sub-entities that would be deleted due to those sub-entities being dependent on an entity being targeted for deletion. An example of this is of a set of devices being deleted, where the devices each carry different sets of components; the bulk-deletion view should show the list of devices the user targeted (as it currently does), and well as a list of the components from those devices that will be deleted (this is the request), with a column showing which device each component is related to.

Use case

When performing atomic-deletion of entities such as devices and components, Netbox warns that sub-entities considered dependent on the entity-being-deleted will also be deleted.

image
(from #15665 (comment))

For purposes of this feature-request, we call such sub-entities "dependently-deleted entities" because their deletion is dependent on their association to the entity targeted for deletion - removing the association between a target entity and a sub-entity before deletion of the target entity is often sufficient to retain the sub-entity.

When performing bulk-deletion of the same kinds of targets, the bulk-deletion menu shows the list of targets, but does not for-each-target show the list of dependently-deleted entities that accompany it. As a result, the deletion of the dependently-deleted entities is therefore both silent and unacknowledged; customers may be unaware that they have removed data from their instance that they may (or may not) have wanted to retain, unless they also review the changelog.

image

This use-case is not limited to the relationship between devices and their components (#14488); it also incorporates relationships between inventory-items (and templates) and other components within a single device (#15665), between interfaces and their assigned IP addresses (#6015 mentions this feature), between virtual-machines and assigned virtual-disks (not mentioned in-issue but observed at demo.netbox.dev), and at numerous other points of entity-intersection throughout the application.

This feature would bring the functionality of the bulk-deletion view to-parity with its atomic-deletion counterpart. Providing a user with a visual reminder that their intended actions may have unintended consequences is important to ensure that customers have and consider their choice of which data to retain and which to discard.

Database changes

None

External dependencies

None

@chadhpaine chadhpaine added status: needs triage This issue is awaiting triage by a maintainer type: feature Introduction of new functionality to the application labels May 18, 2024
@arthanson arthanson removed their assignment May 20, 2024
@arthanson arthanson added status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation and removed status: needs triage This issue is awaiting triage by a maintainer labels May 20, 2024
@jeremystretch jeremystretch added the complexity: high Expected to require a large amont of time and effort to implement relative to other tasks label May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: high Expected to require a large amont of time and effort to implement relative to other tasks status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation type: feature Introduction of new functionality to the application
Projects
None yet
Development

No branches or pull requests

3 participants