Skip to content

Conversation

justinc1
Copy link
Collaborator

@justinc1 justinc1 commented Jan 6, 2025

Modules vm and vm_disk were not able to eject ISO image from CD-ROM device. Reported in #345. PR fixes this. It also adds integration tests for both modules, and an example playbook.

@justinc1 justinc1 requested a review from anazobec January 6, 2025 10:02
@justinc1 justinc1 force-pushed the iso-eject branch 3 times, most recently from 58472e6 to 2cde845 Compare January 17, 2025 16:33
@anazobec
Copy link
Collaborator

Looks fixed to me. Thanks @justinc1. Merge when ready.

Param is "iso_name", not "name"

Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
Reproduce #345

Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
Fixes #345

Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>
@justinc1
Copy link
Collaborator Author

Integ test https://github.com/ScaleComputing/HyperCoreAnsibleCollection/actions/runs/13068024737/job/36463667445

  • virtual_disk - 502 Bad Gateway during upload, not related to this PR
  • vm_disk - looks like temporal error, returned ISO had wrong size, failed at "assertion": "vminfo.records.0.disks.0.size == 356352", test passed when run manually

@justinc1 justinc1 merged commit eeca326 into main Jan 31, 2025
59 of 61 checks passed
@justinc1 justinc1 deleted the iso-eject branch January 31, 2025 07:58
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.

2 participants