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 <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Wed, 16 Oct 2024 09:23:07 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No