Neels Hofmeyr has submitted this change and it was merged.
Change subject: debian: rename libosmo-mgcp-client0 -> libosmo-mgcp-client1
..
debian: rename libosmo-mgcp-client0 -> libosmo-mgcp-client1
The numeral is describing the
Neels Hofmeyr has submitted this change and it was merged.
Change subject: debian: .install: split mgcp-client out of legacy-mgcp
..
debian: .install: split mgcp-client out of legacy-mgcp
Change-Id: I681e9a1299607ff3a30de34c043
Neels Hofmeyr has submitted this change and it was merged.
Change subject: debian: use new dbgsym package instead of old dbg packages.
..
debian: use new dbgsym package instead of old dbg packages.
The new dbgsym packages gets
Neels Hofmeyr has submitted this change and it was merged.
Change subject: debian/rules: compile with debug symbols
..
debian/rules: compile with debug symbols
The debug symbol will be stripped out of the "normal" package.
Cha
Neels Hofmeyr has submitted this change and it was merged.
Change subject: debian: fix paths of examples
..
debian: fix paths of examples
Change-Id: I054e500be9527b1dd0fc0c21cde2b7b2d2fdfbff
---
M debian/osmo-gbproxy.install
M
Patch Set 1: Verified+1
--
To view, visit https://gerrit.osmocom.org/3947
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I054e500be9527b1dd0fc0c21cde2b7b2d2fdfbff
Gerrit-PatchSet: 1
Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-O
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/3947
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I054e500be9527b1dd0fc0c21cde2b7b2d2fdfbff
Gerrit-PatchSet: 1
Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerri
Neels Hofmeyr has submitted this change and it was merged.
Change subject: debian: fix dependency to mgcp library
..
debian: fix dependency to mgcp library
osmo-mgw has split the MGCP client to a separate library in
change-id I
Review at https://gerrit.osmocom.org/3947
debian: fix paths of examples
Change-Id: I054e500be9527b1dd0fc0c21cde2b7b2d2fdfbff
---
M debian/osmo-gbproxy.install
M debian/osmo-gtphub.install
M debian/osmo-sgsn.install
3 files changed, 3 insertions(+), 3 deletions(-)
git pull ssh://gerrit.osmoc
Patch Set 1: Code-Review+1
LGTM
--
To view, visit https://gerrit.osmocom.org/3941
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9c56c218fce2264fe0acae62caed4d9ff4bfb54c
Gerrit-PatchSet: 1
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Patch Set 2: Code-Review-1
Having this ready for the case that the vty tests start failing with "Broken
pipe" again and we're fed up and don't want to fix it.
--
To view, visit https://gerrit.osmocom.org/3922
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: commen
Neels Hofmeyr has abandoned this change.
Change subject: remove contrib/soap.py, seems to apply to osmo-bsc only
..
Abandoned
I don't feel competent, may others move the script or change it.
--
To view, visit https://gerrit.o
Patch Set 1: Code-Review+2
maybe mention that it is related to the CFLAGS = -g ?
--
To view, visit https://gerrit.osmocom.org/3946
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4113c10d7698232217071becddd997fad918ca36
Gerrit-PatchSet:
Patch Set 1: Code-Review+2
(1 comment)
https://gerrit.osmocom.org/#/c/3945/1//COMMIT_MSG
Commit Message:
Line 9: The debug symbol will be stripped out of the "normal" package.
if you say so...
--
To view, visit https://gerrit.osmocom.org/3945
To unsubscribe, visit https://gerrit.osmocom.org/
Patch Set 1: Code-Review+2
the plan is to move to 2.0.0 when the split is complete ... note to self, take
this as a blueprint for then.
--
To view, visit https://gerrit.osmocom.org/3943
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I66
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/3882
to look at the new patch set (#3).
VTY: allow comments in the same line as vty commands
Note: this breaks users' VTY config when a command's argument starts with a '!'
or '#'.
Allow co
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/3944
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I681e9a1299607ff3a30de34c043da499c471bc31
Gerrit-PatchSet: 1
Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/3880
to look at the new patch set (#4).
VTY: implicit node exit by de-indenting, not parent lookup
Note: This will break users' config files if they do not use consistent
indenting. (see bel
Review at https://gerrit.osmocom.org/3945
debian/rules: compile with debug symbols
The debug symbol will be stripped out of the "normal" package.
Change-Id: I2428b922298c312c6071f51f23d7fd5939e5af91
---
M debian/rules
1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://gerrit.os
Review at https://gerrit.osmocom.org/3944
debian: .install: split mgcp-client out of legacy-mgcp
Change-Id: I681e9a1299607ff3a30de34c043da499c471bc31
---
M debian/libosmo-legacy-mgcp-dev.install
M debian/libosmo-legacy-mgcp0.install
A debian/libosmo-mgcp-client-dev.install
A debian/libosmo-mgcp
Review at https://gerrit.osmocom.org/3946
debian: use new dbgsym package instead of old dbg packages.
The new dbgsym packages gets created automatic without
taken care of them.
Change-Id: I4113c10d7698232217071becddd997fad918ca36
---
M debian/control
M debian/rules
2 files changed, 0 insertion
Review at https://gerrit.osmocom.org/3943
debian: rename libosmo-mgcp-client0 -> libosmo-mgcp-client1
The numeral is describing the library major version. libosmo-mgcp-client has
version 1.0.0
Change-Id: I66394109f909cfe0385d4519c14236bc53a10b14
---
M debian/control
1 file changed, 3 insertion
Hello Harald Welte, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/3832
to look at the new patch set (#4).
doc/examples: tweak osmo-bsc.cfg, add osmo-bsc_custom-sccp.cfg
Now osmo-bsc.cfg's SCCP addresses work by internal defaults, while
osmo-b
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/3942
to look at the new patch set (#2).
cosmetic: ipa.c: use msgb_dequeue(), drop local var
Make ipa_server_conn_write() a lot simpler and shorter by using msgb_dequeue()
and avoiding local
Review at https://gerrit.osmocom.org/3942
cosmetic: ipa.c: use msgb_dequeue() instead of code dup
Change-Id: Id968e60d081a6d69de482a908ca3fe5e1044aaa3
---
M src/input/ipa.c
1 file changed, 2 insertions(+), 4 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-abis refs/changes/42/3
Neels Hofmeyr has abandoned this change.
Change subject: add jenkins-make-check-and-run.sh, fix lock_test
..
Abandoned
merged as two separate patches. this one was just to see what the gerrit job
made of it
--
To view, visit
Patch Set 3:
In this discussion I'd also like to plug the forward perspective that all
subscriber management will move to osmo-hlr, where we also are not using libdbi
anymore for its cumbersomeness. So if your problem goes away from using sqlite
directly, consider that accomodating libdbi migh
Patch Set 3:
(2 comments)
https://gerrit.osmocom.org/#/c/3907/3/selftest/suite_test/test_suite/suite.conf
File selftest/suite_test/test_suite/suite.conf:
Line 6: - times: 2
> It really doesn't matter if I add more attributes or not for the test case
it would serve to see which one of them go
Patch Set 3: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/3830
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I97e517f2d98e83894ea707c63489559302ff6bd2
Gerrit-PatchSet: 3
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
G
Harald Welte has submitted this change and it was merged.
Change subject: osmo-auc-gen.c: squelch compiler warnings, move local var
..
osmo-auc-gen.c: squelch compiler warnings, move local var
The compiler thinks that ind or in
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/3908
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9ddcb0525159da520aceaeb6e908a735a003bb5a
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/3934
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifd88fc7ff818ea2a041eae61c5d457926a0df0f2
Gerrit-PatchSet: 1
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit
Patch Set 7: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/3807
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I39d81ab64ff790b9c4c2d0312a574485cd83e755
Gerrit-PatchSet: 7
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/3912
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I211b10b4da59c73d509b719346774515c761886a
Gerrit-PatchSet: 1
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/3929
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia13855877b2145cb57b1646f5562b2af3b87bcfb
Gerrit-PatchSet: 1
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit
Patch Set 5: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/3905
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I64cf78c5cfc78664766f9769dd5cde632dab92b0
Gerrit-PatchSet: 5
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit
Patch Set 2: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/3928
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib8814e4ce49596b586990f20d7d625a719852e90
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Patch Set 1: Verified+1
--
To view, visit https://gerrit.osmocom.org/3925
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4178b4866078f20f7415772255e9c755dcb90efe
Gerrit-PatchSet: 1
Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Ger
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/3925
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4178b4866078f20f7415772255e9c755dcb90efe
Gerrit-PatchSet: 1
Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Harald Welte has submitted this change and it was merged.
Change subject: Add git-review config
..
Add git-review config
Change-Id: I4178b4866078f20f7415772255e9c755dcb90efe
---
A .gitreview
1 file changed, 3 insertions(+), 0 d
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/3920
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I2699ceebf0cbec01652a02fa68ccc9e9419d0293
Gerrit-PatchSet: 2
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/3939
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iefc7fd5207c926fdeb3282d9d5b4708f1b56d20c
Gerrit-PatchSet: 1
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/3941
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9c56c218fce2264fe0acae62caed4d9ff4bfb54c
Gerrit-PatchSet: 1
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit
Neels Hofmeyr has submitted this change and it was merged.
Change subject: fix util.py/FileLock and lock_test
..
fix util.py/FileLock and lock_test
Do not pass os.O_TRUNC to open(), that immediately empties out the lock file.
Neels Hofmeyr has submitted this change and it was merged.
Change subject: add jenkins-make-check-and-run.sh
..
add jenkins-make-check-and-run.sh
Change-Id: I493142bb4cd6c0465e34b4171461d28455315ef4
---
A contrib/jenkins-make-c
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/3940
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I872a3d2548ac84097ac7acf13cb12c36822e076e
Gerrit-PatchSet: 1
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Patch Set 2: Code-Review+2 Verified+1
+V comes from the following patch, #3940.
+2ing to match what jenkins expects to find.
--
To view, visit https://gerrit.osmocom.org/3936
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I493142bb4cd6c0
Review at https://gerrit.osmocom.org/3941
debian: fix dependency to mgcp library
osmo-mgw has split the MGCP client to a separate library in
change-id I8e0b2d2a399b77086a36606f5e427271c6242df1
or commit 97df691307e48c39170ac39b2394a7095d7f0ee5.
Change-Id: I9c56c218fce2264fe0acae62caed4d9ff4bfb
Review at https://gerrit.osmocom.org/3940
fix util.py/FileLock and lock_test
Do not pass os.O_TRUNC to open(), that immediately empties out the lock file.
Fix lock_test to be able to catch this error. So far things were happening too
fast to notice it.
Caught by running 'make check' on a main
Neels Hofmeyr has abandoned this change.
Change subject: fix util.py/FileLock and lock_test
..
Abandoned
how hard can it be to just fire a gerrit build!
--
To view, visit https://gerrit.osmocom.org/3938
To unsubscribe, visit
Review at https://gerrit.osmocom.org/3939
add jenkins-make-check-and-run.sh, fix lock_test
fix util.py/FileLock and lock_test
Do not pass os.O_TRUNC to open(), that immediately empties out the lock file.
Fix lock_test to be able to catch this error. So far things were happening too
fast to no
fix util.py/FileLock and lock_test
Do not pass os.O_TRUNC to open(), that immediately empties out the lock file.
Fix lock_test to be able to catch this error. So far things were happening too
fast to notice it.
Caught by running 'make check' on a main unit APU that has different timing
behavior
Review at https://gerrit.osmocom.org/3938
fix util.py/FileLock and lock_test
Do not pass os.O_TRUNC to open(), that immediately empties out the lock file.
Fix lock_test to be able to catch this error. So far things were happening too
fast to notice it.
Caught by running 'make check' on a main
Neels Hofmeyr has abandoned this change.
Change subject: fix util.py/FileLock and lock_test
..
Abandoned
bloody gerrit won't retrigger unless I resubmit this
--
To view, visit https://gerrit.osmocom.org/3937
To unsubscribe, v
Review at https://gerrit.osmocom.org/3937
fix util.py/FileLock and lock_test
Do not pass os.O_TRUNC to open(), that immediately empties out the lock file.
Fix lock_test to be able to catch this error. So far things were happening too
fast to notice it.
Caught by running 'make check' on a main
Patch Set 2: Code-Review+2
verified by https://jenkins.osmocom.org/jenkins/job/osmo-gsm-tester_run/2907/
--
To view, visit https://gerrit.osmocom.org/3926
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If3226183fe21b4820149b1d03207193dc6
Neels Hofmeyr has submitted this change and it was merged.
Change subject: bts_osmotrx: Fix mkdtemp use in python < 3.5
..
bts_osmotrx: Fix mkdtemp use in python < 3.5
We are currently seeing this issues in prod setup, which us
Review at https://gerrit.osmocom.org/3936
add jenkins-make-check-and-run.sh
Change-Id: I493142bb4cd6c0465e34b4171461d28455315ef4
---
A contrib/jenkins-make-check-and-run.sh
1 file changed, 14 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester refs/changes/
Neels Hofmeyr has abandoned this change.
Change subject: contrib: Fix regexp issue deleting previous artifacts
..
Abandoned
sorry, I didn't see this and already merged the exact same patch just with a
worse commit log message.
Review at https://gerrit.osmocom.org/3935
Simplify TS alloc: move slot check into function
Move timeslot applicability check outside of nested for loop into
separate function and document it.
Change-Id: Ic39e848da47dc11357782362fdf6206d2c1457c2
Related: OS#2282
---
M src/gprs_rlcmac_ts_alloc.c
Review at https://gerrit.osmocom.org/3934
Simplify TS alloc: separate capacity computation
Move TRX capacity computation into separate function and document it.
Change-Id: Ifd88fc7ff818ea2a041eae61c5d457926a0df0f2
Related: OS#2282
---
M src/gprs_rlcmac_ts_alloc.cpp
1 file changed, 28 insertion
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/3914
to look at the new patch set (#3).
Simplify TS alloc: cosmetic, use proper formatting
The find_multi_slots() function has 5 nested for(;;) loops but it's
formatted as one. Before trying
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/3933
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib7e5995aa6914e34f8b7bbb8a8ed46ff9230a913
Gerrit-PatchSet: 1
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Neels Hofmeyr has submitted this change and it was merged.
Change subject: fix build: properly remove artifacts
..
fix build: properly remove artifacts
The jenkins scripts are run by sh, which does not support the {a,b} syntax
Review at https://gerrit.osmocom.org/3933
fix build: properly remove artifacts
The jenkins scripts are run by sh, which does not support the {a,b} syntax
shell glob. As a result, old build artifacts pile up in the workspaces.
Use two separate lines to remove .tgz and .md5.
Change-Id: Ib7e5995a
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/3925
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4178b4866078f20f7415772255e9c755dcb90efe
Gerrit-PatchSet: 1
Gerrit-Project: meta-telephony
Gerrit-Branch: 201705
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/3928
to look at the new patch set (#2).
contrib: Build octphy osmo-bts variant and make it generic
This is a first step to add support for osmo-bts-octpy to
osmo-gsm-tester.
There's no point
Hello Harald Welte, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/3905
to look at the new patch set (#5).
Simplify TS alloc: move slot assignment
Move into separate functions:
* move timeslot reservation
* move UL timeslot assignment
* move D
Review at https://gerrit.osmocom.org/3931
TS alloc: print suggested TRX on allocation errors
If TS allocation fails due to unavailable TFI, print TRX which was
suggested to allocator. This simplifies allocator debugging but requires
cosmetic modifications to test output.
Change-Id: Icaf97d71d7
Hello Harald Welte, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/3895
to look at the new patch set (#14).
TS alloc: expand tests log
* restructure code for easier reading
* use consistent formatting for output
* log essential allocation para
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/3906
to look at the new patch set (#7).
Simplify TS alloc: split allocation
* generalize TS allocation and move it into separate function
* move single-slot allocation into separate function
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/3914
to look at the new patch set (#2).
Simplify TS alloc: cosmetic, use proper formatting
The find_multi_slots() function has 4 nested for(;;) loops but it's
formatted as one. Before trying
Review at https://gerrit.osmocom.org/3930
Simplify TS alloc: split USF/UL allocation
* move USF allocation into separate function
* document USF allocation
This allows to clearly see where selected UL TS is forced into single TS
in algorithm B allocator.
Change-Id: I563dc10827ce68295553f88f3b
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/3913
to look at the new patch set (#2).
TS alloc: enable debugging instrumentation
Previously extra debugging was disabled by preprocessor directives which
means allocation tests were run wi
Review at https://gerrit.osmocom.org/3929
Simplify TS alloc: replace debug printer
Replace unreadable recursive debug printer with simpler functions.
Note: the new printer also correctly handle reserved TS so Control slot
overrides TS for the bits set for both Uplink and Downlink slots. This
d
Review at https://gerrit.osmocom.org/3932
TS alloc: update tests
* restructure code for easier reading
* rearrange tests order to facilitate further UL alloc changes
Change-Id: I6d1df8e96e1dad04ab7cf2655b8363233d6e210f
Related: OS#2282
---
M tests/alloc/AllocTest.cpp
M tests/alloc/AllocTest.er
Review at https://gerrit.osmocom.org/3928
contrib: Build octphy osmo-bts variant and make it generic
This is a first step to add support for osmo-bts-octpy to
osmo-gsm-tester.
There's no point in building separate insts for different osmo-bts
variants which run on the main unit and which come f
Review at https://gerrit.osmocom.org/3927
contrib: Fix regexp issue deleting previous artifacts
Commit 36e0404f456b32898d174b8bdfb363dbe580db2e recently modified the
line to fix an issue, but it seems the new introduced regexp is not
handled properly by jenkins jobs (it works in my terminal loc
Patch Set 2:
A job has been queued with this commit in a branch to test if it solves the
issue:
https://jenkins.osmocom.org/jenkins/view/osmo-gsm-tester/job/osmo-gsm-tester_run/2896/
--
To view, visit https://gerrit.osmocom.org/3926
To unsubscribe, visit https://gerrit.osmocom.org/settings
G
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/3926
to look at the new patch set (#2).
bts_osmotrx: Fix mkdtemp use in python < 3.5
We are currently seeing this issues in prod setup, which uses python
3.4:
File
"/home/jenkins/workspa
Review at https://gerrit.osmocom.org/3926
bts_osmotrx: Fix mkdtemp use inf python < 3.5
We are currently seeing this issues in prod setup, which uses python
3.4:
File
"/home/jenkins/workspace/osmo-gsm-tester_run/osmo-gsm-tester/src/osmo_gsm_tester/bts_osmotrx.py",
line 44, in __init__
Patch Set 1:
Original patch in gerrit was submited and merged against master. However, we
don't user master branch in meta-telephony and it may even disapear at some
point. The patch should be applied against 201705 which is the current base
branch.
--
To view, visit https://gerrit.osmocom.o
Review at https://gerrit.osmocom.org/3925
Add git-review config
Change-Id: I4178b4866078f20f7415772255e9c755dcb90efe
---
A .gitreview
1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/meta-telephony refs/changes/25/3925/1
diff --git a/.gitreview b/.git
Patch Set 3:
(3 comments)
https://gerrit.osmocom.org/#/c/3907/3/selftest/suite_test/test_suite/suite.conf
File selftest/suite_test/test_suite/suite.conf:
Line 6: - times: 2
> this makes little sense besides being an artificial example, maybe. This wa
It really doesn't matter if I add more att
Patch Set 1: Code-Review-2
related: https://osmocom.org/issues/2509
--
To view, visit https://gerrit.osmocom.org/2696
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I602604d25f51b24d87877bc8ac798525b7be61dd
Gerrit-PatchSet: 1
Gerrit-Proj
Neels Hofmeyr has submitted this change and it was merged.
Change subject: vty: Follow links to see what is a file and what is a socket
..
vty: Follow links to see what is a file and what is a socket
Change-Id: I3d94cde8523a2ed
Neels Hofmeyr has submitted this change and it was merged.
Change subject: vty: Print open file descriptors of the VTY test runner
..
vty: Print open file descriptors of the VTY test runner
/proc/self/fd refers to the process o
Patch Set 1: Verified+1
--
To view, visit https://gerrit.osmocom.org/3923
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I638e94f16f4019a7b6026d1fae69e677870fb253
Gerrit-PatchSet: 1
Gerrit-Project: python/osmo-python-tests
Gerrit-Branch:
Patch Set 1: Verified+1
--
To view, visit https://gerrit.osmocom.org/3924
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3d94cde8523a2ed26a66b8b8b3f651606aa17a9d
Gerrit-PatchSet: 1
Gerrit-Project: python/osmo-python-tests
Gerrit-Branch:
Patch Set 5: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/3899
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I0c0bcd919cc3275d491995d17c6a32bb61c6afe1
Gerrit-PatchSet: 5
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-
Neels Hofmeyr has submitted this change and it was merged.
Change subject: libmsc: sms_route_mt_sms() may return uninitialized return value
..
libmsc: sms_route_mt_sms() may return uninitialized return value
If smpp_first is se
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/3916
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I26d7a4351089b73716d193496d7161bf3d9988bb
Gerrit-PatchSet: 1
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/3917
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie9e313727053a411cec2f698c3bd654d27e8013f
Gerrit-PatchSet: 1
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/3918
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I26ddae2124813bfb63a7c97c466e056cd2ef991a
Gerrit-PatchSet: 1
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Patch Set 3: Code-Review-1
(3 comments)
https://gerrit.osmocom.org/#/c/3907/3/selftest/suite_test/test_suite/suite.conf
File selftest/suite_test/test_suite/suite.conf:
Line 6: - times: 2
this makes little sense besides being an artificial example, maybe. This wants
one BTS without specific t
Patch Set 1: Code-Review+2
(1 comment)
https://gerrit.osmocom.org/#/c/3919/1/src/osmo_gsm_tester/config.py
File src/osmo_gsm_tester/config.py:
Line 286: idx += times
(might be more obvious code to create an entirely new list on the side, but
nevermind if it works)
--
To view, vi
Patch Set 7: Code-Review-2
as a reminder to not merge, -2 to keep it sticky. Main reason that all
resources shall be reserved before the tests start running.
--
To view, visit https://gerrit.osmocom.org/3731
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/3924
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3d94cde8523a2ed26a66b8b8b3f651606aa17a9d
Gerrit-PatchSet: 1
Gerrit-Project: python/osmo-python-tests
Gerrit-Branc
Patch Set 1: Code-Review+2
lol that was my fault
--
To view, visit https://gerrit.osmocom.org/3923
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I638e94f16f4019a7b6026d1fae69e677870fb253
Gerrit-PatchSet: 1
Gerrit-Project: python/osmo-py
Patch Set 2: Code-Review+2
If the cancellable object remains valid and usable after calling cancel(), then
it should indeed remain. But if calling self.cancellable.cancel() somehow
invalidates that object / makes it not useful anymore, then please set it to
None whenever you cancel()ed, and al
1 - 100 of 108 matches
Mail list logo