Attention is currently required from: daniel, fixeria, laforge, osmith.

pespin has posted comments on this change by pespin. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41740?usp=email )

Change subject: gbproxy: Fix several tests on titan 11
......................................................................


Patch Set 1:

(6 comments)

File gbproxy/GBProxy_Tests.ttcn:

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41740/comment/ec1c1ca7_63fb51ae?usp=email
 :
PS1, Line 2722: [] pt.receive(BssgpStatusIndication:?) { repeat; }
> cosmetic: shift right
Done


https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41740/comment/8a3ac142_8e06c339?usp=email
 :
PS1, Line 2744: defaults
> It's only one now. […]
Done


https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41740/comment/ac4eddea_be449fb5?usp=email
 :
PS1, Line 2753: PESPIN
> debugging leftovers
Done


https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41740/comment/a22b5aed_36afbb6f?usp=email
 :
PS1, Line 2921: f_get_pcu_idx_ct
> BTW, you can use `__SCOPE__` to print an function/altstep name, i.e. 
> `log(__SCOPE__, "(): ...)`.
Done


https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41740/comment/b317acf0_6937bf44?usp=email
 :
PS1, Line 2954: defaults
> Likewise, don't you need to `deactivate()` it anymore?
Done


https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41740/comment/2d6a3a6f_97162de3?usp=email
 :
PS1, Line 2956:         /* Activate one altsteps for all PCU NSE */
              :         var default d := activate(as_count_bvc0_block(g_roi));
              :
              :         f_sleep(3.0);
              :
              :         d
> Does not look like we need to activate altsteps in background here? […]
Done



--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41740?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ib17c7dcb3d224a6b9e75ce5f9121a05126f81909
Gerrit-Change-Number: 41740
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <[email protected]>
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Attention: osmith <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Attention: daniel <[email protected]>
Gerrit-Comment-Date: Mon, 22 Dec 2025 09:40:57 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <[email protected]>

Reply via email to