-
Notifications
You must be signed in to change notification settings - Fork 75
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
[Bug]: Storage account always showing diff detected #683
Comments
@fherbert, thank you for raising this issue. The issue can be reproduced with the information provided, and the UpToDate condition does not come. 2024-03-26T17:39:09+03:00 DEBUG provider-azure Diff detected {"uid": "ccc1f4cc-3fa5-40b4-865f-b22c6816b3bf", "name": "labtefst", "gvk": "storage.azure.upbound.io/v1beta1, Kind=Account", "instanceDiff": "*terraform.InstanceDiff{mu:sync.Mutex{state:0, sema:0x0}, Attributes:map[string]*terraform.ResourceAttrDiff{\"queue_properties.#\":*terraform.ResourceAttrDiff{Old:\"\", New:\"\", NewComputed:true, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"share_properties.#\":*terraform.ResourceAttrDiff{Old:\"\", New:\"\", NewComputed:true, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}}, Destroy:false, DestroyDeposed:false, DestroyTainted:false, RawConfig:cty.NilVal, RawState:cty.NilVal, RawPlan:cty.NilVal, Meta:map[string]interface {}(nil)}"}
2024-03-26T17:39:09+03:00 DEBUG provider-azure Successfully requested update of external resource {"controller": "managed/storage.azure.upbound.io/v1beta1, kind=account", "request": {"name":"labtefst"}, "uid": "ccc1f4cc-3fa5-40b4-865f-b22c6816b3bf", "version": "23939", "external-name": "labtefst", "requeue-after": "2024-03-26T17:49:36+03:00"} |
I think I am encountering the same issue as the storage account is continuously being reconciled. |
@aurel33, thank you for your interest, are diff in the same fields? If the logs are recorded, can you share them? |
I used the following manifest (I removed some values but they do not seem important for this issue): apiVersion: storage.azure.upbound.io/v1beta2
kind: Account
metadata:
annotations:
crossplane.io/external-name: accounttest
name: account-test
spec:
deletionPolicy: Delete
forProvider:
accessTier: Hot
accountKind: FileStorage
accountReplicationType: LRS
accountTier: Premium
allowNestedItemsToBePublic: false
crossTenantReplicationEnabled: true
enableHttpsTrafficOnly: false
largeFileShareEnabled: true
location: northeurope
minTlsVersion: TLS1_2
networkRules:
bypass:
- AzureServices
defaultAction: Deny
ipRules:
- ...
- ...
virtualNetworkSubnetIds:
- ...
- ...
publicNetworkAccessEnabled: true
queueEncryptionKeyType: Service
resourceGroupName: myrg
shareProperties:
retentionPolicy:
days: 7
smb: {}
sharedAccessKeyEnabled: true
tableEncryptionKeyType: Service
initProvider: {} And in the logs I got the following diff:
So the detected diffs are on the fields Also I notice that in both cases it is the field |
This provider repo does not have enough maintainers to address every issue. Since there has been no activity in the last 90 days it is now marked as |
/fresh |
Is there an existing issue for this?
Affected Resource(s)
storage.azure.upbound.io/v1beta1 - Account
Resource MRs required to reproduce the bug
Steps to Reproduce
kubectl apply -f MR.yaml
What happened?
Diff detected is continuously being logged, crossplane applies changes, the (we presume) azure modifies it back, crossplane detecs diff, rinse and repeat.
Nothing in the provider pod debug logs indicates what is different.
Relevant Error Output Snippet
Crossplane Version
1.15.1
Provider Version
1.0.0
Kubernetes Version
1.27.9
Kubernetes Distribution
AKS
Additional Info
No response
The text was updated successfully, but these errors were encountered: