Attention is currently required from: laforge.

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

Change subject: ggsn: add test TC_pdp4_clients_interact_with_ext_hdr
......................................................................


Patch Set 3:

(1 comment)

Patchset:

PS3:
actually I'm not sure if it makes sense to merge this:
* The test passes with open5gs, but fails with osmo-ggsn currently (with both 
kernel gtp-u and with userspace gtp-u implementation), as mentioned in 
https://osmocom.org/issues/6223#note-9
* I'm not sure if we want to extend osmo-ggsn so it accepts packages with such 
an extension header, or if the desired behavior is that they should get 
rejected and we should adjust the test to expect that.

So waiting for feedback here first:
* https://gerrit.osmocom.org/c/osmo-ggsn/+/38383
* https://osmocom.org/issues/6223



--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38382?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: Ic767f8b50f034171d71cae16512d1d26a07b24bf
Gerrit-Change-Number: 38382
Gerrit-PatchSet: 3
Gerrit-Owner: osmith <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-CC: lynxis lazus <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Comment-Date: Wed, 16 Oct 2024 09:23:07 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No

Reply via email to