Skip to content

[CI] Allow to run each cloud cleanup step independently in DRY_RUN false mode #2581

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.

Sign up for GitHub

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

Merged
merged 9 commits into from
May 8, 2025
Merged

[CI] Allow to run each cloud cleanup step independently in DRY_RUN false mode #2581

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
7054f85
Set the expected COMMAND when DRY_RUN is false
mrodm May 7, 2025
04243b8
Add annotations
mrodm May 7, 2025
7fb14de
Update annotation with step key
mrodm May 7, 2025
e4d3ce1
Update context annotation
mrodm May 7, 2025
464b115
Add exception
mrodm May 7, 2025
479d8c4
Do not show output in grep command
mrodm May 7, 2025
5bd8746
Add header in buildkite output
mrodm May 7, 2025
bb930ec
Restore plan for cloud-reaper
mrodm May 7, 2025
f30e89a
Update annotation
mrodm May 7, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add exception
  • Loading branch information
@mrodm
mrodm committed May 7, 2025
commit 464b115661222d167f0f8a8581cd9db6a312dee1
10 changes: 10 additions & 0 deletions .buildkite/scripts/cloud-cleanup-oidc.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,16 @@ any_resources_to_delete() {
# ⇒ Loading configuration...
# ✓ Succeeded to load configuration
# Scanning resources... ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:00

# FIXME:: When running with DRY_RUN: false there could be more lines.
# In the case, there is nothing to delete, there is one more line:
# ⇒ Nothing to destroy !
# but there are no examples when resources are deleted to add the required logic
if [[ "${DRY_RUN}" == false ]] ; then
if tail -n 1 ${file} | grep "Nothing to destroy" 2> /dev/null ; then
return 1
fi
fi
number=$(tail -n +4 "${file}" | wc -l)
if [ "${number}" -eq 0 ]; then
return 1
Expand Down
10 changes: 10 additions & 0 deletions .buildkite/scripts/cloud-cleanup.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,16 @@ any_resources_to_delete() {
# ⇒ Loading configuration...
# ✓ Succeeded to load configuration
# Scanning resources... ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:00

# FIXME:: When running with DRY_RUN: false there could be more lines.
# In the case, there is nothing to delete, there is one more line:
# ⇒ Nothing to destroy !
# but there are no examples when resources are deleted to add the required logic
if [[ "${DRY_RUN}" == false ]] ; then
if tail -n 1 ${file} | grep "Nothing to destroy" 2> /dev/null ; then
return 1
fi
fi
number=$(tail -n +4 "${file}" | wc -l)
if [ "${number}" -eq 0 ]; then
return 1
Expand Down