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

[Bug]: private repository's user and/or password not supplied to LinuxFunctionApp #838

Open
1 task done
dprts opened this issue Oct 8, 2024 · 0 comments
Open
1 task done
Labels
bug Something isn't working needs:triage

Comments

@dprts
Copy link

dprts commented Oct 8, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Affected Resource(s)

  • web.azure.upbound.io/v1beta1 - LinuxFunctionApp

Resource MRs required to reproduce the bug

No response

Steps to Reproduce

  • create more than one LinuxFunctionApp pulling image from a private repository and observe that user and/or password get removed even though they're properly defined
  • last version of the provider that it works is v1.2.0

What happened?

It appears to have started with v1.3.0 and is still present in v1.7.0

Relevant Error Output Snippet

2024-10-08T13:22:32Z	DEBUG	provider-azure	Diff detected	{"uid": "0957ce88-b38d-40b1-9dc3-3ca036f4e18b", "name": "po-acme-v2-qkg6z-kts67", "gvk": "web.azure.upbound.io/v1beta1, Kind=LinuxFunctionApp", "instanceDiff": "*terraform.InstanceDiff{mu:sync.Mutex{state:0, sema:0x0}, Attributes:map[string]*terraform.ResourceAttrDiff{\"app_settings.%\":*terraform.ResourceAttrDiff{Old:\"18\", New:\"19\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"app_settings.FUNCTIONS_EXTENSION_VERSION\":*terraform.ResourceAttrDiff{Old:\"\", New:\"~4\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"site_config.0.application_stack.0.docker.0.registry_password\":*terraform.ResourceAttrDiff{Old:\"***REDACTED***\", New:\"\", NewComputed:false, NewRemoved:true, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"site_config.0.application_stack.0.docker.0.registry_username\":*terraform.ResourceAttrDiff{Old:\"\", New:\"***REDACTED***\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:true, Type:0x0}}, Destroy:false, DestroyDeposed:false, DestroyTainted:false, RawConfig:cty.NilVal, RawState:cty.NilVal, RawPlan:cty.NilVal, Meta:map[string]interface {}(nil)}"}
2024-10-08T13:23:58Z	DEBUG	provider-azure	Diff detected	{"uid": "0957ce88-b38d-40b1-9dc3-3ca036f4e18b", "name": "po-acme-v2-qkg6z-kts67", "gvk": "web.azure.upbound.io/v1beta1, Kind=LinuxFunctionApp", "instanceDiff": "*terraform.InstanceDiff{mu:sync.Mutex{state:0, sema:0x0}, Attributes:map[string]*terraform.ResourceAttrDiff{\"app_settings.%\":*terraform.ResourceAttrDiff{Old:\"18\", New:\"19\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"app_settings.FUNCTIONS_EXTENSION_VERSION\":*terraform.ResourceAttrDiff{Old:\"\", New:\"~4\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"site_config.0.application_stack.0.docker.0.registry_password\":*terraform.ResourceAttrDiff{Old:\"\", New:\"REDACTED\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:true, Type:0x0}, \"site_config.0.application_stack.0.docker.0.registry_username\":*terraform.ResourceAttrDiff{Old:\"\", New:\"***REDACTED***\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:true, Type:0x0}}, Destroy:false, DestroyDeposed:false, DestroyTainted:false, RawConfig:cty.NilVal, RawState:cty.NilVal, RawPlan:cty.NilVal, Meta:map[string]interface {}(nil)}"}
2024-10-08T13:29:11Z	DEBUG	provider-azure	Diff detected	{"uid": "0957ce88-b38d-40b1-9dc3-3ca036f4e18b", "name": "po-acme-v2-qkg6z-kts67", "gvk": "web.azure.upbound.io/v1beta1, Kind=LinuxFunctionApp", "instanceDiff": "*terraform.InstanceDiff{mu:sync.Mutex{state:0, sema:0x0}, Attributes:map[string]*terraform.ResourceAttrDiff{\"app_settings.%\":*terraform.ResourceAttrDiff{Old:\"18\", New:\"19\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"app_settings.FUNCTIONS_EXTENSION_VERSION\":*terraform.ResourceAttrDiff{Old:\"\", New:\"~4\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"site_config.0.application_stack.0.docker.0.registry_password\":*terraform.ResourceAttrDiff{Old:\"\", New:\"***REDACTED***\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:true, Type:0x0}, \"site_config.0.application_stack.0.docker.0.registry_username\":*terraform.ResourceAttrDiff{Old:\"\", New:\"***REDACTED***\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:true, Type:0x0}}, Destroy:false, DestroyDeposed:false, DestroyTainted:false, RawConfig:cty.NilVal, RawState:cty.NilVal, RawPlan:cty.NilVal, Meta:map[string]interface {}(nil)}"}
2024-10-08T13:52:28Z	DEBUG	provider-azure	Diff detected	{"uid": "0957ce88-b38d-40b1-9dc3-3ca036f4e18b", "name": "po-acme-v2-qkg6z-kts67", "gvk": "web.azure.upbound.io/v1beta1, Kind=LinuxFunctionApp", "instanceDiff": "*terraform.InstanceDiff{mu:sync.Mutex{state:0, sema:0x0}, Attributes:map[string]*terraform.ResourceAttrDiff{\"app_settings.%\":*terraform.ResourceAttrDiff{Old:\"18\", New:\"19\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"app_settings.FUNCTIONS_EXTENSION_VERSION\":*terraform.ResourceAttrDiff{Old:\"\", New:\"~4\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"site_config.0.application_stack.0.docker.0.registry_password\":*terraform.ResourceAttrDiff{Old:\"***REDACTED***\", New:\"\", NewComputed:false, NewRemoved:true, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"site_config.0.application_stack.0.docker.0.registry_username\":*terraform.ResourceAttrDiff{Old:\"\", New:\"***REDACTED***\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:true, Type:0x0}}, Destroy:false, DestroyDeposed:false, DestroyTainted:false, RawConfig:cty.NilVal, RawState:cty.NilVal, RawPlan:cty.NilVal, Meta:map[string]interface {}(nil)}"}
2024-10-08T13:54:36Z	DEBUG	provider-azure	Diff detected	{"uid": "0957ce88-b38d-40b1-9dc3-3ca036f4e18b", "name": "po-acme-v2-qkg6z-kts67", "gvk": "web.azure.upbound.io/v1beta1, Kind=LinuxFunctionApp", "instanceDiff": "*terraform.InstanceDiff{mu:sync.Mutex{state:0, sema:0x0}, Attributes:map[string]*terraform.ResourceAttrDiff{\"app_settings.%\":*terraform.ResourceAttrDiff{Old:\"18\", New:\"19\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"app_settings.FUNCTIONS_EXTENSION_VERSION\":*terraform.ResourceAttrDiff{Old:\"\", New:\"~4\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"site_config.0.application_insights_connection_string\":*terraform.ResourceAttrDiff{Old:\"InstrumentationKey=86bbcce7-7db9-4b22-93b8-2b1cab082b1f;IngestionEndpoint=https://westeurope-5.in.applicationinsights.azure.com/;LiveEndpoint=https://westeurope.livediagnostics.monitor.azure.com/;ApplicationId=7c2075b4-b4d0-4c7b-bb0b-79ee132d0d95\", New:\"\", NewComputed:false, NewRemoved:true, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"site_config.0.application_stack.0.docker.0.registry_password\":*terraform.ResourceAttrDiff{Old:\"***REDACTED***\", New:\"\", NewComputed:false, NewRemoved:true, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"site_config.0.application_stack.0.docker.0.registry_username\":*terraform.ResourceAttrDiff{Old:\"***REDACTED***\", New:\"\", NewComputed:false, NewRemoved:true, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}, \"storage_account_access_key\":*terraform.ResourceAttrDiff{Old:\"TddPr+1OxvTdK/Kqry4u9Q2WFwN5mGcpwCaIH78d8xm6qExkw1XpOq9Ule6viEU/z323hqHHfTA3+AStaN23FQ==\", New:\"\", NewComputed:false, NewRemoved:true, 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)}"}

Crossplane Version

1.17.0

Provider Version

v1.3.0 ... v1.7.0

Kubernetes Version

No response

Kubernetes Distribution

No response

Additional Info

No response

@dprts dprts added bug Something isn't working needs:triage labels Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs:triage
Projects
None yet
Development

No branches or pull requests

1 participant