-
Notifications
You must be signed in to change notification settings - Fork 80
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Failed to Delete Temporary VM Disk - Authentication Not Supported #315
Comments
@wbz-mt Can you please post a simplified template and the full output of your packer build I will add that I believe this is a separate issue #190 was two issues, one was that we tried to delete the KeyVault's disk incorrectly, and the other was related to an auto-rest crash when trying to create a managed image, with the clearer output in v1.4.3 I can see you are failing to delete the temporary data disk, is this happening every time you build? |
@JenGoldstrich I've been tried multiple times with the latest version to create ubuntu20.04, yes, every time since this morning. Please find the simplified template below.
and one output below.
|
Okay I've seen this issue before, The actual error here is
coming back from the Azure API, I am not sure why we're seeing this but will investigate it, thanks for your report |
This issue has been synced to JIRA for planning. |
@JenGoldstrich, since I have no visibilities on JIRA ID: HPR-1234, just wanted to check the status of this issue, I'm still getting such errors every time with a deployment. |
Hey @wbz-mt, I have addressed this in this not yet merged PR #326 and the fix for this will be released in the v2.0.0 release of Packer. We were calling the wrong Delete Disk endpoint for VHD builds, leading to that error message when attempting to delete the OS Disk as we don't initialize the storage account client for builds that don't create VHDs. v2.0.0 does not have any breaking changes that impact the build you're using in the template above but for a full list of breaking changes if you are concerned https://github.com/JenGoldstrich/packer-plugin-azure/releases/tag/v2.0.0-betarc1 the release notes for this temporary beta build have all of the breaking changes that will go in the final release |
Thanks @JenGoldstrich for the feedback and then I will be waiting for the v2.0.0 when it's available. |
Originally posted by @JenGoldstrich in #190 (comment)
I'm still seeing the same issue with the latest packer and azure plugins installed.
The text was updated successfully, but these errors were encountered: