Skip to content
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

Allowed larger numbers for azurerm_shared_image_version name #2301

Merged
merged 1 commit into from
Nov 13, 2018

Conversation

katbyte
Copy link
Collaborator

@katbyte katbyte commented Nov 13, 2018

fixes #2283

@katbyte
Copy link
Collaborator Author

katbyte commented Nov 13, 2018

Tests pass:

==> Fixing source code with gofmt...
gofmt -s -w ./azurerm
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./azurerm -v -test.run=TestAccAzureRMSharedImageVersion_basic -timeout 180m -ldflags="-X=github.com/terraform-providers/terraform-provider-azurerm/version.ProviderVersion=acc"
=== RUN   TestAccAzureRMSharedImageVersion_basic
=== PAUSE TestAccAzureRMSharedImageVersion_basic
=== CONT  TestAccAzureRMSharedImageVersion_basic
--- PASS: TestAccAzureRMSharedImageVersion_basic (1958.02s)
PASS
ok  	github.com/terraform-providers/terraform-provider-azurerm/azurerm	1959.705s

Copy link
Contributor

@tombuildsstuff tombuildsstuff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@tombuildsstuff tombuildsstuff merged commit 4e8c56f into master Nov 13, 2018
@tombuildsstuff tombuildsstuff deleted the b-image-version-name branch November 13, 2018 08:44
tombuildsstuff added a commit that referenced this pull request Nov 13, 2018
@whytoe
Copy link
Contributor

whytoe commented Nov 13, 2018

Thanks you guys are the best! @tombuildsstuff @katbyte . This was causing a bunch of issues because it looks like Azure isnt updating the source -> managedImage -> id within our pipeline(below json put parameter from Azure Rest Browser). Terraform says it updated the Shared_image_version but when the image was provisioned it was the old image.

We had 0.0.1 Hard-coded until this issue was fixed but it may require more investigating, if its you or them on the update function.

"source": {
        "managedImage": {
          "id": "/subscriptions/{subscriptionId}/resourceGroups/{resourceGroup}/providers/Microsoft.Compute/images/{imageName}"

@tombuildsstuff
Copy link
Contributor

@whytoe is this within a VM Scale Set? if so there's an explicit upgrade API endpoint/method that can be called which cycles the machines in the Scale Set

@whytoe
Copy link
Contributor

whytoe commented Nov 13, 2018

@tombuildsstuff no this is just to commit a new Source Managed Image ID to the same version number in the shared gallery

Example 
ID 
"id": "/subscriptions/{subscriptionId}/resourceGroups/{resourceGroup}/providers/Microsoft.Compute/images/ImageName1"

Then I update the ID but I keep the same image version number

"/subscriptions/{subscriptionId}/resourceGroups/{resourceGroup}/providers/Microsoft.Compute/images/ImageName2"

Terraform will state that it has updated the shared image version in 2 minutes (first time it takes about 10-15 minutes). But the Image will not actually update.

@tombuildsstuff
Copy link
Contributor

@whytoe mind opening a separate issue about that?

@ghost
Copy link

ghost commented Mar 6, 2019

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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!

@ghost ghost locked and limited conversation to collaborators Mar 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Shared_Image _Gallery_Version only takes single digit integers
3 participants