Attention is currently required from: laforge, fixeria, pespin.
1 comment:
Patchset:
i settled for dynamic/osmocom and dynamic/ipaccess now.
One thing to consider for this patch, do we really want to change the pchan names on items returned on the CTRL interface? If not, we need to keep the old names as default, meaning also that a VTY write will replace 'dynamic/osmocom' with 'TCH/F_TCH/H_SDCCH8_PDCH'.
I did keep the stat/rate_ctr naming unchanged, to save users having to adjust their stat evaluation to match the naming tweak... there it is still 'OSMO_DYN' and "TCH/F_PDCH' ... so it's completely inconsistent now but backwards compatible. Maybe that's half-arsed and we should rather change to consistent naming everywhere?
An argument for changing all naming everywhere is that we already did change the naming when adding SDCCH8 to TCH/F_TCH/H_PDCH, where VTY and CTRL changed to "TCH/F_TCH/H_SDCCH8_PDCH" and stats/rate_ctr changed to "OSMO_DYN", and no-one complained AFAIK.
any opinions?
To view, visit change 31615. To unsubscribe, or for help writing mail filters, visit settings.