You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When running mix prom_ex.gen.config --datasource "Local Prometheus", the generated docs says the following in step 2:
Add this module to your application supervision tree. It should be one of the first
things that is started so that no Telemetry events are missed. For example, if PromEx
is started after your Repo module, you will miss Ecto's init events and the dashboards
will be missing some data points
However, the README.md for the project mentions the following:
PromEx should be started after the Endpoint, to avoid unnecessary error messages
The supervised children for the application look like the following:
children=[# Start the PromExZeroPhoenix.PromEx,# Start the Telemetry supervisorZeroPhoenixWeb.Telemetry,# Start the Ecto repositoryZeroPhoenix.Repo,# Start the PubSub system{Phoenix.PubSub,name: ZeroPhoenix.PubSub},# Start Finch{Finch,name: ZeroPhoenix.Finch},# Start the Endpoint (http/https)ZeroPhoenixWeb.Endpoint]
I came here seeking the same clarification to the documentation inconsistency raised by @conradwt in this unaddressed issue.
In my experience I'm inclined to start the prom_ex supervision tree before all other applications for the exact reason mentioned in the README and quoted by OP.
If any project authors have the time to chime in here it would be greatly appreciated. ❤️
Describe the bug
When running
mix prom_ex.gen.config --datasource "Local Prometheus"
, the generated docs says the following in step 2:However, the
README.md
for the project mentions the following:The supervised children for the application look like the following:
To Reproduce
Steps to reproduce the behavior:
prom_ex.ex
and project README.mdExpected behavior
I would expect the generated code docs (i.e. prom_ex.ex) and the
README.md
would be consistent here.Environment
Additional context
Just looking for better clarification within the documentation as to the proper location for the PromEx within the supervision tree.
The text was updated successfully, but these errors were encountered: