-
Notifications
You must be signed in to change notification settings - Fork 895
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
ENV_VAR for where to send spans #172
Comments
Is the host similar to Resource API? Maybe dup with the #61 |
In OpenCensus it's described here: https://github.com/census-instrumentation/opencensus-specs/blob/master/resource/Resource.md#from-environment-variables |
No activity from the author, I will close this. Please re-open and answer the questions if you still need this feature. |
It's maybe a bit harsh to close this on Monday when the last question was on Friday. |
Sorry did not see when was asked, so only when the issue was posted. you are correct. |
Everything is in original post if anything is unclear please let me know. |
Can you please take a look if resource API is what you are looking for.
|
The agent host is where the agent runs to send spans to. |
Not sure tbh can you define the agent host with the resource API? |
@naseemkullah I got it. You mean this agent/collector: https://github.com/open-telemetry/opentelemetry-collector There is a quite similar issue that calls for standardizing of a configuration: #142 |
Yes exactly, the agent in particular.
Should we merge the issues then? something like Please see https://github.com/jaegertracing/jaeger-client-node#environment-variables for an example of what is desired ( |
I think this can be as simple as telling our Jaeger exporter to look for A similar thing can be done for each exporter. |
Maybe a generic From my understanding the collector can run as an agent/sidecar (so default value to As a side note I am a little confused by the difference between Jaeger's approach to distinguish |
Jaeger agent allows SDKs to have minimal requirements, e.g. no RPC framework is needed to submit the spans to the agent, and SDKs don't need to have manual configuration across all applications, such as the address of the collectors (because agents are always available on localhost). |
When using OTLP, it would be nice if there is a default across OTel client libraries that uses |
Actually there is a case where Jaeger agents are not available on localhost, in the case of kubernetes where the jaeger-agent is run as a daemonset (as opposed to sidecar), it must be set as follows in the app's pod spec:
|
I guess every exporter should have some standard env var for a host to look for when it is not localhost. In the case of the opentelemetry collector it should be |
We've added As for the otel collector, can we decide on an env var? |
This should be addressed via #572 |
Hello,
an env var for agent "host" would be nice
In JS/open-census for example, i need this in my code, but could there already be a default "TRACING_AGENT_HOST" ?...
... if there was a default env var I would set it ` and I could just have in code:
because in k8s, if using the AGENT is running as a daemonset we need to do something like this for the workload:
Thank you
The text was updated successfully, but these errors were encountered: