Project

General

Profile

action #90038

coordination #80142: [saga][epic] Scale out: Redundant/load-balancing deployments of openQA, easy containers, containers on kubernetes

coordination #80150: [epic] Scale out openQA: Easier openQA setup

Better error handling when reading API key+secret from ~/.config/openqa/client.conf

Added by okurz 4 months ago. Updated 4 months ago.

Status:
Resolved
Priority:
Low
Assignee:
Category:
Feature requests
Target version:
Start date:
2021-03-12
Due date:
% Done:

0%

Estimated time:
Difficulty:
easy

Description

Observation

With a file like ~/.config/openqa/client.conf with slightly incorrect format and names used because the user does not know any better:

[openqa.opensuse.org]
apikey = FOO
apisecret = BAR
$ openqa-clone-job --skip-chained-deps --within-instance https://openqa.opensuse.org/tests/1234
Use of uninitialized value $val in substitution (s///) at /usr/share/openqa/script/../lib/OpenQA/UserAgent.pm line 46.
Use of uninitialized value $val in substitution (s///) at /usr/share/openqa/script/../lib/OpenQA/UserAgent.pm line 46.
Failed to create job: "{\"error\":\"no api key\",\"error_status\":403}" at /usr/bin/openqa-clone-job line 304.

Acceptance criteria

  • AC1: No low-level perl warnings on incorrect usage or user mistakes
  • AC2: There is help accessible from command line explaining the right format for the config file
  • AC3: More helpful error message

History

#1 Updated by mkittler 4 months ago

  • Status changed from Workable to In Progress
  • Assignee set to mkittler

#3 Updated by mkittler 4 months ago

  • Status changed from In Progress to Resolved

The PR has been merged.

Also available in: Atom PDF