Change in osmo-pcu[master]: Move alpha,gamma fields from BTS to PCU

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

laforge gerrit-no-reply at lists.osmocom.org
Thu Jan 14 13:53:44 UTC 2021


laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/22181 )

Change subject: Move alpha,gamma fields from BTS to PCU
......................................................................


Patch Set 1:

> Since we don't have any of that, as of current code, the configs are global (per PCU). If at any point we want to have it per BTS, the stated above needs to be implemented.
> 
> So for now we have configs where they belong as per data structures, otherwise it's a complete mess.

I would argue that we should indeed have the struct/object members where they logically belong.  Whaever the VTY has supported so far or does support today should not prevent us from having the config settings where they belong from a logical point of view.

In the worst case, the current VTY code would have to iterate over all BTSs and sed the value N times.


-- 
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/22181
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I2fdd9c8a7393157183fff64084bb10e2a3b1dc63
Gerrit-Change-Number: 22181
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <laforge at osmocom.org>
Gerrit-Comment-Date: Thu, 14 Jan 2021 13:53:44 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210114/ff72ea50/attachment.htm>


More information about the gerrit-log mailing list