fix(ci): do not silently fail integration tests

[gustavovalverde]

Motivation

After #8594 some test might silently fail as a job dependency was removed for failure-issue, but it was still in the needs list.

Solution

Comment the non existent dependency from failure-issue

Tests

• Integration tests should run here without issue

PR Author's Checklist

• The PR name will make sense to users.
• The solution is tested.
• The documentation is up to date.
• The PR has a priority label.

PR Reviewer's Checklist

• The PR Author's checklist is complete.
• The PR resolves the issue.

[oxarbitrage]

I don't understand the fix. It is enough to have the test commented out instead of just being removed ? Please also note this might be restored in the context of #8585

[arya2]

This looks good, I left a note in #8585 about reverting this if/when the test is restored so we don't forget.

[gustavovalverde]

It is enough to have the test commented out instead of just being removed ?

@oxarbitrage I commented the test from the list, instead of removing it, as the test itself was commented out and not removed. It was, mainly, to keep it consistent with the previous change

[oxarbitrage]

It is enough to have the test commented out instead of just being removed ?

@oxarbitrage I commented the test from the list, instead of removing it, as the test itself was commented out and not removed. It was, mainly, to keep it consistent with the previous change

Got it, thank you.