Skip to content

Fix VM cleanup issue and improve VM deletion logic for containerd periodic jobs#310

Merged
AshwinHIBM merged 1 commit intoppc64le-cloud:mainfrom
bharti-0211:fix-vm-cleanup
Apr 21, 2026
Merged

Fix VM cleanup issue and improve VM deletion logic for containerd periodic jobs#310
AshwinHIBM merged 1 commit intoppc64le-cloud:mainfrom
bharti-0211:fix-vm-cleanup

Conversation

@bharti-0211
Copy link
Copy Markdown
Contributor

@bharti-0211 bharti-0211 commented Feb 11, 2026

Ensures VM deletion in all cases even if the tests don't start running at all because of some early issue like inability to connect to the VM.
Fixes #311

@ppc64le-cloud-bot ppc64le-cloud-bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 11, 2026
@anup-kodlekere
Copy link
Copy Markdown

I am okay with these changes for now. I will raise another PR once this is merged to improve robustness of the scripts.

Comment thread test-containerd/instantiate_vm_and_test.sh Outdated
Comment thread test-containerd/instantiate_vm_and_test.sh Outdated
Comment thread test-containerd/instantiate_vm_and_test.sh Outdated
Comment thread test-containerd/instantiate_vm_and_test.sh Outdated
Comment thread test-containerd/instantiate_vm_and_test.sh Outdated
Comment thread test-containerd/instantiate_vm_and_test.sh Outdated
@AshwinHIBM
Copy link
Copy Markdown
Collaborator

Trying to adapt https://google.github.io/styleguide/shellguide.html in every new PR.

@ppc64le-cloud-bot ppc64le-cloud-bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed do-not-merge/contains-merge-commits size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Mar 2, 2026
@bharti-0211 bharti-0211 requested a review from AshwinHIBM March 4, 2026 11:19
@AshwinHIBM
Copy link
Copy Markdown
Collaborator

Can you squash the commits?

@AshwinHIBM
Copy link
Copy Markdown
Collaborator

/approve

@ppc64le-cloud-bot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: AshwinHIBM, bharti-0211

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ppc64le-cloud-bot ppc64le-cloud-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 31, 2026
@clnperez
Copy link
Copy Markdown
Collaborator

clnperez commented Apr 7, 2026

LGTM

@AshwinHIBM AshwinHIBM merged commit 0523dce into ppc64le-cloud:main Apr 21, 2026
1 check was pending
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

VM not cleaned up when tests fail to start due to some early issues

5 participants