Change in osmo-gsm-tester[master]: srs-ms: allows to run given scripts before the execution of the UE.

alealcon gerrit-no-reply at
Fri Jun 11 06:52:11 UTC 2021

alealcon has posted comments on this change. ( )

Change subject: srs-ms: allows to run given scripts before the execution of the UE.

Patch Set 2:

For me, this case is the same as the srs-enb pre-init scripts.
I agree with you that the best solution is to add these new features to OGT the way you propose (creating specific classes that add this support/features) and that's what we should do in the future.
Right now we're under heavy development, with a lot of changes and testing, so the idea of the pre/post init script is very good to us, because it allows to run utilities/scripts that will help us to debug the system. I see two big advantages with this: we can do it faster with a script that adding this new feature to OGT and some of this utilities may be temporal and don't make sense to add them to OGT(for example the dmesg before and after the launch of the srs-enb).

My proposal is to add the pre/post run scripts feature to the srs-ms and srs-enb. Then we can list and discuss the feature that each script adds and decide whether to add it to OGT or not (sometimes may be not needed because it's temporal, sometimes won't be worth it, because it's too unstable to add it and most of the times we will add them)
Does it sound good to you?

To view, visit
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I718e686b5844f2f07eda820914678052f1008182
Gerrit-Change-Number: 24277
Gerrit-PatchSet: 2
Gerrit-Owner: alealcon <alejandro.leal at>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin at>
Gerrit-CC: srs_andre <andre at>
Gerrit-Comment-Date: Fri, 11 Jun 2021 06:52:11 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the gerrit-log mailing list