Actions
action #90038
closedcoordination #127031: [saga][epic] openQA for SUSE customers
coordination #80150: [epic] Scale out openQA: Easier openQA setup
Better error handling when reading API key+secret from ~/.config/openqa/client.conf
Start date:
2021-03-12
Due date:
% Done:
0%
Estimated time:
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
Updated by mkittler over 3 years ago
- Status changed from Workable to In Progress
- Assignee set to mkittler
I've already fixed the warning in the meantime: https://github.com/os-autoinst/openQA/commit/afd13620b47118fa4e6f2f2700fc7e2aea65c6ba
Updated by mkittler over 3 years ago
PR for remaining ACs: https://github.com/os-autoinst/openQA/pull/3815
Updated by mkittler over 3 years ago
- Status changed from In Progress to Resolved
The PR has been merged.
Updated by okurz over 2 years ago
- Related to action #81890: Extend help for openQA client config, e.g. mention the environment variable OPENQA_CONFIG added
Actions