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

integer values should be passed as is #302

Closed
jkandasa opened this issue Mar 13, 2019 · 1 comment
Closed

integer values should be passed as is #302

jkandasa opened this issue Mar 13, 2019 · 1 comment
Labels
bug Something isn't working

Comments

@jkandasa
Copy link
Member

when we specify options(collector, query, agent) in CR files when creating actual service, options(integer values) are passed in scientific notation format, specific jaeger service throws an exception,

CR file:

apiVersion: jaegertracing.io/v1
kind: Jaeger
metadata:
  name: jaegerqe
spec:
  ingress:
    security: none
  strategy: production
  collector:
    replicas: 1
    image: jaegertracing/jaeger-collector:latest
    options:
      metrics-backend: expvar
      collector:
        num-workers: 50
        queue-size: 2000
      es:
        bulk:
          size: 5000000
          workers: 1
          flush-interval: 200ms
  query:
    replicas: 1
    image: jaegertracing/jaeger-query:latest
    options:
      metrics-backend: expvar
      query:
        port: 16686
  agent:
    strategy: sidecar
    image: jaegertracing/jaeger-agent:latest
    options:
      metrics-backend: expvar
  storage:
    type: elasticsearch
    esIndexCleaner:
      enabled: false
    sparkDependencies:
      enabled: false
    options:
      es:
        server-urls: http://elasticsearch:9200

Exception in collector:

Error: invalid argument "5e+06" for "--es.bulk.size" flag: strconv.ParseInt: parsing "5e+06": invalid syntax
invalid argument "5e+06" for "--es.bulk.size" flag: strconv.ParseInt: parsing "5e+06": invalid syntax
@pavolloffay pavolloffay added the bug Something isn't working label Mar 13, 2019
@pavolloffay
Copy link
Member

@jkandasa as a workaround you can define values as strings "es.bulk.size": "10000000".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants