Bug 1652476

Summary:	beaker-repo-update did not fetch harness for distros
Product:	[Retired] Beaker	Reporter:	Martin Styk <mastyk>
Component:	general	Assignee:	Martin Styk <mastyk>
Status:	CLOSED NEXTRELEASE	QA Contact:	tools-bugs <tools-bugs>
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	26	CC:	bpeck, breilly, cbeer, ed
Target Milestone:	26.2
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2018-11-26 16:41:00 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Martin Styk 2018-11-22 08:22:05 UTC

Description of problem:
beaker-repo-update with default values didn't populate the harness repos for Distros.

Version-Release number of selected component (if applicable):
26

How reproducible:
easily.

Steps to Reproduce:
1. Setup beaker-in-a-box via Ansible
2. Run beaker-repo-update without arguments 

Actual results:
2018-11-21 12:17:21,719 bkr.server.tools.repo_update WARNING Harness packages not found for OS major CentOS6, ignoring
2018-11-21 12:17:21,920 bkr.server.tools.repo_update WARNING Harness packages not found for OS major CentOS7, ignoring
2018-11-21 12:17:22,119 bkr.server.tools.repo_update WARNING Harness packages not found for OS major Fedora27, ignoring
2018-11-21 12:17:22,394 bkr.server.tools.repo_update WARNING Harness packages not found for OS major Fedora28, ignoring
Nothing stored in /var/www/beaker/harness

Expected results:
Test harness and other packages are stored/cached in /var/www/beaker/harness/osmajor/

Additional info:

Comment 1 Martin Styk 2018-11-22 08:57:08 UTC

It seems that issue was introduced in commit 157935e. A check was added to ensure that harness repo exists (status code 200). However, this will not work with the default parameter for BASEURL (http://beaker-project.org/yum/harness/).

% curl -i http://beaker-project.org/yum/harness/                            
HTTP/1.1 301 Moved Permanently
Location: https://beaker-project.org/yum/harness/
Content-Length: 0
Date: Thu, 22 Nov 2018 08:53:53 GMT
Server: lighttpd/1.4.49

Therefore, we should update the default BASEURL to https://beaker-project.org/yum/harness/
% curl -i https://beaker-project.org/yum/harness/
HTTP/1.1 200 OK

Comment 3 Christopher Beer 2018-11-26 17:53:33 UTC

My mistake, I told Martin to close it with the incorrect identifier, I am fixing it.

Comment 4 Christopher Beer 2018-12-05 19:20:57 UTC

*** Bug 1643618 has been marked as a duplicate of this bug. ***