Project

General

Profile

Actions

action #161393

closed

openQA Project - coordination #112862: [saga][epic] Future ideas for easy multi-machine handling: MM-tests as first-class citizens

openQA Project - coordination #111929: [epic] Stable multi-machine tests covering multiple physical workers

gitlab CI jobs do not execute salt-lint and do not fail on missing "find" executable

Added by okurz 27 days ago. Updated 19 days ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Regressions/Crashes
Target version:
Start date:
2024-06-03
Due date:
% Done:

0%

Estimated time:

Description

Observation

https://gitlab.suse.de/pdostal/salt-pillars-openqa/-/jobs/2661772#L24
states

$ salt-lint -v $(if command -v git >/dev/null; then git ls-files "*.sls"; else find -name '*.sls'; fi)
/usr/bin/bash: line 149: find: command not found

so does not find "find" but then succeeds because the problem in the command evaluation is ignored so we never call salt-lint

Acceptance criteria

  • AC1: salt-lint CI jobs succeed again consistently
  • AC2: the CI jobs fail if find or another necessary application do not exist

Suggestions

  • Improve the command to fail if a necessary command does not exist. Possibly we can simplify to just rely on git (or find)
  • Ensure that salt-lint CI jobs succeed again consistently

Related issues 1 (0 open1 closed)

Copied to openQA Infrastructure - action #161396: gitlab CI jobs do not fail on invalid YAML size:SResolvedybonatakis2024-06-032024-06-19

Actions
Actions

Also available in: Atom PDF