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

Turn on Telemetry #313

Open
ric-evans opened this issue Apr 15, 2022 · 0 comments
Open

Turn on Telemetry #313

ric-evans opened this issue Apr 15, 2022 · 0 comments

Comments

@ric-evans
Copy link
Member

ric-evans commented Apr 15, 2022

Turning on telemetry is straightforward. It's already an extra dependency for wipac-rest-tools (wipac-rest-tools[telemetry]). We won't want to enable telemetry by default so any clients (those that treat iceprod as a pip package) can decide for themselves, so let's add an extra dependency "telemetry":

[options.extras_require]
...
telemetry =
	wipac-rest-tools[telemetry]

For running iceprod as an application, we'll want to install iceprod as pip install iceprod[telemetry] (or pip install .[telemetry]) in the Dockerfile.

The telemetry environment variables will also need to be set to enable communication to the telemetry server. Without these set, telemetry will be silently bypassed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant