Project

General

Profile

Feature #2257

prepare split of the openbsc.git in smaller components

Added by neels 5 months ago. Updated 15 days ago.

Status:
Feedback
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
05/15/2017
Due date:
% Done:

100%

Resolution:
Spec Reference:

Description

On the OsmoDevCon 2017, we decided to split openbsc.git in smaller git repositories that match the (core) network component names.
Some code separation still needs to be done to be able to do so.
See the mail threads:

Make sure that in the final result, no #if BEFORE_MSCSPLIT remain (unless for very good reasons).

Abis/BSC related utilities like abisip-find should go with the BSC.


Related issues

Related to Cellular Infrastructure - Feature #2296: jenkins: build jobs for repositories split from openbsc.git (OsmoMSC, OsmoBSC, ...) Closed 05/27/2017
Related to Cellular Infrastructure - Feature #2299: osmo-gsm-manuals: reflect split of separate repositories from openbsc.git (OsmoMSC, OsmoBSC, ...) New 05/27/2017
Related to Cellular Infrastructure - Bug #2298: opkg packaging: package repositories split from openbsc.git (OsmoMSC, OsmoBSC, ...) New 05/27/2017
Related to Cellular Infrastructure - Feature #2297: debian packaging: package repositories split from openbsc.git (OsmoMSC, OsmoBSC, ...) Closed 05/27/2017
Related to OsmoMSC - Feature #2260: "next generation" osmo-bsc_mgcp New 05/16/2017
Related to OsmoBSC - Feature #2389: re-add siemens MRPCI message, dropped during MSCSPLIT New 07/24/2017
Related to OsmoBTS - Feature #2508: Cleanup BSC header use New 09/08/2017
Duplicated by OpenBSC - Feature #2181: Separate BSC Code from MSC code into separate repository Closed 04/25/2017
Duplicated by OpenBSC - Feature #2180: Split GPRS (SGSN/gb_proxy/gtp_hub) into separate repository Closed 04/25/2017
Blocked by OpenBSC - Feature #2017: Cleanup deprecated, make breaking changes for BSC/MSC split New 04/20/2017

History

#1 Updated by neels 5 months ago

  • Related to Feature #2296: jenkins: build jobs for repositories split from openbsc.git (OsmoMSC, OsmoBSC, ...) added

#2 Updated by neels 5 months ago

  • Related to Feature #2299: osmo-gsm-manuals: reflect split of separate repositories from openbsc.git (OsmoMSC, OsmoBSC, ...) added

#3 Updated by neels 5 months ago

  • Related to Bug #2298: opkg packaging: package repositories split from openbsc.git (OsmoMSC, OsmoBSC, ...) added

#4 Updated by neels 5 months ago

  • Related to Feature #2297: debian packaging: package repositories split from openbsc.git (OsmoMSC, OsmoBSC, ...) added

#5 Updated by laforge 3 months ago

  • Assignee changed from Osmocom Developers to neels

#7 Updated by neels 3 months ago

  • Duplicated by Feature #2181: Separate BSC Code from MSC code into separate repository added

#8 Updated by neels 3 months ago

So far I assumed that the osmo-bsc_nat would go in the osmo-bsc repository. The name suggests it. Reading #2181 made me not so sure now.

#9 Updated by neels 3 months ago

  • Duplicated by Feature #2180: Split GPRS (SGSN/gb_proxy/gtp_hub) into separate repository added

#10 Updated by neels 3 months ago

  • Blocked by Feature #2017: Cleanup deprecated, make breaking changes for BSC/MSC split added

#11 Updated by neels 3 months ago

  • Related to Feature #2260: "next generation" osmo-bsc_mgcp added

#12 Updated by neels 3 months ago

  • % Done changed from 30 to 40

A first test run with the separately built OsmoMSC, OsmoBSC, OsmoSGSN and osmo-bsc_mgcp worked.
(It took a bit longer because I was using an old Nokia 1100 for the first time, which provoked two issues unrelated to the code split: an erratic loop exit condition on codec selection present on openbsc.git aoip branch, codec mismatch situations because the nokia does not support HR3, peculiar SMS ACK behavior mismatch from the Nokia.)

Next up is to apply code review from gerrit on osmo-msc.git, rebase the separate file removals onto that and test again.
Each separate repository most probably still contains some unrelated cruft here and there which can be cleaned out.
The osmo-mgw's binary is still called osmo-bsc_mgcp, rename still todo.

#13 Updated by neels 3 months ago

  • Description updated (diff)

#14 Updated by neels 3 months ago

  • Related to Feature #2389: re-add siemens MRPCI message, dropped during MSCSPLIT added

#15 Updated by neels 3 months ago

  • % Done changed from 40 to 60

A patch series is now pending on gerrit, of

  • first up, to have highest scrutiny on the patches being reviewed:
    • 'make distcheck' fixes to be able to run the py tests in distcheck,
    • jenkins.sh adjustments to properly invoke 'make distcheck' with various configurations
  • libvlr addition with extensive MSC+VLR end-to-end test suite,
  • mscsplit (separating libmsc from libbsc)
  • IuCS addition

Next up is squashing the move to new SIGTRAN and addition of AoIP into intelligible patches (rather few large ones instead of numerous back-and-forth errors and fixes) and submitting that to gerrit.

#16 Updated by neels 3 months ago

  • Description updated (diff)
  • Target version set to NITB Split

#17 Updated by neels about 2 months ago

  • % Done changed from 60 to 90

openbsc.git up to and including 84b7e3723b8d7bf9bfd5db7ba5532c3d174c2c77 is now merged to osmo-msc.git and osmo-{bsc,mgw,sgsn}.git.

All patches on gerrit are verified, waiting for CR and merge, at which point we should be able to mark the move to the new repositories.

#18 Updated by neels 28 days ago

  • % Done changed from 90 to 100

The new repositories are in place and also actively used on the osmo-gsm-tester.
(For some reason I am not able to mark this issue resolved.)

#19 Updated by msuraev 18 days ago

#20 Updated by neels 15 days ago

  • Status changed from In Progress to Feedback
  • Assignee changed from neels to laforge

I can't resolve the issue, is it because of the Target version? The Resolution field?

#21 Updated by neels 15 days ago

nm, it's of course because of the subtasks.

#22 Updated by neels 15 days ago

I'm not paying attention... they are related issues, not subtasks. sorry for the noise.

Also available in: Atom PDF