Project

General

Profile

Actions

Feature #2534

closed

move gsup_client and oap_client code out of osmo-{msc,sgsn}.git

Added by neels over 6 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
-
Start date:
10/04/2017
Due date:
% Done:

100%

Spec Reference:

Description

osmo-msc.git and osmo-sgsn.git both use GSUP.
The GSUP protocol definition is in libosmocore already, but both osmo-sgsn and osmo-msc duplicate the gsup_client code.
The gsup_client also depends on oap_client. OAP is currently not used by us but we should keep it active for future use.

gsup_client uses libosmo-abis/ipa as IPA Multiplex implementation.
Hence gsup_client as well as oap_client cannot move to libosmocore, but need to move "at least" to libosmo-abis.
(libosmo-abis/ipa uses libosmocore/gsm/ipa, but the server link code is in libosmo-abis.)

Semantically, libosmo-abis is not the right place for GSUP and OAP: they are not at all related to the Abis interface.

Clarify: what should move where? Then move it to drop the gsup_client code dup between osmo-msc and osmo-sgsn.


Related issues

Related to OsmoMSC - Feature #2528: remove dead code after split from openbsc.gitResolvedneels10/04/2017

Actions
Related to OsmoSGSN - Feature #2530: remove dead code after split from openbsc.gitRejectedneels10/04/2017

Actions
Actions #1

Updated by neels over 6 years ago

  • Subject changed from move gsup_client and oap_client to libosmocore to move gsup_client and oap_client code out of osmo-{msc,sgsn}.git
Actions #2

Updated by neels over 6 years ago

  • Project changed from libosmocore to Cellular Network Infrastructure
Actions #3

Updated by neels over 6 years ago

  • Related to Feature #2528: remove dead code after split from openbsc.git added
Actions #4

Updated by neels over 6 years ago

  • Related to Feature #2530: remove dead code after split from openbsc.git added
Actions #5

Updated by neels over 5 years ago

  • Status changed from New to Feedback
  • Assignee set to laforge

osmo-hlr.git recently got libosmo-gsup-client -- if a dep of {msc,sgsn} -> osmo-hlr is acceptable, this code could move in there?

Actions #6

Updated by laforge over 5 years ago

neels wrote:

osmo-hlr.git recently got libosmo-gsup-client -- if a dep of {msc,sgsn} -> osmo-hlr is acceptable, this code could move in there?

yes, that's the point and in fact https://gerrit.osmocom.org/#/c/osmo-sgsn/+/10284/ is a patch for that, but it somehow fails for not-yet-resolved reasons.

Actions #7

Updated by laforge over 5 years ago

  • Status changed from Feedback to New
Actions #8

Updated by laforge over 5 years ago

  • Assignee changed from laforge to msuraev
Actions #9

Updated by laforge over 5 years ago

  • % Done changed from 0 to 50

at least for osmo-sgsn, libosmo-gsup-client has just been removed by change-id I6f542945403cf2e3ddac419186b09ec0e2d43b69.

In osmo-msc, it has already been removed as part of change-id I6f542945403cf2e3ddac419186b09ec0e2d43b69 back in July.

This leaves the ticket remaining about oap_client.

Actions #10

Updated by msuraev over 5 years ago

  • Status changed from New to In Progress

There's no oap implementation in OsmoSGSN. Related cleanup patch is available in https://gerrit.osmocom.org/#/c/osmo-sgsn/+/11566/

Actions #11

Updated by msuraev over 5 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 50 to 100

OAP client was removed from OsmoMSC some time ago already.

Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)