Refactor #6187
closed
Dynflowize environment delete
Updated by Ivan Necas almost 10 years ago
- Blocks Feature #6180: Implement org destroy added
Updated by David Davis almost 10 years ago
- Status changed from New to Closed
The env destroy doesn't require dynflow. It doesn't touch any backend services. There was some code I wrote/fixed as part of this bug but I'll just move that code to the PR for #6180.
Updated by Ivan Necas over 9 years ago
- Status changed from Closed to New
- Target version deleted (
48)
I don't think this is reason for not having the lifecycle enevironment destroy action. Two reasons for having it:
1. destroying the resoruces under the lifecycle environment: consumers, content view versions
2. having the action recorded in the foreman tasks for auditing purposes
Updated by David Davis over 9 years ago
@ivan, content view versions and consumers aren't tied directly to environments. I'll have to take a look at the code again but I don't believe you can delete an environment if it has content_view_environments.
Updated by David Davis over 9 years ago
Looks like I wrote the code in an another PR but forgot to pull it over to the new PR:
https://github.com/Katello/katello/pull/4324/files#diff-186d05e79248a368ffb5ff36269a9ea2R39
Updated by David Davis over 9 years ago
I've created a bug to address the deleting of environments with associations in the short term.
Longer term, maybe we can develop something that would at least warn users if environments have content hosts, etc. This would of course rely on dynflow. Maybe we could even allow users to choose whether to reassign or destroy associations like the content view remove code.
Updated by David Davis over 9 years ago
- Related to Bug #6815: I can delete a environment even if it has associations added
Updated by Ivan Necas over 9 years ago
- Assignee changed from David Davis to Adam Ruzicka
Ok, assigning this to aruzicka, just for auditing purposes, leaving the deletion of associations untouched for now.
Updated by The Foreman Bot over 9 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/Katello/katello/pull/4571 added
- Pull request deleted (
)
Updated by David Davis over 9 years ago
- Assignee changed from Adam Ruzicka to Partha Aji
- Pull request https://github.com/Katello/katello/pull/4563 added
- Pull request deleted (
https://github.com/Katello/katello/pull/4571)
Updated by Partha Aji over 9 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset katello|c7bfc1592488adf5312e09d84cff082e3e2d29fb.
Updated by Eric Helms over 9 years ago
- translation missing: en.field_release set to 13