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 · 1 comment
Open
1 task done
Labels
bug Something isn't working needs:triage stale

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
Copy link

github-actions bot commented Jan 7, 2025

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 stale. It will be closed in 14 days if no further activity occurs. Leaving a comment starting with /fresh will mark this issue as not stale.

@github-actions github-actions bot added the stale label Jan 7, 2025
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 stale
Projects
None yet
Development

No branches or pull requests

1 participant