Hmm I can't edit labels or reviewers but tagging @sysrex @Wondertan @liamsi
This is my first ADR so I would definitely appreciate feedback on how to improve this one & future ADRs.

Codecov Report

Merging #922 (842ae78) into main (2cc8077) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #922      +/-   ##
==========================================
- Coverage   58.53%   58.46%   -0.08%     
==========================================
  Files         132      132              
  Lines        7947     7947              
==========================================
- Hits         4652     4646       -6     
- Misses       2818     2822       +4     
- Partials      477      479       +2     

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Something for the node team to think about: what would opting in look like? A command-line flag, or an interactive prompt?

what would opting in look like? A command-line flag, or an interactive prompt?

With #907 this looks like a command-line flag: --metrics.endpoint. I think command-line flag is slightly preferable over interactive prompt because the latter is more cumbersome to write scripts against. Thoughts @Wondertan?

My main concern is that if it's some obscure flag that people explicitly have to add, very few people will know about it and help share metrics with us, even if they're ok with it.

Is there a way to explicitly ask people if they want to share metrics or not? Maybe when you run init there can be an interactive prompt. If people want to run a script there can be a --noninteractive option or something to skip the prompt?

@musalbas, @rootulp, I have no strong preferences between either script running over node binary and doing a prompt or the binary itself doing this on init. Both are the same amount of work, and choice can be given to the implementor. Although, if we want to keep this telemetry setup for long in the mainnet, then having it as a part of the binary might be more convenient. Similar to how we hardcode bootstrappers per network we run, we may also hardcode our telemetry endpoint to which data will be sent if users (1) agree to send it, (2)enabled telemetry(agreeing might also enable it by default), and (3) don't set a custom endpoint. Well, including this in binary seems like more work now, but I have no strong preferences.

I also don't have strong opinions here. I'm okay with the interactive prompt if a --noninteractive option exists. I created #1003 to continue the conversation because it seems like an implementation detail that doesn't strictly block this ADR.