Revert "Bump geekyeggo/delete-artifact from 2 to 4"

[vojtapolasek]

Reverts #11407

This change prevents efficient review of CI tests. It requires some changes in workflow files, but it is not clear what changes are required. It should be investigated.

[github-actions]

Start a new ephemeral environment with changes proposed in this pull request:

Fedora Environment

Oracle Linux 8 Environment

[github-actions]

🤖 The image for this PR is available at:
ghcr.io/complianceascode/k8scontent:11481

[codeclimate]

Code Climate has analyzed commit 9a054e6 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 58.5% (0.0% change).

View more on Code Climate.

[marcusburghardt]

The new version doesn't seem to be buggy, but implements a reasonable new feature. We will need to update it any way soon or later. The automatus tests are working properly (the tasks that really matters) but can't remove a file at the end. In practical we have to access these failed tests to confirm if they are not also showing a test scenario error. I don't think there is enough benefits reverting this update now.

Is not possible to simply disable the task which is trying to remove the file, for example?

[vojtapolasek]

You are correct @marcusburghardt .
I created #11482
Feel free to close this PR. I will also create associated issue so that we don't forget about this thing and we investigate it more. I consider this fix to be temporary.

[marcusburghardt]

@Mab879 we can close this in favor of #11482

[Mab879]

@Mab879 we can close this in favor of #11482

Makes sense to me.

[Mab879]

Closing in favor #11482