Change in osmo-ci[master]: repo-install-test: install repo after repo key

[osmith]

osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/23129 )

Change subject: repo-install-test: install repo after repo key
......................................................................

repo-install-test: install repo after repo key

In previous patch 27ee885a689163cede1fc0c7f76584ccbb7abb41, I made sure
that "apt-get update" runs before trying to install wget to download the
repository key. But of course the OBS repository should not be present
before installing the repo key, or else it will fail:

E: The repository 'http://download.opensuse.org/.../Debian_10 ./ InRelease' is not signed.

Fixes: 27ee885 ("repo-install-test: apt update before install wget")
Change-Id: If79484f9ffe2a14ce6481b53867f5aee111aa11b
---
M scripts/repo-install-test/run-inside-docker.sh
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  pespin: Looks good to me, approved
  osmith: Verified



diff --git a/scripts/repo-install-test/run-inside-docker.sh b/scripts/repo-install-test/run-inside-docker.sh
index 2ed78f6..4a5f01a 100755
--- a/scripts/repo-install-test/run-inside-docker.sh
+++ b/scripts/repo-install-test/run-inside-docker.sh
@@ -106,7 +106,6 @@
 	local release_key="/var/cache/apt/${proj}_Release.key"
 
 	echo "Configuring Osmocom repository"
-	echo "deb http://$obs_repo ./" > "/etc/apt/sources.list.d/$proj.list"
 
 	# Add repository key
 	if ! [ -e "$release_key" ]; then
@@ -116,6 +115,7 @@
 	fi
 	apt-key add "$release_key"
 
+	echo "deb http://$obs_repo ./" > "/etc/apt/sources.list.d/$proj.list"
 	apt-get update
 }
 

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

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: If79484f9ffe2a14ce6481b53867f5aee111aa11b
Gerrit-Change-Number: 23129
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith at sysmocom.de>
Gerrit-Reviewer: osmith <osmith at sysmocom.de>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210226/29b70bd0/attachment.htm>