Add Prometheus environment variable

[axiomofjoy]

• add environment variable
• add deprecation warning for command line argument
• create ticket to remove command line argument in an upcoming sprint
• change the CMD in the Dockerfile
• change the CMD in the Arize deployment of Phoenix
• do a full-text search for --enable-prometheus to make sure we go everything

Originally posted by @mikeldking in #2708 (comment)

[axiomofjoy]

@mikeldking Can you clarify the details of this ticket? Do you want to remove the --enable-prometheus command line argument and have an environment variable instead?

[mikeldking]

@mikeldking Can you clarify the details of this ticket? Do you want to remove the --enable-prometheus command line argument and have an environment variable instead?

I think this can be an opt-in feature via env-vars. Not every person that deploys phoenix needs prometheus metrics. To make the integration seamless we should just control it via environment variables rather than a launch command

[axiomofjoy]

If I understand correctly, the user should be able to decide whether to run the container with Prometheus at runtime, e.g.,

docker run -it -e "ENABLE_PROMETHEUS=TRUE" arizephoenix/phoenix:latest

[mikeldking]

If I understand correctly, the user should be able to decide whether to run the container with Prometheus at runtime, e.g.,

docker run -it -e "ENABLE_PROMETHEUS=TRUE" arizephoenix/phoenix:latest

Yes, it should be off by default

[mikeldking]

Bumping back in priority.

[axiomofjoy]

Fully remove --enable-prometheus command line argument.