gitlab-ci-linter is broken since GitLab v16

• POST ci/lint API endpoint deprecated
• https://gitlab.com/devopshq/gitlab-ci-linter/-/issues/4

Whether it get fixed, we find an alternative, or we get rid of this tool.

Alternatives:

• https://pre-commit.com/hooks.html
• https://gitlab.com/smop/pre-commit-hooks
• https://github.com/FalcoSuessgott/lint-gitlab-ci

changed milestone to %1.3.0

added CI/CD Dev Env labels

changed the description

assigned to @moul

changed milestone to %1.2.0

marked this issue as related to #169 (closed)

changed the description

https://github.com/FalcoSuessgott/lint-gitlab-ci has support for GitLab v16 API change. The new issues which come with this tool are:

• We need an HTTP token per project GITLAB_TOKEN. I used to use the same token for Silkaj and DunitePy. Now, we have to have to set the token when working on one or the other project. Since there is no custom env variable. May be it’s not a big deal, if we are changing gitlab-ci.yml, we set the token, get it checked before pushing. Probably GitLab v16 API change related (not the tool it-self). Upstream can support to read different env variable names.
• They are determining the GitLab instance with git remote -v. I had g1_monetary_license remote in my Silkaj local clone for the Ğ1 Monetary License subtree. Therefore, the check was failing, because it was running on the wrong project, token are projects specific now. I can delete this remote. But, having GitLab URL hardcoded is somehow safer. The tool should support to hardcode an URL as https://gitlab.com/devopshq/gitlab-ci-linter/ does. I found that adding grep origin solves the issue.

These topics can be reported and fixed upstream for better support.

In the meantime, this hook seems to be the only one supporting GitLab v16, so let switch to it even though there are known limitations.

In the end I managed to have gitlab-ci-linter working as stated in #4 by specifying --project client/python/$project. I didn’t manage to have it working last time probably because I was using DuniterPy token on Silkaj project as stated in first bullet point above.

mentioned in commit 046ba62f

removed CI/CD label

mentioned in merge request !195 (merged)

mentioned in commit silkaj@10248a3f

closed with merge request !195 (merged)

mentioned in issue silkaj#484 (closed)

mentioned in commit silkaj@95ef7abd