Project

General

Profile

Actions

action #17182

closed

clone_job.pl doesn't work without http(s):// in the hostname

Added by pgeorgiadis almost 8 years ago. Updated over 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Feature requests
Target version:
Start date:
2017-02-17
Due date:
% Done:

100%

Estimated time:

Description

If you do not provide http(s) then you will end up with the following error:

ultron:~ # /usr/share/openqa/script/clone_job.pl --from openqa.opensuse.org --skip-chained-deps --host `hostname -f` -v 356589 TESTDEBUG=1
unexpected return code: 301 Moved Permanently at /usr/share/openqa/script/clone_job.pl line 189.

Workaround, is to use http(s):// in the hostname:

ultron:~ # /usr/share/openqa/script/clone_job.pl --from https://openqa.opensuse.org --skip-chained-deps --host `hostname -f` -v 356589 TESTDEBUG=1

I would like the error message to say something like: please use http(s):// or (better) add the missing http(s):// part automatically in the background.

Actions #1

Updated by szarate almost 8 years ago

  • Category changed from 122 to 168
  • Status changed from New to Rejected
  • Target version set to future

This is more regarding infrastructure than how jobs are being scheduled.

An excerpt from the documentation shows the following, implicitly explaining how to interact with remote hosts.

clone_job.pl --from https://openqa.opensuse.org --host localhost 24 TESTDEBUG=1 SKIPTO=consoletest-yast2_i

I see no point in adding a check for error codes like this (Because then... you have also 404, 500, and so on), I more value would be in adding a pitfall to express similar situations in the docs.

Actions #2

Updated by coolo almost 8 years ago

30X are no errors, they are redirects - and normal HTTP clients follow them. So before you add anything to the docs - rather change the client.

Actions #3

Updated by okurz almost 8 years ago

  • Category changed from 168 to Feature requests
  • Status changed from Rejected to New

agreed. client can be improved and also it's good practice to always try https if not forced by the user to not use it.

Actions #4

Updated by dheidler almost 8 years ago

  • Status changed from New to In Progress
  • Assignee set to dheidler
Actions #5

Updated by dheidler almost 8 years ago

  • % Done changed from 0 to 100

PR to simply follow the redirect: https://github.com/os-autoinst/openQA/pull/1258

Actions #6

Updated by dheidler almost 8 years ago

  • Status changed from In Progress to Resolved

merged.

Actions #7

Updated by okurz over 6 years ago

  • Target version changed from future to future
Actions

Also available in: Atom PDF