Skip to content

[Bug]: Multiple usage requests sent during helm install/upgrade #7360

@mattdesmarais

Description

@mattdesmarais

Version

edge

What Kubernetes platforms are you running on?

Other

Steps to reproduce

When installing or upgrading NIC (plus >= R33 ) with the helm chart nginx-ingress-0.0.0-edge, two consecutive usage reports are sent, one without the deployment_context followed by one with deployment_context to the management plane (by default NGINX One Cloud Console or controller.mgmt.usageReport.endpoint for an on-prem NGINX Instance Manager deployment)

Reproduction:

helm install nic1 oci://ghcr.io/nginxinc/charts/nginx-ingress --version 0.0.0-edge -n nginx-ingress-1  \  
--set controller.image.repository=private-registry.nginx.com/nginx-ic/nginx-plus-ingress \
--set controller.image.tag=edge \
--set controller.nginxplus=true\ 
--set controller.serviceAccount.imagePullSecretName=nginx-plus-registry-secret \
--set controller.mgmt.usageReport.endpoint=<endpoint> \
--set controller.mgmt.usageReport.sslVerify=false \
--set controller.config.entries.error-log-level=debug \
--set controller.replicaCount=1 \
--set controller.watchNamespace=nginx-ingress-1 \
--set controller.ingressClass.name=ingress-1

Debug Logs:

2025/02/11 17:47:20 [info] 18#18: usage report was sent
...
I20250211 17:47:21.915888   1 main.go:105] Event(v1.ObjectReference{Kind:"Pod", Namespace:"nginx-ingress", Name:"nic-nginx-ingress-controller-564dc9d6-gztq7", UID:"8910f55b-c060-4322-82bf-e69dfd757600", APIVersion:"v1", ResourceVersion:"3684680", FieldPath:""}): type: 'Normal' reason: 'SecretUpdated' the special Secret nginx-ingress/license-token was updated
I20250211 17:47:22.284486   1 main.go:105] Event(v1.ObjectReference{Kind:"Pod", Namespace:"nginx-ingress", Name:"nic-nginx-ingress-controller-564dc9d6-gztq7", UID:"8910f55b-c060-4322-82bf-e69dfd757600", APIVersion:"v1", ResourceVersion:"3684680", FieldPath:""}): type: 'Normal' reason: 'SecretUpdated' the special Secret nginx-ingress/license-token was updated
2025/02/11 17:47:22 [notice] 17#17: signal 1 (SIGHUP) received from 34, reconfiguring
2025/02/11 17:47:22 [notice] 17#17: reconfiguring
...
2025/02/11 17:47:22 [info] 35#35: usage report was sent

Suggested Fix:
NIC to embed a default /etc/nginx/reporting/tracking.info :

{
    "integration": "nic",
}
  • Ensure Dockerfile is updated with comments to give context into why there is a default tracking.info file

Metadata

Metadata

Labels

bugAn issue reporting a potential bug

Type

Projects

Status

Done 🚀

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions