Change in ...osmo-pcu[master]: Introduce osmo_tdef infra and timer VTY commands

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
Fri Sep 6 19:03:41 UTC 2019


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

Change subject: Introduce osmo_tdef infra and timer VTY commands
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.osmocom.org/#/c/15425/2//COMMIT_MSG 
Commit Message:

https://gerrit.osmocom.org/#/c/15425/2//COMMIT_MSG@9 
PS2, Line 9: even
           : overwriting whatever came from BTS
I actually consider that rather problematic.  There should be one point where a given timer is configured, and if that is in the BSC or the BTS (and then passed to the PCU), the PCU should just execute that and not allow for any override to it.  This is confusing and can easily lead to inconsistencies.



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

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I5cfb9ef01706124be262d4536617b9edb4601dd5
Gerrit-Change-Number: 15425
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <axilirator at gmail.com>
Gerrit-Reviewer: lynxis lazus <lynxis at fe80.eu>
Gerrit-Reviewer: osmith <osmith at sysmocom.de>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-CC: laforge <laforge at gnumonks.org>
Gerrit-Comment-Date: Fri, 06 Sep 2019 19:03:41 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190906/7f335071/attachment.htm>


More information about the gerrit-log mailing list