Attention is currently required from: neels.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-hnbgw/+/32715
to look at the new patch set (#4).
Change subject: vty test: show missing write-back of 'rnc-id'
......................................................................
vty test: show missing write-back of 'rnc-id'
Change-Id: I9eecda8678c32abbeaf39344ccdb14ed5a0828ce
---
M tests/osmo-hnbgw.vty
1 file changed, 42 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-hnbgw refs/changes/15/32715/4
--
To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/32715
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: I9eecda8678c32abbeaf39344ccdb14ed5a0828ce
Gerrit-Change-Number: 32715
Gerrit-PatchSet: 4
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-MessageType: newpatchset
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/32339 )
Change subject: Fix DL_TBF PACCH ass done on UL_TBF already scheduled to tx last PKT CTRL ACK
......................................................................
Patch Set 2:
(1 comment)
This change is ready for review.
Patchset:
PS2:
Manual testing also looks good. I plan to get this merged as soon as we fix bug in osmocom-bb modem app appearing once this patch is in place.
--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/32339
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I62ae91b494e4fd0ade3f4a3ba3817bcaedbdebf5
Gerrit-Change-Number: 32339
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Comment-Date: Tue, 16 May 2023 10:49:10 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: osmith, daniel.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-cbc/+/32723 )
Change subject: sbcap: Update asn1c skeleton files
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
> Yeah having that in the README in a follow-up patch would be very useful, thanks!
done
--
To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/32723
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-cbc
Gerrit-Branch: master
Gerrit-Change-Id: I385d14d4be308b991a48ce1033087f07bd1f47ea
Gerrit-Change-Number: 32723
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 16 May 2023 10:41:09 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: osmith <osmith(a)sysmocom.de>
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: pespin.
osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-cbc/+/32730 )
Change subject: README.md: Document generation of SBcAP code from ASN.1 files
......................................................................
Patch Set 2:
(2 comments)
File README.md:
https://gerrit.osmocom.org/c/osmo-cbc/+/32730/comment/adeb132e_295e86cd
PS1, Line 75: Upstream master as1nc from [vlm](https://github.com/vlm/asn1c) [doesn't support APER encoding](https://github.com/vlm/asn1c/issues/452). Nevertheless, the upstream fork maintained by a big contributor [mouse07410](https://github.com/mouse07410/asn1c) does support it, and it is used in osmo-cbc to generate the SBc-AP code from ASN.1 files present in src/sbcap/asn1/.
> break lines at a similar length as above for consistency (maybe ~80)?
Done
https://gerrit.osmocom.org/c/osmo-cbc/+/32730/comment/39b95fef_904c88f6
PS1, Line 77: regente
> regenerate
Done
--
To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/32730
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-cbc
Gerrit-Branch: master
Gerrit-Change-Id: Id6edb2dcbd39ca101110f1ca57fcd67bf08c0ef0
Gerrit-Change-Number: 32730
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 16 May 2023 10:36:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: osmith <osmith(a)sysmocom.de>
Gerrit-MessageType: comment