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

zipkin-sample-rate of 0 still samples every request #2949

Closed
Stono opened this issue Aug 16, 2018 · 7 comments
Closed

zipkin-sample-rate of 0 still samples every request #2949

Stono opened this issue Aug 16, 2018 · 7 comments

Comments

@Stono
Copy link
Contributor

Stono commented Aug 16, 2018

I have zipkin-sample-rate: "0.0" in my configmap, which is setting the opentracing config correctly:

{
  "service_name": "ingress-nginx-external",
  "collector_host": "zipkin.istio-system.svc.cluster.local",
  "collector_port": 9411,
  "sample_rate": 0
}

However I still observe every single request get sampled:

❯ curl -kvL app.platform-testing.testing.k8.atcloud.io 2>/dev/null | grep -i sampled
    "x-b3-sampled": "1",
/Users/karl.stoney/git/autotrader/charts/helm-infra-calico git/master
❯ curl -kvL app.platform-testing.testing.k8.atcloud.io 2>/dev/null | grep -i sampled
    "x-b3-sampled": "1",
@Stono
Copy link
Contributor Author

Stono commented Aug 16, 2018

Nginx log:

sample.log

@Stono
Copy link
Contributor Author

Stono commented Aug 16, 2018

@aledbf whilst we're looking at metrics, any ideas about this one?

@aledbf
Copy link
Member

aledbf commented Aug 17, 2018

@Stono please test quay.io/aledbf/nginx-ingress-controller:0.412
(contains rnburn/zipkin-cpp-opentracing#27)

@Stono
Copy link
Contributor Author

Stono commented Aug 17, 2018

Fixed! Thanks @aledbf, however I now have another problem.... I can't "force" a sample:

❯ curl -kvL -H "x-b3-sampled: 1" -H "x-b3-flags: 1" app.platform-testing.testing.k8.atcloud.io 2>/dev/null | grep -i sampled
    "x-b3-sampled": "0",

Any ideas?

@aledbf
Copy link
Member

aledbf commented Aug 17, 2018

Any ideas?

No. Is this supported?
I just opened opentracing-contrib/nginx-opentracing#54

@Stono
Copy link
Contributor Author

Stono commented Aug 17, 2018

It is, sending x-b3-flags: 1 should force a sample

@aledbf
Copy link
Member

aledbf commented Aug 21, 2018

Closing. This is not an issue with the ingress controller itself but related to the nginx-opentracing module.

@aledbf aledbf closed this as completed Aug 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants