Commit Graph

214 Commits

Author SHA1 Message Date
Vadim Yanitskiy 9f47bedf37 build: include README.md into the release tarball
Change-Id: I1fcd14371dd2576f45c9c0798fd5c3628b631e13
2024-01-26 23:36:41 +07:00
Andreas Eversberg 13b41bc206 Use uniform log format for default config files
Related: OS#6272
Change-Id: I0754f45561a45557d0f9fb3218813a2c8e83559c
2023-12-01 12:48:59 +01:00
Pau Espin 88b5dc63a9 Bump version: 0.4.1.9-7f4d-dirty → 0.4.2
Change-Id: I26ebcdcad2e7913d6a76277917ce4210bff20e1f
2023-09-12 15:50:50 +02:00
Vadim Yanitskiy 7f4debf1b9 tests: $(BUILT_SOURCES) is not defined
Change-Id: I81dc30f6d6e3b25723f00d945c87b9c01c648cbf
2023-06-12 19:39:52 +07:00
Oliver Smith 2ec703f7c8 systemd: depend on networking-online.target
Related: SYS#6400
Change-Id: Ib6c78c76c5f13b9482428ce653a61b03b2aca1d3
2023-05-26 14:10:46 +02:00
Oliver Smith b8d16e4e44 debian: set compat level to 10
Related: OS#5958
Change-Id: I9776c48f4f913dd7f7cde3afc6fe32bd0e9ba1b0
2023-04-25 16:48:30 +02:00
Max 17da7b38b9 CI/CD: drop travis support
It hasn't been tried in years and has likely bit rot by now.

Change-Id: Ia3c725e94ecd544adb8899c05d66828387906625
2023-04-15 10:54:12 +00:00
Pau Espin 3092167e4e server: Call osmo_fd_unregister() before closing and changing bfd->fd
Change-Id: I44405687dae79a13db5617867c669ee4840485ba
2023-03-14 11:50:02 +01:00
Daniel Willmann fa47880575 cosmetic: Remove trailing whitespace
Change-Id: I33c8747f1f9c1020232e91d5559f0a06a31857d8
2023-02-28 11:39:05 +01:00
Daniel Willmann 4e934bb546 osmo_{client,server}_main: Remove is_config_node in vty_app_info
This member is deprcated/unused so simply remove it:

> 'is_config_node' is deprecated: Implicit parent node tracking has
> replaced the use of this callback. This callback is no longer called,
> ever, and can be left NULL.

Change-Id: I34295950557d9009b0d8d3378deb3b1263476f82
2023-02-28 11:39:05 +01:00
Daniel Willmann 50cc638ac7 osmo_{client,server}_main: Remove tall_ctr_ctx and tall_msgb_ctx
tall_ctr_ctx is only used by the obsolete osmo_counters which are not
used anymore.
tall_msgb_ctx is now set through msgb_tallox_ctx_init().

Fixes undefined reference errors since libosmocore.map was introduced in
I13169c00a59fb59513dfc598de5a71d094492422:

```
osmo-pcap/src/osmo_client_main.c:226: undefined reference to `tall_msgb_ctx'
/usr/bin/ld: osmo-pcap/src/osmo_client_main.c:226: undefined reference to `tall_msgb_ctx'
/usr/bin/ld: osmo-pcap/src/osmo_client_main.c:227: undefined reference to `tall_ctr_ctx'
/usr/bin/ld: osmo-pcap/src/osmo_client_main.c:227: undefined reference to `tall_ctr_ctx'
```

Change-Id: Idcbb96fd070f5ad20d3091ce886dc6130968538f
2023-02-28 11:07:59 +01:00
arehbein 9d7556a73e Transition to use of 'telnet_init_default'
Related: OS#5809
Change-Id: Icc57c68337d55c6594c1c36e9bf41624d11dab0a
2023-02-25 17:48:58 +01:00
Pau Espin 97c5c0c15c Bump version: 0.4.0.2-af54 → 0.4.1
Change-Id: I98918597c0e3823f14aacd9e84d8b9b10f68e71d
2023-02-07 17:11:46 +01:00
Vadim Yanitskiy af54579bd4 doc/manuals: update git URLs (git -> https; gitea)
Change-Id: I5d510f324f59303c79c3a0fb2c15f79e7cb3a8c4
2022-10-18 17:18:56 +07:00
Max 36061e0d37 Set working directory in systemd service file
By default systemd will execute service with root directory
(or home directory for user instance) which might result in
attempts to create files in unexpected place. Let's set it
to 'osmocom' subdir of state directory
(/var/lib for system instance) instead.

Related: OS#4821
Change-Id: I39c48167cbe74b599c8edc3659e787792cbcfdfd
2022-09-09 22:47:13 +07:00
Pau Espin bddebaecb2 Bump version: 0.3.0.5-5364-dirty → 0.4.0
Change-Id: Id9d5ef30ddbeb644cfbd98383a8764435576cbfe
2022-06-28 17:36:44 +02:00
Pau Espin 5364393417 .gitignore: blacklist configure~
Change-Id: Ic31044e997c1d2213a89e5c7c051814373c454bd
2022-06-28 17:36:02 +02:00
Harald Welte 57be1059bd update git URLs (git -> https; gitea)
Change-Id: I6e679bde473cebb98e9521b35dc3c5da3672e5a9
2022-06-18 12:29:01 +02:00
Pau Espin a55253f438 client: Add 'wqueue max-length <0-4294967295>' VTY command
This allows setting a suitable write-queue max length per client. The
desired value can be different based on a lot of variables, like memory
availabilty, network and CPU load, input/output link state, etc.

Related: SYS#5921
Change-Id: I4e9d5d836ddda215f9e7a075aa8e10d2d3854dd2
2022-04-14 18:32:23 +02:00
Pau Espin 50dcc15a2e client: Log wqueue capacity when failing to enqueue
Related: SYS#5921
Change-Id: Ie5f39fe679906a15f74490cd8554b18b9d1e0664
2022-04-14 18:30:33 +02:00
Pau Espin 6bbda89308 client: Increase wqueue transmit length
Having a length of 10 packets is too low, it can be filled easily under
high load or really bursty traffic, where the input path could be polled
multiple times while the output path (write socket poll) is not called.

Related: SYS#5921
Change-Id: I72babfcc31e12624f30c16450dafd988192148be
2022-04-14 17:57:14 +02:00
Pau Espin d72581492f Bump version: 0.2.1.4-b9be → 0.3.0
Change-Id: I7a4c45817dd3db7471b98b4db3b7bd2d4f3c240d
2022-01-13 10:06:14 +01:00
Pau Espin b9be6767ab server: Add vty command file-permission-mask
Related: SYS#5792
Change-Id: I78e0b56b38de438ee5fb679ae41c65b02ea2e722
2022-01-12 17:44:38 +01:00
Oliver Smith 235eba3e9b Revert "configure.ac: don't depend on libosmogb."
gprs header files from libosmogb are needed for osmo-pcap. This was the
reason why it was added as dependency to configure.ac in
I9a8fa03cef1efc9fdaea65ee63ca9b3379993989.

  $ git grep "osmocom/gprs"
  src/osmo_client_core.c:#include <osmocom/gprs/gprs_bssgp.h>
  src/osmo_client_core.c:#include <osmocom/gprs/protocol/gsm_08_16.h>
  src/osmo_client_core.c:#include <osmocom/gprs/protocol/gsm_08_18.h>

The package is only still building for debian, because in debian
libosmocore is not split up, and for centos, because the rpm spec file
still lists the dependency.

This reverts commit 2c2eadcadf.

Change-Id: Ic358956521230a2f6525533779c27417b2e86833
2021-12-13 12:39:13 +01:00
Harald Welte 2c2eadcadf configure.ac: don't depend on libosmogb.
There is nothing in osmo-pcap that relates to the Gb interface
(NS/BSSGP).

Change-Id: I079013b288c0585c32a3e3883a585460b4c3e616
2021-12-11 10:50:38 +01:00
Vadim Yanitskiy b9392bfde0 debian/control: minimum version for libzmq3-dev is 3.2.2
We require this version in 'contrib/osmo-pcap.spec.in', and should here too.

Change-Id: If76b4fcb8863ad5468a77113ad25142fe1145cda
2021-11-16 12:57:16 +00:00
Pau Espin 02d9d05143 Bump version: 0.2.0.4-30bc-dirty → 0.2.1
Change-Id: I345f288218a6459ffbadc5cae9feaf708c2188eb
2021-11-16 13:40:07 +01:00
Pau Espin 30bc1885e4 Explicitly depend on required libosmogb
the spec.in files already stated so expicitly, since some files include
osmocom/gprs/*.h. Only some data types from there are used, so there's
no need in linking the lib. Even more, doing so makes the build fail
because there soft-linking symbols required to be implemented by
libosmogb are not implemented here.

Related: OS#5311
Change-Id: I9a8fa03cef1efc9fdaea65ee63ca9b3379993989
2021-11-16 13:36:14 +01:00
Oliver Smith eedd83c113 Change default ports of client, server
Resolve conflict with other Osmocom software by moving ports:

osmo-pcap-client 4237 -> 4227
osmo-pcap-server 4238 -> 4228

Notably the defines OSMO_VTY_PORT_PCAP_CLIENT and _SERVER are currently
not provided by libosmocore, but will be added with the related commit.

Related: OS#5203
Related: libosmocore I098f211370bba61995ce424ab64029bdd4c49e2d
Change-Id: I3b18b1fbbca2799f868f28104e2f5d513f93d46c
2021-08-05 19:24:02 +02:00
Oliver Smith 22f3d95ac9 README.md: fix typo
Change-Id: I624e491dda95449b879f362ed98784adf604ba2b
2021-08-05 18:52:43 +02:00
Pau Espin ef92a539e8 Use new stat item/ctr getter APIs
Generated with spatch:
"""
@@
expression E1, E2;
@@
- &E2->ctr[E1]
+ rate_ctr_group_get_ctr(E2, E1)
"""

Change-Id: Ic860db04489e9ee7312edb008497186116f30bfc
2021-06-04 18:01:06 +02:00
Harald Welte 6e564a97b9 Bump version: 0.1.3 → 0.2.0
Change-Id: Ic36b541fa4000c352927e721397a081ccc709b73
2021-04-24 23:03:34 +02:00
Harald Welte 8b73a2a530 vty: call telnet_init_dynif() after config file is read
In case the config file specifies a vty bind address, we must read
it before we start the telnet server.

Change-Id: I44561754d4beaad5c74cb66994ca4ef38960ea78
2021-04-24 16:09:09 +02:00
Harald Welte dd1389c3d3 use telnet_init_dynif() to allow VTY bind to non-loopack address
Prior to this patch, you could configure 'bind 1.2.3.4' in the
config file, but the telnet interface still binds to 127.0.0.1.

Change-Id: I9b86b2cf6949917c55ea15277619cfa2b745185d
2021-04-24 13:00:41 +02:00
Harald Welte 9148d49841 client: Ensure the "file" header is sent on connect
A non-blocking STREAM socket connect() will mark the socket as
write-able once the connection succeeds.  However, as we first
call osmo_fd_setup() and then osmo_sock_init2_ofd(), the latter
will force the 'when' to OSMO_FD_READ and hence the write callback
will not be called.

Change-Id: I44c484b48966a985a9b85fb16122a17df5666bc1
2021-04-23 14:02:11 +02:00
Harald Welte 831494ed34 use osmo_wqueue_enqueue_quiet() as we log anyway
There's no need in both libosmocore and osmo-pcap-client logging
the same queue overflow twice.

Change-Id: I43881f2bd0b0870a51f9ca5dad2d8cba40d1e54b
2021-04-23 13:27:11 +02:00
Vadim Yanitskiy 69ce121c6c contrib/jenkins.sh: fix: pass '--enable-manuals' to configure
Currently `make publish` fails on Jenkins because there is nothing
to publish.  In [1] we simply forgot to pass '--enable-manuals'
if $WITH_MANUALS is set to 1, so no PDFs are built.  Fix this.

Change-Id: I85dfecf2025a0466fccfe5a1e63cda788f85992e
Fixes: [1] I28353f51de798535a3bb6efdc6c2da443d96ddfb
2021-04-18 21:14:21 +02:00
Harald Welte a663ed23b1 Add user manual for osmo-pcap
This adds one common user manual for both osmo-pcap-client
and osmo-pcap-server.

The manual is still basic in nature, but already contains useful
information regarding the setup of both clients and servers.

Change-Id: I66182fc55f1ee323eba45e7a7fc59db55bff520e
2021-04-18 15:25:01 +02:00
Harald Welte 8a119642ac update copyright statement; Holger worked on it until 2017
Change-Id: I7a39d4780d309f2dba99272c17bf1b9639760199
2021-04-18 14:26:07 +02:00
Harald Welte b0b2a2d542 add "--version" to osmo-pcap-client and -server
Change-Id: Ib82ee1a784a657f7274d78ba5f7a8afe6cfab395
2021-04-18 14:24:15 +02:00
Vadim Yanitskiy ff3f4b3bef vty_{client,server}_init(): remove unused argument
Change-Id: I65256c8602e91bcdbe5fe9e532b6ce7d347d3980
2021-04-16 20:30:18 +02:00
Vadim Yanitskiy d59783e601 vty: register commands for talloc context introspection
Change-Id: I3774cfd483102993340bd6e96f8512af61085679
2021-04-16 19:36:29 +02:00
Joachim Steiger 52f06fd54b manuals: generate VTY reference for osmo-pcap-{client,server}
Change-Id: I28353f51de798535a3bb6efdc6c2da443d96ddfb
Tweaked-By: Vadim Yanitskiy <vyanitskiy@sysmocom.de>
2021-04-16 19:29:31 +02:00
Harald Welte 611fd19fa3 vty: Add space after prompt, as customary
In all other programs, we have a space after the VTY prompt.

Change-Id: I7916c15cc34e41738a5716e4fefbb15e18f02fa7
2021-04-12 16:07:17 +02:00
Pau Espin Pedrol 06303a6dc5 Bump version: 0.1.2.9-288c-dirty → 0.1.3
Change-Id: I9287411a7c1f69ddc6211c2557d41dd533c84ff9
2021-02-23 13:19:37 +01:00
Oliver Smith 288c39be10 configure.ac: set -std=gnu11
Change-Id: I52c86f461edd96d810fb85a60c68be581caa1799
2021-01-27 17:52:22 +01:00
Harald Welte 72cc2a508d reformat debian/control for osmo-release.sh compatibility
If we have more than one dependency in the line, osmo-release.sh
fails with
ERROR: configure.ac <libosmocore, 0.11.0> does NOT match debian/control <Build-Depends:debhelper, 7.0.50~dh-autoreconfautotools-devlibpcap0.8-devpkg-configlibosmocore-devlibgnutls28-devlibzmq3-dev>!

Change-Id: I8ab9a24af6ac0e4610ecc0eca1d5f5b9e03ad445
2021-01-06 17:14:13 +01:00
Pau Espin 716a4db54c main: generate coredump and exit upon SIGABRT received
Previous code relied on abort() switching sigaction to SIG_FDL +
retriggering SIGABRT in case the signal handler returns, which would
then generate the coredump + terminate the process.
However, if a SIGABRT is received from somewhere else (kill -SIGABRT),
then the process would print the talloc report and continue running,
which is not desired.

Fixes: OS#4865
Change-Id: I39367aa480445fe961dcfa308789b3fc0cf759a1
2020-11-25 19:11:43 +01:00
Harald Welte 5899cb4fbf Use osmo_fd_*_{disable,enable}
Change-Id: I97e100b34e9d26cf5acaa57be12879787d4d2d2b
Depends: libosmocore.git Idb89ba7bc7c129a6304a76900d17f47daf54d17d
2020-10-19 13:07:06 +02:00
Harald Welte a9600141b8 Use osmo_fd_setup() whenever applicable
Change-Id: I02097760e7c6a5694290f4b877013aee9d92fd0e
2020-10-19 13:06:44 +02:00