Skip to content

fix: extract instance id and disk id from packer build and use them for cleanup in case of cancellation#6539

Open
librarian wants to merge 5 commits into
mainfrom
users/librarian/packer-build-always
Open

fix: extract instance id and disk id from packer build and use them for cleanup in case of cancellation#6539
librarian wants to merge 5 commits into
mainfrom
users/librarian/packer-build-always

Conversation

@librarian

Copy link
Copy Markdown
Collaborator

Initial thought was to use always() to prevent packer build from interrupting and thus allowing it to cleanup after himself. But this is problematic and would require force-cancel to cancel workflow.

Now we parse logs that are provided by packer and catch instance ids and disk ids and run clean up action after.

@github-actions

Copy link
Copy Markdown
Contributor

status: 🟢

action-validator - 🟢 log
actionlint - 🟢 log

@github-actions

Copy link
Copy Markdown
Contributor

status: 🔴

shellcheck - 🟢 log
shfmt - 🔴 log

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Note

This is an automated comment that will be appended during run.

Note

All workloads for linux-x86_64-relwithdebinfo have completed.

Tip

Planned checks for linux-x86_64-relwithdebinfo.

@github-actions

Copy link
Copy Markdown
Contributor

status: 🔴 GA scripts pytest result (artifact) GA scripts pytest report (artifact) GA scripts pytest report (s3) GA scripts pytest results (s3)

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
191 191 0 0 0 0 0

black - 🟢 log
flake8 - 🔴 log

@github-actions

Copy link
Copy Markdown
Contributor

status: 🟢

action-validator - 🟢 log
actionlint - 🟢 log

@github-actions

Copy link
Copy Markdown
Contributor

status: 🟢

shellcheck - 🟢 log
shfmt - 🟢 log

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Note

This is an automated comment that will be appended during run.

Note

All workloads for linux-x86_64-relwithdebinfo have completed.

Tip

Planned checks for linux-x86_64-relwithdebinfo.

@github-actions

Copy link
Copy Markdown
Contributor

status: 🔴 GA scripts pytest result (artifact) GA scripts pytest report (artifact) GA scripts pytest report (s3) GA scripts pytest results (s3)

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
191 191 0 0 0 0 0

black - 🟢 log
flake8 - 🔴 log

@github-actions

Copy link
Copy Markdown
Contributor

status: 🟢

action-validator - 🟢 log
actionlint - 🟢 log

@github-actions

Copy link
Copy Markdown
Contributor

status: 🟢

shellcheck - 🟢 log
shfmt - 🟢 log

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Note

This is an automated comment that will be appended during run.

Note

All workloads for linux-x86_64-relwithdebinfo have completed.

Tip

Planned checks for linux-x86_64-relwithdebinfo.

@github-actions

Copy link
Copy Markdown
Contributor

status: 🟢 GA scripts pytest result (artifact) GA scripts pytest report (artifact) GA scripts pytest report (s3) GA scripts pytest results (s3)

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
191 191 0 0 0 0 0

black - 🟢 log
flake8 - 🟢 log

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant