Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: fix clean-runner job #1181

Merged
merged 2 commits into from
Jan 26, 2024
Merged

ci: fix clean-runner job #1181

merged 2 commits into from
Jan 26, 2024

Conversation

ldevulder
Copy link
Contributor

@ldevulder ldevulder commented Jan 25, 2024

In case of SPOT in GCP we should delete the Qase run/report in clean-runner otherwise we have no runner where to execute our commands.

Verification runs:

  • CLI-K3s-OBS_Dev with cancelled job, so Qase run/report should be deleted
  • CLI-K3s-OBS_Dev with GCP runner killed (to simulate a SPOT), so Qase run/report should be deleted
  • CLI-RKE2-OBS_Dev with test ending (success of failure, we don't care), so Qase run/report should be kept and publish
  • UI-K3s-OBS_Dev with no Qase ID defined, so nothing should be done

NOTE: most of the VRs are failing for another reason not related to this PR (in fact the whole CI is red since 3-4 days).

@ldevulder ldevulder self-assigned this Jan 25, 2024
@ldevulder ldevulder requested a review from a team as a code owner January 25, 2024 17:04
@ldevulder ldevulder force-pushed the ci-fix-cleanup-of-qase-results branch 7 times, most recently from cbe95d3 to cfd8939 Compare January 26, 2024 13:31
In case of SPOT in GCP we should delete the Qase run/report in
clean-runner otherwise we have no runner where to execute our commands.

Signed-off-by: Loic Devulder <ldevulder@suse.com>
@ldevulder ldevulder force-pushed the ci-fix-cleanup-of-qase-results branch 3 times, most recently from ed9bf07 to dde1ca1 Compare January 26, 2024 14:27
Signed-off-by: Loic Devulder <ldevulder@suse.com>
@ldevulder ldevulder force-pushed the ci-fix-cleanup-of-qase-results branch from dde1ca1 to 889dbdc Compare January 26, 2024 14:37
@ldevulder ldevulder merged commit 5764e97 into main Jan 26, 2024
5 checks passed
@ldevulder ldevulder deleted the ci-fix-cleanup-of-qase-results branch January 26, 2024 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants