[valid+CI] check for required and unwanted files in input/

To detect such issues before they are merged into master.

added 1 commit

• 4bd1dab8 - [valid+CI] check for required and unwanted files in input/

Compare with previous version

J'ai testé la MR, elle a l'air de fonctionner correctement.

Par contre on a un soucis qui est lié au moment où la CI fait son test: Si le rake valid:required-unwanted-files est en échec lorsqu'il y a un fichier nodes.yaml.erb, ça nous empêche de déployer l'API, et donc de faire les premières validation de déploiement avec kadeploy.

On peut contourner ce problème de la façon suivante:

• Si le check est fait uniquement dans une Merge Request et dans la branche master, on s'assure qu'on ne laissera pas passer de templates dans la branche master. (Je ne sais pas si il est possible d'avoir une CI particulière qui s'exécute dans le cas d'une MR. Je crois qu'actuellement, dans le cas d'une MR de branch vers master c'est la pipeline de branch qui tourne).
• Une solution moins élégante serait de tester la présence d'unwanted files après avoir déployé l'API, mais je ne trouve pas ça satisfaisant. A mon avis on devrait toujours faire nos tests avant de déployer.

Je pense que ce problème est lié de manière générale à une limitation de notre workflow avec le ref-repo. A mon sens, il faudrait qu'on revoie complètement la manière de gérer nos artéfacts (ne plus commiter le répertoire data, faire les tests, générer la ref-api, générer automatiquement les fichiers de conf hiera, déployer l'API automatiquement, déployer les pages wiki automatiquement...). Mais je sais bien qu'avec la charge de travail qu'on a actuellement, et les différents projets lié à la création du site web et doc' ABACA, ça n'est pas le moment de se lancer dans la refonte d'un composant aussi critique que le ref-repo.

J'ai créé une branche test-check-unwanted-files pour illustrer le problème et tester différents scénarios si besoin.

added 1 commit

• 6adc4d6f - [ci] reorganize jobs

Compare with previous version

added 8 commits

• 6adc4d6f...68dd38c8 - 6 commits from branch master
• 4f92ca34 - [valid+CI] check for required and unwanted files in input/
• 24794036 - [ci] reorganize jobs

Compare with previous version

resolved all threads

merged

mentioned in commit 83a57cff