-
Notifications
You must be signed in to change notification settings - Fork 95
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
chore(tests): remove unnecessary AddCdromCalled
#352
Conversation
AddCdromCalled
✅ CI Passed. Run echo "Testing with Go 1.20.10"
Testing with Go 1.20.10
? github.com/hashicorp/packer-plugin-vsphere [no test files]
ok github.com/hashicorp/packer-plugin-vsphere/builder/vsphere/clone 0.112s
? github.com/hashicorp/packer-plugin-vsphere/builder/vsphere/common/testing [no test files]
? github.com/hashicorp/packer-plugin-vsphere/builder/vsphere/examples/driver [no test files]
ok github.com/hashicorp/packer-plugin-vsphere/builder/vsphere/common 1.073s
? github.com/hashicorp/packer-plugin-vsphere/version [no test files]
ok github.com/hashicorp/packer-plugin-vsphere/builder/vsphere/driver 4.[6](https://github.com/hashicorp/packer-plugin-vsphere/actions/runs/7330363187/job/19965407964?pr=352#step:4:7)92s
ok github.com/hashicorp/packer-plugin-vsphere/builder/vsphere/iso 0.092s
ok github.com/hashicorp/packer-plugin-vsphere/builder/vsphere/supervisor 2.193s
ok github.com/hashicorp/packer-plugin-vsphere/post-processor/vsphere 0.062s
ok github.com/hashicorp/packer-plugin-vsphere/post-processor/vsphere-template 0.09[7](https://github.com/hashicorp/packer-plugin-vsphere/actions/runs/7330363187/job/19965407964?pr=352#step:4:8)s |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. But I suggest merging this after the changes for reattaching the CD-ROM and another CI pass,
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this change. I agree it is not needed given the existence of AddCdromCalledTimes
There is an AddCdromCalledTimes which serves the same purpose
c4a18bc
to
8082cd5
Compare
@Hi-Angel I rebased and pushed the latest changes from main to fix the reattach CDROMs tests. I will merged once all goes green. |
Thank you! |
There is an AddCdromCalledTimes which serves the same purpose