Project

General

Profile

Actions

action #116845

closed

salt-states-openqa CI complains about "fatal: Authentication failed for 'https://gitlab.suse.de/openqa/salt-pillars-openqa/'" but no useful hint size:M

Added by okurz over 1 year ago. Updated over 1 year ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
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

Related issues 2 (1 open1 closed)

Related to openQA Infrastructure - action #115553: salt-states-openqa pipeline failed: Authentication failed for 'https://gitlab.suse.de/openqa/salt-pillars-openqa/'Resolvedjbaier_cz2022-08-19

Actions
Related to openQA Infrastructure - action #135005: Reduce duplication in salt-pillars-openqa openqa/workerconf.sls with advanced YAML/jinja featuresNew2023-09-01

Actions
Actions #1

Updated by okurz over 1 year ago

  • Related to action #115553: salt-states-openqa pipeline failed: Authentication failed for 'https://gitlab.suse.de/openqa/salt-pillars-openqa/' added
Actions #2

Updated by okurz over 1 year 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
Actions #3

Updated by okurz over 1 year ago

  • Description updated (diff)
Actions #4

Updated by mkittler over 1 year ago

  • Assignee set to mkittler
Actions #5

Updated by mkittler over 1 year ago

  • Status changed from Workable to In Progress
Actions #6

Updated by openqa_review over 1 year ago

  • Due date set to 2022-11-05

Setting due date based on mean cycle time of SUSE QE Tools

Actions #7

Updated by mkittler over 1 year 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.

Actions #8

Updated by okurz over 1 year ago

  • Status changed from Resolved to Feedback
Actions #10

Updated by okurz over 1 year ago

  • Due date deleted (2022-11-05)
  • Status changed from Feedback to Resolved

MR merged.

Actions #11

Updated by okurz 8 months ago

  • Related to action #135005: Reduce duplication in salt-pillars-openqa openqa/workerconf.sls with advanced YAML/jinja features added
Actions

Also available in: Atom PDF