Container SSA: splitting pod deletion

[enoodle]

This is just separating the pod deletion and skipping it altogether if no pod was created before.

cc @simon3z

[miq-bot]

Checked commit enoodle@31a8cb0 with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
1 file checked, 1 offense detected

app/models/manageiq/providers/kubernetes/container_manager/scanning/job.rb

• ❕ - Line 176, Col 3 - Metrics/AbcSize - Assignment Branch Condition size for delete_pod is too high. [34.73/20]

[roliveri]

@enoodle Should this be back ported to euwe and/or darga?

[enoodle]

@roliveri this doesn't change much feature, but the user experience: instead of waiting 5/10 minutes to see the failed scan it will be more immediate and the error will be more clear.
I think that it is important to add to euwe and darge.
@miq-bot add_label euwe/yes

cc @simon3z

[simon3z]

Just euwe. cc @Loicavenel in case he thinks otherwise.

[simon3z]

Temporarily removing darga (we can re-add in case @Loicavenel thinks otherwise).

@miq-bot remove_label darga/yes

[simaishi]

@enoodle is there a BZ for this?

[enoodle]

@simaishi This is not a bugfix so no. I don't think its important to backport to euwe. I would have been nice back then but now its not important anymore.
@miq-bot remove_label euwe/yes