Actions
action #116845
closedsalt-states-openqa CI complains about "fatal: Authentication failed for 'https://gitlab.suse.de/openqa/salt-pillars-openqa/'" but no useful hint size:M
Start date:
2022-09-20
Due date:
% Done:
0%
Estimated time:
Description
Observation¶
CI jobs like https://gitlab.suse.de/okurz/salt-states-openqa/-/jobs/1147783 complain with
$ git clone --depth 3 https://osd_deployment_ci:${PILLARS_CLONE_TOKEN}@gitlab.suse.de/openqa/salt-pillars-openqa /srv/pillar || echo "Copy pillar access token to …"
Cloning into '/srv/pillar'...
remote: HTTP Basic: Access denied
fatal: Authentication failed for 'https://gitlab.suse.de/openqa/salt-pillars-openqa/'
I thought I would have such credentials, maybe something got lost or expired. It turned out I need to But there is no mention in the README nor helpful error message pointing me to what to do.
Acceptance criteria¶
- AC1: CI jobs for salt-states-openqa work without users needing to set any salt pillar credentials or clear instructions exist
Suggestions¶
- Make sure tests run without cloning salt pillars in merge requests, e.g. with a mocked pillar directory that is symlinked to /srv/pillar
- Alternative: Add something to the README or at best within the CI script fix the
echo "Copy pillar access token
which should be telling the user with a hint what to do and consider how people who don't have access would be able to contribute - Alternative: Encrypt the pillar sensitive data so that we can make the pillar repo public again because tests don't need to care about individual values
Updated by okurz about 2 years ago
- Related to action #115553: salt-states-openqa pipeline failed: Authentication failed for 'https://gitlab.suse.de/openqa/salt-pillars-openqa/' added
Updated by okurz about 2 years ago
- Subject changed from salt-states-openqa CI complains about "fatal: Authentication failed for 'https://gitlab.suse.de/openqa/salt-pillars-openqa/'" but no useful hint to salt-states-openqa CI complains about "fatal: Authentication failed for 'https://gitlab.suse.de/openqa/salt-pillars-openqa/'" but no useful hint size:M
- Description updated (diff)
- Status changed from New to Workable
Updated by mkittler almost 2 years ago
- Status changed from Workable to In Progress
Updated by openqa_review almost 2 years ago
- Due date set to 2022-11-05
Setting due date based on mean cycle time of SUSE QE Tools
Updated by mkittler almost 2 years ago
- Status changed from In Progress to Resolved
The SR has been merged with a passing CI pipeline that is now independent from production pillars.
Updated by okurz almost 2 years ago
- Status changed from Resolved to Feedback
I think https://github.com/os-autoinst/salt-states-openqa/#ci-tests still needs to be updated?
Updated by mkittler almost 2 years ago
Updated by okurz almost 2 years ago
- Due date deleted (
2022-11-05) - Status changed from Feedback to Resolved
MR merged.
Updated by okurz about 1 year ago
- Related to action #135005: Reduce duplication in salt-pillars-openqa openqa/workerconf.sls with advanced YAML/jinja features added
Updated by jbaier_cz 5 months ago
- Copied to action #160883: salt-states-openqa CI complains about "Authentication failed for 'https://gitlab.suse.de/openqa/salt-pillars-openqa.git/'" size:S added
Actions