[S] Change in osmo-ci[master]: repo-install-test: ignore limesuite-images

2023-11-02 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ci/+/34955?usp=email )

Change subject: repo-install-test: ignore limesuite-images
..

repo-install-test: ignore limesuite-images

Don't attempt to install limesuite-images. It runs a post-install script
that downloads files from an external server and fails currently, as
there are no images for 23.10. While we have limesuite-images packages
in our OBS repository, this is just a side-effect of building limesuite.
What we are really interested in is liblimesuite for osmo-trx, as I
understand.

Add --no-install-recommends to the apt-get install line in
install_repo_packages_debian, because the main limesuite package has
limesuite-images in recommends and would pull it in otherwise.

Change-Id: I237408c805977c831f352a57a301ea45753d1ac1
---
M scripts/repo-install-test/blacklist_debian10.txt
M scripts/repo-install-test/blacklist_debian11.txt
M scripts/repo-install-test/blacklist_debian12.txt
M scripts/repo-install-test/run-inside.sh
4 files changed, 29 insertions(+), 1 deletion(-)

Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified
  pespin: Looks good to me, but someone else must approve




diff --git a/scripts/repo-install-test/blacklist_debian10.txt 
b/scripts/repo-install-test/blacklist_debian10.txt
index 6570de6..93dac75 100644
--- a/scripts/repo-install-test/blacklist_debian10.txt
+++ b/scripts/repo-install-test/blacklist_debian10.txt
@@ -19,6 +19,10 @@
 osmocom-nitb
 osmocom-nitb-dbg

+# Not relevant for the test
+limesuite-images
+limesuite-images23.10
+
 # SoapySDR is not used anymore (see OS#3542)
 soapysdr-module-lms7
 soapysdr0.6-module-lms7
diff --git a/scripts/repo-install-test/blacklist_debian11.txt 
b/scripts/repo-install-test/blacklist_debian11.txt
index 53187dc..c962ffc 100644
--- a/scripts/repo-install-test/blacklist_debian11.txt
+++ b/scripts/repo-install-test/blacklist_debian11.txt
@@ -31,6 +31,8 @@
 libulfius2.7-dbgsym
 libyder2.0-dbgsym
 limesuite-dbgsym
+limesuite-images
+limesuite-images23.10
 soapysdr0.7-module-lms7-dbgsym

 # Depends on specific verions 0.5.4.38.0847 of rtl-sdr, which we won't install
diff --git a/scripts/repo-install-test/blacklist_debian12.txt 
b/scripts/repo-install-test/blacklist_debian12.txt
index b470d44..9b5f5e7 100644
--- a/scripts/repo-install-test/blacklist_debian12.txt
+++ b/scripts/repo-install-test/blacklist_debian12.txt
@@ -32,6 +32,8 @@
 libulfius2.7-dbgsym
 libyder2.0-dbgsym
 limesuite-dbgsym
+limesuite-images
+limesuite-images23.10
 soapysdr0.7-module-lms7
 soapysdr0.7-module-lms7-dbgsym
 soapysdr-module-lms7
diff --git a/scripts/repo-install-test/run-inside.sh 
b/scripts/repo-install-test/run-inside.sh
index e08338f..d7730ff 100755
--- a/scripts/repo-install-test/run-inside.sh
+++ b/scripts/repo-install-test/run-inside.sh
@@ -329,7 +329,7 @@
cat osmocom_packages_all.txt

filter_packages_txt
-   apt install -y $(cat osmocom_packages.txt)
+   apt-get install -y --no-install-recommends $(cat osmocom_packages.txt)
 }

 install_repo_packages_centos() {

--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/34955?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I237408c805977c831f352a57a301ea45753d1ac1
Gerrit-Change-Number: 34955
Gerrit-PatchSet: 2
Gerrit-Owner: osmith 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


[S] Change in osmo-ci[master]: repo-install-test: ignore limesuite-images

2023-11-02 Thread laforge
Attention is currently required from: osmith.

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ci/+/34955?usp=email )

Change subject: repo-install-test: ignore limesuite-images
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/34955?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I237408c805977c831f352a57a301ea45753d1ac1
Gerrit-Change-Number: 34955
Gerrit-PatchSet: 1
Gerrit-Owner: osmith 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Attention: osmith 
Gerrit-Comment-Date: Thu, 02 Nov 2023 21:42:06 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


[S] Change in osmo-ci[master]: repo-install-test: ignore limesuite-images

2023-11-02 Thread pespin
Attention is currently required from: osmith.

pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ci/+/34955?usp=email )

Change subject: repo-install-test: ignore limesuite-images
..


Patch Set 1: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/34955?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I237408c805977c831f352a57a301ea45753d1ac1
Gerrit-Change-Number: 34955
Gerrit-PatchSet: 1
Gerrit-Owner: osmith 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Attention: osmith 
Gerrit-Comment-Date: Thu, 02 Nov 2023 16:01:17 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


[S] Change in osmo-ci[master]: repo-install-test: ignore limesuite-images

2023-11-02 Thread osmith
osmith has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-ci/+/34955?usp=email )


Change subject: repo-install-test: ignore limesuite-images
..

repo-install-test: ignore limesuite-images

Don't attempt to install limesuite-images. It runs a post-install script
that downloads files from an external server and fails currently, as
there are no images for 23.10. While we have limesuite-images packages
in our OBS repository, this is just a side-effect of building limesuite.
What we are really interested in is liblimesuite for osmo-trx, as I
understand.

Add --no-install-recommends to the apt-get install line in
install_repo_packages_debian, because the main limesuite package has
limesuite-images in recommends and would pull it in otherwise.

Change-Id: I237408c805977c831f352a57a301ea45753d1ac1
---
M scripts/repo-install-test/blacklist_debian10.txt
M scripts/repo-install-test/blacklist_debian11.txt
M scripts/repo-install-test/blacklist_debian12.txt
M scripts/repo-install-test/run-inside.sh
4 files changed, 29 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/55/34955/1

diff --git a/scripts/repo-install-test/blacklist_debian10.txt 
b/scripts/repo-install-test/blacklist_debian10.txt
index 6570de6..93dac75 100644
--- a/scripts/repo-install-test/blacklist_debian10.txt
+++ b/scripts/repo-install-test/blacklist_debian10.txt
@@ -19,6 +19,10 @@
 osmocom-nitb
 osmocom-nitb-dbg

+# Not relevant for the test
+limesuite-images
+limesuite-images23.10
+
 # SoapySDR is not used anymore (see OS#3542)
 soapysdr-module-lms7
 soapysdr0.6-module-lms7
diff --git a/scripts/repo-install-test/blacklist_debian11.txt 
b/scripts/repo-install-test/blacklist_debian11.txt
index 53187dc..c962ffc 100644
--- a/scripts/repo-install-test/blacklist_debian11.txt
+++ b/scripts/repo-install-test/blacklist_debian11.txt
@@ -31,6 +31,8 @@
 libulfius2.7-dbgsym
 libyder2.0-dbgsym
 limesuite-dbgsym
+limesuite-images
+limesuite-images23.10
 soapysdr0.7-module-lms7-dbgsym

 # Depends on specific verions 0.5.4.38.0847 of rtl-sdr, which we won't install
diff --git a/scripts/repo-install-test/blacklist_debian12.txt 
b/scripts/repo-install-test/blacklist_debian12.txt
index b470d44..9b5f5e7 100644
--- a/scripts/repo-install-test/blacklist_debian12.txt
+++ b/scripts/repo-install-test/blacklist_debian12.txt
@@ -32,6 +32,8 @@
 libulfius2.7-dbgsym
 libyder2.0-dbgsym
 limesuite-dbgsym
+limesuite-images
+limesuite-images23.10
 soapysdr0.7-module-lms7
 soapysdr0.7-module-lms7-dbgsym
 soapysdr-module-lms7
diff --git a/scripts/repo-install-test/run-inside.sh 
b/scripts/repo-install-test/run-inside.sh
index e08338f..d7730ff 100755
--- a/scripts/repo-install-test/run-inside.sh
+++ b/scripts/repo-install-test/run-inside.sh
@@ -329,7 +329,7 @@
cat osmocom_packages_all.txt

filter_packages_txt
-   apt install -y $(cat osmocom_packages.txt)
+   apt-get install -y --no-install-recommends $(cat osmocom_packages.txt)
 }

 install_repo_packages_centos() {

--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/34955?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I237408c805977c831f352a57a301ea45753d1ac1
Gerrit-Change-Number: 34955
Gerrit-PatchSet: 1
Gerrit-Owner: osmith 
Gerrit-MessageType: newchange