Attention is currently required from: pespin.
fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27854 )

Change subject: pgw: Add TC_gy_charging_cc_time
......................................................................


Patch Set 1:

(2 comments)

File pgw/PGW_Tests.ttcn:

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27854/comment/59603b2f_c6b6c63d
PS1, Line 688: /* create a session, then execute a ping command on the user 
plane */
This test is not only about creating session and running ping, right? This 
comment needs to be clarified a bit.


https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27854/comment/49d6de41_69de6a8f
PS1, Line 715: control {
Is it intentional that you don't add TC_gy_charging_cc_time to the control 
section?



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

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ie4e567f62bbbdba276e6b3221111c8d9f82cde14
Gerrit-Change-Number: 27854
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-CC: fixeria <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Comment-Date: Thu, 21 Apr 2022 22:40:06 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to