What issue your PR is related to.
Make vhdSize argument as optional for unpack command. When vhdSize will not be provided, 4x of source package size in MB will be used as recommended size of vhd.
What change your PR adds.
Logic to check if vhdSize is not provided and then calculating the recommended vhdSize.
How you tested your change.
Checked for a bunch of apps, what is the minimum vhdSize that those apps were able to get unpacked with and local app attach was working for them. Outcome of this exercise was recommended vhdSize, which came out to be 4x of source package size in MB.
After adding the logic to code, tested the vhdSize creation flow for a bunch of apps.
What issue your PR is related to. Make vhdSize argument as optional for unpack command. When vhdSize will not be provided, 4x of source package size in MB will be used as recommended size of vhd.
What change your PR adds. Logic to check if vhdSize is not provided and then calculating the recommended vhdSize.
How you tested your change. Checked for a bunch of apps, what is the minimum vhdSize that those apps were able to get unpacked with and local app attach was working for them. Outcome of this exercise was recommended vhdSize, which came out to be 4x of source package size in MB. After adding the logic to code, tested the vhdSize creation flow for a bunch of apps.