Attention is currently required from: fixeria, laforge, pespin.
Timur Davydov has posted comments on this change by Timur Davydov. ( https://gerrit.osmocom.org/c/osmo-bts/+/42704?usp=email )
Change subject: trx: add JSON stats export for WebSDR API ......................................................................
Patch Set 8:
(1 comment)
Patchset:
PS8:
As mentioned in the osmo-trx similar commit, I doubt this has to be merged into the repository. […]
Similar to the osmo-trx case, this is only built when --with-websdr is enabled, so it does not affect the default/native builds.
In the WebSDR setup, VTY/CTRL/stats interfaces are not directly accessible from the web frontend. While it is technically possible to emulate VTY access via callbacks (e.g. trigger a VTY command, capture its textual output and forward it to the frontend for parsing), this would require modifying core VTY paths (e.g. vty_out) under conditional compilation.
That approach would be more invasive and harder to maintain than keeping this separate implementation. Also, in this case I only need a limited subset of information (mainly stats from osmo-bts-trx), so introducing a full VTY round-trip + parsing would be unnecessary overhead.
So the intention here is to keep the core code untouched and provide a minimal, isolated path for the WebSDR use case.
Of course, I’m open to alternative approaches if there is a cleaner way to achieve this.