Refactor deletion of Git-based domains

[miq-bot]

Refactor based on comments in: ManageIQ/manageiq#12539 (comment)

---

This issue was moved to this repository from ManageIQ/manageiq#12696, originally opened by @gmcculloug

[miq-bot]

This issue has been automatically marked as stale because it has not been updated for at least 6 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions!

[miq-bot]

This issue has been automatically marked as stale because it has not been updated for at least 3 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the traige process documentation.

[Fryguy]

@NickLaMuro Is this already done via the work we did with embedded ansible?

[NickLaMuro]

@Fryguy My guess is "no" since I think the issues you had that caused this issue to be created were with naming/queue conventions outside the context of app/models/git_repository.rb and lib/git_worktree.rb.

ManageIQ/manageiq#12539 (comment)

I'm ok with merging as is, but there are some inconsistencies with the existing queueing (or task) pattern.

• The model's destroy method should do the actual destroy (not on the queue)
• The model should have a destroy_queue that calls destroy but through the queue.
• The API and UI code should call destroy_queue
• The external facing API endpoint should just be destroy (that is the User has no idea it's running off of the queue other than the fact that they get a task back)