https://pulumi.com logo
Powered by
Title
b

boundless-airport-99052

12/07/2020, 12:54 PM
Hello, I’m trying to debug an issue introduced by the 
terraform azurerm - v2.34.0
provider. I downgraded to 
@pulumi/azure - v3.28.0
which uses 
terraform azurerm v2.33.0
according to https://github.com/pulumi/pulumi-azure/blob/master/CHANGELOG.md But I still got the same error message 
error: azure:operationalinsights/analyticsWorkspace:AnalyticsWorkspace resource 'log-f-stack-s-a-sb' has a problem: expected daily_quota_gb to be at least (0.000000), got -1.000000
The 
daily_quota_gb
property was not present before`terraform azurerm v2.34.0` so I wonder why I still got this error. Terraform changelog: https://github.com/terraform-providers/terraform-provider-azurerm/releases/tag/v2.34.0 
azurerm_log_analytics_workspace
 - support for the 
daily_quota_gb
 property (#8861)
I can also confirm this with this commit which in only present in v2.34
b

broad-dog-22463

12/07/2020, 12:59 PM
Hey @boundless-airport-99052 So this issue is this line: 
if workspaceCapping := resp.WorkspaceCapping; workspaceCapping != nil {
		d.Set("daily_quota_gb", resp.WorkspaceCapping.DailyQuotaGb)
	} else {
		d.Set("daily_quota_gb", utils.Float(-1))
	}
notice it's being set to -1 here
b

boundless-airport-99052

12/07/2020, 1:00 PM
hi paul
b

broad-dog-22463

12/07/2020, 1:00 PM
the way around this for you would be to use ignoreChanges on that property
then we can open an issue and help work through it
b

boundless-airport-99052

12/07/2020, 1:01 PM
the code you mentioned is pulumi or terrform code?
b

broad-dog-22463

12/07/2020, 1:01 PM
TF code
we don't have any explicit mapping of that field
wait a minute, I linked to the wrong code but it's still the same issue: 
"daily_quota_gb": {
				Type:             schema.TypeFloat,
				Optional:         true,
				Default:          -1.0,
				DiffSuppressFunc: dailyQuotaGbDiffSuppressFunc,
				ValidateFunc:     validation.FloatAtLeast(0),
			},
b

boundless-airport-99052

12/07/2020, 1:03 PM
you’re right
but this code has been added by this commit : https://github.com/terraform-providers/terraform-provider-azurerm/commit/539b9e33c7191a2205ab836b8b4d328219ece7b8#diff-9789004cd3332a59bb68e7e189a450189f96d532fd9e4bfb99e53634dcbaaebe
which is not in terraform v2.33
it’s why I try dowgraded to @pulumi/azure - v3.28.0
b

broad-dog-22463

12/07/2020, 1:04 PM
I am on master and see this in the provider
https://github.com/terraform-providers/terraform-provider-azurerm/blob/master/azurerm/internal/services/loganalytics/log_analytics_workspace_resource.go#L96
b

boundless-airport-99052

12/07/2020, 1:04 PM
is pulumi using the terraform master branch?
b

broad-dog-22463

12/07/2020, 1:04 PM
nope this code was added 27 days ago
https://github.com/terraform-providers/terraform-provider-azurerm/commit/07938e516ee958d31c98c34dbdaec120ccbad255
that's the commit that added it
which was released in v2.39.0 of AzureRM TF provide
b

boundless-airport-99052

12/07/2020, 1:05 PM
yes
and before this commit https://github.com/terraform-providers/terraform-provider-azurerm/commit/539b9e33c7191a2205ab836b8b4d328219ece7b8 the property 
daily_quota_gb
was not existing at all
b

broad-dog-22463

12/07/2020, 1:06 PM
agreed - this is the bug for sure
so setting ignoreChanges on this should work for you for now till we can fix it up
b

boundless-airport-99052

12/07/2020, 1:07 PM
Yes, but if I use @pulumi/azure v3.28, I should use terraform v2.33, isn’t it?
b

broad-dog-22463

12/07/2020, 1:12 PM
those are correct versions
yes
b

boundless-airport-99052

12/07/2020, 1:15 PM
(Note that I didn’t set the 
dailyQuotaGb
in the pulumi code. But this property is returned by azure - I try to add the 
ignoreChanges
property and I still got this error)
I created a new pulumi project (
@pulumi/azure v3.32.0
) with a new AnalyticsWorkspace resource. I was able to reproduce the error message. Then I downgraded to 
@pulumi/azure v3.28.0
and I didn’t have the error anymore 👍 But on my real project, after having downgraded to 
@pulumi/azure v3.28.0
I still got the error 🤔 The only difference I see is that in my real project the AnalyticsWorkspace resource already exists. Do you have any idea?
b

broad-dog-22463

12/07/2020, 1:35 PM
mmmhhhh that's super weird! If azure is returning that then it's a potential breakage in their API - can you set the value to 0 for now?
or whatever makes sense for your project to ensure you are not blocked
b

boundless-airport-99052

12/07/2020, 1:40 PM
I set the verbose to 
9
and got this: 
I1207 14:37:59.165317   71348 rpc.go:247] Unmarshaling property for RPC[Provider[azure, 0xc0013b7c80].Check(urn:pulumi:sandbox::foundations::maif:subscription$azure:operationalinsights/analyticsWorkspace:AnalyticsWorkspace::log-f-stack-s-a-sb).inputs]: __defaults={[{dailyQuotaGb} {internetIngestionEnabled} {internetQueryEnabled} {location}]}
I1207 14:37:59.165358   71348 rpc.go:247] Unmarshaling property for RPC[Provider[azure, 0xc0013b7c80].Check(urn:pulumi:sandbox::foundations::maif:subscription$azure:operationalinsights/analyticsWorkspace:AnalyticsWorkspace::log-f-stack-s-a-sb).inputs]: dailyQuotaGb={-1}
I1207 14:37:59.165376   71348 rpc.go:247] Unmarshaling property for RPC[Provider[azure, 0xc0013b7c80].Check(urn:pulumi:sandbox::foundations::maif:subscription$azure:operationalinsights/analyticsWorkspace:AnalyticsWorkspace::log-f-stack-s-a-sb).inputs]: internetIngestionEnabled={true}
I1207 14:37:59.165390   71348 rpc.go:247] Unmarshaling property for RPC[Provider[azure, 0xc0013b7c80].Check(urn:pulumi:sandbox::foundations::maif:subscription$azure:operationalinsights/analyticsWorkspace:AnalyticsWorkspace::log-f-stack-s-a-sb).inputs]: internetQueryEnabled={true}
I1207 14:37:59.165403   71348 rpc.go:247] Unmarshaling property for RPC[Provider[azure, 0xc0013b7c80].Check(urn:pulumi:sandbox::foundations::maif:subscription$azure:operationalinsights/analyticsWorkspace:AnalyticsWorkspace::log-f-stack-s-a-sb).inputs]: location={francecentral}
I1207 14:37:59.165413   71348 rpc.go:247] Unmarshaling property for RPC[Provider[azure, 0xc0013b7c80].Check(urn:pulumi:sandbox::foundations::maif:subscription$azure:operationalinsights/analyticsWorkspace:AnalyticsWorkspace::log-f-stack-s-a-sb).inputs]: name={log-f-stack-s-a-sb-yh}
I1207 14:37:59.165439   71348 rpc.go:247] Unmarshaling property for RPC[Provider[azure, 0xc0013b7c80].Check(urn:pulumi:sandbox::foundations::maif:subscription$azure:operationalinsights/analyticsWorkspace:AnalyticsWorkspace::log-f-stack-s-a-sb).inputs]: resourceGroupName={rg-cloud-platform}
I1207 14:37:59.165465   71348 rpc.go:247] Unmarshaling property for RPC[Provider[azure, 0xc0013b7c80].Check(urn:pulumi:sandbox::foundations::maif:subscription$azure:operationalinsights/analyticsWorkspace:AnalyticsWorkspace::log-f-stack-s-a-sb).inputs]: retentionInDays={30}
I1207 14:37:59.165488   71348 rpc.go:247] Unmarshaling property for RPC[Provider[azure, 0xc0013b7c80].Check(urn:pulumi:sandbox::foundations::maif:subscription$azure:operationalinsights/analyticsWorkspace:AnalyticsWorkspace::log-f-stack-s-a-sb).inputs]: sku={PerGB2018}
I1207 14:37:59.165509   71348 provider_plugin.go:574] Provider[azure, 0xc0013b7c80].Check(urn:pulumi:sandbox::foundations::maif:subscription$azure:operationalinsights/analyticsWorkspace:AnalyticsWorkspace::log-f-stack-s-a-sb) success: inputs=#9 failures=#1
I1207 14:37:59.165530   71348 eventsink.go:86] eventSink::Error(<{%reset%}>azure:operationalinsights/analyticsWorkspace:AnalyticsWorkspace resource 'log-f-stack-s-a-sb' has a problem: expected daily_quota_gb to be at least (0.000000), got -1.000000<{%reset%}>)
How can I debug it more?
I also checkout the tag 
v2.28.0
of 
terraform-provider-azurerm
and no code mention 
daily_quota_gb
in my 
node_modules
folder, I grep for 
dailyQuotaGb
or `daily_quota_qb`and didn’t find anything neither
I set the value to 100000, so I’m not blocked anymore. I will open an issue on terraform BUT I don’t understand why I got the error after having downgrade to @pulumi/azure v3.28.0. Do you have any idea? How can I debug this?
b

broad-dog-22463

12/07/2020, 2:33 PM
Open an issue on pulumi-azure and we can look into it
it seems to be an issue on our side TBH
b

boundless-airport-99052

12/07/2020, 3:29 PM
great, I’ll do
thanks for your time
https://github.com/pulumi/pulumi-azure/issues/783
I also created this: https://github.com/pulumi/pulumi-azure/issues/784 (which is related)
3 Views
#azureJoin Slack
Powered by