Packer: VirtualBox 6.0 Beta 3 fails with bad parameter "sataportcount"

[ripleymj]

virtualbox-iso builder fails when run with VirtualBox 6.0 Beta 3 because it passes an invalid parameter sataportcount to VBoxManage. It looks like #2429 and #2438 addressed this by adding a version check for 4.3 and 5.x.

I just recently started using Packer, I'm not sure if this is best addressed by adding a check for 6.x, checking for anything greater than 4.3, or if it's just time to deprecate < 4.3 functionality.

[SwampDragons]

I've written a patch that'll check for 6.x in addition to 5.x; would you be willing to test it for me?

Linux build:
packer.zip

osx build:
packer.zip

windows build:
packer.zip

[ripleymj]

Yep, that looks better. Thanks for the quick fix!

Just out of curiosity, it seems like only VirtualBox 5.1 and 5.2 are currently supported, with 6.0 coming soon. At what point would it be safe to delete the <= 4.2 code and prevent this error from occurring on newer versions?

[svpernova09]

@SwampDragons thanks for the build! fixed the issue for me.

[breisig]

@SwampDragons Thanks, that build also works for me.

[hashibot[bot]]

I'm going to lock this issue because it has been closed for _30 days_ ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.